Re: [Qemu-devel] [PULL 00/14] VFIO updates 2016-02-19

[Alex Williamson]

On Fri, 19 Feb 2016 10:39:31 -0700
Alex Williamson <address@hidden> 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