[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v26 00/20] i386 cleanup PART 2
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v26 00/20] i386 cleanup PART 2 |
Date: |
Wed, 5 May 2021 14:24:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 |
On 5/5/21 2:15 PM, Philippe Mathieu-Daudé wrote:
> On 5/5/21 12:04 PM, Alex Bennée wrote:
>> Claudio Fontana <cfontana@suse.de> writes:
>>> On 3/8/21 3:02 PM, Alex Bennée wrote:
>>>> Claudio Fontana <cfontana@suse.de> writes:
>>>>
>>>>> Hi,
>>>>>
>>>>> anything else for me to do here?
>>>>
>>>> It looks to me that this series is looking pretty good. Every patch has
>>>> at least one review so I think it's just waiting on the maintainers to
>>>> pick it up.
>>>>
>>>> Paolo/Richard - are you intending to take the series as is or are you
>>>> waiting for something else? I'd like to see the patch delta reduced for
>>>> the ARM cleanup work which is still ongoing.
>>>
>>> I am a bit at a loss here, as this has been reviewed for a while, but
>>> nothing is happening.
>>> Rebasing is starting to become more and more draining;
> I didn't noticed the following patch had its content changed:
> Revert "target/arm: Restrict v8M IDAU to TCG"
> So now this is not a full revert, only the TYPE_IDAU_INTERFACE
> type is moved back.
Correcting myself. This is a plain revert, and I disagree with it
because if TCG is not available, TCG-specific devices/machines
shouldn't be loaded into QEMU. Cortex-M and v8M IDAU *are* TCG
specific and no such object should be built/registered, so the
interface shouldn't be there neither.
Below is a possible way to get there. Reverting the commit and
build Cortex-M devices in a non-TCG build is a kludge / shortcut
which allows Claudio to remove a big chunk of non-TCG code, so
is probably acceptable, and we can keep cleaning later.
> More generally I think more code should be automatically stripped
> out by Kconfig instead. In [1,2] I suggested to tie accel-specific
> Kconfig selectors:
>
> config ARM_V7R
> bool
> depends on TCG && ARM
>
> config ARM_V7M
> bool
> depends on TCG && ARM
> select PTIMER
>
> config XLNX_ZYNQMP_ARM
> bool
> default y if TCG && AARCH64
>
> But this can certainly be done on top of Claudio's work.
>
> [1] https://www.mail-archive.com/qemu-devel@nongnu.org/msg777710.html
> [2] https://www.mail-archive.com/qemu-devel@nongnu.org/msg777719.html
>
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Claudio Fontana, 2021/05/05
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Alex Bennée, 2021/05/05
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Philippe Mathieu-Daudé, 2021/05/05
- Re: [PATCH v26 00/20] i386 cleanup PART 2,
Philippe Mathieu-Daudé <=
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Eduardo Habkost, 2021/05/05
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Claudio Fontana, 2021/05/06
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Philippe Mathieu-Daudé, 2021/05/06
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Philippe Mathieu-Daudé, 2021/05/06
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Philippe Mathieu-Daudé, 2021/05/12
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Claudio Fontana, 2021/05/17
- Re: [PATCH v26 00/20] i386 cleanup PART 2, Philippe Mathieu-Daudé, 2021/05/19