From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46334) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aWp9t-0001aJ-8M for qemu-devel@nongnu.org; Fri, 19 Feb 2016 12:48:02 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aWp9o-0001Vo-CE for qemu-devel@nongnu.org; Fri, 19 Feb 2016 12:48:01 -0500 Received: from mx1.redhat.com ([209.132.183.28]:41055) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aWp9o-0001Vd-4b for qemu-devel@nongnu.org; Fri, 19 Feb 2016 12:47:56 -0500 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (Postfix) with ESMTPS id A3131C000707 for ; Fri, 19 Feb 2016 17:47:55 +0000 (UTC) Received: from t450s.home (ovpn-113-36.phx2.redhat.com [10.3.113.36]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u1JHltO6006047 for ; Fri, 19 Feb 2016 12:47:55 -0500 Date: Fri, 19 Feb 2016 10:47:55 -0700 From: Alex Williamson Message-ID: <20160219104755.57ab5504@t450s.home> In-Reply-To: <20160219173708.16497.23357.stgit@gimli.home> References: <20160219173708.16497.23357.stgit@gimli.home> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PULL 00/14] VFIO updates 2016-02-19 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org On Fri, 19 Feb 2016 10:39:31 -0700 Alex Williamson wrote: > The following changes since commit 1b3337bb1d1c3125a2140c47629f36540ac57605: > > Merge remote-tracking branch 'remotes/armbru/tags/pull-error-2016-02-19' into staging (2016-02-19 15:19:13 +0000) > > are available in the git repository at: > > > git://github.com/awilliam/qemu-vfio.git tags/vfio-update-20160219.1 > > for you to fetch changes up to b58b17f744b5465d0fc76eba1be549a9f5704bab: > > vfio/pci: use PCI_MSIX_FLAGS on retrieving the MSIX entries (2016-02-19 09:42:32 -0700) > > ---------------------------------------------------------------- > VFIO updates 2016-02-19 > > - AER pre-enable and misc fixes (Cao jin and Chen Fan) > - PCI_CAP_LIST_NEXT & PCI_MSIX_FLAGS cleanup (Wei Yang) > - AMD XGBE KVM platform passthrough (Eric Auger) > > ---------------------------------------------------------------- > Chen Fan (4): > pcie: modify the capability size assert > vfio: make the 4 bytes aligned for capability size > aer: impove pcie_aer_init to support vfio device > pcie_aer: expose pcie_aer_msg() interface > > Eric Auger (8): > hw/vfio/platform: amd-xgbe device > device_tree: introduce load_device_tree_from_sysfs > device_tree: introduce qemu_fdt_node_path > device_tree: qemu_fdt_getprop converted to use the error API > device_tree: qemu_fdt_getprop_cell converted to use the error API > hw/arm/sysbus-fdt: helpers for clock node generation > hw/arm/sysbus-fdt: enable amd-xgbe dynamic instantiation > hw/arm/sysbus-fdt: remove qemu_fdt_setprop returned value check > > Wei Yang (2): > vfio/pci: replace 1 with PCI_CAP_LIST_NEXT to make code self-explain > vfio/pci: use PCI_MSIX_FLAGS on retrieving the MSIX entries > > device_tree.c | 182 +++++++++++++++++++-- > hw/arm/boot.c | 6 +- > hw/arm/sysbus-fdt.c | 319 +++++++++++++++++++++++++++++++++++-- > hw/arm/vexpress.c | 6 +- > hw/pci-bridge/ioh3420.c | 2 +- > hw/pci-bridge/xio3130_downstream.c | 2 +- > hw/pci-bridge/xio3130_upstream.c | 2 +- > hw/pci/pcie.c | 2 +- > hw/pci/pcie_aer.c | 6 +- > hw/vfio/Makefile.objs | 1 + > hw/vfio/amd-xgbe.c | 55 +++++++ > hw/vfio/pci.c | 13 +- > include/hw/pci/pcie_aer.h | 3 +- > include/hw/vfio/vfio-amd-xgbe.h | 51 ++++++ > include/sysemu/device_tree.h | 53 +++++- > 15 files changed, 660 insertions(+), 43 deletions(-) > create mode 100644 hw/vfio/amd-xgbe.c > create mode 100644 include/hw/vfio/vfio-amd-xgbe.h To be clear, this is a respin of the 2/18 pull with Eric's v8 making the following change: + g_slist_free_full(path_list, g_free); becomes: + for (iter = path_list; iter; iter = iter->next) { + g_free(iter->data); + } + g_slist_free(path_list); Also pulled in a trivial patch from Wei Yang. Thanks, Alex