Re: [PATCH] tools: include: sync include/api/linux/kvm.h

[Paolo Bonzini]

On 10/22/22 13:47, Marc Zyngier wrote:
> > Provide a definition of KVM_CAP_DIRTY_LOG_RING_ACQ_REL.
> >
> > Fixes: 4b3402f1f4d9 ("KVM: selftests: dirty-log: Use KVM_CAP_DIRTY_LOG_RING_ACQ_REL if available")
> > Cc: Marc Zyngier<maz@xxxxxxxxxx>
> > Signed-off-by: Paolo Bonzini<pbonzini@xxxxxxxxxx>
> > ---
> >   tools/include/uapi/linux/kvm.h | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/tools/include/uapi/linux/kvm.h b/tools/include/uapi/linux/kvm.h
> > index eed0315a77a6..0d5d4419139a 100644
> > --- a/tools/include/uapi/linux/kvm.h
> > +++ b/tools/include/uapi/linux/kvm.h
> > @@ -1177,6 +1177,7 @@ struct kvm_ppc_resize_hpt {
> >   #define KVM_CAP_VM_DISABLE_NX_HUGE_PAGES 220
> >   #define KVM_CAP_S390_ZPCI_OP 221
> >   #define KVM_CAP_S390_CPU_TOPOLOGY 222
> > +#define KVM_CAP_DIRTY_LOG_RING_ACQ_REL 223
> >   
> >   #ifdef KVM_CAP_IRQ_ROUTING
> >   
> Huh, I wonder how I missed that one, as the test were compiling here.
>
> Acked-by: Marc Zyngier<maz@xxxxxxxxxx>

5 minutes later -- The tests do not use that file, they use usr/include/ 
in the build tree.  So the right Fixes tag is 17601bfed909 ("KVM: Add 
KVM_CAP_DIRTY_LOG_RING_ACQ_REL capability and config option").

Paolo