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.
What about "Unbinding PCI device %s (stub driver %s)"?
> Hope they help some day!
They have certainly helped me while working on the rest of the
series ;)
I've tried to reply to each of your comments exactly once, please let me
know if I've missed any in the process. Thanks again for taking the time
to look at this.
Cheers.
--
Andrea Bolognani
Software Engineer - Virtualization Team
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list