linux-kernel - Re: [PATCH 22/23] clocksource: new clock lookup method

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <1170266874.9781.150.camel@imap.mvista.com>
Date:	Wed, 31 Jan 2007 10:07:54 -0800
From:	Daniel Walker <dwalker@...sta.com>
To:	tglx@...utronix.de
Cc:	Ingo Molnar <mingo@...e.hu>, akpm@...l.org,
	linux-kernel@...r.kernel.org, johnstul@...ibm.com
Subject: Re: [PATCH 22/23] clocksource: new clock lookup method

On Wed, 2007-01-31 at 18:55 +0100, Thomas Gleixner wrote:
> On Wed, 2007-01-31 at 09:39 -0800, Daniel Walker wrote:
> > > please read my reply above! To repeat: such flags tend to get forgotten, 
> > > resulting in a less safe default behavior. Clock hardware and thus 
> > > clocksources are fundamentally fragile so we want to default to the 
> > > safest behavior. I.e. if the IS_CONTINOUS flag is 'forgotten', the clock 
> > > wont be usable as a clock verification base for example. The flag has to 
> > > be affirmatively set to mark the clocksource continous.
> > 
> > I don't see this as an issue .. Your assuming that not continuous clocks
> > will be prevalent which they aren't ..
> 
> And how does this change Ingo's statement ? Such beasts exist and having
> them default to the safe side is good. 
> 
> Also I really do not see the "huge burden" for a clocksource coder to
> add this flag. If he forgets it, then he can not use highres / dynticks
> and nothing breaks.

We're not talking about a rare system runtime configuration . If that
were the case I would agree safe is better..

I'm assuming that programmers will test their code, and others will
review the code .. Catering to any other situation doesn't make sense to
me. On top of that those clocks are rare, and not desirable ..

Daniel

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/