login
Login
/
Register
Search
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2007
»
April
»
2
Re: [patch 17/17] Add a sched_clock paravirt_op
view
thread
!MAILaRCHIVE_VOTE_RePLACE
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
[view in full thread]
From:
Andi Kleen <ak@...>
To: Jeremy Fitzhardinge <jeremy@...>
Cc: Andrew Morton <akpm@...>, <virtualization@...>, lkml <linux-kernel@...>, Zachary Amsden <zach@...>, Dan Hecht <dhecht@...>, john stultz <johnstul@...>
Subject:
Re: [patch 17/17] Add a sched_clock paravirt_op
Date: Monday, April 2, 2007 - 2:50 am
On Monday 02 April 2007 08:47, Jeremy Fitzhardinge wrote:
quoted text
> Andi Kleen wrote: > > I think it would be much cleaner if you didn't implement your own sched_clock, > > but you adjust ns_base/last_tsc to account for your lost cycles. > > This could be done cleanly by adding a new function to sched-clock.c > > Possibly such a function could be used by other parts of the kernel > > in the future too. > > > > Cleaner how? This seems pretty to me. Xen can return a > clock measuring unstolen nanoseconds,
Do you also get a clock for stolen nanoseconds?
quoted text
> which maps directly to the > sched_clock interface, doesn't need any of the existing sched_clock > code. I suppose I could map the Xen interface onto some abstract > "cycles" notion and hook it into the tsc machinery, but it seems like it > would be a forced fit. In general, my approach has been to choose the > higher-level interface over a lower-level one, all other things being equal.
No need for cycles, you could just subtract the stolen ns if you can get those. -Andi -
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
Messages in current thread:
[patch 17/17] Add a sched_clock paravirt_op
, Jeremy Fitzhardinge
, (Mon Apr 2, 1:57 am)
Re: [patch 17/17] Add a sched_clock paravirt_op
, Andi Kleen
, (Mon Apr 2, 2:09 am)
Re: [patch 17/17] Add a sched_clock paravirt_op
, Jeremy Fitzhardinge
, (Mon Apr 2, 2:47 am)
Re: [patch 17/17] Add a sched_clock paravirt_op
, Andi Kleen
, (Mon Apr 2, 2:50 am)
Re: [patch 17/17] Add a sched_clock paravirt_op
, Jeremy Fitzhardinge
, (Mon Apr 2, 3:06 am)
Navigation
Create content
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Greg Kroah-Hartman
[PATCH 004/196] Chinese: add translation of SubmittingPatches
Jeff Garzik
Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3
Paul E. McKenney
[PATCH RFC 3/9] RCU: Preemptible RCU
James Bottomley
Re: Integration of SCST in the mainstream Linux kernel
git
:
linux-netdev
:
Gerrit Renker
[PATCH 13/37] dccp: Deprecate Ack Ratio sysctl
Patrick McHardy
Re: [GIT]: Networking
Jarek Poplawski
Re: [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock().
openbsd-misc
:
Colocation donated by:
Who's online
There are currently
1 user
and
828 guests
online.
Online users
strcmp
Syndicate