From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56016) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c6BbX-0006Wy-5e for qemu-devel@nongnu.org; Mon, 14 Nov 2016 02:23:00 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c6BbU-00041Z-Ir for qemu-devel@nongnu.org; Mon, 14 Nov 2016 02:22:59 -0500 Received: from [59.151.112.132] (port=17791 helo=heian.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c6BbU-0003zR-5m for qemu-devel@nongnu.org; Mon, 14 Nov 2016 02:22:56 -0500 From: Cao jin Date: Mon, 14 Nov 2016 15:25:30 +0800 Message-ID: <1479108340-3453-1-git-send-email-caoj.fnst@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain Subject: [Qemu-devel] [PATCH v7 00/10] Convert msix_init() to error List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Jiri Pirko , Gerd Hoffmann , Dmitry Fleytman , Jason Wang , "Michael S. Tsirkin" , Hannes Reinecke , Paolo Bonzini , Alex Williamson , Markus Armbruster , Marcel Apfelbaum v7 changelog: 1. fix the segfaut bug in patch 2. So drop the all the R-b of it, please take a look, there is detailed description in the patch. 2. add the R-b from Hannes Reinecke Test: 1. make check: pass 2. After applied all the patch, command line test for all the affected devices, just make sure device realize process is ok, no crash, but no further use of device. CC: Jiri Pirko CC: Gerd Hoffmann CC: Dmitry Fleytman CC: Jason Wang CC: Michael S. Tsirkin CC: Hannes Reinecke CC: Paolo Bonzini CC: Alex Williamson CC: Markus Armbruster CC: Marcel Apfelbaum Cao jin (10): msix: Follow CODING_STYLE hcd-xhci: check & correct param before using it pci: Convert msix_init() to Error and fix callers to check it megasas: change behaviour of msix switch hcd-xhci: change behaviour of msix switch megasas: remove unnecessary megasas_use_msix() megasas: undo the overwrites of msi user configuration vmxnet3: fix reference leak issue vmxnet3: remove unnecessary internal msix flag msi_init: convert assert to return -errno hw/block/nvme.c | 5 +++- hw/misc/ivshmem.c | 8 +++--- hw/net/e1000e.c | 6 ++++- hw/net/rocker/rocker.c | 7 ++++- hw/net/vmxnet3.c | 46 +++++++++++++++------------------ hw/pci/msi.c | 9 ++++--- hw/pci/msix.c | 42 +++++++++++++++++++++++++----- hw/scsi/megasas.c | 49 ++++++++++++++++++++--------------- hw/usb/hcd-xhci.c | 69 ++++++++++++++++++++++++++++++-------------------- hw/vfio/pci.c | 8 ++++-- hw/virtio/virtio-pci.c | 11 ++++---- include/hw/pci/msix.h | 5 ++-- 12 files changed, 164 insertions(+), 101 deletions(-) -- 2.1.0