From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48001) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YYsRR-0006lT-3o for qemu-devel@nongnu.org; Fri, 20 Mar 2015 04:38:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YYsRQ-0005Nw-8q for qemu-devel@nongnu.org; Fri, 20 Mar 2015 04:38:05 -0400 Received: from mx1.redhat.com ([209.132.183.28]:60359) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YYsRP-0005Nq-WE for qemu-devel@nongnu.org; Fri, 20 Mar 2015 04:38:04 -0400 Message-ID: <550BDC55.7080505@redhat.com> Date: Fri, 20 Mar 2015 10:37:41 +0200 From: Marcel Apfelbaum MIME-Version: 1.0 References: <1426791181-23831-1-git-send-email-marcel@redhat.com> <550B4387.1030308@redhat.com> In-Reply-To: <550B4387.1030308@redhat.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH V6 for-2.3 00/26] hw/pc: implement multiple primary busses for pc machines List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini , qemu-devel@nongnu.org Cc: quintela@redhat.com, mst@redhat.com, agraf@suse.de, alex.williamson@redhat.com, kevin@koconnor.net, kraxel@redhat.com, amit.shah@redhat.com, imammedo@redhat.com, rth@twiddle.net, leon.alrae@imgtec.com, aurelien@aurel32.net, hare@suse.de On 03/19/2015 11:45 PM, Paolo Bonzini wrote: > > > On 19/03/2015 19:52, Marcel Apfelbaum wrote: >> hw/pci-host/piix.c | 128 ++++++++- >> hw/pci-host/ppce500.c | 1 + >> hw/pci-host/q35.c | 5 + >> hw/pci-host/uninorth.c | 1 + >> hw/pci/Makefile.objs | 2 +- >> hw/pci/pci.c | 493 +--------------------------------- >> hw/pci/pci_bus.c | 517 ++++++++++++++++++++++++++++++++++++ >> hw/pci/pci_host.c | 42 +++ >> hw/ppc/ppc4xx_pci.c | 1 + >> hw/scsi/megasas.c | 1 + >> hw/sh4/r2d.c | 1 + >> hw/sh4/sh_pci.c | 1 + >> hw/vfio/pci.c | 1 + >> hw/xen/xen_pt.c | 1 + >> include/hw/acpi/aml-build.h | 8 + >> include/hw/i386/pc.h | 8 +- >> include/hw/pci/pci.h | 6 +- >> include/hw/pci/pci_bus.h | 35 +++ >> include/hw/pci/pci_host.h | 34 +++ >> include/sysemu/sysemu.h | 1 + >> 32 files changed, 1508 insertions(+), 510 deletions(-) > > I think this should not be committed to 2.3 after -rc0. While I agree that looking at the above the series do not fit for 2.3, I still think the risk is not big because: - The biggest amount of modifications are in pci_bus/pci_host and *is only* a movement of code into a new file, no changes at all (This cover almost half of the series changes) - The series does not intervene with usual process, meaning if pxb is not used, it will work exactly like before - The bios already supports pxb for this release, is a pity to wait for another QEMU version. - The series were in mailing list for more than a month before rc, but the real review process started at rc time. (begging the question what a non-maintainer must do to get its code in time) - The series is mature, rebased on the latest QEMU and was reviewed in depth and tested by the community. The above being said, I'll agree with whatever the maintainers will decide. Thanks, Marcel > > Paolo >