Re: [RFC PATCH 0/9] Remove useless on_each_cpu return value
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- To: Gilad Ben-Yossef <gilad@xxxxxxxxxxxxx>
- Subject: Re: [RFC PATCH 0/9] Remove useless on_each_cpu return value
- From: Peter Zijlstra <a.p.zijlstra@xxxxxxxxx>
- Date: Fri, 06 Jan 2012 14:03:01 +0100
- Cc: linux-kernel@xxxxxxxxxxxxxxx, Michal Nazarewicz <mina86@xxxxxxxxxx>, David Airlie <airlied@xxxxxxxx>, dri-devel@xxxxxxxxxxxxxxxxxxxxx, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>, Paul Mackerras <paulus@xxxxxxxxx>, Grant Likely <grant.likely@xxxxxxxxxxxx>, Rob Herring <rob.herring@xxxxxxxxxxx>, linuxppc-dev@xxxxxxxxxxxxxxxx, devicetree-discuss@xxxxxxxxxxxxxxxx, Richard Henderson <rth@xxxxxxxxxxx>, Ivan Kokshaysky <ink@xxxxxxxxxxxxxxxxxxxx>, Matt Turner <mattst88@xxxxxxxxx>, linux-alpha@xxxxxxxxxxxxxxx, Thomas Gleixner <tglx@xxxxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, "H. Peter Anvin" <hpa@xxxxxxxxx>, x86@xxxxxxxxxx, Tony Luck <tony.luck@xxxxxxxxx>, Fenghua Yu <fenghua.yu@xxxxxxxxx>, linux-ia64@xxxxxxxxxxxxxxx, Will Deacon <will.deacon@xxxxxxx>, Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxxxxxxx>, Russell King <linux@xxxxxxxxxxxxxxxx>, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
- In-reply-to: <1325600353-10895-1-git-send-email-gilad@benyossef.com>
- List-id: <linux-ia64.vger.kernel.org>
- References: <1325600353-10895-1-git-send-email-gilad@benyossef.com>
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 <a.p.zijlstra@xxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Index of Archives]
[Linux Kernel]
[Sparc Linux]
[DCCP]
[Linux ARM]
[Yosemite News]
[Linux SCSI]
[Linux x86_64]
[Linux for Ham Radio]