tools: include: sync include/api/linux/kvm.h

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

Posted by Paolo Bonzini 3 years, 5 months ago

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@kernel.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 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
 
-- 
2.31.1

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

Posted by Marc Zyngier 3 years, 5 months ago

On Sat, 22 Oct 2022 12:44:23 +0100,
Paolo Bonzini <pbonzini@redhat.com> 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@kernel.org>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  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@kernel.org>

	M.

-- 
Without deviation from the norm, progress is not possible.

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

Posted by Paolo Bonzini 3 years, 5 months ago

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@kernel.org>
>> Signed-off-by: Paolo Bonzini<pbonzini@redhat.com>
>> ---
>>   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@kernel.org>

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