diff mbox series

s390x/kvm: remove unused gs handling

Message ID 20210602125050.492500-1-cohuck@redhat.com (mailing list archive)
State New, archived
Headers show
Series s390x/kvm: remove unused gs handling | expand

Commit Message

Cornelia Huck June 2, 2021, 12:50 p.m. UTC
With commit 0280b3eb7c05 ("s390x/kvm: use cpu model for gscb on
compat machines"), we removed any calls to kvm_s390_get_gs()
in favour of a different mechanism.

Let's remove the unused kvm_s390_get_gs(), and with it the now
unneeded cap_gs as well.

Signed-off-by: Cornelia Huck <cohuck@redhat.com>
---
 target/s390x/kvm-stub.c  |  5 -----
 target/s390x/kvm.c       | 10 +---------
 target/s390x/kvm_s390x.h |  1 -
 3 files changed, 1 insertion(+), 15 deletions(-)

Comments

Thomas Huth June 2, 2021, 1:07 p.m. UTC | #1
On 02/06/2021 14.50, Cornelia Huck wrote:
> With commit 0280b3eb7c05 ("s390x/kvm: use cpu model for gscb on
> compat machines"), we removed any calls to kvm_s390_get_gs()
> in favour of a different mechanism.
> 
> Let's remove the unused kvm_s390_get_gs(), and with it the now
> unneeded cap_gs as well.
> 
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>   target/s390x/kvm-stub.c  |  5 -----
>   target/s390x/kvm.c       | 10 +---------
>   target/s390x/kvm_s390x.h |  1 -
>   3 files changed, 1 insertion(+), 15 deletions(-)

Reviewed-by: Thomas Huth <thuth@redhat.com>
Philippe Mathieu-Daudé June 2, 2021, 3:13 p.m. UTC | #2
On 6/2/21 2:50 PM, Cornelia Huck wrote:
> With commit 0280b3eb7c05 ("s390x/kvm: use cpu model for gscb on
> compat machines"), we removed any calls to kvm_s390_get_gs()
> in favour of a different mechanism.
> 
> Let's remove the unused kvm_s390_get_gs(), and with it the now
> unneeded cap_gs as well.
> 
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>  target/s390x/kvm-stub.c  |  5 -----
>  target/s390x/kvm.c       | 10 +---------
>  target/s390x/kvm_s390x.h |  1 -
>  3 files changed, 1 insertion(+), 15 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Christian Borntraeger June 7, 2021, 8:11 a.m. UTC | #3
On 02.06.21 14:50, Cornelia Huck wrote:
> With commit 0280b3eb7c05 ("s390x/kvm: use cpu model for gscb on
> compat machines"), we removed any calls to kvm_s390_get_gs()
> in favour of a different mechanism.
> 
> Let's remove the unused kvm_s390_get_gs(), and with it the now
> unneeded cap_gs as well.
> 
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>

Reviewed-by: Christian Borntraeger <borntraeger@de.ibm.com>

> ---
>   target/s390x/kvm-stub.c  |  5 -----
>   target/s390x/kvm.c       | 10 +---------
>   target/s390x/kvm_s390x.h |  1 -
>   3 files changed, 1 insertion(+), 15 deletions(-)
> 
> diff --git a/target/s390x/kvm-stub.c b/target/s390x/kvm-stub.c
> index 9970b5a8c705..8a308cfebb68 100644
> --- a/target/s390x/kvm-stub.c
> +++ b/target/s390x/kvm-stub.c
> @@ -49,11 +49,6 @@ int kvm_s390_get_ri(void)
>       return 0;
>   }
>   
> -int kvm_s390_get_gs(void)
> -{
> -    return 0;
> -}
> -
>   int kvm_s390_get_clock(uint8_t *tod_high, uint64_t *tod_low)
>   {
>       return -ENOSYS;
> diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
> index 4fb3bbfef506..23889245877a 100644
> --- a/target/s390x/kvm.c
> +++ b/target/s390x/kvm.c
> @@ -154,7 +154,6 @@ static int cap_async_pf;
>   static int cap_mem_op;
>   static int cap_s390_irq;
>   static int cap_ri;
> -static int cap_gs;
>   static int cap_hpage_1m;
>   static int cap_vcpu_resets;
>   static int cap_protected;
> @@ -369,9 +368,7 @@ int kvm_arch_init(MachineState *ms, KVMState *s)
>           }
>       }
>       if (cpu_model_allowed()) {
> -        if (kvm_vm_enable_cap(s, KVM_CAP_S390_GS, 0) == 0) {
> -            cap_gs = 1;
> -        }
> +        kvm_vm_enable_cap(s, KVM_CAP_S390_GS, 0);
>       }
>   
>       /*
> @@ -2039,11 +2036,6 @@ int kvm_s390_get_ri(void)
>       return cap_ri;
>   }
>   
> -int kvm_s390_get_gs(void)
> -{
> -    return cap_gs;
> -}
> -
>   int kvm_s390_set_cpu_state(S390CPU *cpu, uint8_t cpu_state)
>   {
>       struct kvm_mp_state mp_state = {};
> diff --git a/target/s390x/kvm_s390x.h b/target/s390x/kvm_s390x.h
> index 25bbe98b2514..05a5e1e6f46d 100644
> --- a/target/s390x/kvm_s390x.h
> +++ b/target/s390x/kvm_s390x.h
> @@ -27,7 +27,6 @@ void kvm_s390_vcpu_interrupt_pre_save(S390CPU *cpu);
>   int kvm_s390_vcpu_interrupt_post_load(S390CPU *cpu);
>   int kvm_s390_get_hpage_1m(void);
>   int kvm_s390_get_ri(void);
> -int kvm_s390_get_gs(void);
>   int kvm_s390_get_clock(uint8_t *tod_high, uint64_t *tod_clock);
>   int kvm_s390_get_clock_ext(uint8_t *tod_high, uint64_t *tod_clock);
>   int kvm_s390_set_clock(uint8_t tod_high, uint64_t tod_clock);
>
Cornelia Huck June 8, 2021, 7:40 a.m. UTC | #4
On Wed, Jun 02 2021, Cornelia Huck <cohuck@redhat.com> wrote:

> With commit 0280b3eb7c05 ("s390x/kvm: use cpu model for gscb on
> compat machines"), we removed any calls to kvm_s390_get_gs()
> in favour of a different mechanism.
>
> Let's remove the unused kvm_s390_get_gs(), and with it the now
> unneeded cap_gs as well.
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>  target/s390x/kvm-stub.c  |  5 -----
>  target/s390x/kvm.c       | 10 +---------
>  target/s390x/kvm_s390x.h |  1 -
>  3 files changed, 1 insertion(+), 15 deletions(-)

Queued to s390-next.
diff mbox series

Patch

diff --git a/target/s390x/kvm-stub.c b/target/s390x/kvm-stub.c
index 9970b5a8c705..8a308cfebb68 100644
--- a/target/s390x/kvm-stub.c
+++ b/target/s390x/kvm-stub.c
@@ -49,11 +49,6 @@  int kvm_s390_get_ri(void)
     return 0;
 }
 
-int kvm_s390_get_gs(void)
-{
-    return 0;
-}
-
 int kvm_s390_get_clock(uint8_t *tod_high, uint64_t *tod_low)
 {
     return -ENOSYS;
diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
index 4fb3bbfef506..23889245877a 100644
--- a/target/s390x/kvm.c
+++ b/target/s390x/kvm.c
@@ -154,7 +154,6 @@  static int cap_async_pf;
 static int cap_mem_op;
 static int cap_s390_irq;
 static int cap_ri;
-static int cap_gs;
 static int cap_hpage_1m;
 static int cap_vcpu_resets;
 static int cap_protected;
@@ -369,9 +368,7 @@  int kvm_arch_init(MachineState *ms, KVMState *s)
         }
     }
     if (cpu_model_allowed()) {
-        if (kvm_vm_enable_cap(s, KVM_CAP_S390_GS, 0) == 0) {
-            cap_gs = 1;
-        }
+        kvm_vm_enable_cap(s, KVM_CAP_S390_GS, 0);
     }
 
     /*
@@ -2039,11 +2036,6 @@  int kvm_s390_get_ri(void)
     return cap_ri;
 }
 
-int kvm_s390_get_gs(void)
-{
-    return cap_gs;
-}
-
 int kvm_s390_set_cpu_state(S390CPU *cpu, uint8_t cpu_state)
 {
     struct kvm_mp_state mp_state = {};
diff --git a/target/s390x/kvm_s390x.h b/target/s390x/kvm_s390x.h
index 25bbe98b2514..05a5e1e6f46d 100644
--- a/target/s390x/kvm_s390x.h
+++ b/target/s390x/kvm_s390x.h
@@ -27,7 +27,6 @@  void kvm_s390_vcpu_interrupt_pre_save(S390CPU *cpu);
 int kvm_s390_vcpu_interrupt_post_load(S390CPU *cpu);
 int kvm_s390_get_hpage_1m(void);
 int kvm_s390_get_ri(void);
-int kvm_s390_get_gs(void);
 int kvm_s390_get_clock(uint8_t *tod_high, uint64_t *tod_clock);
 int kvm_s390_get_clock_ext(uint8_t *tod_high, uint64_t *tod_clock);
 int kvm_s390_set_clock(uint8_t tod_high, uint64_t tod_clock);