Re: [PATCH v1 1/1] x86/percpu: Cast -1 to argument type when comparing in percpu_add_op()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 16, 2024 at 12:44:18PM -0700, Dave Hansen wrote:

> Would anybody hate if we broke this up a bit, like:
> 
>         const typeof(var) _val = val;
>         const int paoconst = __builtin_constant_p(val);
>         const int paoinc   = paoconst && ((_val) == 1);
>         const int paodec   = paoconst && ((_val) == (typeof(var))-1);
> 
> and then did
> 
> 	if (paoinc)
> 		percpu_unary_op(size, qual, "inc", var);
> 	...

I think that is an overall improvement. Proceed! :-)




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux