On 03/25/2014 03:27 PM, Serge Hallyn wrote: > Quoting Cedric Bosdonnat (cbosdonnat@xxxxxxxx): >> On Tue, 2014-03-25 at 10:40 -0500, Serge Hallyn wrote: >>> Quoting Cédric Bosdonnat (cbosdonnat@xxxxxxxx): >>>> See lp#1276719 for the bug description. As virt-aa-helper doesn't know >>>> the VFIO groups to use for the guest, allow access to all >>>> /dev/vfio/[0-9]* and /dev/vfio/vfio files if there is a potential need >>>> for vfio >>>> --- >>> >>> Thanks, Cédric! Looks good to me. Still needs a signed-off-by from you >>> (I assume), but >>> >>> Acked-by: Serge E. Hallyn <serge.hallyn@xxxxxxxxxx> >> >> I wasn't aware I needed to sign-off my patches, but I can resubmit with >> it ;) > > Actually it looks like I'm wrong, libvirt doesn't require them: > > http://libvirt.org/hacking.html (point 3) > > I've pushed this patch to ppa:ubuntu-virt/candidate, which is meant to > go into trusty when qemu 2.0 is released. > + if (needsVfio) { > + virBufferAsprintf(&buf, " /dev/vfio/vfio rw,\n"); > + virBufferAsprintf(&buf, " /dev/vfio/[0-9]* rw,\n"); virBufferAsprintf should only be used with % format strings. This fails 'make syntax-check': prohibit_virBufferAsprintf_with_string_literal src/security/virt-aa-helper.c:1107: virBufferAsprintf(&buf, " /dev/vfio/vfio rw,\n"); src/security/virt-aa-helper.c:1108: virBufferAsprintf(&buf, " /dev/vfio/[0-9]* rw,\n"); maint.mk: use virBufferAddLit, not virBufferAsprintf, with a string literal I made the obvious change, and pushed in time for 1.2.3. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list