public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
* [kvm-unit-tests PATCH] x86: Drop "enabled" field from "struct kvm_vcpu_pv_apf_data"
@ 2025-02-21 22:57 Sean Christopherson
  2025-02-24  3:02 ` Xiaoyao Li
  2025-02-24 17:24 ` Sean Christopherson
  0 siblings, 2 replies; 3+ messages in thread
From: Sean Christopherson @ 2025-02-21 22:57 UTC (permalink / raw)
  To: Paolo Bonzini; +Cc: kvm, Xiaoyao Li, Sean Christopherson

Remove the now-defunct (and never used in KVM-Unit-Tests) "enabled" field
from kvm_vcpu_pv_apf_data.  The field was removed from KVM by commit
ccb2280ec2f9 ("x86/kvm: Use separate percpu variable to track the enabling
of asyncpf").

Suggested-by: Xiaoyao Li <xiaoyao.li@intel.com>
Signed-off-by: Sean Christopherson <seanjc@google.com>
---
 x86/asyncpf.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/x86/asyncpf.c b/x86/asyncpf.c
index 6474fede..0e6eb6ff 100644
--- a/x86/asyncpf.c
+++ b/x86/asyncpf.c
@@ -50,7 +50,6 @@ struct kvm_vcpu_pv_apf_data {
       uint32_t  token;
 
       uint8_t  pad[56];
-      uint32_t  enabled;
 } apf_reason __attribute__((aligned(64)));
 
 char *buf;

base-commit: f77fb696cfd0e4a5562cdca189be557946bf522f
-- 
2.48.1.601.g30ceb7b040-goog


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [kvm-unit-tests PATCH] x86: Drop "enabled" field from "struct kvm_vcpu_pv_apf_data"
  2025-02-21 22:57 [kvm-unit-tests PATCH] x86: Drop "enabled" field from "struct kvm_vcpu_pv_apf_data" Sean Christopherson
@ 2025-02-24  3:02 ` Xiaoyao Li
  2025-02-24 17:24 ` Sean Christopherson
  1 sibling, 0 replies; 3+ messages in thread
From: Xiaoyao Li @ 2025-02-24  3:02 UTC (permalink / raw)
  To: Sean Christopherson, Paolo Bonzini; +Cc: kvm

On 2/22/2025 6:57 AM, Sean Christopherson wrote:
> Remove the now-defunct (and never used in KVM-Unit-Tests) "enabled" field
> from kvm_vcpu_pv_apf_data.  The field was removed from KVM by commit
> ccb2280ec2f9 ("x86/kvm: Use separate percpu variable to track the enabling
> of asyncpf").
> 
> Suggested-by: Xiaoyao Li <xiaoyao.li@intel.com>
> Signed-off-by: Sean Christopherson <seanjc@google.com>

Reviewed-by: Xiaoyao Li <xiaoyao.li@intel.com>

> ---
>   x86/asyncpf.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/x86/asyncpf.c b/x86/asyncpf.c
> index 6474fede..0e6eb6ff 100644
> --- a/x86/asyncpf.c
> +++ b/x86/asyncpf.c
> @@ -50,7 +50,6 @@ struct kvm_vcpu_pv_apf_data {
>         uint32_t  token;
>   
>         uint8_t  pad[56];
> -      uint32_t  enabled;
>   } apf_reason __attribute__((aligned(64)));
>   
>   char *buf;
> 
> base-commit: f77fb696cfd0e4a5562cdca189be557946bf522f


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [kvm-unit-tests PATCH] x86: Drop "enabled" field from "struct kvm_vcpu_pv_apf_data"
  2025-02-21 22:57 [kvm-unit-tests PATCH] x86: Drop "enabled" field from "struct kvm_vcpu_pv_apf_data" Sean Christopherson
  2025-02-24  3:02 ` Xiaoyao Li
@ 2025-02-24 17:24 ` Sean Christopherson
  1 sibling, 0 replies; 3+ messages in thread
From: Sean Christopherson @ 2025-02-24 17:24 UTC (permalink / raw)
  To: Sean Christopherson, Paolo Bonzini; +Cc: kvm, Xiaoyao Li

On Fri, 21 Feb 2025 14:57:44 -0800, Sean Christopherson wrote:
> Remove the now-defunct (and never used in KVM-Unit-Tests) "enabled" field
> from kvm_vcpu_pv_apf_data.  The field was removed from KVM by commit
> ccb2280ec2f9 ("x86/kvm: Use separate percpu variable to track the enabling
> of asyncpf").
> 
> 

Applied to kvm-x86 next (and now pulled by Paolo), thanks!

[1/1] x86: Drop "enabled" field from "struct kvm_vcpu_pv_apf_data"
      https://github.com/kvm-x86/kvm-unit-tests/commit/5f4bb7408590

--
https://github.com/kvm-x86/kvm-unit-tests/tree/next

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-02-24 17:26 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-21 22:57 [kvm-unit-tests PATCH] x86: Drop "enabled" field from "struct kvm_vcpu_pv_apf_data" Sean Christopherson
2025-02-24  3:02 ` Xiaoyao Li
2025-02-24 17:24 ` Sean Christopherson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox