qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Brad Smith <brad@comstyle.com>
To: qemu-devel@nongnu.org
Cc: Paolo Bonzini <pbonzini@redhat.com>
Subject: Re: [Qemu-devel] [PATCH] Add configure clock_gettime() monotonic time test
Date: Tue, 2 Apr 2013 18:28:55 -0400	[thread overview]
Message-ID: <20130402222854.GK26146@rox.home.comstyle.com> (raw)
In-Reply-To: <515B0B1D.9050202@redhat.com>

On Tue, Apr 02, 2013 at 06:45:17PM +0200, Paolo Bonzini wrote:
> Il 02/04/2013 17:42, Brad Smith ha scritto:
> > Replace the hardcoded list of OS's utilizing clock_gettime() for monotonic
> > time with a configure test. This is to fix the use of monotonic time on
> > OpenBSD but allows for other POSIX compliant OS's such as NetBSD to also
> > utilize clock_gettime().
> 
> I thought the list of OSes was supposed to filter out those that somehow
> had a broken CLOCK_MONOTONIC.
> 
> Otherwise, you might as well use "#ifdef CLOCK_MONOTONIC" and skip the
> configure test completely.

I only wrote the test as that was what was suggested and even if there
are OS's that have broken monotonic support it is better to put any
exclusion logic in the configure script over having more hardcoded
lists of OS's directly in the code. What OS's have broken monotonic
support?

I would have no problem going in that direction if no one can actually
come up with an OS (or more) that have monotonic support but it is
somehow broken; as in having clock_gettime() and the CLOCK_MONOTONIC
define but not functioning properly.

-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

  reply	other threads:[~2013-04-02 22:29 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-02 15:42 [Qemu-devel] [PATCH] Add configure clock_gettime() monotonic time test Brad Smith
2013-04-02 16:45 ` Paolo Bonzini
2013-04-02 22:28   ` Brad Smith [this message]
2013-04-02 22:52   ` Peter Maydell
2013-04-03  8:18     ` Paolo Bonzini
2013-04-03 19:22       ` Brad Smith
2013-04-03 20:00         ` Paolo Bonzini
2013-04-03 20:07         ` Brad Smith
2013-04-04 17:21         ` Brad Smith
2013-04-05  0:22           ` Anthony Liguori

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130402222854.GK26146@rox.home.comstyle.com \
    --to=brad@comstyle.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).