[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 43/49] qapi: make s390 commands depend on TAR
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH v3 43/49] qapi: make s390 commands depend on TARGET_S390X |
Date: |
Thu, 22 Mar 2018 17:10:23 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 22.03.2018 10:41, Marc-André Lureau wrote:
> Hi
>
> On Thu, Mar 22, 2018 at 6:42 AM, Thomas Huth <address@hidden> wrote:
>> On 21.03.2018 12:52, Marc-André Lureau wrote:
[...]
>>> diff --git a/qmp.c b/qmp.c
>>> index d8f80cb04e..14972b78df 100644
>>> --- a/qmp.c
>>> +++ b/qmp.c
>>> @@ -601,20 +601,6 @@ CpuModelExpansionInfo
>>> *qmp_query_cpu_model_expansion(CpuModelExpansionType type,
>>> return arch_query_cpu_model_expansion(type, model, errp);
>>> }
>>>
>>> -CpuModelCompareInfo *qmp_query_cpu_model_comparison(CpuModelInfo *modela,
>>> - CpuModelInfo *modelb,
>>> - Error **errp)
>>> -{
>>> - return arch_query_cpu_model_comparison(modela, modelb, errp);
>>> -}
>>> -
>>> -CpuModelBaselineInfo *qmp_query_cpu_model_baseline(CpuModelInfo *modela,
>>> - CpuModelInfo *modelb,
>>> - Error **errp)
>>> -{
>>> - return arch_query_cpu_model_baseline(modela, modelb, errp);
>>> -}
>>
>> Not sure, but couldn't these two commands be implemented on other
>> architectures in the long run, too? So removing them now here seems
>> somewhat counterproductive?
>
> They would have modify the qapi ifdef and implement the qmp handler in
> their target, similar to what would be done by implementing arch_query
> handlers. How counterproductive is that? The benefit is that we don't
> have to have stubs and "de-register" the commands.
Yes, thinking about that again, I guess you're right, this should be
fine. I first thought that there would likely be some common code
between the targets finally, but it's more likely that this CPU stuff is
completely different everywhere. So never mind!
Thomas
- [Qemu-devel] [PATCH v3 37/49] qapi: add conditions to REPLICATION type/commands on the schema, (continued)
- [Qemu-devel] [PATCH v3 37/49] qapi: add conditions to REPLICATION type/commands on the schema, Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 38/49] build-sys: move qmp-introspect per target, Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 39/49] qapi-commands: don't initialize command list in qmp_init_marshall(), Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 40/49] RFC: qapi: learn to split schema by 'top-unit', Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 41/49] qapi: add a top-unit 'target' schema, Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 44/49] target.json: add a note about query-cpu* not being s390x-specific, Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 42/49] qapi: make rtc-reset-reinjection and SEV depend on TARGET_I386, Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 43/49] qapi: make s390 commands depend on TARGET_S390X, Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 45/49] qapi: make query-gic-capabilities depend on TARGET_ARM, Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 46/49] qapi: make query-cpu-model-expansion depend on s390 or x86, Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 47/49] qapi: make query-cpu-definitions depend on specific targets, Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 48/49] qapi: remove qmp_unregister_command(), Marc-André Lureau, 2018/03/21
- [Qemu-devel] [PATCH v3 49/49] RFC: make RTC_CHANGE per-target, Marc-André Lureau, 2018/03/21
- Re: [Qemu-devel] [PATCH v3 00/49] qapi: add #if pre-processor conditions to generated code, no-reply, 2018/03/21
- Re: [Qemu-devel] [PATCH v3 00/49] qapi: add #if pre-processor conditions to generated code, no-reply, 2018/03/21
- Re: [Qemu-devel] [PATCH v3 00/49] qapi: add #if pre-processor conditions to generated code, no-reply, 2018/03/21
- Re: [Qemu-devel] [PATCH v3 00/49] qapi: add #if pre-processor conditions to generated code, no-reply, 2018/03/22