From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:33763) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UNMMA-0004n6-D2 for qemu-devel@nongnu.org; Wed, 03 Apr 2013 08:00:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UNMM1-0001D2-2g for qemu-devel@nongnu.org; Wed, 03 Apr 2013 07:59:58 -0400 Received: from mx1.redhat.com ([209.132.183.28]:12113) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UNMM0-0001Cn-Rg for qemu-devel@nongnu.org; Wed, 03 Apr 2013 07:59:49 -0400 Date: Wed, 3 Apr 2013 15:00:41 +0300 From: "Michael S. Tsirkin" Message-ID: <20130403120041.GA22200@redhat.com> 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> <515C1719.903@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <515C1719.903@redhat.com> 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: Paolo Bonzini Cc: gleb@redhat.com, mtosatti@redhat.com, qemu-devel@nongnu.org, kvm@vger.kernel.org On Wed, Apr 03, 2013 at 01:48:41PM +0200, Paolo Bonzini wrote: > 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), Well host bridges often do lots of things besides pci on the same chip. > ISA host devices > are in hw/isa, etc. what do you mean "host ISA device"? > 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. This makes no sense really. Pls add hw/misc or just keep misc stuff in hw/ We don't need to have everything in subdirectories. > 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 This messes up things. pci core is separate from devices using it, and it's important to me. Really just add hw/bridge/ and put all kind of bridge devices there. > > > >> Another comment is that you should add a description in docs/specs. > >> > >> Paolo > > > > pc-test has none ... I'll see what I can do. > >