[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 05/10] s390x/cpumodel: vector enhancements
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PATCH 05/10] s390x/cpumodel: vector enhancements |
Date: |
Thu, 18 Apr 2019 15:04:00 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 |
On 18.04.19 14:56, David Hildenbrand wrote:
> On 18.04.19 13:31, Christian Borntraeger wrote:
>> Add vector enhancements to the cpu model.
>>
>> Signed-off-by: Christian Borntraeger <address@hidden>
>> ---
>> target/s390x/cpu_features.c | 2 ++
>> target/s390x/cpu_features_def.h | 2 ++
>> 2 files changed, 4 insertions(+)
>>
>> diff --git a/target/s390x/cpu_features.c b/target/s390x/cpu_features.c
>> index 4d624b2256..ed3f6aa969 100644
>> --- a/target/s390x/cpu_features.c
>> +++ b/target/s390x/cpu_features.c
>> @@ -108,6 +108,8 @@ static const S390FeatDef s390_features[] = {
>> FEAT_INIT("irbm", S390_FEAT_TYPE_STFL, 145,
>> "Insert-reference-bits-multiple facility"),
>> FEAT_INIT("msa8-base", S390_FEAT_TYPE_STFL, 146,
>> "Message-security-assist-extension-8 facility (excluding subfunctions)"),
>> FEAT_INIT("cmmnt", S390_FEAT_TYPE_STFL, 147, "CMM: ESSA-enhancement (no
>> translate) facility"),
>> + FEAT_INIT("vxeh2", S390_FEAT_TYPE_STFL, 148, "Vector Enhancements
>> facility 2"),
>> + FEAT_INIT("vxbeh", S390_FEAT_TYPE_STFL, 152, "Vector BCD enhancements
>> facility 1"),
>> FEAT_INIT("msa9-base", S390_FEAT_TYPE_STFL, 155,
>> "Message-security-assist-extension-9 facility (excluding subfunctions)"),
>> FEAT_INIT("etoken", S390_FEAT_TYPE_STFL, 156, "Etoken facility"),
>
> BTW, to make this less error prone I thought about converting this into
> something like
>
> FEAT_INIT(VECTOR_ENH2, "vxeh2", S390_FEAT_TYPE_STFL, 148, "Vector
> Enhancements facility 2"),
>
> and internally use
>
> .[S390_FEAT_##FEAT] = { ...
>
> Or similar. But don't have time for that right now.
Yes, we can make the model code simpler. Lots of ideas.
>
>>
>> diff --git a/target/s390x/cpu_features_def.h
>> b/target/s390x/cpu_features_def.h
>> index 030784811b..ce2223c9d7 100644
>> --- a/target/s390x/cpu_features_def.h
>> +++ b/target/s390x/cpu_features_def.h
>> @@ -96,6 +96,8 @@ typedef enum {
>> S390_FEAT_INSERT_REFERENCE_BITS_MULT,
>> S390_FEAT_MSA_EXT_8,
>> S390_FEAT_CMM_NT,
>> + S390_FEAT_VECTOR_ENH2,
>> + S390_FEAT_VECTOR_BCD_ENH,
>> S390_FEAT_MSA_EXT_9,
>> S390_FEAT_ETOKEN,
>>
>>
>
> Reviewed-by: David Hildenbrand <address@hidden>
>
- [Qemu-devel] [PATCH 00/10] s390x: new guest features, Christian Borntraeger, 2019/04/18
- [Qemu-devel] [PATCH 07/10] s390x/cpumodel: deflate, Christian Borntraeger, 2019/04/18
- [Qemu-devel] [PATCH 03/10] s390x/cpumodel: Miscellaneous-Instruction-Extensions Facility 3, Christian Borntraeger, 2019/04/18
- [Qemu-devel] [PATCH 05/10] s390x/cpumodel: vector enhancements, Christian Borntraeger, 2019/04/18
- [Qemu-devel] [PATCH 09/10] s390x/cpumodel: wire up 8561 and 8562 as gen15 machines, Christian Borntraeger, 2019/04/18
- [Qemu-devel] [PATCH 06/10] s390x/cpumodel: enhanced sort facility, Christian Borntraeger, 2019/04/18
- [Qemu-devel] [PATCH 04/10] s390x/cpumodel: msa9 facility, Christian Borntraeger, 2019/04/18
- [Qemu-devel] [PATCH 10/10] s390x/cpumodel: do not claim csske for expanded models in qmp, Christian Borntraeger, 2019/04/18
- [Qemu-devel] [PATCH 02/10] s390x/cpumodel: remove CSSKE from base model, Christian Borntraeger, 2019/04/18