From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43620) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c5Fn4-0005FQ-Lq for qemu-devel@nongnu.org; Fri, 11 Nov 2016 12:39:03 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c5Fn1-0003Ki-Gn for qemu-devel@nongnu.org; Fri, 11 Nov 2016 12:39:02 -0500 Received: from mx1.redhat.com ([209.132.183.28]:59108) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1c5Fn1-0003Ka-BF for qemu-devel@nongnu.org; Fri, 11 Nov 2016 12:38:59 -0500 Date: Fri, 11 Nov 2016 19:38:56 +0200 From: "Michael S. Tsirkin" Message-ID: <20161111193808-mutt-send-email-mst@kernel.org> References: <1478311649-22027-1-git-send-email-caoj.fnst@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1478311649-22027-1-git-send-email-caoj.fnst@cn.fujitsu.com> Subject: Re: [Qemu-devel] [PATCH v6 00/10] Convert msix_init() to error List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cao jin Cc: qemu-devel@nongnu.org, Jiri Pirko , Gerd Hoffmann , Dmitry Fleytman , Jason Wang , Hannes Reinecke , Paolo Bonzini , Alex Williamson , Markus Armbruster , Marcel Apfelbaum On Sat, Nov 05, 2016 at 10:07:19AM +0800, Cao jin wrote: > v6 changelog: > 1. re-spin vfio-pci related code on patch 3: for -ENOTSUP, report & free Error, > for other error, propagate the Error. (Marcel) I dropped this patchset for now, pls submit either patches that were tested, or note otherwise. > 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 | 71 ++++++++++++++++++++++++++++++-------------------- > hw/vfio/pci.c | 8 ++++-- > hw/virtio/virtio-pci.c | 11 ++++---- > include/hw/pci/msix.h | 5 ++-- > 12 files changed, 165 insertions(+), 102 deletions(-) > > -- > 2.1.0 > >