qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v9 00/13] Add param Error ** for msi_init()--par


From: Cao jin
Subject: Re: [Qemu-devel] [PATCH v9 00/13] Add param Error ** for msi_init()--part2
Date: Tue, 28 Jun 2016 19:19:31 +0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0

ping

On 06/20/2016 02:13 PM, Cao jin wrote:
rebased against upstream, and passed make check.

changelog:
1. vmw_pvscsi: for compatibility, leave the field msi_used alone.
2. since patch "msi_init: change return value to 0 on success" has been adopted
    first, the patch "megasas: Fix check for msi_init() failure" isn't necessary
    anymore, so drop it.
3. fix failure of make check. It is actually not a bug, test case
    "/ahci/hba_spec" always think the 1st capability pointed by Capabilities
    pointer should be MSI, and the patch changed the order of adding capability.
    Since we don`t pass a error object to msi_init() in ich9ahci and return on
    its error, also and PCIDeviceClass->exit function is enough to free all the
    resource even if .realize() returns on msi_init() failure, so, revert to the
    position where we added msi capability to make "make check" happy.

cc: Gerd Hoffmann <address@hidden>
cc: John Snow <address@hidden>
cc: Dmitry Fleytman <address@hidden>
cc: Jason Wang <address@hidden>
cc: Michael S. Tsirkin <address@hidden>
cc: Hannes Reinecke <address@hidden>
cc: Paolo Bonzini <address@hidden>
cc: Alex Williamson <address@hidden>
cc: Markus Armbruster <address@hidden>
cc: Marcel Apfelbaum <address@hidden>

Cao jin (13):
   change pvscsi_init_msi() type to void
   mptsas: change .realize function name
   usb xhci: change msi/msix property type
   intel-hda: change msi property type
   mptsas: change msi property type
   megasas: change msi/msix property type
   pci bridge dev: change msi property type
   pci: Convert msi_init() to Error and fix callers to check it
   megasas: remove unnecessary megasas_use_msi()
   mptsas: remove unnecessary internal msi state flag
   vmxnet3: remove unnecessary internal msi state flag
   e1000e: remove unnecessary internal msi state flag
   vmw_pvscsi: remove unnecessary internal msi state flag

  hw/audio/intel-hda.c               | 29 +++++++++++++++----
  hw/ide/ich.c                       |  7 +++--
  hw/net/e1000e.c                    | 37 +++++-------------------
  hw/net/vmxnet3.c                   | 52 +++++++++++----------------------
  hw/pci-bridge/ioh3420.c            |  6 +++-
  hw/pci-bridge/pci_bridge_dev.c     | 31 ++++++++++++++------
  hw/pci-bridge/xio3130_downstream.c |  6 +++-
  hw/pci-bridge/xio3130_upstream.c   |  6 +++-
  hw/pci/msi.c                       | 11 +++++--
  hw/scsi/megasas.c                  | 59 ++++++++++++++++++++------------------
  hw/scsi/mptsas.c                   | 40 +++++++++++++++++---------
  hw/scsi/mptsas.h                   |  5 ++--
  hw/scsi/vmw_pvscsi.c               | 15 ++++------
  hw/usb/hcd-xhci.c                  | 35 ++++++++++++++++------
  hw/vfio/pci.c                      |  7 +++--
  include/hw/pci/msi.h               |  3 +-
  16 files changed, 194 insertions(+), 155 deletions(-)


--
Yours Sincerely,

Cao jin





reply via email to

[Prev in Thread] Current Thread [Next in Thread]