public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: John Daiker <jdaiker@osdl.org>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: John Hawkes <hawkes@sgi.com>,
	Arjan van de Ven <arjan@infradead.org>,
	Tony Luck <tony.luck@gmail.com>, Andrew Morton <akpm@osdl.org>,
	linux-ia64@vger.kernel.org, linux-kernel@vger.kernel.org,
	Jack Steiner <steiner@sgi.com>, Dan Higgins <djh@sgi.com>,
	Jeremy Higdon <jeremy@sgi.com>
Subject: Re: [PATCH] ia64: change usermode HZ to 250
Date: Wed, 28 Jun 2006 09:11:23 -0700	[thread overview]
Message-ID: <44A2AA2B.3090101@osdl.org> (raw)
In-Reply-To: <1151511668.15166.34.camel@localhost.localdomain>

Alan Cox wrote:
> Ar Mer, 2006-06-28 am 08:43 -0700, ysgrifennodd John Hawkes:
>   
>>> #define HZ sysconf(_SC_CLK_TCK)
>>>       
>> That did occur to me.  It obviously does get the correct value.  The downside
>> is that one of those crufty apps that thinks it is using "HZ" as a constant
>> will instead be invoking a more costly syscall.  Should we care about the
>> resulting performance impact?
>>     
>
> Given that HZ can be cached by glibc the performance impact is minimal
> for most cases. The bigger problem will be code that does things with HZ
> that only work on compile time evaluation. At least for those you'll
> break at compile time.
>
> Either way its kind of irrelevant, the ABI set HZ. Its done, there are
> plenty of ways to change the kernel HZ without confusing userspace.
>
> Alan
>
>   
Alan, I agree with Arjan's solution as well.  From a very novice point 
of view, it makes sense to #define HZ as a syscall (which it technically 
should be anyway, right?).  Any performance hit isn't our problem... 
people should have been using the syscall to begin with... we're just 
forcing it on them this way!  :-)  That's my $0.02

John Daiker

  reply	other threads:[~2006-06-28 16:11 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-27 22:01 [PATCH] ia64: change usermode HZ to 250 hawkes
2006-06-28  8:39 ` Arjan van de Ven
2006-06-28 15:43   ` John Hawkes
2006-06-28 16:21     ` Alan Cox
2006-06-28 16:11       ` John Daiker [this message]
2006-06-28 16:50         ` Alan Cox
  -- strict thread matches above, loose matches on Subject: below --
2006-06-27 22:26 Luck, Tony
2006-06-27 23:09 ` Lee Revell
2006-06-28  8:43 ` Arjan van de Ven
2006-06-28 10:47   ` Alan Cox
2006-06-28 10:34     ` Arjan van de Ven
2006-06-28 14:46       ` Christoph Lameter
2006-06-28 17:36 Luck, Tony
2006-06-29  9:37 ` Jes Sorensen
2006-06-29 11:02   ` Alan Cox
2006-06-29 10:48     ` Jes Sorensen
2006-06-29 10:55       ` Arjan van de Ven
2006-06-29 12:56         ` Jes Sorensen
2006-07-08  0:14         ` Jeremy Higdon
2006-07-08  2:51           ` Tony Luck
2006-07-08  6:42           ` Arjan van de Ven
2006-07-08 13:07             ` David Mosberger-Tang
2006-07-10 20:22               ` Jeremy Higdon
2006-07-11  3:01                 ` David Mosberger-Tang
2006-07-11 10:10                   ` Alan Cox
2006-07-11 18:37                   ` Jeremy Higdon
2006-07-12  2:02                     ` David Mosberger-Tang
2006-06-29 11:34       ` Alan Cox
2006-06-29 14:09 Albert Cahalan
2006-07-09 19:18 Luck, Tony

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=44A2AA2B.3090101@osdl.org \
    --to=jdaiker@osdl.org \
    --cc=akpm@osdl.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=arjan@infradead.org \
    --cc=djh@sgi.com \
    --cc=hawkes@sgi.com \
    --cc=jeremy@sgi.com \
    --cc=linux-ia64@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=steiner@sgi.com \
    --cc=tony.luck@gmail.com \
    /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