Re: [Xen-devel] [PATCH v8 03/24] x86: refactor psr: implement main data structures.

[Roger Pau Monn� <roger.pau@xxxxxxxxxx>]

On Wed, Mar 01, 2017 at 01:54:00AM -0700, Jan Beulich wrote:
> >>> On 01.03.17 at 09:49, <roger.pau@xxxxxxxxxx> wrote:
> > What I was proposing is to make feat_node an array of pointers, so if a 
> > feature
> > is not supported that specific pointer would be NULL (ie: if feat_mask &
> > PSR_SOCKET_L3_CDP == 0, features[PSR_SOCKET_L3_CDP] == NULL). And then you 
> > can
> > avoid all the list traversing code.
> 
> In which case the feature flags would look to become redundant too.

I guess so, I'm just not sure if you can have a feature available (so
feat_mask bit set) but not enabled (so features[...] == NULL). Not sure if that
makes sense or is desirable at all.

Roger.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel