Re: [PATCH 1/2] AMD errata checking framework

H. Peter Anvin
2010-06-15T18:17:18Z

On 06/14/2010 05:33 AM, Hans Rosenfeld wrote:
>
>> Also, the use of divide and modulo when shifts and masks work are
>> generally frowned upon in Linux as a matter of style.
>
> While I agree in principle, I think that in this special case the
> divide and modulo by 64 make it easier to understand what the code
> does on the first look. But if you insist I will send another patch
> set that replaces divide and modulo with shifts and masks.
>

I do insist.

-hpa