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?