From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:59350) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UNMBR-0006HH-4T for qemu-devel@nongnu.org; Wed, 03 Apr 2013 07:48:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UNMBM-0005Lb-8g for qemu-devel@nongnu.org; Wed, 03 Apr 2013 07:48:53 -0400 Received: from mx1.redhat.com ([209.132.183.28]:25377) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UNMBL-0005LT-Vp for qemu-devel@nongnu.org; Wed, 03 Apr 2013 07:48:48 -0400 Message-ID: <515C1719.903@redhat.com> Date: Wed, 03 Apr 2013 13:48:41 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <515BF657.7000904@redhat.com> <20130403094529.GA18179@redhat.com> <515BFC37.4000802@redhat.com> <20130403102229.GD18179@redhat.com> <515C037E.5030706@redhat.com> <20130403103301.GG18179@redhat.com> <515C05B0.2000401@redhat.com> <20130403103846.GB18725@redhat.com> In-Reply-To: <20130403103846.GB18725@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: gleb@redhat.com, mtosatti@redhat.com, qemu-devel@nongnu.org, kvm@vger.kernel.org Il 03/04/2013 12:38, Michael S. Tsirkin ha scritto: > On Wed, Apr 03, 2013 at 12:34:24PM +0200, Paolo Bonzini wrote: >> Il 03/04/2013 12:33, Michael S. Tsirkin ha scritto: >>>> Because someone wasn't looking. :) The hw/ reorganization patches I've >>>> posted fix that. >>> >>> Still not sure how to merge this, if it goes in through my tree >>> and that's the only comment, I'll just fix it silently ... >> >> Indeed. In fact, considering my hw/ reorganization patches will be >> committed soon, please put it in hw/pci (matching hw/isa/pc-testdev.c). > > Paolo, hw/pci is pci core, I haven't looked at your reorg patches, > but please do not move devices there. > Sorting devices by connection is also wrong I think, by function > would be better. Indeed that's how most devices are sorted. For example, PCI host devices/bridges/etc. are in hw/pci (together with the PCI core, making hw/pci basically all that goes in through your tree), ISA host devices are in hw/isa, etc. However, there are a few exceptions. You cannot really sort out 600 files without exceptions. All USB devices are in hw/usb, and the existing test devices (debugexit, testdev) are in hw/isa. There is only one exception you should care about, namely that VFIO and ivshmem are also in hw/pci. Here is the list of files in hw/pci: host-apb.c host-bonito.c host-dec.c host-dec.h host-grackle.c host-gt64xxx.c host-piix.c host-ppc4xx.c host-ppce500.c host-prep.c host-q35.c host-sh.c host-spapr.c host-uninorth.c host-versatile.c i82801b11.c ioh3420.c ioh3420.h ivshmem.c msi.c msi.h msix.c msix.h pam.c pci-hotplug.c pci-stub.c pci.c pci.h pci_bridge.c pci_bridge.h pci_bridge_dev.c pci_bus.h pci_host.c pci_host.h pci_ids.h pci_regs.h pcie.c pcie.h pcie_aer.c pcie_aer.h pcie_host.c pcie_host.h pcie_port.c pcie_port.h pcie_regs.h shpc.c shpc.h slotid_cap.c slotid_cap.h vfio.c xio3130_downstream.c xio3130_downstream.h xio3130_upstream.c xio3130_upstream.h Paolo > >> Another comment is that you should add a description in docs/specs. >> >> Paolo > > pc-test has none ... I'll see what I can do. >