From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38254) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c0yrB-0007Jl-TV for qemu-devel@nongnu.org; Sun, 30 Oct 2016 18:45:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c0yr7-0008VX-Ol for qemu-devel@nongnu.org; Sun, 30 Oct 2016 18:45:37 -0400 Received: from mx1.redhat.com ([209.132.183.28]:59686) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1c0yr7-0008VF-Ju for qemu-devel@nongnu.org; Sun, 30 Oct 2016 18:45:33 -0400 Date: Mon, 31 Oct 2016 00:45:29 +0200 From: "Michael S. Tsirkin" Message-ID: <20161031004459-mutt-send-email-mst@kernel.org> References: <1476859665-30133-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: <1476859665-30133-1-git-send-email-caoj.fnst@cn.fujitsu.com> Subject: Re: [Qemu-devel] [PATCH v4 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 Wed, Oct 19, 2016 at 02:47:35PM +0800, Cao jin wrote: > v4 changelog > 1. add the missed comment of "errp" in the msix_init's function comment > 2. fix typo: msic --> msix > 3. fix a build failure due to "copy&paste" error, in patch > "megasas: change behaviour of msix switch" > 4. separate the issus-fix part of vmxnet3's patch > 5. a new patch added in the end > 6. rebase on Eric Auger's "Convert VFIO-PCI to realize" series Markus, you reviewed previous revisions, would you like to ack this one? > 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 | 11 +++++--- > hw/pci/msix.c | 42 ++++++++++++++++++++++++----- > hw/scsi/megasas.c | 49 +++++++++++++++++++--------------- > hw/usb/hcd-xhci.c | 71 ++++++++++++++++++++++++++++++-------------------- > hw/vfio/pci.c | 4 ++- > hw/virtio/virtio-pci.c | 11 ++++---- > include/hw/pci/msix.h | 5 ++-- > 12 files changed, 164 insertions(+), 101 deletions(-) > > -- > 2.1.0 > >