Re: [this_cpu_xx V8 09/16] Allow arch to provide inc/dec functionalityfor each size separately

[Tejun Heo]

On 12/19/2009 07:26 AM, Christoph Lameter wrote:
> Current this_cpu ops only allow an arch to specify add RMW operations or inc
> and dec for all sizes. Some arches can do more efficient inc and dec
> operations. Allow size specific override of fallback functions like with
> the other operations.

Wouldn't it be better to use __builtin_constant_p() and switching in
arch add/dec macros?  It just seems a bit extreme to define all those
different variants from generic header.  I'm quite unsure whether
providing overrides for all the different size variants from generic
header is necessary at all.  If an arch is gonna override the
operation, it's probably best to just let it override the whole thing.

Thanks.

-- 
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/