[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130928084412.GC26167@gmail.com>
Date: Sat, 28 Sep 2013 10:44:12 +0200
From: Ingo Molnar <mingo@...nel.org>
To: Christoph Lameter <cl@...ux.com>
Cc: Tejun Heo <tj@...nel.org>, akpm@...uxfoundation.org,
linux-arch@...r.kernel.org, Steven Rostedt <srostedt@...hat.com>,
linux-kernel@...r.kernel.org,
Peter Zijlstra <peterz@...radead.org>,
Thomas Gleixner <tglx@...utronix.de>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>
Subject: Re: [pchecks v2 2/2] percpu: Add preemption checks to __this_cpu ops
* Christoph Lameter <cl@...ux.com> wrote:
> > The blind __this_cpu conversions without proper preempt debugging
> > cannot continue without first fixing all the fallout of the missing
> > debug checks to begin with.
>
> That will take some time as the feedback from the other patchset
> suggests.
That's the reason why we insisted on __this_cpu*() primitives growing
these essential debug checks early on - which you resisted. I had to bring
out NAKs for you to see sense and start fixing the mess already - next
time around I'll probably have to NAK your changes earlier to prevent such
mishaps.
( Note that some false positives were possibly fixed by the use of the
lib/smp_processor_id.c methods to check for preemptability, so the
situation might not be as dire as your first series suggests. )
Thanks,
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists