From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Zijlstra Subject: Re: [RFC PATCH 0/9] Remove useless on_each_cpu return value Date: Fri, 06 Jan 2012 14:03:01 +0100 Message-ID: <1325854981.2442.25.camel@twins> References: <1325600353-10895-1-git-send-email-gilad@benyossef.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1325600353-10895-1-git-send-email-gilad@benyossef.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Gilad Ben-Yossef Cc: linux-ia64@vger.kernel.org, David Airlie , Benjamin Herrenschmidt , Will Deacon , dri-devel@lists.freedesktop.org, Grant Likely , Paul Mackerras , "H. Peter Anvin" , Russell King , x86@kernel.org, Ingo Molnar , Arnaldo Carvalho de Melo , Matt Turner , Fenghua Yu , devicetree-discuss@lists.ozlabs.org, Michal Nazarewicz , Ivan Kokshaysky , Rob Herring , Thomas Gleixner , linux-arm-kernel@lists.infradead.org, Richard Henderson , Tony Luck , linux-kernel@vger.kernel.org, linux-alpha@vger.kernel.org, linuxppc-dev@lists.o List-Id: devicetree@vger.kernel.org On Tue, 2012-01-03 at 16:19 +0200, Gilad Ben-Yossef wrote: > on_each_cpu() returns as its own return value the return value of > smp_call_function(). smp_call_function() in turn returns a hard > coded value of zero. > > Some callers to on_each_cpu() waste cycles and bloat code space > by checking the return value to on_each_cpu(), probably for > historical reasons. hysterical raisins,, yumm! :-) > This patch set refactors callers to not test on_each_cpu() > (fixed) return value and then refactors on_each_cpu to > return void to avoid confusing future users. > Seems sane enough, fwiw: Acked-by: Peter Zijlstra