lkml.org 
[lkml]   [2017]   [Oct]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v9 2/5] x86/cpuid: Add generic table for cpuid dependencies

* Ingo Molnar <mingo@kernel.org> wrote:

> > + bool changed;
> > + __u32 disable[NCAPINTS + NBUGINTS];
> > + unsigned long *disable_mask = (unsigned long *)disable;
> > + const struct cpuid_dep *d;
>
> The constant forced types are really ugly and permeate the whole code. Please
> introduce some suitably named helpers in the x86 bitops file that work on local
> variables:
>
> var &= ~(1<<bit);
> var |= 1<<bit;

So instead of adding helpers the 1<< ops can be written out explicitly - they are
easy to read after all. Can also be:

var &= ~BIT(bit);
var |= BIT(bit);

Thanks,

Ingo

\
 
 \ /
  Last update: 2017-10-12 10:17    [W:0.132 / U:0.656 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site