[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops |
Date: |
Fri, 11 Dec 2020 19:54:54 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.5.0 |
On 12/11/20 7:51 PM, Claudio Fontana wrote:
> On 12/11/20 7:26 PM, Philippe Mathieu-Daudé wrote:
>> On 12/11/20 7:22 PM, Richard Henderson wrote:
>>> On 12/11/20 12:15 PM, Claudio Fontana wrote:
>>>> Should I return this file to the original state (without the extra
>>>> #includes that pretend it to be a standalone header file,
>>>> and call it
>>>>
>>>> tcg-cpu-ops.h.inc
>>>>
>>>> ?
>>>
>>> If this header can work with qemu/typedefs.h, then no, because the
>>> circularity
>>> has been resolved. Otherwise, yes.
>>
>> My editor got confused with TranslationBlock, which is why I asked
>> to include its declaration.
>>
>> Easier to forward-declare TranslationBlock in qemu/typedefs.h?
>>
>> Regards,
>>
>> Phil.
>>
>
> Hello Philippe,
>
> ok you propose to move the existing fwd declaration of TranslationBlock from
> cpu.h to qemu/typedefs.h .
I'll let that answer to Richard =)
>
> And what about #include "exec/memattrs.h"?
>
> I assume you propose to put struct MemTxAttrs there as a fwd declaration too,
>
> and this concludes our experiment here?
Well, there is no circular problem here, right?
(and Richard already reviewed patch #24 :P )
>
> Thanks,
>
> Claudio
>
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, (continued)
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Richard Henderson, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Claudio Fontana, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Claudio Fontana, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Richard Henderson, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Claudio Fontana, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Richard Henderson, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Claudio Fontana, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Richard Henderson, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Philippe Mathieu-Daudé, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Claudio Fontana, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops,
Philippe Mathieu-Daudé <=
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Eduardo Habkost, 2020/12/11
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Claudio Fontana, 2020/12/12
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Claudio Fontana, 2020/12/12
- Re: [PATCH v11 18/25] cpu: Move synchronize_from_tb() to tcg_ops, Eduardo Habkost, 2020/12/14
[PATCH v11 10/25] i386: move TCG cpu class initialization out of helper.c, Claudio Fontana, 2020/12/11
[PATCH v11 17/25] accel/tcg: split TCG-only code from cpu_exec_realizefn, Claudio Fontana, 2020/12/11
[PATCH v11 24/25] cpu: move cc->transaction_failed to tcg_ops, Claudio Fontana, 2020/12/11