From: Collin Walling <walling@linux.ibm.com>
To: Markus Armbruster <armbru@redhat.com>
Cc: qemu-s390x@nongnu.org, qemu-devel@nongnu.org, thuth@redhat.com,
david@redhat.com, wangyanan55@huawei.com, philmd@linaro.org,
marcel.apfelbaum@gmail.com, eduardo@habkost.net
Subject: Re: [PATCH v3 1/2] target/s390x: report deprecated-props in cpu-model-expansion reply
Date: Fri, 26 Apr 2024 16:11:56 -0400 [thread overview]
Message-ID: <f966a24e-dd7e-408c-b394-cc8650786713@linux.ibm.com> (raw)
In-Reply-To: <8c265c38-d9aa-43bf-a0b6-f70a52c5acf4@linux.ibm.com>
On 4/26/24 13:35, Collin Walling wrote:
> On 4/26/24 04:42, Markus Armbruster wrote:
>> Collin Walling <walling@linux.ibm.com> writes:
>>
>>> Retain a list of deprecated features disjoint from any particular
>>> CPU model. A query-cpu-model-expansion reply will now provide a list of
>>> properties (i.e. features) that are flagged as deprecated. Example:
>>>
>>> {
>>> "return": {
>>> "model": {
>>> "name": "z14.2-base",
>>> "deprecated-props": [
>>> "bpb",
>>> "csske"
>>> ],
>>> "props": {
>>> "pfmfi": false,
>>> "exrl": true,
>>> ...a lot more props...
>>> "skey": false,
>>> "vxpdeh2": false
>>> }
>>> }
>>> }
>>> }
>>>
>>> It is recommended that s390 guests operate with these features
>>> explicitly disabled to ensure compatability with future hardware.
>>>
>>> Signed-off-by: Collin Walling <walling@linux.ibm.com>
>>> ---
>>> qapi/machine-target.json | 5 ++++-
>>> target/s390x/cpu_features.c | 14 ++++++++++++++
>>> target/s390x/cpu_features.h | 1 +
>>> target/s390x/cpu_models_sysemu.c | 6 ++++++
>>> 4 files changed, 25 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/qapi/machine-target.json b/qapi/machine-target.json
>>> index 29e695aa06..3799a60e3d 100644
>>> --- a/qapi/machine-target.json
>>> +++ b/qapi/machine-target.json
>>> @@ -20,11 +20,14 @@
>>> #
>>> # @props: a dictionary of QOM properties to be applied
>>> #
>>> +# @deprecated-props: a list of QOM properties that are flagged as deprecated
>>
>> Deprecated by whom? QEMU? The CPU vendor?
>>
>
> The CPU vendor would be the one who decides which props are deprecated.
> How about:
>
> # @deprecated-props: a list of QOM properties that are flagged as
> deprecated by the CPU vendor
>
^ let's ignore the incorrect indentation here.
Actually, I may be wildly incorrect with my description by referring to
this as "a list of QOM properties", when in fact this is just an array
of strings. Also, the deprecated props may not always reflect the
features that are found by a static expansion, so I added another
sentence to describe that they are a part of a full model expansion.
# @deprecated-props: a list of properties that are flagged as deprecated
# by the CPU vendor. These props are a subset of the model's full
# definition list of properties. (since X.Y)
I may need some help with the wording on the 2nd sentence.
[...]
>
>>> +#
>>> # Since: 2.8
>>> ##
>>> { 'struct': 'CpuModelInfo',
>>> 'data': { 'name': 'str',
>>> - '*props': 'any' } }
>>> + '*props': 'any',
>>> + '*deprecated-props': ['str'] } }
>>>
>>> ##
>>> # @CpuModelExpansionType:
>>
>> [...]
>>
>>
>
--
Regards,
Collin
next prev parent reply other threads:[~2024-04-26 20:15 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-24 21:56 [PATCH v3 0/2] query-cpu-model-expansion: report deprecated features Collin Walling
2024-04-24 21:56 ` [PATCH v3 1/2] target/s390x: report deprecated-props in cpu-model-expansion reply Collin Walling
2024-04-26 8:42 ` Markus Armbruster
2024-04-26 17:35 ` Collin Walling
2024-04-26 20:11 ` Collin Walling [this message]
2024-04-26 17:44 ` David Hildenbrand
2024-04-26 17:45 ` David Hildenbrand
2024-04-26 19:28 ` Collin Walling
2024-04-24 21:56 ` [PATCH v3 2/2] target/s390x: flag te and cte as deprecated Collin Walling
2024-04-24 22:05 ` [PATCH v3 0/2] query-cpu-model-expansion: report deprecated features Collin Walling
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f966a24e-dd7e-408c-b394-cc8650786713@linux.ibm.com \
--to=walling@linux.ibm.com \
--cc=armbru@redhat.com \
--cc=david@redhat.com \
--cc=eduardo@habkost.net \
--cc=marcel.apfelbaum@gmail.com \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=thuth@redhat.com \
--cc=wangyanan55@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.