On Mon, Apr 30, 2018 at 04:09:40PM +0200, Jiri Denemark wrote: > On Fri, Apr 27, 2018 at 16:25:07 +0100, Daniel P. Berrangé wrote: > > The vm name is not needed for any functional requirement, but it will be > > useful when debugging problems to identify which VM is associated with a > > filter, since UUID is not human friendly. > > > > Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx> > > --- > > src/conf/domain_nwfilter.c | 5 +++-- > > src/conf/domain_nwfilter.h | 6 ++++-- > > src/lxc/lxc_process.c | 2 +- > > src/nwfilter/nwfilter_driver.c | 3 ++- > > src/qemu/qemu_hotplug.c | 6 ++++-- > > src/qemu/qemu_interface.c | 4 ++-- > > src/qemu/qemu_process.c | 2 +- > > src/uml/uml_conf.c | 2 +- > > 8 files changed, 18 insertions(+), 12 deletions(-) > > Reviewed-by: Jiri Denemark <jdenemar@xxxxxxxxxx> FYI, I have pushed upto & including this patch. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
