From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id A49ECB6EEE for ; Tue, 29 Mar 2011 19:13:20 +1100 (EST) Subject: Re: [PATCH 15/18] powerpc/smp: Don't expose per-cpu "cpu_state" array From: Benjamin Herrenschmidt To: michael@ellerman.id.au In-Reply-To: <1301377040.32738.14.camel@concordia> References: <1299566250-10516-1-git-send-email-benh@kernel.crashing.org> <1299566250-10516-16-git-send-email-benh@kernel.crashing.org> <1301377040.32738.14.camel@concordia> Content-Type: text/plain; charset="UTF-8" Date: Tue, 29 Mar 2011 19:13:08 +1100 Message-ID: <1301386388.2402.672.camel@pasglop> Mime-Version: 1.0 Cc: linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 2011-03-29 at 16:37 +1100, Michael Ellerman wrote: > Takes a cpu parameter .. > > > +void generic_set_cpu_dead(unsigned int cpu) > > +{ > > + __get_cpu_var(cpu_state) = CPU_DEAD; > > +} > > But doesn't use it. Yeah that's a bug. Fortunately it's always called on the current CPU :-) I'll fix it tho. Thanks, Ben.