Re: [Qemu-devel] [PATCH v6 0/7] vfio: Prepare for SPAPR

[Alex Williamson]

On Fri, 2014-05-23 at 14:59 +1000, Alexey Kardashevskiy wrote:
> Yet another try with VFIO on SPAPR (server PPC64).
> 
> After a previous try, the series was split into few smaller ones,
> first one was "spapr_pci: Prepare for VFIO" which prepares spapr_pci.
> This one is the second one and prepares vfio. Third one will be posted
> after series 1 and 2 meet in some external tree.
> 
> Changelog is in individual patches.
> 
> Please comment. Thanks!
> 
> 
> Alexey Kardashevskiy (3):
>   int128: Add int128_exts64()
>   vfio: Fix 128 bit handling
>   vfio: Rework to have error paths
> 
> David Gibson (4):
>   memory: Sanity check that no listeners remain on a destroyed
>     AddressSpace
>   vfio: Introduce VFIO address spaces
>   vfio: Create VFIOAddressSpace objects as needed
>   vfio: Add guest side IOMMU support
> 
>  hw/misc/vfio.c        | 286 
> +++++++++++++++++++++++++++++++++++++++++---------
>  include/qemu/int128.h |   5 +
>  memory.c              |   7 ++
>  3 files changed, 249 insertions(+), 49 deletions(-)
> 

I'm down to one nit for this series[1] and given Paolo's reviewed-bys
for the minor tweaks to common code, I think we're finally about ready
for liftoff on this code.  I'll let AlexG decided whether he's satisfied
with the discussion on 5/7 before I pull them in though.  Thanks,

Alex

[1] 4/7: all the error_reports stay in place except the
listener_release_exit goto.  why?