Re: [RFC][PATCH 01/31] locking: Flip arguments to atomic_fetch_or
From: Peter Zijlstra
Date: Fri Apr 22 2016 - 10:20:43 EST
On Fri, Apr 22, 2016 at 01:09:38PM +0200, Geert Uytterhoeven wrote:
> On Fri, Apr 22, 2016 at 11:04 AM, Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
> > All the atomic operations have their arguments the wrong way around;
Nah, I find they really are the wrong way around. I forever write:
atomic_add(&v, val); and then have the compiler yell at me.
> > make atomic_fetch_or() consistent and flip them.
> BTW, there are a few other inconsistencies:
> git grep "\<atomic_.*atomic_t\>.*\<int\>"
Yes, but fixing those will be much more pain :/ atomic_fetch_or() was
freshly introduced and only has a few callers, furthermore, the
following patches would require it to be in line with the other
atomic_$op() due to them all being generated from the same 'template'.