From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41548) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c2B70-00045F-7s for qemu-devel@nongnu.org; Thu, 03 Nov 2016 02:02:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c2B6x-0004Pg-5m for qemu-devel@nongnu.org; Thu, 03 Nov 2016 02:02:54 -0400 Received: from [59.151.112.132] (port=22066 helo=heian.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c2B6w-0004Or-RA for qemu-devel@nongnu.org; Thu, 03 Nov 2016 02:02:51 -0400 References: <1478145997-28865-1-git-send-email-caoj.fnst@cn.fujitsu.com> From: Cao jin Message-ID: <581AD39B.9030705@cn.fujitsu.com> Date: Thu, 3 Nov 2016 14:05:15 +0800 MIME-Version: 1.0 In-Reply-To: <1478145997-28865-1-git-send-email-caoj.fnst@cn.fujitsu.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v5 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: "Michael S. Tsirkin" , Markus Armbruster sorry I forget to cc maintainers in cover letter. On 11/03/2016 12:06 PM, Cao jin wrote: > v5 changelog: > 1. Address to all comments of Markus > 2. Add the given R-b to each commit message. > > 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(-) > -- Yours Sincerely, Cao jin