Patchwork [PULL] pci,net,misc infrastructure

login
register
mail settings
Submitter Michael S. Tsirkin
Date Dec. 13, 2012, 7:26 a.m.
Message ID <20121213072606.GA833@redhat.com>
Download mbox
Permalink /patch/205772/
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony

Comments

Michael S. Tsirkin - Dec. 13, 2012, 7:26 a.m.
The following changes since commit 1c97e303d4ea80a2691334b0febe87a50660f99d:

  Merge remote-tracking branch 'afaerber/qom-cpu' into staging (2012-12-10 08:35:15 -0600)

are available in the git repository at:


  git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony

for you to fetch changes up to 5c1ad98d71923b83a530e5db4c2110564b84e11d:

  pci_bus.h: tweak include guards (2012-12-12 23:41:04 +0200)

----------------------------------------------------------------
pci,net,misc infrastructure

This rearranges pci core into hw/pci/
so it's separate from actual devices.
Also included are a couple of misc fixes:
    - a fix for tap bug reported by alex on kvm forum
    - a couple of misc updates: get_maintainer.pl,
      and license update for the q35 files

I've addressed the comments Blue Swirl had
for the rename patches - would like to merge ASAP
to avoid any conflicts, and we can do code style
fixups later - note this is style of existing files.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

----------------------------------------------------------------
Jason Baron (1):
      Fixup q35/ich9 Licenses

Michael S. Tsirkin (12):
      tap: reset vnet header size on open
      get_maintainer.pl: update from linix 3.8
      pci: prepare makefiles for pci code reorganization
      MAINTAINERS: add hw/pci/ to list of PCI files
      pci: move pci core code to hw/pci
      pci: update all users to look in pci/
      pci: fix path for include paths
      Revert "pci: prepare makefiles for pci code reorganization"
      pci: rename pci_internals.h pci_bus.h
      pci_bus: update comment
      msi: add API to get notified about pending bit poll
      pci_bus.h: tweak include guards

 MAINTAINERS                           |  1 +
 arch_init.c                           |  2 +-
 hw/Makefile.objs                      | 10 ++-------
 hw/ac97.c                             |  2 +-
 hw/acpi_ich9.c                        | 15 +++++++------
 hw/acpi_piix4.c                       |  2 +-
 hw/alpha_sys.h                        |  4 ++--
 hw/apb_pci.c                          |  8 +++----
 hw/apic.c                             |  2 +-
 hw/bonito.c                           |  4 ++--
 hw/cirrus_vga.c                       |  2 +-
 hw/dec_pci.c                          |  8 +++----
 hw/e1000.c                            |  2 +-
 hw/eepro100.c                         |  2 +-
 hw/es1370.c                           |  2 +-
 hw/esp-pci.c                          |  2 +-
 hw/grackle_pci.c                      |  4 ++--
 hw/gt64xxx.c                          |  4 ++--
 hw/hda-audio.c                        |  2 +-
 hw/i386/Makefile.objs                 |  2 +-
 hw/i82378.c                           |  2 +-
 hw/i82801b11.c                        |  2 +-
 hw/ich9.h                             |  8 +++----
 hw/ide.h                              |  2 +-
 hw/ide/ahci.c                         |  4 ++--
 hw/ide/cmd646.c                       |  2 +-
 hw/ide/core.c                         |  2 +-
 hw/ide/ich.c                          |  4 ++--
 hw/ide/pci.c                          |  2 +-
 hw/ide/piix.c                         |  2 +-
 hw/ide/via.c                          |  2 +-
 hw/intel-hda.c                        |  4 ++--
 hw/ioh3420.c                          |  6 +++---
 hw/ioh3420.h                          |  2 +-
 hw/ivshmem.c                          |  4 ++--
 hw/kvm/apic.c                         |  2 +-
 hw/kvm/pci-assign.c                   |  4 ++--
 hw/lpc_ich9.c                         | 40 +++++++++++------------------------
 hw/lsi53c895a.c                       |  2 +-
 hw/macio.c                            |  2 +-
 hw/megasas.c                          |  4 ++--
 hw/mips_fulong2e.c                    |  2 +-
 hw/mips_malta.c                       |  2 +-
 hw/ne2000.c                           |  2 +-
 hw/openpic.c                          |  2 +-
 hw/pc.c                               |  4 ++--
 hw/pc_piix.c                          |  4 ++--
 hw/pci/Makefile.objs                  |  6 ++++++
 hw/{ => pci}/msi.c                    |  2 +-
 hw/{ => pci}/msi.h                    |  2 +-
 hw/{ => pci}/msix.c                   | 19 ++++++++++++-----
 hw/{ => pci}/msix.h                   |  5 +++--
 hw/{ => pci}/pci-hotplug.c            | 12 +++++------
 hw/{ => pci}/pci-stub.c               |  2 +-
 hw/{ => pci}/pci.c                    | 14 ++++++------
 hw/{ => pci}/pci.h                    | 14 +++++++-----
 hw/{ => pci}/pci_bridge.c             |  4 ++--
 hw/{ => pci}/pci_bridge.h             |  2 +-
 hw/{pci_internals.h => pci/pci_bus.h} | 16 ++++++--------
 hw/{ => pci}/pci_host.c               |  4 ++--
 hw/{ => pci}/pci_host.h               |  2 +-
 hw/{ => pci}/pci_ids.h                |  0
 hw/{ => pci}/pci_regs.h               |  0
 hw/{ => pci}/pcie.c                   | 12 +++++------
 hw/{ => pci}/pcie.h                   |  8 +++----
 hw/{ => pci}/pcie_aer.c               | 12 +++++------
 hw/{ => pci}/pcie_aer.h               |  2 +-
 hw/{ => pci}/pcie_host.c              |  6 +++---
 hw/{ => pci}/pcie_host.h              |  2 +-
 hw/{ => pci}/pcie_port.c              |  2 +-
 hw/{ => pci}/pcie_port.h              |  4 ++--
 hw/{ => pci}/pcie_regs.h              |  0
 hw/{ => pci}/shpc.c                   |  8 +++----
 hw/{ => pci}/shpc.h                   |  0
 hw/{ => pci}/slotid_cap.c             |  4 ++--
 hw/{ => pci}/slotid_cap.h             |  0
 hw/pci_bridge_dev.c                   | 12 +++++------
 hw/pcnet-pci.c                        |  2 +-
 hw/piix4.c                            |  2 +-
 hw/piix_pci.c                         |  4 ++--
 hw/ppc/Makefile.objs                  |  2 +-
 hw/ppc/e500.c                         |  2 +-
 hw/ppc440_bamboo.c                    |  2 +-
 hw/ppc4xx.h                           |  2 +-
 hw/ppc4xx_pci.c                       |  4 ++--
 hw/ppc_newworld.c                     |  2 +-
 hw/ppc_oldworld.c                     |  2 +-
 hw/ppc_prep.c                         |  4 ++--
 hw/ppce500_pci.c                      |  4 ++--
 hw/prep_pci.c                         |  4 ++--
 hw/q35.h                              |  4 ++--
 hw/qxl.h                              |  2 +-
 hw/r2d.c                              |  2 +-
 hw/realview.c                         |  2 +-
 hw/rtl8139.c                          |  2 +-
 hw/serial-pci.c                       |  2 +-
 hw/sga.c                              |  2 +-
 hw/sh_pci.c                           |  4 ++--
 hw/smbus_ich9.c                       | 16 ++++++++------
 hw/spapr.c                            |  4 ++--
 hw/spapr_pci.c                        | 10 ++++-----
 hw/spapr_pci.h                        |  4 ++--
 hw/sun4u.c                            |  2 +-
 hw/unin_pci.c                         |  4 ++--
 hw/usb/hcd-ehci-pci.c                 |  2 +-
 hw/usb/hcd-ohci.c                     |  2 +-
 hw/usb/hcd-uhci.c                     |  2 +-
 hw/usb/hcd-xhci.c                     |  6 +++---
 hw/versatile_pci.c                    |  4 ++--
 hw/versatilepb.c                      |  2 +-
 hw/vfio_pci.c                         |  8 +++----
 hw/vga-pci.c                          |  2 +-
 hw/vga.c                              |  2 +-
 hw/virtio-balloon.h                   |  2 +-
 hw/virtio-net.h                       |  2 +-
 hw/virtio-pci.c                       |  9 ++++----
 hw/virtio-scsi.h                      |  2 +-
 hw/vmware_vga.c                       |  2 +-
 hw/vt82c686.c                         |  4 ++--
 hw/wdt_i6300esb.c                     |  2 +-
 hw/xen-host-pci-device.h              |  2 +-
 hw/xen_apic.c                         |  2 +-
 hw/xen_platform.c                     |  2 +-
 hw/xen_pt.c                           |  2 +-
 hw/xen_pt.h                           |  2 +-
 hw/xio3130_downstream.c               |  6 +++---
 hw/xio3130_downstream.h               |  2 +-
 hw/xio3130_upstream.c                 |  6 +++---
 hw/xio3130_upstream.h                 |  2 +-
 kvm-all.c                             |  2 +-
 kvm-stub.c                            |  2 +-
 monitor.c                             |  2 +-
 net/tap-linux.c                       |  8 +++++++
 scripts/get_maintainer.pl             | 25 ++++++++++++++--------
 target-i386/kvm.c                     |  2 +-
 xen-all.c                             |  2 +-
 136 files changed, 303 insertions(+), 289 deletions(-)
 create mode 100644 hw/pci/Makefile.objs
 rename hw/{ => pci}/msi.c (99%)
 rename hw/{ => pci}/msi.h (98%)
 rename hw/{ => pci}/msix.c (96%)
 rename hw/{ => pci}/msix.h (93%)
 rename hw/{ => pci}/pci-hotplug.c (98%)
 rename hw/{ => pci}/pci-stub.c (98%)
 rename hw/{ => pci}/pci.c (99%)
 rename hw/{ => pci}/pci.h (98%)
 rename hw/{ => pci}/pci_bridge.c (99%)
 rename hw/{ => pci}/pci_bridge.h (98%)
 rename hw/{pci_internals.h => pci/pci_bus.h} (79%)
 rename hw/{ => pci}/pci_host.c (99%)
 rename hw/{ => pci}/pci_host.h (99%)
 rename hw/{ => pci}/pci_ids.h (100%)
 rename hw/{ => pci}/pci_regs.h (100%)
 rename hw/{ => pci}/pcie.c (99%)
 rename hw/{ => pci}/pcie.h (98%)
 rename hw/{ => pci}/pcie_aer.c (99%)
 rename hw/{ => pci}/pcie_aer.h (99%)
 rename hw/{ => pci}/pcie_host.c (98%)
 rename hw/{ => pci}/pcie_host.h (98%)
 rename hw/{ => pci}/pcie_port.c (99%)
 rename hw/{ => pci}/pcie_port.h (96%)
 rename hw/{ => pci}/pcie_regs.h (100%)
 rename hw/{ => pci}/shpc.c (99%)
 rename hw/{ => pci}/shpc.h (100%)
 rename hw/{ => pci}/slotid_cap.c (95%)
 rename hw/{ => pci}/slotid_cap.h (100%)
Anthony Liguori - Dec. 13, 2012, 8:31 p.m.
"Michael S. Tsirkin" <mst@redhat.com> writes:

> The following changes since commit 1c97e303d4ea80a2691334b0febe87a50660f99d:
>
>   Merge remote-tracking branch 'afaerber/qom-cpu' into staging (2012-12-10 08:35:15 -0600)
>
> are available in the git repository at:
>
>
>   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
>
> for you to fetch changes up to 5c1ad98d71923b83a530e5db4c2110564b84e11d:
>
>   pci_bus.h: tweak include guards (2012-12-12 23:41:04 +0200)

Doesn't build:

  CC    hw/apm.o
/home/anthony/git/qemu/hw/apm.c:25:17: fatal error: pci.h: No such file or directory
compilation terminated.

And there's really no good reason for this.  apm's part of target-i386
so you couldn't have built this prior to doing a pull request.

Please setup your tree in buildbot and wait for a full run before
sending a pull requests in the future.

Regards,

Anthony Liguori

>
> ----------------------------------------------------------------
> pci,net,misc infrastructure
>
> This rearranges pci core into hw/pci/
> so it's separate from actual devices.
> Also included are a couple of misc fixes:
>     - a fix for tap bug reported by alex on kvm forum
>     - a couple of misc updates: get_maintainer.pl,
>       and license update for the q35 files
>
> I've addressed the comments Blue Swirl had
> for the rename patches - would like to merge ASAP
> to avoid any conflicts, and we can do code style
> fixups later - note this is style of existing files.
>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
>
> ----------------------------------------------------------------
> Jason Baron (1):
>       Fixup q35/ich9 Licenses
>
> Michael S. Tsirkin (12):
>       tap: reset vnet header size on open
>       get_maintainer.pl: update from linix 3.8
>       pci: prepare makefiles for pci code reorganization
>       MAINTAINERS: add hw/pci/ to list of PCI files
>       pci: move pci core code to hw/pci
>       pci: update all users to look in pci/
>       pci: fix path for include paths
>       Revert "pci: prepare makefiles for pci code reorganization"
>       pci: rename pci_internals.h pci_bus.h
>       pci_bus: update comment
>       msi: add API to get notified about pending bit poll
>       pci_bus.h: tweak include guards
>
>  MAINTAINERS                           |  1 +
>  arch_init.c                           |  2 +-
>  hw/Makefile.objs                      | 10 ++-------
>  hw/ac97.c                             |  2 +-
>  hw/acpi_ich9.c                        | 15 +++++++------
>  hw/acpi_piix4.c                       |  2 +-
>  hw/alpha_sys.h                        |  4 ++--
>  hw/apb_pci.c                          |  8 +++----
>  hw/apic.c                             |  2 +-
>  hw/bonito.c                           |  4 ++--
>  hw/cirrus_vga.c                       |  2 +-
>  hw/dec_pci.c                          |  8 +++----
>  hw/e1000.c                            |  2 +-
>  hw/eepro100.c                         |  2 +-
>  hw/es1370.c                           |  2 +-
>  hw/esp-pci.c                          |  2 +-
>  hw/grackle_pci.c                      |  4 ++--
>  hw/gt64xxx.c                          |  4 ++--
>  hw/hda-audio.c                        |  2 +-
>  hw/i386/Makefile.objs                 |  2 +-
>  hw/i82378.c                           |  2 +-
>  hw/i82801b11.c                        |  2 +-
>  hw/ich9.h                             |  8 +++----
>  hw/ide.h                              |  2 +-
>  hw/ide/ahci.c                         |  4 ++--
>  hw/ide/cmd646.c                       |  2 +-
>  hw/ide/core.c                         |  2 +-
>  hw/ide/ich.c                          |  4 ++--
>  hw/ide/pci.c                          |  2 +-
>  hw/ide/piix.c                         |  2 +-
>  hw/ide/via.c                          |  2 +-
>  hw/intel-hda.c                        |  4 ++--
>  hw/ioh3420.c                          |  6 +++---
>  hw/ioh3420.h                          |  2 +-
>  hw/ivshmem.c                          |  4 ++--
>  hw/kvm/apic.c                         |  2 +-
>  hw/kvm/pci-assign.c                   |  4 ++--
>  hw/lpc_ich9.c                         | 40 +++++++++++------------------------
>  hw/lsi53c895a.c                       |  2 +-
>  hw/macio.c                            |  2 +-
>  hw/megasas.c                          |  4 ++--
>  hw/mips_fulong2e.c                    |  2 +-
>  hw/mips_malta.c                       |  2 +-
>  hw/ne2000.c                           |  2 +-
>  hw/openpic.c                          |  2 +-
>  hw/pc.c                               |  4 ++--
>  hw/pc_piix.c                          |  4 ++--
>  hw/pci/Makefile.objs                  |  6 ++++++
>  hw/{ => pci}/msi.c                    |  2 +-
>  hw/{ => pci}/msi.h                    |  2 +-
>  hw/{ => pci}/msix.c                   | 19 ++++++++++++-----
>  hw/{ => pci}/msix.h                   |  5 +++--
>  hw/{ => pci}/pci-hotplug.c            | 12 +++++------
>  hw/{ => pci}/pci-stub.c               |  2 +-
>  hw/{ => pci}/pci.c                    | 14 ++++++------
>  hw/{ => pci}/pci.h                    | 14 +++++++-----
>  hw/{ => pci}/pci_bridge.c             |  4 ++--
>  hw/{ => pci}/pci_bridge.h             |  2 +-
>  hw/{pci_internals.h => pci/pci_bus.h} | 16 ++++++--------
>  hw/{ => pci}/pci_host.c               |  4 ++--
>  hw/{ => pci}/pci_host.h               |  2 +-
>  hw/{ => pci}/pci_ids.h                |  0
>  hw/{ => pci}/pci_regs.h               |  0
>  hw/{ => pci}/pcie.c                   | 12 +++++------
>  hw/{ => pci}/pcie.h                   |  8 +++----
>  hw/{ => pci}/pcie_aer.c               | 12 +++++------
>  hw/{ => pci}/pcie_aer.h               |  2 +-
>  hw/{ => pci}/pcie_host.c              |  6 +++---
>  hw/{ => pci}/pcie_host.h              |  2 +-
>  hw/{ => pci}/pcie_port.c              |  2 +-
>  hw/{ => pci}/pcie_port.h              |  4 ++--
>  hw/{ => pci}/pcie_regs.h              |  0
>  hw/{ => pci}/shpc.c                   |  8 +++----
>  hw/{ => pci}/shpc.h                   |  0
>  hw/{ => pci}/slotid_cap.c             |  4 ++--
>  hw/{ => pci}/slotid_cap.h             |  0
>  hw/pci_bridge_dev.c                   | 12 +++++------
>  hw/pcnet-pci.c                        |  2 +-
>  hw/piix4.c                            |  2 +-
>  hw/piix_pci.c                         |  4 ++--
>  hw/ppc/Makefile.objs                  |  2 +-
>  hw/ppc/e500.c                         |  2 +-
>  hw/ppc440_bamboo.c                    |  2 +-
>  hw/ppc4xx.h                           |  2 +-
>  hw/ppc4xx_pci.c                       |  4 ++--
>  hw/ppc_newworld.c                     |  2 +-
>  hw/ppc_oldworld.c                     |  2 +-
>  hw/ppc_prep.c                         |  4 ++--
>  hw/ppce500_pci.c                      |  4 ++--
>  hw/prep_pci.c                         |  4 ++--
>  hw/q35.h                              |  4 ++--
>  hw/qxl.h                              |  2 +-
>  hw/r2d.c                              |  2 +-
>  hw/realview.c                         |  2 +-
>  hw/rtl8139.c                          |  2 +-
>  hw/serial-pci.c                       |  2 +-
>  hw/sga.c                              |  2 +-
>  hw/sh_pci.c                           |  4 ++--
>  hw/smbus_ich9.c                       | 16 ++++++++------
>  hw/spapr.c                            |  4 ++--
>  hw/spapr_pci.c                        | 10 ++++-----
>  hw/spapr_pci.h                        |  4 ++--
>  hw/sun4u.c                            |  2 +-
>  hw/unin_pci.c                         |  4 ++--
>  hw/usb/hcd-ehci-pci.c                 |  2 +-
>  hw/usb/hcd-ohci.c                     |  2 +-
>  hw/usb/hcd-uhci.c                     |  2 +-
>  hw/usb/hcd-xhci.c                     |  6 +++---
>  hw/versatile_pci.c                    |  4 ++--
>  hw/versatilepb.c                      |  2 +-
>  hw/vfio_pci.c                         |  8 +++----
>  hw/vga-pci.c                          |  2 +-
>  hw/vga.c                              |  2 +-
>  hw/virtio-balloon.h                   |  2 +-
>  hw/virtio-net.h                       |  2 +-
>  hw/virtio-pci.c                       |  9 ++++----
>  hw/virtio-scsi.h                      |  2 +-
>  hw/vmware_vga.c                       |  2 +-
>  hw/vt82c686.c                         |  4 ++--
>  hw/wdt_i6300esb.c                     |  2 +-
>  hw/xen-host-pci-device.h              |  2 +-
>  hw/xen_apic.c                         |  2 +-
>  hw/xen_platform.c                     |  2 +-
>  hw/xen_pt.c                           |  2 +-
>  hw/xen_pt.h                           |  2 +-
>  hw/xio3130_downstream.c               |  6 +++---
>  hw/xio3130_downstream.h               |  2 +-
>  hw/xio3130_upstream.c                 |  6 +++---
>  hw/xio3130_upstream.h                 |  2 +-
>  kvm-all.c                             |  2 +-
>  kvm-stub.c                            |  2 +-
>  monitor.c                             |  2 +-
>  net/tap-linux.c                       |  8 +++++++
>  scripts/get_maintainer.pl             | 25 ++++++++++++++--------
>  target-i386/kvm.c                     |  2 +-
>  xen-all.c                             |  2 +-
>  136 files changed, 303 insertions(+), 289 deletions(-)
>  create mode 100644 hw/pci/Makefile.objs
>  rename hw/{ => pci}/msi.c (99%)
>  rename hw/{ => pci}/msi.h (98%)
>  rename hw/{ => pci}/msix.c (96%)
>  rename hw/{ => pci}/msix.h (93%)
>  rename hw/{ => pci}/pci-hotplug.c (98%)
>  rename hw/{ => pci}/pci-stub.c (98%)
>  rename hw/{ => pci}/pci.c (99%)
>  rename hw/{ => pci}/pci.h (98%)
>  rename hw/{ => pci}/pci_bridge.c (99%)
>  rename hw/{ => pci}/pci_bridge.h (98%)
>  rename hw/{pci_internals.h => pci/pci_bus.h} (79%)
>  rename hw/{ => pci}/pci_host.c (99%)
>  rename hw/{ => pci}/pci_host.h (99%)
>  rename hw/{ => pci}/pci_ids.h (100%)
>  rename hw/{ => pci}/pci_regs.h (100%)
>  rename hw/{ => pci}/pcie.c (99%)
>  rename hw/{ => pci}/pcie.h (98%)
>  rename hw/{ => pci}/pcie_aer.c (99%)
>  rename hw/{ => pci}/pcie_aer.h (99%)
>  rename hw/{ => pci}/pcie_host.c (98%)
>  rename hw/{ => pci}/pcie_host.h (98%)
>  rename hw/{ => pci}/pcie_port.c (99%)
>  rename hw/{ => pci}/pcie_port.h (96%)
>  rename hw/{ => pci}/pcie_regs.h (100%)
>  rename hw/{ => pci}/shpc.c (99%)
>  rename hw/{ => pci}/shpc.h (100%)
>  rename hw/{ => pci}/slotid_cap.c (95%)
>  rename hw/{ => pci}/slotid_cap.h (100%)
Blue Swirl - Dec. 14, 2012, 8:26 p.m.
On Thu, Dec 13, 2012 at 7:26 AM, Michael S. Tsirkin <mst@redhat.com> wrote:
> The following changes since commit 1c97e303d4ea80a2691334b0febe87a50660f99d:
>
>   Merge remote-tracking branch 'afaerber/qom-cpu' into staging (2012-12-10 08:35:15 -0600)
>
> are available in the git repository at:
>
>
>   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
>
> for you to fetch changes up to 5c1ad98d71923b83a530e5db4c2110564b84e11d:
>
>   pci_bus.h: tweak include guards (2012-12-12 23:41:04 +0200)
>
> ----------------------------------------------------------------
> pci,net,misc infrastructure
>
> This rearranges pci core into hw/pci/
> so it's separate from actual devices.
> Also included are a couple of misc fixes:
>     - a fix for tap bug reported by alex on kvm forum
>     - a couple of misc updates: get_maintainer.pl,
>       and license update for the q35 files
>
> I've addressed the comments Blue Swirl had
> for the rename patches - would like to merge ASAP
> to avoid any conflicts, and we can do code style
> fixups later - note this is style of existing files.

Usually the sequence is to fix style first, then move. But in this
case that would be unreasonable thing to ask, applying checkpatch.pl
to the movement patch reports a huge number of problems:
total: 649 errors, 122 warnings, 9048 lines checked

Also after the move, there's no urgent need to fix anything wrt.
style, it can done be one step at a time as the code is touched
anyway.

>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
>
> ----------------------------------------------------------------
> Jason Baron (1):
>       Fixup q35/ich9 Licenses
>
> Michael S. Tsirkin (12):
>       tap: reset vnet header size on open
>       get_maintainer.pl: update from linix 3.8
>       pci: prepare makefiles for pci code reorganization
>       MAINTAINERS: add hw/pci/ to list of PCI files
>       pci: move pci core code to hw/pci
>       pci: update all users to look in pci/
>       pci: fix path for include paths
>       Revert "pci: prepare makefiles for pci code reorganization"
>       pci: rename pci_internals.h pci_bus.h
>       pci_bus: update comment
>       msi: add API to get notified about pending bit poll
>       pci_bus.h: tweak include guards
>
>  MAINTAINERS                           |  1 +
>  arch_init.c                           |  2 +-
>  hw/Makefile.objs                      | 10 ++-------
>  hw/ac97.c                             |  2 +-
>  hw/acpi_ich9.c                        | 15 +++++++------
>  hw/acpi_piix4.c                       |  2 +-
>  hw/alpha_sys.h                        |  4 ++--
>  hw/apb_pci.c                          |  8 +++----
>  hw/apic.c                             |  2 +-
>  hw/bonito.c                           |  4 ++--
>  hw/cirrus_vga.c                       |  2 +-
>  hw/dec_pci.c                          |  8 +++----
>  hw/e1000.c                            |  2 +-
>  hw/eepro100.c                         |  2 +-
>  hw/es1370.c                           |  2 +-
>  hw/esp-pci.c                          |  2 +-
>  hw/grackle_pci.c                      |  4 ++--
>  hw/gt64xxx.c                          |  4 ++--
>  hw/hda-audio.c                        |  2 +-
>  hw/i386/Makefile.objs                 |  2 +-
>  hw/i82378.c                           |  2 +-
>  hw/i82801b11.c                        |  2 +-
>  hw/ich9.h                             |  8 +++----
>  hw/ide.h                              |  2 +-
>  hw/ide/ahci.c                         |  4 ++--
>  hw/ide/cmd646.c                       |  2 +-
>  hw/ide/core.c                         |  2 +-
>  hw/ide/ich.c                          |  4 ++--
>  hw/ide/pci.c                          |  2 +-
>  hw/ide/piix.c                         |  2 +-
>  hw/ide/via.c                          |  2 +-
>  hw/intel-hda.c                        |  4 ++--
>  hw/ioh3420.c                          |  6 +++---
>  hw/ioh3420.h                          |  2 +-
>  hw/ivshmem.c                          |  4 ++--
>  hw/kvm/apic.c                         |  2 +-
>  hw/kvm/pci-assign.c                   |  4 ++--
>  hw/lpc_ich9.c                         | 40 +++++++++++------------------------
>  hw/lsi53c895a.c                       |  2 +-
>  hw/macio.c                            |  2 +-
>  hw/megasas.c                          |  4 ++--
>  hw/mips_fulong2e.c                    |  2 +-
>  hw/mips_malta.c                       |  2 +-
>  hw/ne2000.c                           |  2 +-
>  hw/openpic.c                          |  2 +-
>  hw/pc.c                               |  4 ++--
>  hw/pc_piix.c                          |  4 ++--
>  hw/pci/Makefile.objs                  |  6 ++++++
>  hw/{ => pci}/msi.c                    |  2 +-
>  hw/{ => pci}/msi.h                    |  2 +-
>  hw/{ => pci}/msix.c                   | 19 ++++++++++++-----
>  hw/{ => pci}/msix.h                   |  5 +++--
>  hw/{ => pci}/pci-hotplug.c            | 12 +++++------
>  hw/{ => pci}/pci-stub.c               |  2 +-
>  hw/{ => pci}/pci.c                    | 14 ++++++------
>  hw/{ => pci}/pci.h                    | 14 +++++++-----
>  hw/{ => pci}/pci_bridge.c             |  4 ++--
>  hw/{ => pci}/pci_bridge.h             |  2 +-
>  hw/{pci_internals.h => pci/pci_bus.h} | 16 ++++++--------
>  hw/{ => pci}/pci_host.c               |  4 ++--
>  hw/{ => pci}/pci_host.h               |  2 +-
>  hw/{ => pci}/pci_ids.h                |  0
>  hw/{ => pci}/pci_regs.h               |  0
>  hw/{ => pci}/pcie.c                   | 12 +++++------
>  hw/{ => pci}/pcie.h                   |  8 +++----
>  hw/{ => pci}/pcie_aer.c               | 12 +++++------
>  hw/{ => pci}/pcie_aer.h               |  2 +-
>  hw/{ => pci}/pcie_host.c              |  6 +++---
>  hw/{ => pci}/pcie_host.h              |  2 +-
>  hw/{ => pci}/pcie_port.c              |  2 +-
>  hw/{ => pci}/pcie_port.h              |  4 ++--
>  hw/{ => pci}/pcie_regs.h              |  0
>  hw/{ => pci}/shpc.c                   |  8 +++----
>  hw/{ => pci}/shpc.h                   |  0
>  hw/{ => pci}/slotid_cap.c             |  4 ++--
>  hw/{ => pci}/slotid_cap.h             |  0
>  hw/pci_bridge_dev.c                   | 12 +++++------
>  hw/pcnet-pci.c                        |  2 +-
>  hw/piix4.c                            |  2 +-
>  hw/piix_pci.c                         |  4 ++--
>  hw/ppc/Makefile.objs                  |  2 +-
>  hw/ppc/e500.c                         |  2 +-
>  hw/ppc440_bamboo.c                    |  2 +-
>  hw/ppc4xx.h                           |  2 +-
>  hw/ppc4xx_pci.c                       |  4 ++--
>  hw/ppc_newworld.c                     |  2 +-
>  hw/ppc_oldworld.c                     |  2 +-
>  hw/ppc_prep.c                         |  4 ++--
>  hw/ppce500_pci.c                      |  4 ++--
>  hw/prep_pci.c                         |  4 ++--
>  hw/q35.h                              |  4 ++--
>  hw/qxl.h                              |  2 +-
>  hw/r2d.c                              |  2 +-
>  hw/realview.c                         |  2 +-
>  hw/rtl8139.c                          |  2 +-
>  hw/serial-pci.c                       |  2 +-
>  hw/sga.c                              |  2 +-
>  hw/sh_pci.c                           |  4 ++--
>  hw/smbus_ich9.c                       | 16 ++++++++------
>  hw/spapr.c                            |  4 ++--
>  hw/spapr_pci.c                        | 10 ++++-----
>  hw/spapr_pci.h                        |  4 ++--
>  hw/sun4u.c                            |  2 +-
>  hw/unin_pci.c                         |  4 ++--
>  hw/usb/hcd-ehci-pci.c                 |  2 +-
>  hw/usb/hcd-ohci.c                     |  2 +-
>  hw/usb/hcd-uhci.c                     |  2 +-
>  hw/usb/hcd-xhci.c                     |  6 +++---
>  hw/versatile_pci.c                    |  4 ++--
>  hw/versatilepb.c                      |  2 +-
>  hw/vfio_pci.c                         |  8 +++----
>  hw/vga-pci.c                          |  2 +-
>  hw/vga.c                              |  2 +-
>  hw/virtio-balloon.h                   |  2 +-
>  hw/virtio-net.h                       |  2 +-
>  hw/virtio-pci.c                       |  9 ++++----
>  hw/virtio-scsi.h                      |  2 +-
>  hw/vmware_vga.c                       |  2 +-
>  hw/vt82c686.c                         |  4 ++--
>  hw/wdt_i6300esb.c                     |  2 +-
>  hw/xen-host-pci-device.h              |  2 +-
>  hw/xen_apic.c                         |  2 +-
>  hw/xen_platform.c                     |  2 +-
>  hw/xen_pt.c                           |  2 +-
>  hw/xen_pt.h                           |  2 +-
>  hw/xio3130_downstream.c               |  6 +++---
>  hw/xio3130_downstream.h               |  2 +-
>  hw/xio3130_upstream.c                 |  6 +++---
>  hw/xio3130_upstream.h                 |  2 +-
>  kvm-all.c                             |  2 +-
>  kvm-stub.c                            |  2 +-
>  monitor.c                             |  2 +-
>  net/tap-linux.c                       |  8 +++++++
>  scripts/get_maintainer.pl             | 25 ++++++++++++++--------
>  target-i386/kvm.c                     |  2 +-
>  xen-all.c                             |  2 +-
>  136 files changed, 303 insertions(+), 289 deletions(-)
>  create mode 100644 hw/pci/Makefile.objs
>  rename hw/{ => pci}/msi.c (99%)
>  rename hw/{ => pci}/msi.h (98%)
>  rename hw/{ => pci}/msix.c (96%)
>  rename hw/{ => pci}/msix.h (93%)
>  rename hw/{ => pci}/pci-hotplug.c (98%)
>  rename hw/{ => pci}/pci-stub.c (98%)
>  rename hw/{ => pci}/pci.c (99%)
>  rename hw/{ => pci}/pci.h (98%)
>  rename hw/{ => pci}/pci_bridge.c (99%)
>  rename hw/{ => pci}/pci_bridge.h (98%)
>  rename hw/{pci_internals.h => pci/pci_bus.h} (79%)
>  rename hw/{ => pci}/pci_host.c (99%)
>  rename hw/{ => pci}/pci_host.h (99%)
>  rename hw/{ => pci}/pci_ids.h (100%)
>  rename hw/{ => pci}/pci_regs.h (100%)
>  rename hw/{ => pci}/pcie.c (99%)
>  rename hw/{ => pci}/pcie.h (98%)
>  rename hw/{ => pci}/pcie_aer.c (99%)
>  rename hw/{ => pci}/pcie_aer.h (99%)
>  rename hw/{ => pci}/pcie_host.c (98%)
>  rename hw/{ => pci}/pcie_host.h (98%)
>  rename hw/{ => pci}/pcie_port.c (99%)
>  rename hw/{ => pci}/pcie_port.h (96%)
>  rename hw/{ => pci}/pcie_regs.h (100%)
>  rename hw/{ => pci}/shpc.c (99%)
>  rename hw/{ => pci}/shpc.h (100%)
>  rename hw/{ => pci}/slotid_cap.c (95%)
>  rename hw/{ => pci}/slotid_cap.h (100%)
>
Blue Swirl - Dec. 14, 2012, 9:37 p.m.
On Thu, Dec 13, 2012 at 7:26 AM, Michael S. Tsirkin <mst@redhat.com> wrote:
> The following changes since commit 1c97e303d4ea80a2691334b0febe87a50660f99d:
>
>   Merge remote-tracking branch 'afaerber/qom-cpu' into staging (2012-12-10 08:35:15 -0600)
>
> are available in the git repository at:
>
>
>   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
>
> for you to fetch changes up to 5c1ad98d71923b83a530e5db4c2110564b84e11d:
>
>   pci_bus.h: tweak include guards (2012-12-12 23:41:04 +0200)
>
> ----------------------------------------------------------------
> pci,net,misc infrastructure
>
> This rearranges pci core into hw/pci/
> so it's separate from actual devices.
> Also included are a couple of misc fixes:
>     - a fix for tap bug reported by alex on kvm forum
>     - a couple of misc updates: get_maintainer.pl,
>       and license update for the q35 files
>
> I've addressed the comments Blue Swirl had
> for the rename patches - would like to merge ASAP
> to avoid any conflicts, and we can do code style
> fixups later - note this is style of existing files.
>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
>
> ----------------------------------------------------------------
> Jason Baron (1):
>       Fixup q35/ich9 Licenses
>
> Michael S. Tsirkin (12):
>       tap: reset vnet header size on open
>       get_maintainer.pl: update from linix 3.8
>       pci: prepare makefiles for pci code reorganization
>       MAINTAINERS: add hw/pci/ to list of PCI files
>       pci: move pci core code to hw/pci
>       pci: update all users to look in pci/
>       pci: fix path for include paths
>       Revert "pci: prepare makefiles for pci code reorganization"
>       pci: rename pci_internals.h pci_bus.h
>       pci_bus: update comment
>       msi: add API to get notified about pending bit poll
>       pci_bus.h: tweak include guards
>
>  MAINTAINERS                           |  1 +
>  arch_init.c                           |  2 +-
>  hw/Makefile.objs                      | 10 ++-------
>  hw/ac97.c                             |  2 +-
>  hw/acpi_ich9.c                        | 15 +++++++------
>  hw/acpi_piix4.c                       |  2 +-
>  hw/alpha_sys.h                        |  4 ++--
>  hw/apb_pci.c                          |  8 +++----
>  hw/apic.c                             |  2 +-
>  hw/bonito.c                           |  4 ++--
>  hw/cirrus_vga.c                       |  2 +-
>  hw/dec_pci.c                          |  8 +++----
>  hw/e1000.c                            |  2 +-
>  hw/eepro100.c                         |  2 +-
>  hw/es1370.c                           |  2 +-
>  hw/esp-pci.c                          |  2 +-
>  hw/grackle_pci.c                      |  4 ++--
>  hw/gt64xxx.c                          |  4 ++--
>  hw/hda-audio.c                        |  2 +-
>  hw/i386/Makefile.objs                 |  2 +-
>  hw/i82378.c                           |  2 +-
>  hw/i82801b11.c                        |  2 +-
>  hw/ich9.h                             |  8 +++----
>  hw/ide.h                              |  2 +-
>  hw/ide/ahci.c                         |  4 ++--
>  hw/ide/cmd646.c                       |  2 +-
>  hw/ide/core.c                         |  2 +-
>  hw/ide/ich.c                          |  4 ++--
>  hw/ide/pci.c                          |  2 +-
>  hw/ide/piix.c                         |  2 +-
>  hw/ide/via.c                          |  2 +-
>  hw/intel-hda.c                        |  4 ++--
>  hw/ioh3420.c                          |  6 +++---
>  hw/ioh3420.h                          |  2 +-
>  hw/ivshmem.c                          |  4 ++--
>  hw/kvm/apic.c                         |  2 +-
>  hw/kvm/pci-assign.c                   |  4 ++--
>  hw/lpc_ich9.c                         | 40 +++++++++++------------------------
>  hw/lsi53c895a.c                       |  2 +-
>  hw/macio.c                            |  2 +-
>  hw/megasas.c                          |  4 ++--
>  hw/mips_fulong2e.c                    |  2 +-
>  hw/mips_malta.c                       |  2 +-
>  hw/ne2000.c                           |  2 +-
>  hw/openpic.c                          |  2 +-
>  hw/pc.c                               |  4 ++--
>  hw/pc_piix.c                          |  4 ++--
>  hw/pci/Makefile.objs                  |  6 ++++++
>  hw/{ => pci}/msi.c                    |  2 +-
>  hw/{ => pci}/msi.h                    |  2 +-
>  hw/{ => pci}/msix.c                   | 19 ++++++++++++-----
>  hw/{ => pci}/msix.h                   |  5 +++--
>  hw/{ => pci}/pci-hotplug.c            | 12 +++++------
>  hw/{ => pci}/pci-stub.c               |  2 +-
>  hw/{ => pci}/pci.c                    | 14 ++++++------
>  hw/{ => pci}/pci.h                    | 14 +++++++-----
>  hw/{ => pci}/pci_bridge.c             |  4 ++--

Perhaps the use of '-' vs. '_' in file names could be unified while
renaming. I think most new files use dash.

>  hw/{ => pci}/pci_bridge.h             |  2 +-
>  hw/{pci_internals.h => pci/pci_bus.h} | 16 ++++++--------
>  hw/{ => pci}/pci_host.c               |  4 ++--
>  hw/{ => pci}/pci_host.h               |  2 +-
>  hw/{ => pci}/pci_ids.h                |  0
>  hw/{ => pci}/pci_regs.h               |  0
>  hw/{ => pci}/pcie.c                   | 12 +++++------
>  hw/{ => pci}/pcie.h                   |  8 +++----
>  hw/{ => pci}/pcie_aer.c               | 12 +++++------
>  hw/{ => pci}/pcie_aer.h               |  2 +-
>  hw/{ => pci}/pcie_host.c              |  6 +++---
>  hw/{ => pci}/pcie_host.h              |  2 +-
>  hw/{ => pci}/pcie_port.c              |  2 +-
>  hw/{ => pci}/pcie_port.h              |  4 ++--
>  hw/{ => pci}/pcie_regs.h              |  0
>  hw/{ => pci}/shpc.c                   |  8 +++----
>  hw/{ => pci}/shpc.h                   |  0
>  hw/{ => pci}/slotid_cap.c             |  4 ++--
>  hw/{ => pci}/slotid_cap.h             |  0
>  hw/pci_bridge_dev.c                   | 12 +++++------
>  hw/pcnet-pci.c                        |  2 +-
>  hw/piix4.c                            |  2 +-
>  hw/piix_pci.c                         |  4 ++--
>  hw/ppc/Makefile.objs                  |  2 +-
>  hw/ppc/e500.c                         |  2 +-
>  hw/ppc440_bamboo.c                    |  2 +-
>  hw/ppc4xx.h                           |  2 +-
>  hw/ppc4xx_pci.c                       |  4 ++--
>  hw/ppc_newworld.c                     |  2 +-
>  hw/ppc_oldworld.c                     |  2 +-
>  hw/ppc_prep.c                         |  4 ++--
>  hw/ppce500_pci.c                      |  4 ++--
>  hw/prep_pci.c                         |  4 ++--
>  hw/q35.h                              |  4 ++--
>  hw/qxl.h                              |  2 +-
>  hw/r2d.c                              |  2 +-
>  hw/realview.c                         |  2 +-
>  hw/rtl8139.c                          |  2 +-
>  hw/serial-pci.c                       |  2 +-
>  hw/sga.c                              |  2 +-
>  hw/sh_pci.c                           |  4 ++--
>  hw/smbus_ich9.c                       | 16 ++++++++------
>  hw/spapr.c                            |  4 ++--
>  hw/spapr_pci.c                        | 10 ++++-----
>  hw/spapr_pci.h                        |  4 ++--
>  hw/sun4u.c                            |  2 +-
>  hw/unin_pci.c                         |  4 ++--
>  hw/usb/hcd-ehci-pci.c                 |  2 +-
>  hw/usb/hcd-ohci.c                     |  2 +-
>  hw/usb/hcd-uhci.c                     |  2 +-
>  hw/usb/hcd-xhci.c                     |  6 +++---
>  hw/versatile_pci.c                    |  4 ++--
>  hw/versatilepb.c                      |  2 +-
>  hw/vfio_pci.c                         |  8 +++----
>  hw/vga-pci.c                          |  2 +-
>  hw/vga.c                              |  2 +-
>  hw/virtio-balloon.h                   |  2 +-
>  hw/virtio-net.h                       |  2 +-
>  hw/virtio-pci.c                       |  9 ++++----
>  hw/virtio-scsi.h                      |  2 +-
>  hw/vmware_vga.c                       |  2 +-
>  hw/vt82c686.c                         |  4 ++--
>  hw/wdt_i6300esb.c                     |  2 +-
>  hw/xen-host-pci-device.h              |  2 +-
>  hw/xen_apic.c                         |  2 +-
>  hw/xen_platform.c                     |  2 +-
>  hw/xen_pt.c                           |  2 +-
>  hw/xen_pt.h                           |  2 +-
>  hw/xio3130_downstream.c               |  6 +++---
>  hw/xio3130_downstream.h               |  2 +-
>  hw/xio3130_upstream.c                 |  6 +++---
>  hw/xio3130_upstream.h                 |  2 +-
>  kvm-all.c                             |  2 +-
>  kvm-stub.c                            |  2 +-
>  monitor.c                             |  2 +-
>  net/tap-linux.c                       |  8 +++++++
>  scripts/get_maintainer.pl             | 25 ++++++++++++++--------
>  target-i386/kvm.c                     |  2 +-
>  xen-all.c                             |  2 +-
>  136 files changed, 303 insertions(+), 289 deletions(-)
>  create mode 100644 hw/pci/Makefile.objs
>  rename hw/{ => pci}/msi.c (99%)
>  rename hw/{ => pci}/msi.h (98%)
>  rename hw/{ => pci}/msix.c (96%)
>  rename hw/{ => pci}/msix.h (93%)
>  rename hw/{ => pci}/pci-hotplug.c (98%)
>  rename hw/{ => pci}/pci-stub.c (98%)
>  rename hw/{ => pci}/pci.c (99%)
>  rename hw/{ => pci}/pci.h (98%)
>  rename hw/{ => pci}/pci_bridge.c (99%)
>  rename hw/{ => pci}/pci_bridge.h (98%)
>  rename hw/{pci_internals.h => pci/pci_bus.h} (79%)
>  rename hw/{ => pci}/pci_host.c (99%)
>  rename hw/{ => pci}/pci_host.h (99%)
>  rename hw/{ => pci}/pci_ids.h (100%)
>  rename hw/{ => pci}/pci_regs.h (100%)
>  rename hw/{ => pci}/pcie.c (99%)
>  rename hw/{ => pci}/pcie.h (98%)
>  rename hw/{ => pci}/pcie_aer.c (99%)
>  rename hw/{ => pci}/pcie_aer.h (99%)
>  rename hw/{ => pci}/pcie_host.c (98%)
>  rename hw/{ => pci}/pcie_host.h (98%)
>  rename hw/{ => pci}/pcie_port.c (99%)
>  rename hw/{ => pci}/pcie_port.h (96%)
>  rename hw/{ => pci}/pcie_regs.h (100%)
>  rename hw/{ => pci}/shpc.c (99%)
>  rename hw/{ => pci}/shpc.h (100%)
>  rename hw/{ => pci}/slotid_cap.c (95%)
>  rename hw/{ => pci}/slotid_cap.h (100%)
>
Michael S. Tsirkin - Dec. 16, 2012, 2:41 p.m.
On Thu, Dec 13, 2012 at 02:31:53PM -0600, Anthony Liguori wrote:
> "Michael S. Tsirkin" <mst@redhat.com> writes:
> 
> > The following changes since commit 1c97e303d4ea80a2691334b0febe87a50660f99d:
> >
> >   Merge remote-tracking branch 'afaerber/qom-cpu' into staging (2012-12-10 08:35:15 -0600)
> >
> > are available in the git repository at:
> >
> >
> >   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
> >
> > for you to fetch changes up to 5c1ad98d71923b83a530e5db4c2110564b84e11d:
> >
> >   pci_bus.h: tweak include guards (2012-12-12 23:41:04 +0200)
> 
> Doesn't build:
> 
>   CC    hw/apm.o
> /home/anthony/git/qemu/hw/apm.c:25:17: fatal error: pci.h: No such file or directory
> compilation terminated.
> 
> And there's really no good reason for this.  apm's part of target-i386
> so you couldn't have built this prior to doing a pull request.
> 
> Please setup your tree in buildbot and wait for a full run before
> sending a pull requests in the future.
> 
> Regards,
> 
> Anthony Liguori

Ugh.
I'm not sure but I think I see how this happened: I re-run make after
each commit, before the last one apm.c did find the header it depends
on, the last commit only removes the -I flag to find the header
but neither the header itself not the C file changed so it didn't
need to rebuild.
Michael S. Tsirkin - Dec. 16, 2012, 2:46 p.m.
On Fri, Dec 14, 2012 at 09:37:17PM +0000, Blue Swirl wrote:
> Perhaps the use of '-' vs. '_' in file names could be unified while
> renaming. I think most new files use dash.

I can do this in a follow up patch but first let's put this rule in
coding style file. As it is more files use _ than -.
And I'm not sure what's the point of using dash, as opposed
to underscore: underscore seems more consistent.
Michael S. Tsirkin - Dec. 16, 2012, 3:23 p.m.
> Please setup your tree in buildbot and wait for a full run before
> sending a pull requests in the future.

I actually waited 24 hours which is normally enough for
buildbot to run the tree, and assumed pass means OK.
Turns out buildbot was passing, but only because it was building
some old commits:

http://buildbot.b1-systems.de/qemu/builders/pci_x86_64_debian_6_0/builds/480

Built f1219091edd20e3b92544025c2b6dd5e4d98b61b from May on Dec 15.
I don't understand why: both the pci branch and the for_anthony
tag on my kernel.org tree are updated.
Michael S. Tsirkin - Dec. 16, 2012, 9:10 p.m.
On Thu, Dec 13, 2012 at 02:31:53PM -0600, Anthony Liguori wrote:
> "Michael S. Tsirkin" <mst@redhat.com> writes:
> 
> > The following changes since commit 1c97e303d4ea80a2691334b0febe87a50660f99d:
> >
> >   Merge remote-tracking branch 'afaerber/qom-cpu' into staging (2012-12-10 08:35:15 -0600)
> >
> > are available in the git repository at:
> >
> >
> >   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
> >
> > for you to fetch changes up to 5c1ad98d71923b83a530e5db4c2110564b84e11d:
> >
> >   pci_bus.h: tweak include guards (2012-12-12 23:41:04 +0200)
> 
> Doesn't build:
> 
>   CC    hw/apm.o
> /home/anthony/git/qemu/hw/apm.c:25:17: fatal error: pci.h: No such file or directory
> compilation terminated.
> 
> And there's really no good reason for this.  apm's part of target-i386
> so you couldn't have built this prior to doing a pull request.
> 
> Please setup your tree in buildbot and wait for a full run before
> sending a pull requests in the future.
> 
> Regards,
> 
> Anthony Liguori

I've pushed a new version out will resend a pull request:
I did make clean this time so it should be fine.
Will look into fixing buildbot too.
Markus Armbruster - Dec. 17, 2012, 10:59 a.m.
"Michael S. Tsirkin" <mst@redhat.com> writes:

> On Fri, Dec 14, 2012 at 09:37:17PM +0000, Blue Swirl wrote:
>> Perhaps the use of '-' vs. '_' in file names could be unified while
>> renaming. I think most new files use dash.

Yes, please!

> I can do this in a follow up patch but first let's put this rule in
> coding style file. As it is more files use _ than -.
> And I'm not sure what's the point of using dash, as opposed
> to underscore: underscore seems more consistent.

Actually, '-' is more common:

$ git-ls-files | wc -l
2839
$ git-ls-files | grep [-_] | wc -l
2150
$ git-ls-files | grep _ | wc -l
1124
$ git-ls-files | grep -- - | wc -l
1567
Michael S. Tsirkin - Dec. 17, 2012, 11:19 a.m.
On Mon, Dec 17, 2012 at 11:59:59AM +0100, Markus Armbruster wrote:
> "Michael S. Tsirkin" <mst@redhat.com> writes:
> 
> > On Fri, Dec 14, 2012 at 09:37:17PM +0000, Blue Swirl wrote:
> >> Perhaps the use of '-' vs. '_' in file names could be unified while
> >> renaming. I think most new files use dash.
> 
> Yes, please!
> 
> > I can do this in a follow up patch but first let's put this rule in
> > coding style file. As it is more files use _ than -.
> > And I'm not sure what's the point of using dash, as opposed
> > to underscore: underscore seems more consistent.
> 
> Actually, '-' is more common:
> 
> $ git-ls-files | wc -l
> 2839
> $ git-ls-files | grep [-_] | wc -l
> 2150
> $ git-ls-files | grep _ | wc -l
> 1124
> $ git-ls-files | grep -- - | wc -l
> 1567

I think you counted target-XXX linux-user etc multiple times.
Markus Armbruster - Dec. 17, 2012, 12:30 p.m.
"Michael S. Tsirkin" <mst@redhat.com> writes:

> On Mon, Dec 17, 2012 at 11:59:59AM +0100, Markus Armbruster wrote:
>> "Michael S. Tsirkin" <mst@redhat.com> writes:
>> 
>> > On Fri, Dec 14, 2012 at 09:37:17PM +0000, Blue Swirl wrote:
>> >> Perhaps the use of '-' vs. '_' in file names could be unified while
>> >> renaming. I think most new files use dash.
>> 
>> Yes, please!
>> 
>> > I can do this in a follow up patch but first let's put this rule in
>> > coding style file. As it is more files use _ than -.
>> > And I'm not sure what's the point of using dash, as opposed
>> > to underscore: underscore seems more consistent.
>> 
>> Actually, '-' is more common:
>> 
>> $ git-ls-files | wc -l
>> 2839
>> $ git-ls-files | grep [-_] | wc -l
>> 2150
>> $ git-ls-files | grep _ | wc -l
>> 1124
>> $ git-ls-files | grep -- - | wc -l
>> 1567
>
> I think you counted target-XXX linux-user etc multiple times.

If you want just basenames, no directories:

$ git-ls-files | sed 's#.*/##g' | grep [-_] | wc -l
1698
$ git-ls-files | sed 's#.*/##g' | grep _ | wc -l
1121
$ git-ls-files | sed 's#.*/##g' | grep -- - | wc -l
588

If we exclude tests/, which has tons of '_', we get:

$ git-ls-files | grep -v ^tests/ | sed 's#.*/##g' | grep _ | wc -l
486
$ git-ls-files | grep -v ^tests/ | sed 's#.*/##g' | grep -- - | wc -l
546

Either way, precedence isn't really conclusive.  It's a matter of taste
anyway.  I happen to prefer '-'.