On 02.01.2025 09:45, Tu Dinh wrote: > --- a/xen/arch/x86/include/asm/msr-index.h > +++ b/xen/arch/x86/include/asm/msr-index.h > @@ -112,6 +112,8 @@ > #define MCU_OPT_CTRL_GDS_MIT_DIS (_AC(1, ULL) << 4) > #define MCU_OPT_CTRL_GDS_MIT_LOCK (_AC(1, ULL) << 5) > > +#define MSR_LER_INFO 0x000001e0 > + > #define MSR_RTIT_OUTPUT_BASE 0x00000560 > #define MSR_RTIT_OUTPUT_MASK 0x00000561 > #define MSR_RTIT_CTL 0x00000570 > @@ -193,6 +195,16 @@ > #define MSR_UARCH_MISC_CTRL 0x00001b01 > #define UARCH_CTRL_DOITM (_AC(1, ULL) << 0) > > +/* Architectural LBR state MSRs */ > +#define MSR_LBR_INFO(n) (0x00001200 + (n)) > +#define MSR_LBR_CTL 0x000014ce > +#define LBR_CTL_VALID _AC(0x7f000f, ULL)
While I can see that such a value may be useful at some point, I think it wants introducing when needed and composing of definitions for the individual bits. Jan
