[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v14 15/22] cpu: tcg_ops: move to tcg-cpu-ops.h, keep a pointe
From: |
Claudio Fontana |
Subject: |
Re: [PATCH v14 15/22] cpu: tcg_ops: move to tcg-cpu-ops.h, keep a pointer in CPUClass |
Date: |
Tue, 2 Feb 2021 11:27:01 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 |
On 2/2/21 11:01 AM, Claudio Fontana wrote:
> On 1/29/21 1:31 AM, Richard Henderson wrote:
>> On 1/27/21 11:28 PM, Claudio Fontana wrote:
>>> + /*
>>> + * NB: this should be covered by CONFIG_TCG, but it is unsafe to do it
>>> here,
>>> + * as this header is included by both ss_specific and ss_common code,
>>> + * leading to potential differences in the data structure between
>>> modules.
>>> + * We could always keep it last, but it seems safer to just leave this
>>> + * pointer NULL for non-TCG.
>>> + */
>>> + struct TCGCPUOps *tcg_ops;
>>
>> Sorry, I'm going to unqueue the patch set.
>>
>> I first thought this was fixing up something done already, fixing an
>> existing bug.
>>
>> But it's something done in patch 1, and therefore the patch set needs to be
>> re-worked to use this pointer to begin, for the exact reasons detailed above.
>> Otherwise it would appear this breaks bisection.
>>
>>
>> r~
>>
>
> Hello Richard,
>
> I reworked this as suggested by Eduardo:, by not making the fields
> conditional at all in the first part of the series.
>
> This should remove any doubt about potential issues. In no case is bisection
> compromised.
>
> Are you ok with this?
>
> Thanks,
>
> Claudio
>
Respinned with this change as v15:
https://lists.gnu.org/archive/html/qemu-devel/2021-02/msg00045.html
Thanks,
Claudio