On Mon, Apr 25, 2022 at 9:52 AM Raghavendra Rao Ananta <rananta@xxxxxxxxxx> wrote: [...] > > > +#define KVM_REG_ARM_VENDOR_HYP_BMAP_BIT_MAX 1 > > > > Nit: IMHO perhaps it might be more convenient to define the MAX macro > > in arch/arm64/include/uapi/asm/kvm.h like below for maintenance ? > > (The same comments are applied to other KVM_REG_ARM_*_BMAP_BIT_MAX) > > > > #define KVM_REG_ARM_VENDOR_HYP_BIT_MAX KVM_REG_ARM_VENDOR_HYP_BIT_PTP > > > We have been going back and forth on this :) > It made sense for me to keep it in uapi as well, but I took Oliver's > suggestion of keeping it outside of uapi since this is something that > could be constantly changing [1]. The maximum set of features in a given bitmap register is a property of the running system, not the headers chosen at compile time. There is an illusion of ABI breakage when adding new bits to the registers if we've declared the max bit in UAPI. We also define KVM_VCPU_MAX_FEATURES outside of UAPI, even though it is related to the KVM_ARM_VCPU_INIT ioctl. -- Thanks, Oliver
