All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: David Miller <davem@davemloft.net>
Cc: a.p.zijlstra@chello.nl, efault@gmx.de, elendil@planet.nl,
	parag.warudkar@gmail.com, linux-kernel@vger.kernel.org,
	guichaz@yahoo.fr, andi@firstfloor.org
Subject: Re: 'global' rq->clock
Date: Sat, 3 May 2008 12:10:16 +0200	[thread overview]
Message-ID: <20080503101016.GA18801@elte.hu> (raw)
In-Reply-To: <20080503.020502.226847930.davem@davemloft.net>


* David Miller <davem@davemloft.net> wrote:

> > there's already such a mechanism in sched-devel.git (and has been 
> > there for a week or two): an architecture can set time_sync_thresh 
> > to -1LL during early bootup and essentially disable all the 
> > synchronization logic.
> 
> Does it remove all of the code too? :-)
> 
> Please give us a config boolean.  The only platform for which a 
> run-time knob is even necessary is x86.

yeah, will try something like that too.

the thing is, core kernel folks have to resist such pressures 
_somewhat_.

Architecture maintainers will put pressure on us from two directions: if 
a particular generic feature only helps _another_ architecture, they 
find it a nuisance and want it to be gone as much as possible.

If a particular feature helps them, they want it supported and 
default-enabled as much as possible. In fact, complaints come if a 
generic-looking feature shows up in one architecture only. (recent 
example: inlining optimizations ;-)

And you are totally right about sched_clock() being dead on accurate an 
globally synchronous on sparc64 - and you are right to find _any_ issue 
about it a nuisance. I totally envy you that sparc64's sched_clock() is 
so simple - it should have been like that on x86 years ago, if hw 
designers werent so impotent about it.

( although please note that the growing generalization that goes on
  _did_ find a subtle nohz problem on sparc64 early in the merge window,
  so it's not like these changes are totally useless to you. )

but it all goes in the other direction as well: many folks find 
endianness problems a nuisance on x86, many folks find TLB and explicit 
cache coherence complications a nuisance on x86 as well. The bus-to-phys 
complication which is an identity on x86. Etc. etc.

But the core kernel is a conscious and intelligent union of all 
architecture's needs, and often we maintain complications even if they 
make no sense on the most popular platform. I think it makes strategic 
sense because it keep the kernel truly generic and truly clean. It also 
keeps architectures honest: even today the x86 architecture is still not 
as clean as sparc64 for example.

so please be patient, we are working on it :)

	Ingo

  reply	other threads:[~2008-05-03 10:10 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-02  0:14 Horrendous Audio Stutter - current git Parag Warudkar
2008-05-02  8:34 ` Peter Zijlstra
2008-05-02 10:32   ` Frans Pop
2008-05-02 10:35     ` Peter Zijlstra
2008-05-02 11:08       ` Peter Zijlstra
2008-05-02 11:37         ` Frans Pop
2008-05-02 11:39           ` Peter Zijlstra
2008-05-02 11:45             ` Frans Pop
2008-05-02 11:51               ` Peter Zijlstra
2008-05-02 12:06             ` Frans Pop
2008-05-02 12:22               ` Parag Warudkar
2008-05-02 13:21                 ` Dhaval Giani
2008-05-02 11:10   ` Parag Warudkar
2008-05-02 12:09     ` Mike Galbraith
2008-05-02 12:21       ` Parag Warudkar
2008-05-02 12:37         ` Mike Galbraith
2008-05-02 15:02           ` Mike Galbraith
2008-05-02 15:49             ` Frans Pop
2008-05-02 18:53               ` Frans Pop
2008-05-02 19:27                 ` Mike Galbraith
2008-05-02 19:56                   ` 'global' rq->clock (was Re: Horrendous Audio Stutter - current git) Peter Zijlstra
2008-05-02 20:38                     ` Guillaume Chazarain
2008-05-02 20:46                       ` Peter Zijlstra
2008-05-02 22:00                         ` 'global' rq->clock David Miller
2008-05-02 21:48                     ` David Miller
2008-05-02 10:09                       ` Arjan van de Ven
2008-05-04 12:12                         ` Peter Zijlstra
2008-05-02 22:07                       ` Peter Zijlstra
2008-05-03  8:28                         ` Ingo Molnar
2008-05-03  9:05                           ` David Miller
2008-05-03 10:10                             ` Ingo Molnar [this message]
2008-05-03 19:27                               ` David Miller
2008-05-03 19:37                                 ` Ingo Molnar
2008-05-03 22:30                                 ` Benjamin Herrenschmidt
2008-05-03 22:38                                   ` Ingo Molnar
2008-05-03 23:04                                     ` David Miller
2008-05-03 23:36                                       ` David Miller
2008-05-03 23:38                                         ` Ingo Molnar
2008-05-03 23:40                                           ` David Miller
2008-05-03 23:47                                             ` Ingo Molnar
2008-05-04  2:22                                     ` Benjamin Herrenschmidt
2008-05-03 19:28                               ` David Miller
2008-05-03 19:39                                 ` Ingo Molnar
2008-05-03  6:20                     ` 'global' rq->clock (was Re: Horrendous Audio Stutter - current git) Mike Galbraith
2008-05-02 19:38               ` Horrendous Audio Stutter - current git Mike Galbraith
2008-05-03  7:13           ` Frans Pop
2008-05-03  7:39             ` Mike Galbraith
2008-05-07  8:26   ` Frans Pop
2008-05-07  8:32     ` Ingo Molnar

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=20080503101016.GA18801@elte.hu \
    --to=mingo@elte.hu \
    --cc=a.p.zijlstra@chello.nl \
    --cc=andi@firstfloor.org \
    --cc=davem@davemloft.net \
    --cc=efault@gmx.de \
    --cc=elendil@planet.nl \
    --cc=guichaz@yahoo.fr \
    --cc=linux-kernel@vger.kernel.org \
    --cc=parag.warudkar@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 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.