From: Cao jin <caoj.fnst@cn.fujitsu.com>
To: qemu-devel@nongnu.org
Cc: Gerd Hoffmann <kraxel@redhat.com>, John Snow <jsnow@redhat.com>,
Dmitry Fleytman <dmitry@daynix.com>,
Jason Wang <jasowang@redhat.com>,
"Michael S. Tsirkin" <mst@redhat.com>,
Hannes Reinecke <hare@suse.de>,
Paolo Bonzini <pbonzini@redhat.com>,
Alex Williamson <alex.williamson@redhat.com>,
Markus Armbruster <armbru@redhat.com>,
Marcel Apfelbaum <marcel@redhat.com>
Subject: [Qemu-devel] [PATCH v9 00/13] Add param Error ** for msi_init()--part2
Date: Mon, 20 Jun 2016 14:13:31 +0800 [thread overview]
Message-ID: <1466403224-16604-1-git-send-email-caoj.fnst@cn.fujitsu.com> (raw)
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 <kraxel@redhat.com>
cc: John Snow <jsnow@redhat.com>
cc: Dmitry Fleytman <dmitry@daynix.com>
cc: Jason Wang <jasowang@redhat.com>
cc: Michael S. Tsirkin <mst@redhat.com>
cc: Hannes Reinecke <hare@suse.de>
cc: Paolo Bonzini <pbonzini@redhat.com>
cc: Alex Williamson <alex.williamson@redhat.com>
cc: Markus Armbruster <armbru@redhat.com>
cc: Marcel Apfelbaum <marcel@redhat.com>
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(-)
--
2.1.0
next reply other threads:[~2016-06-20 6:08 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-20 6:13 Cao jin [this message]
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 01/13] change pvscsi_init_msi() type to void Cao jin
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 02/13] mptsas: change .realize function name Cao jin
2016-07-06 3:02 ` Cao jin
2016-09-14 7:57 ` Michael Tokarev
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 03/13] usb xhci: change msi/msix property type Cao jin
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 04/13] intel-hda: change msi " Cao jin
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 05/13] mptsas: " Cao jin
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 06/13] megasas: change msi/msix " Cao jin
2016-06-20 6:12 ` Hannes Reinecke
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 07/13] pci bridge dev: change msi " Cao jin
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 08/13] pci: Convert msi_init() to Error and fix callers to check it Cao jin
2016-06-20 6:14 ` Hannes Reinecke
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 09/13] megasas: remove unnecessary megasas_use_msi() Cao jin
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 10/13] mptsas: remove unnecessary internal msi state flag Cao jin
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 11/13] vmxnet3: " Cao jin
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 12/13] e1000e: " Cao jin
2016-06-29 13:42 ` Markus Armbruster
2016-06-20 6:13 ` [Qemu-devel] [PATCH v9 13/13] vmw_pvscsi: " Cao jin
2016-06-29 13:44 ` Markus Armbruster
2016-06-28 11:19 ` [Qemu-devel] [PATCH v9 00/13] Add param Error ** for msi_init()--part2 Cao jin
2016-06-29 1:24 ` Cao jin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1466403224-16604-1-git-send-email-caoj.fnst@cn.fujitsu.com \
--to=caoj.fnst@cn.fujitsu.com \
--cc=alex.williamson@redhat.com \
--cc=armbru@redhat.com \
--cc=dmitry@daynix.com \
--cc=hare@suse.de \
--cc=jasowang@redhat.com \
--cc=jsnow@redhat.com \
--cc=kraxel@redhat.com \
--cc=marcel@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.