Re: [Qemu-devel] [PATCH v3 0/8] hw/pci: set irq without selecting INTx pin

[Michael S. Tsirkin]

On Mon, Oct 07, 2013 at 10:36:33AM +0300, Marcel Apfelbaum wrote:
> Interrupt pin is selected and saved into PCI_INTERRUPT_PIN
> register during device initialization. Devices should not call
> directly qemu_set_irq and specify the INTx pin.
> 
> Added pci_* wrappers to replace qemu_set_irq, qemu_irq_raise,
> qemu_irq_lower and qemu_irq_pulse, setting the irq
> based on PCI_INTERRUPT_PIN.
> 
> Added interface to allocate and free single irq.
> Added pci_allocate_irq wrapper to be used by devices that
> still need PCIDevice infrastructure to assert irqs.
> 
> Removed irq field from PCIDevice, not needed anymore.
> 
> Special cases of replacements were done in separate patches,
> all others in one patch "hw: set interrupts using pci irq wrappers"

OK so 
Acked-by: Michael S. Tsirkin <address@hidden>

Let's wait for some acks, then I'll merge this.

> Changes from v2:
>  - Addressed Michael S. Tsirkin's comments:
>    - Terminate comments by "."
>    - Add "fixme" comment to pci_irq_pulse
>  - Addressed Paolo Bonzini's comment:
>    - fixed implementation of pci_irq_pulse
>  - Addressed Alex Williamson's comments
>    - replaced pci_irq_raise/lower with
>      pci_irq_assert/deassert
>    - replaced calls to pci_set_irq with
>      pci_irq_assert/deassert when possible
>  - Addressed Gerd Hoffmann's comment
>    - removed irq_pin from UHCIState state
>      because it is not used anymore
>      
>  - Fixed a bug in vmxnet3 (deassert was replaced by an assert)
> 
> Changes from v1:
>  - Addressed Michael S. Tsirkin's comments:
>    - pci_set_irq directly calls pci_irq handler
>    - removed irq field from PCIDevice
>  - Added qemu interface to allocate single irq
>  - Added pci wrappers to allocate and free pci irq
>  - Added pci irq wrappers for all qemu methods
>    setting irq and not only qemu_set_irq 
>  - Replace all qemu irq setters with pci
>    wrappers
> 
> Marcel Apfelbaum (9):
>   hw/core: Add interface to allocate and free a single IRQ
>   hw/pci: add pci wrappers for allocating and asserting irqs
>   hw/pci-bridge: set PCI_INTERRUPT_PIN register before shpc init
>   hw/vmxnet3: set interrupts using pci irq wrappers
>   hw/vfio: set interrupts using pci irq wrappers
>   hw: set interrupts using pci irq wrappers
>   hw/pcie: AER and hot-plug events must use device's interrupt
>   hw/pci: removed irq field from PCIDevice
> 
>  hw/audio/ac97.c                |  4 ++--
>  hw/audio/es1370.c              |  4 ++--
>  hw/audio/intel-hda.c           |  2 +-
>  hw/block/nvme.c                |  2 +-
>  hw/char/serial-pci.c           |  5 +++--
>  hw/char/tpci200.c              |  8 ++++----
>  hw/core/irq.c                  | 16 ++++++++++++++++
>  hw/display/qxl.c               |  2 +-
>  hw/ide/cmd646.c                |  2 +-
>  hw/ide/ich.c                   |  3 ++-
>  hw/isa/vt82c686.c              |  2 +-
>  hw/misc/ivshmem.c              |  2 +-
>  hw/misc/vfio.c                 | 11 ++++++-----
>  hw/net/e1000.c                 |  2 +-
>  hw/net/eepro100.c              |  4 ++--
>  hw/net/ne2000.c                |  3 ++-
>  hw/net/pcnet-pci.c             |  3 ++-
>  hw/net/rtl8139.c               |  2 +-
>  hw/net/vmxnet3.c               | 13 +++++++++++--
>  hw/pci-bridge/pci_bridge_dev.c |  2 +-
>  hw/pci/pci.c                   | 26 +++++++++++++++++++++-----
>  hw/pci/pcie.c                  |  4 ++--
>  hw/pci/pcie_aer.c              |  4 ++--
>  hw/pci/shpc.c                  |  2 +-
>  hw/scsi/esp-pci.c              |  3 ++-
>  hw/scsi/lsi53c895a.c           |  2 +-
>  hw/scsi/megasas.c              |  6 +++---
>  hw/scsi/vmw_pvscsi.c           |  2 +-
>  hw/usb/hcd-ehci-pci.c          |  2 +-
>  hw/usb/hcd-ohci.c              |  2 +-
>  hw/usb/hcd-uhci.c              |  6 ++----
>  hw/usb/hcd-xhci.c              |  7 ++-----
>  hw/virtio/virtio-pci.c         |  4 ++--
>  include/hw/irq.h               |  7 +++++++
>  include/hw/pci/pci.h           | 26 +++++++++++++++++++++++---
>  include/hw/pci/pcie.h          | 18 ------------------
>  36 files changed, 132 insertions(+), 81 deletions(-)
> 
> -- 
> 1.8.3.1