On 04/25/2013 01:57 PM, Laine Stump wrote:
This *almost* completes VFIO support. I still need to add a couple of xml/args test cases.
I actually added the test cases as a separate patch in this series, but then squashed them into the earlier patch that constructs a qemu commandline with vfio support before pushing
Laine Stump (8): util: new function virPCIDeviceGetVFIOGroupDev security: update hostdev labelling functions for VFIO
qemu: add QEMU_CAPS_VFIO_PCI_BOOTINDEX qemu: use vfio-pci on commandline when appropriate
I pushed the above 4 patches, but didn't push the ones below
[PATCH 3/8] qemu: add VFIO devices to cgroup ACL
Eric questioned whether 3/8 was doing the right thing, and I'd like to get danpb's opinion.
[PATCH 6/8] util: new virCommandSetMax(MemLock|Processes|Files) [PATCH 7/8] qemu: use new virCommandSetMax(Processes|Files) [PATCH 8/8] qemu: set qemu process' RLIMIT_MEMLOCK when VFIO is used
Eric pointed out several problems with 6/8, which I've addressed, and 7/8 and 8/8 depend on 6/8 so I can't push them yet. To make things easier, I'm going to close out this thread and repost a new thread with just these 4 remaining patches. If they are ACKed and DV wants to freeze before I wake up, please feel free to push them. Thanks for all the quick reviews!