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

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

 



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!

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]