>>> On 30.05.18 at 15:27, <luwei.k...@intel.com> wrote: > --- a/xen/include/public/arch-x86/cpufeatureset.h > +++ b/xen/include/public/arch-x86/cpufeatureset.h > @@ -215,6 +215,7 @@ XEN_CPUFEATURE(SMAP, 5*32+20) /*S Supervisor > Mode Access Prevention */ > XEN_CPUFEATURE(AVX512IFMA, 5*32+21) /*A AVX-512 Integer Fused Multiply > Add */ > XEN_CPUFEATURE(CLFLUSHOPT, 5*32+23) /*A CLFLUSHOPT instruction */ > XEN_CPUFEATURE(CLWB, 5*32+24) /*A CLWB instruction */ > +XEN_CPUFEATURE(IPT, 5*32+25) /*H Intel Processor Trace */
Btw - introducing the feature flag here is certainly fine, but I think you should add the H annotation only once functionality is complete. That would then e.g. also reduce the impact of patch 8 adding functionality otherwise strictly necessary already in patch 7. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel