> -----Original Message----- > From: Jim Mattson <jmattson@xxxxxxxxxx> > Sent: Wednesday, July 29, 2020 4:06 PM > To: Moger, Babu <Babu.Moger@xxxxxxx> > Cc: Paolo Bonzini <pbonzini@xxxxxxxxxx>; Vitaly Kuznetsov > <vkuznets@xxxxxxxxxx>; Wanpeng Li <wanpengli@xxxxxxxxxxx>; Sean > Christopherson <sean.j.christopherson@xxxxxxxxx>; kvm list > <kvm@xxxxxxxxxxxxxxx>; Joerg Roedel <joro@xxxxxxxxxx>; the arch/x86 > maintainers <x86@xxxxxxxxxx>; LKML <linux-kernel@xxxxxxxxxxxxxxx>; Ingo > Molnar <mingo@xxxxxxxxxx>; Borislav Petkov <bp@xxxxxxxxx>; H . Peter Anvin > <hpa@xxxxxxxxx>; Thomas Gleixner <tglx@xxxxxxxxxxxxx> > Subject: Re: [PATCH v3 05/11] KVM: SVM: Modify 64 bit intercept field to two 32 > bit vectors > > On Tue, Jul 28, 2020 at 4:38 PM Babu Moger <babu.moger@xxxxxxx> wrote: > > > > Convert all the intercepts to one array of 32 bit vectors in > > vmcb_control_area. This makes it easy for future intercept vector > > additions. Also update trace functions. > > > > Signed-off-by: Babu Moger <babu.moger@xxxxxxx> > > --- > > > @@ -128,8 +129,7 @@ enum { > > > > struct __attribute__ ((__packed__)) vmcb_control_area { > > u32 intercepts[MAX_VECTORS]; > > - u64 intercept; > > - u8 reserved_1[40]; > > + u8 reserved_1[60 - (MAX_VECTORS * 4)]; > > Perhaps this could be simplified to 'u32 reserved_1[15 - MAX_VECTORS];' Ok. Will change it. thanks > > > u16 pause_filter_thresh; > > u16 pause_filter_count; > > u64 iopm_base_pa; > > Reviewed-by: Jim Mattson <jmattson@xxxxxxxxxx>