[PATCH 0/3] Alternative proposal to revert the removal of logging of errors from virNetDevSendEthtoolIoctl

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

 



Rather than revert as proposed here:

http://www.redhat.com/archives/libvir-list/2015-November/msg00082.html

Let's make some adjustments to the existing algorithm.

The only question I don't have answered in my mind is whether an
unprivileged daemon that has an interface with the feature bits
defined should be allowed to start. Historically we haven't stopped
it, so doing so now may not be right/good. Conversely, someone may
"believe" their bits are set on their device when in fact they are
not. Of course it's not documented on formatnode.html that setting
the feature bits requires root. I did find through various searches
that SIOCETHTOOL doesn't work unless it's run under root.

John Ferlan (3):
  virnetdev: Check correct return value for virNetDevFeatureAvailable
  virnetdev: Message in virNetDevSendEthtoolIoctl rather than caller.
  virnetdev: Add check for unprivileged daemon

 src/util/virnetdev.c | 38 ++++++++++++++++++++++++--------------
 1 file changed, 24 insertions(+), 14 deletions(-)

-- 
2.1.0

--
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]