On Tue, 2016-01-26 at 19:00 -0500, John Ferlan wrote:
>
> > - VIR_DEBUG("Found stub driver %s", driver);
> > + VIR_DEBUG("Found stub driver %s for PCI device %s", driver, dev->name);
> > + VIR_DEBUG("Unbinding PCI device %s", dev->name);
>
> Redundant - How about "Found stub driver %s to unbind PCI device %s" or
> "Unbinding PCI device %s for stub driver %s" (don't forget to change
> order of args ;-))
>
> IOW: No need to have two messages.
>
> Hope they help some day!
>
> ACK -
I ended up using "Unbinding PCI device %s from stub driver %s" after all.
Pushed.
Cheers.
--
Andrea Bolognani
Software Engineer - Virtualization Team
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list