git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 8/9] autoconf: Check for timer_settime

From
Keller, Jacob E <jacob.e.keller@intel.com>
Date
Sep 10, 2014, 21:13 UTC
Message-ID
<1410383601.2649.5.camel@jekeller-desk1.amr.corp.intel.com>
In-Reply-To
<xmqq38bzgp4j.fsf@gitster.dls.corp.google.com>
On Wed, 2014-09-10 at 14:08 -0700, Junio C Hamano wrote:
Show 17 quoted lines
> Karsten Blees <karsten.blees@gmail.com> writes:
> 
> > While the timer extension (timer_settime) has graduated to mandatory in
> > the current POSIX spec, the monotonic clock extension is still optional
> > today (i.e. not necessarily supported even on newer Unices). In contrast
> > to this, the XSI extensions seem to be widely supported.
> >
> > IMO the 'obsolescent' marker in the current POSIX spec is no reason to
> > spring into action (at least not yet). E.g. utimes (also in <sys/time.h>)
> > has been marked LEGACY in the 2004 version and is no longer LEGACY today.
> > Btw., we'd also have to find a replacement for gettimeofday and probably
> > a lot of other stuff...
> >
> > Therefore I tend to agree with Hannes that we should stick with setitimer
> > and emulate it on systems that don't have it (as we do on Windows).
> 
> Sounds sensible to me.

Ya that makes sense. Was thinking too much about compatability code for Linux kernel drivers, which is a different issue.

Let's go with re-implementing settimer in terms of timer_settime on the one system we have information that doesn't support settimer.

I could spin that patch, but I think the original author should, since he's the one who could actually test it.

Regards, Jake

> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous: Junio C HamanoNext: Jacob Keller
Message 14 of 23 in “git-compat-util.h: Add missing semicolon after struct itimerval”
  1. 1/9 git-compat-util.h: Add missing semicolon after struct itimervalJacob Keller, Aug 29, 2014
  2. 2/9 autoconf: Check for struct itimervalJacob Keller, Aug 29, 2014
  3. 3/9 autoconf: Check for setitimerJacob Keller, Aug 29, 2014
  4. 4/9 autoconf: Check for timer_tJacob Keller, Aug 29, 2014
  5. Jonas 'Sortie' TermansenAug 29, 2014
  6. 5/9 autoconf: Check for struct timespecJacob Keller, Aug 29, 2014
  7. 6/9 autoconf: Check for struct sigeventJacob Keller, Aug 29, 2014
  8. 7/9 autoconf: Check for struct itimerspecJacob Keller, Aug 29, 2014
  9. 8/9 autoconf: Check for timer_settimeJacob Keller, Aug 29, 2014
  10. Johannes SixtAug 29, 2014
  11. Keller, Jacob EAug 29, 2014
  12. Karsten BleesSep 10, 2014
  13. Junio C HamanoSep 10, 2014
  14. Keller, Jacob ESep 10, 2014
  15. 9/9 Use timer_settime for new platformsJacob Keller, Aug 29, 2014
  16. Junio C HamanoAug 29, 2014
  17. Keller, Jacob EAug 29, 2014
  18. Junio C HamanoAug 29, 2014
  19. Keller, Jacob EAug 29, 2014
  20. Jonas 'Sortie' TermansenAug 29, 2014
  21. Junio C HamanoAug 29, 2014
  22. Jonas 'Sortie' TermansenAug 29, 2014
  23. Keller, Jacob ESep 3, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.