From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Jan Beulich <JBeulich@suse.com>,
George Dunlap <George.Dunlap@eu.citrix.com>
Cc: xen-devel <xen-devel@lists.xenproject.org>,
Dario Faggioli <dario.faggioli@citrix.com>
Subject: Re: PRI_stime
Date: Tue, 20 Oct 2015 10:46:05 +0100 [thread overview]
Message-ID: <56260D5D.9040505@citrix.com> (raw)
In-Reply-To: <562618B902000078000AC9F4@prv-mh.provo.novell.com>
On 20/10/15 09:34, Jan Beulich wrote:
>>>> On 20.10.15 at 10:10, <George.Dunlap@eu.citrix.com> wrote:
>> On Tue, Oct 20, 2015 at 7:26 AM, Jan Beulich <JBeulich@suse.com> wrote:
>>> George, Dario,
>>>
>>> it being mostly used in scheduler code, and me considering it quite a
>>> bit easier to compare such big numbers when shown in hex I wonder:
>>> Do you prefer this to stay PRId64, or would you accept it to be
>>> changed to PRIx64 (allowing it to be used in a few other places)?
>> Personally I've never taken the time to familiarize myself with the
>> magnitude of hex numbers vs decimal numbers; so in the case of time, I
>> could easily see that 10000000 nanoseconds is about 1ms; but I don't
>> have a good sense of how long 0x1000000 nanoseconds is. The fact that
>> our times are based on base 10 instead of base 2 is I think as good an
>> argument as any for leaving it as a decimal.
> Well, as long as the number of seconds the value represents is small,
> this indeed is a good argument for using dec. However, already
> when we get into the hour range we're talking about 12-digit values
> without any separators, and at least for me this means counting
> from either end to find a place where to put a mental separator. So
> if staying with dec, perhaps we should make these second based,
> i.e. <ssss>.<nnnnnnnnn>?
Having a custom %p for this would be very useful.
As for the original hex vs dec question, I am really not fussed, so long
as we are consistent (which is not the case currently).
~Andrew
next prev parent reply other threads:[~2015-10-20 9:46 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-20 6:26 PRI_stime Jan Beulich
2015-10-20 8:10 ` PRI_stime George Dunlap
2015-10-20 8:22 ` PRI_stime Juergen Gross
2015-10-20 8:34 ` PRI_stime Jan Beulich
2015-10-20 9:46 ` Andrew Cooper [this message]
2015-10-20 9:59 ` PRI_stime Jan Beulich
2015-10-20 10:18 ` PRI_stime Dario Faggioli
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=56260D5D.9040505@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=George.Dunlap@eu.citrix.com \
--cc=JBeulich@suse.com \
--cc=dario.faggioli@citrix.com \
--cc=xen-devel@lists.xenproject.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.