Re: [patch 1/2] add ALL_CPUS option to stop_machine_run()

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Ingo Molnar <mingo@...>
Cc: Jason Baron <jbaron@...>, Mathieu Desnoyers <mathieu.desnoyers@...>, <akpm@...>, <linux-kernel@...>, Rusty Russell <rusty@...>, Peter Zijlstra <a.p.zijlstra@...>
Date: Friday, February 29, 2008 - 2:24 pm

Ingo Molnar wrote:
Sure I started the discussion but I suppose you missed Andi's and other 
replies. All I said that people should think twice before relying on it.
btw I'm ok if I _am_ the _one_ who has to convert those pieces of code, that's 
part of the fun :). But if people keep adding stuff which uses stom_machine 
that may be pretty difficult :).

btw Being an RT guy you do not think that stop machine is evil ? I mean from 
the overhead and especially latency perspective. By overhead I mean if you 
have 100+ cpu box that Paul and other guys have mentioned here. Every single 
CPU has to be frozen. You said it's reasonably fast. I guess it depends what's 
reasonable. And from the latency perspective all bets are off. We have no 
guaranties whatsoever as to hold long it will take for cpu X to get frozen 
(there numerous factors here) and all the other cpus have to wait for it.
As I said for some things there is just no other way but to use the 
stop_machine but we should try to minimize that as much as possible.

Max

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[patch 1/7] Immediate Values - Architecture Independent Code, Mathieu Desnoyers, (Sat Feb 2, 5:08 pm)
Re: [patch 1/7] Immediate Values - Architecture Independent ..., Mathieu Desnoyers, (Wed Feb 27, 3:05 pm)
Re: [patch 2/2] implement immediate updating via stop_machin..., Mathieu Desnoyers, (Fri Feb 29, 9:43 am)
[patch 1/2] add ALL_CPUS option to stop_machine_run(), Jason Baron, (Thu Feb 28, 12:33 pm)
Re: [patch 1/2] add ALL_CPUS option to stop_machine_run(), Max Krasnyanskiy, (Thu Feb 28, 6:09 pm)
Re: [patch 1/2] add ALL_CPUS option to stop_machine_run(), Max Krasnyanskiy, (Fri Feb 29, 2:24 pm)
Re: [patch 1/2] add ALL_CPUS option to stop_machine_run(), Rusty Russell, (Mon Mar 3, 12:12 am)
Re: [patch 1/2] add ALL_CPUS option to stop_machine_run(), Max Krasnyanskiy, (Mon Mar 3, 8:30 pm)
Re: [patch 1/2] add ALL_CPUS option to stop_machine_run(), Rusty Russell, (Mon Mar 3, 10:36 pm)
Re: [patch 1/2] add ALL_CPUS option to stop_machine_run(), Max Krasnyansky, (Tue Mar 4, 12:11 am)
Re: [patch 1/2] add ALL_CPUS option to stop_machine_run(), Max Krasnyanskiy, (Fri Feb 29, 3:58 pm)
Re: [patch 1/2] add ALL_CPUS option to stop_machine_run(), Mathieu Desnoyers, (Thu Feb 28, 6:14 pm)
Re: [patch 1/7] Immediate Values - Architecture Independent ..., Mathieu Desnoyers, (Tue Feb 26, 7:12 pm)
Re: [patch 1/7] Immediate Values - Architecture Independent ..., Mathieu Desnoyers, (Tue Feb 26, 7:34 pm)