public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: David Mosberger <davidm@napali.hpl.hp.com>
To: linux-ia64@vger.kernel.org
Subject: Re: Timer patches (nsec support + fastcalls for gettod/clock_gettime
Date: Mon, 12 Jul 2004 23:48:48 +0000	[thread overview]
Message-ID: <16627.9056.572246.317538@napali.hpl.hp.com> (raw)
In-Reply-To: <Pine.LNX.4.58.0407091545250.5720@schroedinger.engr.sgi.com>

>>>>> On Mon, 12 Jul 2004 16:40:13 -0700 (PDT), Christoph Lameter <clameter@sgi.com> said:

  Christoph> Ok. Will do all of what you said that I did not quote. Do
  Christoph> I really need to save predicates? There are no docs on
  Christoph> saving pr and the old gettimeofday assembly routine did
  Christoph> not save the predicates and worked without a problem.

Preserved registers _always_ need to be preserved by fsys-handlers.
If you avoid using preserved predicates, there is no need to save
them.  See the software conventions & runtime architecture guide for
details (as far as predicate registers go, p1-p5 and p16-p63 are
"preserved").

  Christoph> I was not able to find the patch but I will talk to
  Christoph> George about this.  However, gettimeofday with a timeval
  Christoph> is an established standard. George's current patches on
  Christoph> sf.net use clock_gettime to retrieve higher precision
  Christoph> timer values.

The kernel-internal gettimeofday() can return anything it wants (as
long as all platforms do it consistently.  As to whether or not that's
a good idea, i don't really care (but I do not think it's reasonable
or necessary to have two platform routines with the only difference
being whether they return timeval or timespec).

  Christoph> What could be done is to change do_gettimeofday to use a
  Christoph> timespec and then have sys_gettimeofday to the proper
  Christoph> scaling. Is that what you were thinking?

Yup, that's what I meant.

  Christoph> But do_gettimeofday is used a gazillion times all over
  Christoph> the linux sources and changing that would result in a
  Christoph> huge patch that might not be acceptable either.

That can be OK, if there is agreement that it's The Right Thing.
Hence it's definitely something that should be done in coordination
with George and lkml in general (and it also needs to be done
separately from the ia64-specific changes; figuring out how to break
the patch up is one thing we'll need to sort out eventually).

	--david

  parent reply	other threads:[~2004-07-12 23:48 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-09 22:58 Timer patches (nsec support + fastcalls for gettod/clock_gettime Christoph Lameter
2004-07-12 17:45 ` Timer patches (nsec support + fastcalls for gettod/clock_gettime for all clocks) Chen, Kenneth W
2004-07-12 19:15 ` Christoph Lameter
2004-07-12 20:22 ` Christoph Lameter
2004-07-12 22:01 ` David Mosberger
2004-07-12 23:40 ` Timer patches (nsec support + fastcalls for gettod/clock_gettime Christoph Lameter
2004-07-12 23:48 ` David Mosberger [this message]
2004-07-13  1:19 ` Christoph Lameter

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=16627.9056.572246.317538@napali.hpl.hp.com \
    --to=davidm@napali.hpl.hp.com \
    --cc=linux-ia64@vger.kernel.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