[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [RFC for-2.9 00/11] Move target-* CPU file into a targe

From: Thomas Huth
Subject: Re: [Qemu-devel] [RFC for-2.9 00/11] Move target-* CPU file into a target/ folder
Date: Wed, 30 Nov 2016 11:12:53 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0

On 30.11.2016 11:10, Laurent Vivier wrote:
> Le 30/11/2016 à 10:47, Thomas Huth a écrit :
>> We've currently got 18 architectures in QEMU, and thus 18 target-xxx
>> folders in the root folder of the QEMU source tree. More architectures
>> (e.g. RISC-V, AVR) are likely to be included soon, too, so the main
>> folder of the QEMU sources slowly gets quite overcrowded with the
>> target-xxx folders.
>> Thus I'd like to suggest that we move the target-xxx folders into
>> a dedicated target/ folder, so that target-xxx/ simply becomes
>> target/xxx/ instead.
>> The first patch is the most important one here, it prepares the
>> build system to be able to deal with both, target-xxx and target/xxx
>> folders. Once that has been applied, each target architecture
>> could be moved to the target/ folder when it is convenient for the
>> subsytem maintainer to do the move (i.e. we do not have to move all
>> files immediately in one go). So if we agree to do this change and
>> the first patch has been committed, I could rather send the following
>> patches step by step to the maintainers so they can queue the patch
>> for their subsystem for a good point in time. And new architectures
>> like RISC-V and AVR could start in the target/ folder immediately
>> instead.
>> So please provide some feedback: Is this a good idea? Or do you
>> consider this rather just as unnecessary code churn instead?
> IMHO, it seems a good idea.
> Nevertheless, I'm not sure to be able to manage both in the makefile is
> a good idea: it can become a little bit messy.

That's of course only a temporary solution. Once all targets have been
moved, the Makefile should be changed to only look into the target/xxx


reply via email to

[Prev in Thread] Current Thread [Next in Thread]