Thread (27 messages) 27 messages, 7 authors, 2012-09-08
STALE5017d

[PATCH] ARM: formalize an IPI for CPU wake-ups

From: nico@fluxnic.net (Nicolas Pitre)
Date: 2012-06-11 19:11:05

On Mon, 11 Jun 2012, Russell King - ARM Linux wrote:
On Mon, Jun 11, 2012 at 02:40:24PM -0400, Nicolas Pitre wrote:
quoted
On Mon, 11 Jun 2012, Russell King - ARM Linux wrote:
quoted
On Mon, Jun 11, 2012 at 01:18:28PM -0400, Nicolas Pitre wrote:
quoted
And that message certainly has to go as it may be triggered multiple 
times per second in some use cases here.
That, I interpret, as an attempt to use CPU hotplugging for power
saving.

No.  Do not do that.  Under any circumstances.  Not only is the CPU
hotplugging code racy in places, it also totally destroys the ability
to have threads bound to CPUs, or even interrupts to CPUs.
[ more comments ommitted ]

Russell, let me suggest you take the time to read this:

	http://lwn.net/Articles/481055/

then let's restart this conversation if you please.
Same comments apply for exactly the reasons I stated.
No they don't.  They can't. The generic kernel code doesn't know what is 
happening behind the scene and may not even care if at the low level a 
complete CPU replacement happened.  Trust me, I happen to know one thing 
or two about those issues you mentioned and the code I'm working on will 
be posted in due course.

But that is not the point of this thread.  The point I want to bring 
forward is:

- IPIs are needed to wake up remote CPUs

- we should formalize a number for it and the actual number is the least 
  of my concern

- the warning must go

- and we shouldn't wait until the entire world comes forth to confirm 
  any arbitrary SGI number works for them or nothing will ever happen.

Can we make some progress with this please?


Nicolas
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help