Andrew Morton wrote:
Nick Piggin <[email protected]> wrote:
Is there a need? Do they (except calc_load) use multiple values at
the same time?
Don't know. It might happen in the future. And the additional cost is
practically zero.
Unless it happens to hit another cacheline (cachelines for all other
CPUs but our own will most likely be invalid on this cpu). In which
case the cost could double quite easily.
And then give get_sched_stuff() a hotplug handler (probably unneeded) and
What would the hotplug handler do?
Move the stats from the going-away CPU into the current CPU's runqueue.
Oh, that. Yeah that is handled already for nr_uninterruptible (although,
ironically, it isn't needed because of the for_each_cpu loop there!)
I think it need only iterate over possible CPUs.
Someone who has four online CPUs, sixteen present CPUs and 128 possible
CPUs would be justifiably disappointed, no?
Yes. Ingo? This can be changed, right?
IOW: this code's an inefficient mess and needs some caring for.
What are the performance critical places that call the nr_blah() functions?
That depends upon the frequency with which userspace reads /proc/loadavg,
/proc/stat or /proc/future-stuff.
I think it might be better to leave it for the moment. If something comes
up we can always take a look at it then (it isn't particularly tricky code).
--
SUSE Labs, Novell Inc.
Send instant messages to your online friends http://au.messenger.yahoo.com
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
