1. Patchew
  2. QEMU
  3. [Qemu-devel] [PATCH v1 0/5] target/s390x: cpu model cleanups + improvements
  4. View patch

[Qemu-devel] [PATCH v1 2/5] target/s390x: indicate query subfunction in s390_fill_feat_block

David Hildenbrand posted 5 patches 8 years, 6 months ago
There is a newer version of this series
[Qemu-devel] [PATCH v1 2/5] target/s390x: indicate query subfunction in s390_fill_feat_block
Posted by David Hildenbrand 8 years, 6 months ago 
We'll have to do the same for TCG, so let's just move it in there.

Signed-off-by: David Hildenbrand <david@redhat.com>
---
 target/s390x/cpu_features.c | 31 ++++++++++++++++++++++++++-----
 target/s390x/kvm.c          | 13 -------------
 2 files changed, 26 insertions(+), 18 deletions(-)

diff --git a/target/s390x/cpu_features.c b/target/s390x/cpu_features.c
index fa887d9..8b14917 100644
--- a/target/s390x/cpu_features.c
+++ b/target/s390x/cpu_features.c
@@ -336,11 +336,32 @@ void s390_fill_feat_block(const S390FeatBitmap features, S390FeatType type,
     S390Feat feat;
     int bit_nr;
 
-    if (type == S390_FEAT_TYPE_STFL && test_bit(S390_FEAT_ZARCH, features)) {
-        /* Features that are always active */
-        data[0] |= 0x20;  /* z/Architecture */
-        data[17] |= 0x20; /* Configuration-z-architectural-mode */
-    }
+    switch (type) {
+    case S390_FEAT_TYPE_STFL:
+        if (test_bit(S390_FEAT_ZARCH, features)) {
+            /* Features that are always active */
+            data[0] |= 0x20;  /* z/Architecture */
+            data[17] |= 0x20; /* Configuration-z-architectural-mode */
+        }
+        break;
+    case S390_FEAT_TYPE_PTFF:
+    case S390_FEAT_TYPE_KMAC:
+    case S390_FEAT_TYPE_KMC:
+    case S390_FEAT_TYPE_KM:
+    case S390_FEAT_TYPE_KIMD:
+    case S390_FEAT_TYPE_KLMD:
+    case S390_FEAT_TYPE_PCKMO:
+    case S390_FEAT_TYPE_KMCTR:
+    case S390_FEAT_TYPE_KMF:
+    case S390_FEAT_TYPE_KMO:
+    case S390_FEAT_TYPE_PCC:
+    case S390_FEAT_TYPE_PPNO:
+    case S390_FEAT_TYPE_KMA:
+        data[0] |= 0x80; /* query is always available */
+        break;
+    default:
+        break;
+    };
 
     feat = find_first_bit(features, S390_FEAT_MAX);
     while (feat < S390_FEAT_MAX) {
diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
index 831492f..999ea57 100644
--- a/target/s390x/kvm.c
+++ b/target/s390x/kvm.c
@@ -2506,41 +2506,28 @@ static int configure_cpu_subfunc(const S390FeatBitmap features)
     s390_fill_feat_block(features, S390_FEAT_TYPE_PLO, prop.plo);
     if (test_bit(S390_FEAT_TOD_CLOCK_STEERING, features)) {
         s390_fill_feat_block(features, S390_FEAT_TYPE_PTFF, prop.ptff);
-        prop.ptff[0] |= 0x80; /* query is always available */
     }
     if (test_bit(S390_FEAT_MSA, features)) {
         s390_fill_feat_block(features, S390_FEAT_TYPE_KMAC, prop.kmac);
-        prop.kmac[0] |= 0x80; /* query is always available */
         s390_fill_feat_block(features, S390_FEAT_TYPE_KMC, prop.kmc);
-        prop.kmc[0] |= 0x80; /* query is always available */
         s390_fill_feat_block(features, S390_FEAT_TYPE_KM, prop.km);
-        prop.km[0] |= 0x80; /* query is always available */
         s390_fill_feat_block(features, S390_FEAT_TYPE_KIMD, prop.kimd);
-        prop.kimd[0] |= 0x80; /* query is always available */
         s390_fill_feat_block(features, S390_FEAT_TYPE_KLMD, prop.klmd);
-        prop.klmd[0] |= 0x80; /* query is always available */
     }
     if (test_bit(S390_FEAT_MSA_EXT_3, features)) {
         s390_fill_feat_block(features, S390_FEAT_TYPE_PCKMO, prop.pckmo);
-        prop.pckmo[0] |= 0x80; /* query is always available */
     }
     if (test_bit(S390_FEAT_MSA_EXT_4, features)) {
         s390_fill_feat_block(features, S390_FEAT_TYPE_KMCTR, prop.kmctr);
-        prop.kmctr[0] |= 0x80; /* query is always available */
         s390_fill_feat_block(features, S390_FEAT_TYPE_KMF, prop.kmf);
-        prop.kmf[0] |= 0x80; /* query is always available */
         s390_fill_feat_block(features, S390_FEAT_TYPE_KMO, prop.kmo);
-        prop.kmo[0] |= 0x80; /* query is always available */
         s390_fill_feat_block(features, S390_FEAT_TYPE_PCC, prop.pcc);
-        prop.pcc[0] |= 0x80; /* query is always available */
     }
     if (test_bit(S390_FEAT_MSA_EXT_5, features)) {
         s390_fill_feat_block(features, S390_FEAT_TYPE_PPNO, prop.ppno);
-        prop.ppno[0] |= 0x80; /* query is always available */
     }
     if (test_bit(S390_FEAT_MSA_EXT_8, features)) {
         s390_fill_feat_block(features, S390_FEAT_TYPE_KMA, prop.kma);
-        prop.kma[0] |= 0x80; /* query is always available */
     }
     return kvm_vm_ioctl(kvm_state, KVM_SET_DEVICE_ATTR, &attr);
 }
-- 
2.9.4
Re: [Qemu-devel] [PATCH v1 2/5] target/s390x: indicate query subfunction in s390_fill_feat_block
Posted by Christian Borntraeger 8 years, 6 months ago 
On 07/19/2017 11:50 AM, David Hildenbrand wrote:
> We'll have to do the same for TCG, so let's just move it in there.
> 
> Signed-off-by: David Hildenbrand <david@redhat.com>

looks sane.

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

> ---
>  target/s390x/cpu_features.c | 31 ++++++++++++++++++++++++++-----
>  target/s390x/kvm.c          | 13 -------------
>  2 files changed, 26 insertions(+), 18 deletions(-)
> 
> diff --git a/target/s390x/cpu_features.c b/target/s390x/cpu_features.c
> index fa887d9..8b14917 100644
> --- a/target/s390x/cpu_features.c
> +++ b/target/s390x/cpu_features.c
> @@ -336,11 +336,32 @@ void s390_fill_feat_block(const S390FeatBitmap features, S390FeatType type,
>      S390Feat feat;
>      int bit_nr;
> 
> -    if (type == S390_FEAT_TYPE_STFL && test_bit(S390_FEAT_ZARCH, features)) {
> -        /* Features that are always active */
> -        data[0] |= 0x20;  /* z/Architecture */
> -        data[17] |= 0x20; /* Configuration-z-architectural-mode */
> -    }
> +    switch (type) {
> +    case S390_FEAT_TYPE_STFL:
> +        if (test_bit(S390_FEAT_ZARCH, features)) {
> +            /* Features that are always active */
> +            data[0] |= 0x20;  /* z/Architecture */
> +            data[17] |= 0x20; /* Configuration-z-architectural-mode */
> +        }
> +        break;
> +    case S390_FEAT_TYPE_PTFF:
> +    case S390_FEAT_TYPE_KMAC:
> +    case S390_FEAT_TYPE_KMC:
> +    case S390_FEAT_TYPE_KM:
> +    case S390_FEAT_TYPE_KIMD:
> +    case S390_FEAT_TYPE_KLMD:
> +    case S390_FEAT_TYPE_PCKMO:
> +    case S390_FEAT_TYPE_KMCTR:
> +    case S390_FEAT_TYPE_KMF:
> +    case S390_FEAT_TYPE_KMO:
> +    case S390_FEAT_TYPE_PCC:
> +    case S390_FEAT_TYPE_PPNO:
> +    case S390_FEAT_TYPE_KMA:
> +        data[0] |= 0x80; /* query is always available */
> +        break;
> +    default:
> +        break;
> +    };
> 
>      feat = find_first_bit(features, S390_FEAT_MAX);
>      while (feat < S390_FEAT_MAX) {
> diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
> index 831492f..999ea57 100644
> --- a/target/s390x/kvm.c
> +++ b/target/s390x/kvm.c
> @@ -2506,41 +2506,28 @@ static int configure_cpu_subfunc(const S390FeatBitmap features)
>      s390_fill_feat_block(features, S390_FEAT_TYPE_PLO, prop.plo);
>      if (test_bit(S390_FEAT_TOD_CLOCK_STEERING, features)) {
>          s390_fill_feat_block(features, S390_FEAT_TYPE_PTFF, prop.ptff);
> -        prop.ptff[0] |= 0x80; /* query is always available */
>      }
>      if (test_bit(S390_FEAT_MSA, features)) {
>          s390_fill_feat_block(features, S390_FEAT_TYPE_KMAC, prop.kmac);
> -        prop.kmac[0] |= 0x80; /* query is always available */
>          s390_fill_feat_block(features, S390_FEAT_TYPE_KMC, prop.kmc);
> -        prop.kmc[0] |= 0x80; /* query is always available */
>          s390_fill_feat_block(features, S390_FEAT_TYPE_KM, prop.km);
> -        prop.km[0] |= 0x80; /* query is always available */
>          s390_fill_feat_block(features, S390_FEAT_TYPE_KIMD, prop.kimd);
> -        prop.kimd[0] |= 0x80; /* query is always available */
>          s390_fill_feat_block(features, S390_FEAT_TYPE_KLMD, prop.klmd);
> -        prop.klmd[0] |= 0x80; /* query is always available */
>      }
>      if (test_bit(S390_FEAT_MSA_EXT_3, features)) {
>          s390_fill_feat_block(features, S390_FEAT_TYPE_PCKMO, prop.pckmo);
> -        prop.pckmo[0] |= 0x80; /* query is always available */
>      }
>      if (test_bit(S390_FEAT_MSA_EXT_4, features)) {
>          s390_fill_feat_block(features, S390_FEAT_TYPE_KMCTR, prop.kmctr);
> -        prop.kmctr[0] |= 0x80; /* query is always available */
>          s390_fill_feat_block(features, S390_FEAT_TYPE_KMF, prop.kmf);
> -        prop.kmf[0] |= 0x80; /* query is always available */
>          s390_fill_feat_block(features, S390_FEAT_TYPE_KMO, prop.kmo);
> -        prop.kmo[0] |= 0x80; /* query is always available */
>          s390_fill_feat_block(features, S390_FEAT_TYPE_PCC, prop.pcc);
> -        prop.pcc[0] |= 0x80; /* query is always available */
>      }
>      if (test_bit(S390_FEAT_MSA_EXT_5, features)) {
>          s390_fill_feat_block(features, S390_FEAT_TYPE_PPNO, prop.ppno);
> -        prop.ppno[0] |= 0x80; /* query is always available */
>      }
>      if (test_bit(S390_FEAT_MSA_EXT_8, features)) {
>          s390_fill_feat_block(features, S390_FEAT_TYPE_KMA, prop.kma);
> -        prop.kma[0] |= 0x80; /* query is always available */
>      }
>      return kvm_vm_ioctl(kvm_state, KVM_SET_DEVICE_ATTR, &attr);
>  }
>

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.