linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Matt Sealey <matt@genesi-usa.com>
Cc: Daniel Walker <dwalker@mvista.com>,
	linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org,
	Paul Mackerras <paulus@samba.org>,
	tglx@linutronix.de, mingo@elte.hu
Subject: Re: [PATCH 2.6.21-rt2] PowerPC: decrementer clockevent driver
Date: Mon, 21 May 2007 07:25:38 +1000	[thread overview]
Message-ID: <1179696339.32247.568.camel@localhost.localdomain> (raw)
In-Reply-To: <4650632B.1010702@genesi-usa.com>

> So.. if we get enough clocksources into the tree, can any of those
> parts of the code be reworked to use clocksources/clockevents and
> hrtimers quickly and easily? I noticed the patch just posted does
> some of it.. but not as much as Ben just mentioned.

Well, some of these are expected to be small & fast and work in all sort
of crazy circumstances, like udelay etc... I'd rather keep that on top
of the TB. Do we have actual examples where the TB freq is changing ?
Beside, on powerpc, we don't have another clock source that is as fast
to access and we have userland using the TB for gettimeofday via the
vdso, so I'd say bad idea ... Just keep the damn thing fixed frequency.

> Or is it a development nightmare?
> 
> I'm fairly sure on a PPC970 box even though the decrementer is
> monotonic and never changes frequency, one day it just might, and
> it would be better to anticipate this (and allow people to
> distribute their timing requirements across an entire system
> and not just the CPU core anyway, which I think is probably a
> good thing from a system integration and possibly the point of
> view of redundancy..)

On a -sane- 970 box (which seems to be the case of all of them that
matter so far), the TB is sourced externally specifically for that
reason : to avoid it changing, The DEC is always derived the TB, so it's
not changing.

I don't have any plan to support somebody coming up with a HW design
broken enough to have a variable TB/DEC speed. If they do it, they
support it and they come up with patches that are acceptable (hint: that
will be hard !). Beside, it means the vDSO will not be useable for
gettimeofday on such a platform, which means it will have to fallback to
the syscall which is much slower.

Ben.

  reply	other threads:[~2007-05-20 21:26 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-17 17:42 [PATCH 2.6.21-rt2] PowerPC: decrementer clockevent driver Sergei Shtylyov
2007-05-17 17:47 ` Kumar Gala
2007-05-17 18:07   ` Sergei Shtylyov
2007-05-17 18:17     ` Kumar Gala
2007-05-17 18:26       ` Sergei Shtylyov
2007-05-17 18:31         ` Kumar Gala
2007-05-17 18:42           ` Sergei Shtylyov
2007-05-18 12:39             ` Matt Sealey
2007-05-18 13:45               ` Sergei Shtylyov
2007-05-18 14:28                 ` Matt Sealey
2007-05-18 14:48                   ` Thomas Gleixner
2007-05-18 15:37                     ` Matt Sealey
2007-05-18 16:31                     ` Kumar Gala
2007-05-18 16:50                       ` Thomas Gleixner
2007-05-18 16:55                         ` Matt Sealey
2007-05-18 16:52                       ` Matt Sealey
2007-07-10 10:52                         ` Gabriel Paubert
2007-07-10 14:24                           ` Segher Boessenkool
2007-07-10 23:31                             ` Paul Mackerras
2007-07-11  0:10                               ` Segher Boessenkool
2007-05-18 20:27                       ` David Miller
2007-05-18 14:56       ` Daniel Walker
2007-05-18 15:06         ` Sergei Shtylyov
2007-05-18 15:39           ` Daniel Walker
2007-05-18 15:48             ` Sergei Shtylyov
2007-05-19  3:33             ` Paul Mackerras
2007-05-20  2:43               ` Daniel Walker
2007-05-20  3:03                 ` Benjamin Herrenschmidt
2007-05-20 15:03                   ` Matt Sealey
2007-05-20 21:25                     ` Benjamin Herrenschmidt [this message]
2007-05-20 16:02                   ` Segher Boessenkool
2007-05-20 21:26                     ` Benjamin Herrenschmidt
2007-05-17 19:27 ` Segher Boessenkool
2007-05-17 19:43   ` Sergei Shtylyov
2007-05-17 19:50     ` Segher Boessenkool
2007-05-17 20:00       ` Sergei Shtylyov
2007-05-17 20:53         ` Segher Boessenkool
2007-05-18  5:04           ` Benjamin Herrenschmidt
2007-05-18  5:39             ` Dave Liu
2007-05-18  7:08               ` Benjamin Herrenschmidt
2007-05-18 13:41                 ` Sergei Shtylyov
2007-05-18 13:48                   ` Sergei Shtylyov
2007-05-18 23:27                   ` Benjamin Herrenschmidt
2007-05-18 23:49                     ` Segher Boessenkool
2007-05-19  0:13                       ` Benjamin Herrenschmidt
2007-05-30 15:56                     ` Milton Miller
2007-05-19  3:34                   ` Paul Mackerras
2007-05-24 18:25 ` [PATCH 2.6.21-rt7] PowerPC: fix clockevents for classic CPUs Sergei Shtylyov
2007-05-24 19:10   ` Segher Boessenkool
2007-10-18 14:45 ` [PATCH 2.6.21-rt2] PowerPC: decrementer clockevent driver Sergei Shtylyov
  -- strict thread matches above, loose matches on Subject: below --
2007-05-18  3:24 Albert Cahalan
2007-05-18 14:52 ` Sergei Shtylyov
2007-05-19  1:45   ` Albert Cahalan
2007-05-19 12:28     ` Segher Boessenkool
2007-05-19 18:22       ` Albert Cahalan

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=1179696339.32247.568.camel@localhost.localdomain \
    --to=benh@kernel.crashing.org \
    --cc=dwalker@mvista.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=matt@genesi-usa.com \
    --cc=mingo@elte.hu \
    --cc=paulus@samba.org \
    --cc=tglx@linutronix.de \
    /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).