Re: [Nouveau] [BUG/PATCH] x86 mmiotrace: dynamically disable non-boot CPUs

2008-07-24 Thread Stephane Marchesin
On Wed, Apr 16, 2008 at 6:59 PM, Pekka Paalanen [EMAIL PROTECTED] wrote: On Wed, 16 Apr 2008 13:46:09 +0200 Ingo Molnar [EMAIL PROTECTED] wrote: * Pekka Paalanen [EMAIL PROTECTED] wrote: we should fix this restriction ASAP. Forcibly dropping to UP will cause mmiotrace to be much less

Re: [Nouveau] [BUG/PATCH] x86 mmiotrace: dynamically disable non-boot CPUs

2008-04-17 Thread Steven Rostedt
On Wed, 16 Apr 2008, Ingo Molnar wrote: so lets fix those preemptability bugs. They show that the cpu-up/cpu-down ops are called from atomic context - it should normally be straightforward to sort out - there's no particular reason why the -open()/-close() methods of an ftrace plugin should

Re: [Nouveau] [BUG/PATCH] x86 mmiotrace: dynamically disable non-boot CPUs

2008-04-16 Thread Pekka Paalanen
On Wed, 16 Apr 2008 13:46:09 +0200 Ingo Molnar [EMAIL PROTECTED] wrote: * Pekka Paalanen [EMAIL PROTECTED] wrote: we should fix this restriction ASAP. Forcibly dropping to UP will cause mmiotrace to be much less useful for diagnostic purposes of Linux Ok, how do you propose

Re: [Nouveau] [BUG/PATCH] x86 mmiotrace: dynamically disable non-boot CPUs

2008-04-16 Thread Ingo Molnar
* Pekka Paalanen [EMAIL PROTECTED] wrote: yeah - it looks complex. Not a showstopper for now :-) but given that Xorg is usually just a single task, do we _really_ need this? We're not tracing Xorg at all. Mmiotrace still cannot catch accesses originating in user space. It is

Re: [Nouveau] [BUG/PATCH] x86 mmiotrace: dynamically disable non-boot CPUs

2008-04-16 Thread Pekka Paalanen
On Wed, 16 Apr 2008 20:32:58 +0200 Ingo Molnar [EMAIL PROTECTED] wrote: * Pekka Paalanen [EMAIL PROTECTED] wrote: yeah - it looks complex. Not a showstopper for now :-) but given that Xorg is usually just a single task, do we _really_ need this? We're not tracing Xorg at

Re: [Nouveau] [BUG/PATCH] x86 mmiotrace: dynamically disable non-boot CPUs

2008-04-16 Thread Ingo Molnar
* Pekka Paalanen [EMAIL PROTECTED] wrote: So i think we need to fix your automatic-cpudown/cpuup patch. I tried that and it worked very intuitively and the cpus were disabled/enabled without any trouble - with ftrace based mmiotrace we now basically have something that most distros