On Wed, Mar 01, 2017 at 01:54:00AM -0700, Jan Beulich wrote: > >>> On 01.03.17 at 09:49, <roger....@citrix.com> 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@lists.xen.org https://lists.xen.org/xen-devel
