[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: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder |
Date: |
Wed, 14 Dec 2016 11:33:40 +0000 |
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 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.
(My gut feeling here is "meh", but that was also how I felt about
Paolo's rearrangement of everything into subdirectories of hw/,
and in hindsight that was clearly the right thing. So I'm assuming
my gut is wrong here too.)
thanks
-- PMM
- [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder, Thomas Huth, 2016/12/14
- Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder, Markus Armbruster, 2016/12/14
- Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder, Thomas Huth, 2016/12/14
- Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder, Markus Armbruster, 2016/12/14
- Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder, Paolo Bonzini, 2016/12/14
- Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder, Eduardo Habkost, 2016/12/14
- Re: [Qemu-devel] [PATCH for-2.9 v2] Move target-* CPU file into a target/ folder, Paolo Bonzini, 2016/12/14