From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH v8 4/8] smp: add func to IPI cpus based on parameter func Date: Wed, 8 Feb 2012 16:03:44 -0800 Message-ID: <20120208160344.88d187e5.akpm@linux-foundation.org> References: <1328448800-15794-1-git-send-email-gilad@benyossef.com> <1328449722-15959-3-git-send-email-gilad@benyossef.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: linux-kernel@vger.kernel.org, "Gilad Ben-Yossef" , "Chris Metcalf" , "Christoph Lameter" , "Frederic Weisbecker" , "Russell King" , linux-mm@kvack.org, "Pekka Enberg" , "Matt Mackall" , "Sasha Levin" , "Rik van Riel" , "Andi Kleen" , "Alexander Viro" , linux-fsdevel@vger.kernel.org, "Avi Kivity" , "Kosaki Motohiro" , "Milton Miller" To: "Michal Nazarewicz" Return-path: In-Reply-To: Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On Wed, 08 Feb 2012 10:30:51 +0100 "Michal Nazarewicz" wrote: > > } while (0) > > +/* > > + * Preemption is disabled here to make sure the > > + * cond_func is called under the same condtions in UP > > + * and SMP. > > + */ > > +#define on_each_cpu_cond(cond_func, func, info, wait, gfp_flags) \ > > + do { \ > > How about: > > void *__info = (info); > > as to avoid double execution. Yup. How does this look? From: Andrew Morton Subject: smp-add-func-to-ipi-cpus-based-on-parameter-func-update-fix - avoid double-evaluation of `info' (per Michal) - parenthesise evaluation of `cond_func' Cc: "Srivatsa S. Bhat" Cc: Gilad Ben-Yossef Cc: Michal Nazarewicz Signed-off-by: Andrew Morton --- include/linux/smp.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) --- a/include/linux/smp.h~smp-add-func-to-ipi-cpus-based-on-parameter-func-update-fix +++ a/include/linux/smp.h @@ -168,10 +168,11 @@ static inline int up_smp_call_function(s */ #define on_each_cpu_cond(cond_func, func, info, wait, gfp_flags)\ do { \ + void *__info = (info); \ preempt_disable(); \ - if (cond_func(0, info)) { \ + if ((cond_func)(0, __info)) { \ local_irq_disable(); \ - (func)(info); \ + (func)(__info); \ local_irq_enable(); \ } \ preempt_enable(); \ _ -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org