From: Vojtech Pavlik <vojtech@suse.cz>
To: Christoph Hellwig <hch@infradead.org>,
Vojtech Pavlik <vojtech@suse.cz>,
linux-kernel@vger.kernel.org
Subject: Re: asm/timex.h
Date: Fri, 24 May 2002 20:23:33 +0200 [thread overview]
Message-ID: <20020524202333.B21716@ucw.cz> (raw)
In-Reply-To: <20020524193345.A21559@ucw.cz> <20020524184903.B24780@infradead.org>
On Fri, May 24, 2002 at 06:49:03PM +0100, Christoph Hellwig wrote:
> On Fri, May 24, 2002 at 07:33:45PM +0200, Vojtech Pavlik wrote:
> > Hi!
> >
> > I have several questions about asm/timex.h
> >
> > 1) Who uses it? The kernel certainly doesn't. Perhaps NTP?
>
> The kernel does. Thanks to the sched.h mess it's even implictly
> included in almost any file..
>
> > 3) What if an architecture doesn't have a compile-time known
> > CLOCK_TICK_RATE? I suppose I cannot just #define it to a variable,
> > because the kernel doesn't use it, and that probably means userland
> > does ...
>
> The kernel DOES use it. grep(1) is your friend.
# grep -rw FINETUNE * | grep -v timex.h
# grep -rw CLOCK_TICK_FACTOR * | grep -v timex.h
# grep -rw CLOCK_TICK_RATE * | grep -v timex.h
drivers/char/ftape/lowlevel/ftape-calibr.c: return (10000 * count) / ((CLOCK_TICK_RATE + 50) / 100);
The only interesting place where is CLOCK_TICK_RATE used is in linux/timex.h
to define LATCH. Which subsequently is only used in arch specific code
and ftape again.
Ftape's use of it is buggy, because it lacks the needed locking.
So, it seems like CLOCK_TICK_RATE isn't needed by non-arch-specific
code.
And FINETUNE isn't used at all.
But ... the fact that kernel uses it ... probably doesn't mean that
userland doesn't.
--
Vojtech Pavlik
SuSE Labs
prev parent reply other threads:[~2002-05-24 18:23 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-05-24 17:33 asm/timex.h Vojtech Pavlik
2002-05-24 17:49 ` asm/timex.h Christoph Hellwig
2002-05-24 18:23 ` Vojtech Pavlik [this message]
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=20020524202333.B21716@ucw.cz \
--to=vojtech@suse.cz \
--cc=hch@infradead.org \
--cc=linux-kernel@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
all inboxes | Powered by JetHome®