linux-kernel - Re: + clocksource-add-generic-sched

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: <1160343221.3693.154.camel@c-67-180-230-165.hsd1.ca.comcast.net>
Date:	Sun, 08 Oct 2006 14:33:41 -0700
From:	Daniel Walker <dwalker@...sta.com>
To:	tglx@...utronix.de
Cc:	akpm@...l.org, johnstul@...ibm.com, mingo@...e.hu,
	zippel@...ux-m68k.org, LKML <linux-kernel@...r.kernel.org>
Subject: Re: + clocksource-add-generic-sched_clock.patch added to -mm tree

On Sun, 2006-10-08 at 23:21 +0200, Thomas Gleixner wrote:
> On Sun, 2006-10-08 at 14:15 -0700, Daniel Walker wrote:
> > On Sun, 2006-10-08 at 20:45 +0200, Thomas Gleixner wrote:
> > 
> > > > I'm not moving the kernel/timer.c clocksource user back into
> > > > kernel/time/clocksource.c . That code completely belongs with the
> > > > generic time of day changes. The code is directly coupled, and in fact
> > > > it improves the timekeeping clock switching code to have it that way.
> > > 
> > > I don't see any reason, why it must be added to timer.c. You can achieve
> > > the same result with calling the code outside, except that the compiler
> > > might miss some inline optimization. The switch clock code is not a
> > > hotpath and so it does not matter whether it is called here or there.
> > 
> > It wouldn't be as clean to integrate the two. The hotpath is improved
> > (which is what I was referring too above.)
> 
> Sorry, where is which hotpath improved ?

The hotpath in update_wall_time() kernel/timer.c which involves clock
switching.

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/