[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 0/8] VFIO support part 3

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: 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!

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]