[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device |
Date: |
Wed, 3 Apr 2013 15:00:41 +0300 |
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.
> >
- [Qemu-devel] [PATCH 4/4] pci: add pci test device, (continued)
- [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Michael S. Tsirkin, 2013/04/03
- Re: [Qemu-devel] [PATCH 4/4] pci: add pci test device, Paolo Bonzini, 2013/04/03