[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder |
Date: |
Wed, 14 Dec 2016 13:05:11 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 |
On 14.12.2016 12:33, Peter Maydell wrote:
> On 14 December 2016 at 11:21, Markus Armbruster <address@hidden> wrote:
>> Thomas Huth <address@hidden> writes:
>>
>>> 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.
>>> To disburden the main folder a little bit, let's move the target-xxx
>>> folders into a dedicated target/ folder, so that target-xxx/ simply
>>> becomes target/xxx/ instead.
>>
>> The root directory is crowded, but hardly because of its 18 target-*
>> entries:
>>
>> $ git-ls-tree --name-only v2.8.0-rc3 | wc -l
>> 167
>>
>> Getting the 18 target-* out of the way would help a little, but I hate
>> making the directory tree deeper.
We already got folders in folders in a lot of places, so this is not
increasing the maximum depths ;-)
>> We have 110 files in the root directory. Moving some of them to
>> subdirectories looks more promising to me.
>
> True, but Thomas has done the work here already, so I don't think
> we lose anything by merging it rather than throwing it away.
Yes, and please consider that there are more targets coming soon ...
RISC-V, AVR, ... and I am pretty sure that we won't stop there.
With regards to the other *.c files in the main directory, I agree with
Markus, we should move some of them to suitable subdirectories, too. I
could have a look at that when I've got some spare minutes...
Thomas