diff mbox

[3/9] s390x: use kvm_enable_cap_vcpu()

Message ID 1397494110-5756-4-git-send-email-cornelia.huck@de.ibm.com
State New
Headers show

Commit Message

Cornelia Huck April 14, 2014, 4:48 p.m. UTC
Make kvm_s390_enable_css_support() use new interface.

Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com>
---
 target-s390x/kvm.c |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Alexander Graf April 16, 2014, 11:23 a.m. UTC | #1
On 14.04.14 18:48, Cornelia Huck wrote:
> Make kvm_s390_enable_css_support() use new interface.
>
> Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com>

What I was trying to indicate with the "convert users" comment last time 
was to convert non-s390 users :). After all, ENABLE_CAP is a 
cross-architecture interface and we should treat it as such.


Alex

> ---
>   target-s390x/kvm.c |    4 +---
>   1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/target-s390x/kvm.c b/target-s390x/kvm.c
> index 56b9af7..fbdc1bb 100644
> --- a/target-s390x/kvm.c
> +++ b/target-s390x/kvm.c
> @@ -910,12 +910,10 @@ void kvm_s390_crw_mchk(S390CPU *cpu)
>   
>   void kvm_s390_enable_css_support(S390CPU *cpu)
>   {
> -    struct kvm_enable_cap cap = {};
>       int r;
>   
>       /* Activate host kernel channel subsystem support. */
> -    cap.cap = KVM_CAP_S390_CSS_SUPPORT;
> -    r = kvm_vcpu_ioctl(CPU(cpu), KVM_ENABLE_CAP, &cap);
> +    r = kvm_enable_cap_vcpu(CPU(cpu), KVM_CAP_S390_CSS_SUPPORT);
>       assert(r == 0);
>   }
>
diff mbox

Patch

diff --git a/target-s390x/kvm.c b/target-s390x/kvm.c
index 56b9af7..fbdc1bb 100644
--- a/target-s390x/kvm.c
+++ b/target-s390x/kvm.c
@@ -910,12 +910,10 @@  void kvm_s390_crw_mchk(S390CPU *cpu)
 
 void kvm_s390_enable_css_support(S390CPU *cpu)
 {
-    struct kvm_enable_cap cap = {};
     int r;
 
     /* Activate host kernel channel subsystem support. */
-    cap.cap = KVM_CAP_S390_CSS_SUPPORT;
-    r = kvm_vcpu_ioctl(CPU(cpu), KVM_ENABLE_CAP, &cap);
+    r = kvm_enable_cap_vcpu(CPU(cpu), KVM_CAP_S390_CSS_SUPPORT);
     assert(r == 0);
 }