From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54025) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d2jm3-000354-CP for qemu-devel@nongnu.org; Mon, 24 Apr 2017 15:35:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d2jlz-00047d-B6 for qemu-devel@nongnu.org; Mon, 24 Apr 2017 15:35:51 -0400 Received: from mx1.redhat.com ([209.132.183.28]:44988) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1d2jlz-00046g-1S for qemu-devel@nongnu.org; Mon, 24 Apr 2017 15:35:47 -0400 Date: Mon, 24 Apr 2017 16:35:43 -0300 From: Eduardo Habkost Message-ID: <20170424193543.GD3482@thinpad.lan.raisama.net> References: <1482255793-19057-1-git-send-email-ehabkost@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [RFC 0/7] Move accel, KVM, Xen, qtest files to accel/ subdir List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Thomas Huth Cc: qemu-devel@nongnu.org, Stefano Stabellini , kvm@vger.kernel.org, xen-devel@lists.xensource.com, Anthony Perard , Paolo Bonzini , Christoffer Dall On Mon, Apr 24, 2017 at 12:40:07PM +0200, Thomas Huth wrote: > On 20.12.2016 18:43, Eduardo Habkost wrote: > > This moves the KVM and Xen files to the an accel/ subdir. > > > > Instead of moving the *-stubs.c file to accel/ as-is, I tried to > > move most of the stub code to libqemustub.a. This way the obj-y > > logic for accel/ is simpler: obj-y includes accel/ only if > > CONFIG_SOFTMMU is set. > > > > The Xen stubs could be moved completely to stubs/, but some of > > the KVM stubs depend on cpu.h. So most of the kvm-stub.c code was > > moved to stubs/kvm.c, but some of that code was kept in > > accel/kvm-stub.c. > > > > About TCG: > > ---------- > > > > It is not obvious to me which TCG-related files could be moved to > > accel/, so this series don't move any of them yet. > > > > About other CONFIG_SOFTMMU top-level files: > > ------------------------------------------- > > > > I would like to know what we should do with the top-level > > CONFIG_SOFTMMU-only files that don't belong to hw/. Some > > candidates: arch_init.c cpus.c monitor.c gdbstub.c balloon.c > > ioport.c bootdevice.c memory.c cputlb.c memory_mapping.c dump.c. > > > > Maybe a sysemu/ subdir? In that case, should we still create an > > accel/ subdir, or move xen-*, kvm-* and friends to sysemu/ too? > > > > Cc: Paolo Bonzini > > Cc: kvm@vger.kernel.org > > Cc: Christoffer Dall > > Cc: Anthony Perard > > Cc: Stefano Stabellini > > Cc: xen-devel@lists.xensource.com > > > > Eduardo Habkost (7): > > xen: Move xen-*-stub.c to stubs/ > > xen: Move xen files to accel/ > > kvm: Move some kvm-stub.c code to stubs/kvm.c > > kvm: Include kvm-stub.o only on CONFIG_SOFTMMU > > kvm: Move kvm*.c files to accel/ > > accel: Move accel.c to accel/ > > accel: Move qtest.c to accel/ > > > > Makefile.objs | 2 +- > > Makefile.target | 10 ++---- > > accel.c => accel/accel.c | 0 > > kvm-all.c => accel/kvm-common.c | 0 > > kvm-stub.c => accel/kvm-stub.c | 51 -------------------------- > > qtest.c => accel/qtest.c | 0 > > xen-common.c => accel/xen-common.c | 0 > > xen-hvm.c => accel/xen-hvm.c | 0 > > xen-mapcache.c => accel/xen-mapcache.c | 0 > > stubs/kvm.c | 65 ++++++++++++++++++++++++++++++++++ > > xen-hvm-stub.c => stubs/xen-hvm.c | 0 > > xen-common-stub.c => stubs/xen.c | 0 > > MAINTAINERS | 4 +-- > > accel/Makefile.objs | 9 +++++ > > stubs/Makefile.objs | 2 ++ > > 15 files changed, 80 insertions(+), 63 deletions(-) > > rename accel.c => accel/accel.c (100%) > > rename kvm-all.c => accel/kvm-common.c (100%) > > rename kvm-stub.c => accel/kvm-stub.c (71%) > > rename qtest.c => accel/qtest.c (100%) > > rename xen-common.c => accel/xen-common.c (100%) > > rename xen-hvm.c => accel/xen-hvm.c (100%) > > rename xen-mapcache.c => accel/xen-mapcache.c (100%) > > rename xen-hvm-stub.c => stubs/xen-hvm.c (100%) > > rename xen-common-stub.c => stubs/xen.c (100%) > > create mode 100644 accel/Makefile.objs > > Now that the development tree is open again ... any chance that we could > get this series into 2.10 ? I remember there were some suggestions about the code movements, especially about the files being moved inside stubs/. I never took the time to make a v2 implementing those suggestions, so if anybody wants to volunteer to address the feedback on this RFC and redo the series, please be my guest. :) -- Eduardo