[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP
From: |
David Hildenbrand |
Subject: |
Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support |
Date: |
Wed, 18 Apr 2018 13:03:13 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 18.04.2018 12:55, Halil Pasic wrote:
>
>
> On 04/15/2018 09:07 PM, Tony Krowiak wrote:
>> A new CPU model feature and two new CPU model facilities are
>> introduced to support AP devices for a KVM guest.
> [..]
>> diff --git a/target/s390x/cpu_features.c b/target/s390x/cpu_features.c
>> index 3b9e274..5ee3a2d 100644
>> --- a/target/s390x/cpu_features.c
>> +++ b/target/s390x/cpu_features.c
>> @@ -40,8 +40,10 @@ static const S390FeatDef s390_features[] = {
>> FEAT_INIT("srs", S390_FEAT_TYPE_STFL, 9, "Sense-running-status
>> facility"),
>> FEAT_INIT("csske", S390_FEAT_TYPE_STFL, 10, "Conditional-SSKE
>> facility"),
>> FEAT_INIT("ctop", S390_FEAT_TYPE_STFL, 11, "Configuration-topology
>> facility"),
>> + FEAT_INIT("apqci", S390_FEAT_TYPE_STFL, 12, "Query AP Configuration
>> facility"),
>
> Why did you change this form qci to apqci. Too may people found the
> qci good?
If the facility is called "Query AP Configuration facility" it should be
qapc
Where does the term "qci" come from ?
>
>> FEAT_INIT("ipter", S390_FEAT_TYPE_STFL, 13, "IPTE-range facility"),
>> FEAT_INIT("nonqks", S390_FEAT_TYPE_STFL, 14, "Nonquiescing key-setting
>> facility"),
>> + FEAT_INIT("apft", S390_FEAT_TYPE_STFL, 15, "Adjunct Processor
>> Facilities Test facility"),
>> FEAT_INIT("etf2", S390_FEAT_TYPE_STFL, 16, "Extended-translation
>> facility 2"),
>> FEAT_INIT("msa-base", S390_FEAT_TYPE_STFL, 17,
>> "Message-security-assist facility (excluding subfunctions)"),
>> FEAT_INIT("ldisp", S390_FEAT_TYPE_STFL, 18, "Long-displacement
>> facility"),
>> @@ -129,6 +131,7 @@ static const S390FeatDef s390_features[] = {
>>
>> FEAT_INIT_MISC("dateh2", "DAT-enhancement facility 2"),
>> FEAT_INIT_MISC("cmm", "Collaborative-memory-management facility"),
>> + FEAT_INIT_MISC("ap", "AP facilities installed"),
>
> Why plural ('facilities')? Would not s/facilities/instructions be more
> end-user
> friendly?
>
> Regards,
> Halil
>
--
Thanks,
David / dhildenb
- Re: [Qemu-devel] [PATCH v4 2/5] s390x/ap: base Adjunct Processor (AP) object, (continued)
- [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Tony Krowiak, 2018/04/15
- Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, David Hildenbrand, 2018/04/16
- Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Tony Krowiak, 2018/04/17
- Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Christian Borntraeger, 2018/04/17
- Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Cornelia Huck, 2018/04/18
- Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, David Hildenbrand, 2018/04/18
- Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Tony Krowiak, 2018/04/22
- Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Tony Krowiak, 2018/04/22
Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Halil Pasic, 2018/04/18
Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Tony Krowiak, 2018/04/22
Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Halil Pasic, 2018/04/22
Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Tony Krowiak, 2018/04/22
Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Tony Krowiak, 2018/04/22
Re: [Qemu-devel] [PATCH v4 3/5] s390x/cpumodel: Set up CPU model for AP device support, Halil Pasic, 2018/04/22
[Qemu-devel] [PATCH v4 4/5] s390x/vfio: ap: Introduce VFIO AP device, Tony Krowiak, 2018/04/15