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

Reply via email to