[PATCH 0/5] Revert virNetDevSendEthtoolIoctl logging errors and adjust code after

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

 



It's partially a reposting, partially some new stuff, and perhaps a v2
of something else.

Patch 1 is Dan's revert patch - just because it's easier to have it
done first.

Patches 2 & 3 just adjust comments - no functional changes

Patch 4 adds a check for root in virNetDevGetFeatures (rather than
both of the callers). If not root, an empty bitmap would be returned
(somewhat similar to what a platform without the features does). I'm
not against moving the check to before the virBitmapNew; however, I
was considering the case of xml parsing which does a virBitmapFree,
then just calls virNetDevGetFeatures. Not filling in the *out with
either NULL or an empty bitmap would result in that caller referencing
a net.features which no longer exists. If the check should be done
before the virBitmapNew, then an "*out = NULL" would need to be
added before returning 0 just to be 'safe'...

As an aside, it just felt better to add one check rather than two.

Patch 5 restore the virNetDevSetupControl from the reverted patch
back into virNetDevSendEthtoolIoctl. Since we cannot be called unless
we're privileged and the ioctl shouldn't fail any with EPERM, that
was also removed from the list of filtered failure errno values.

Daniel P. Berrange (1):
  Revert "utils: Remove the logging of errors from
    virNetDevSendEthtoolIoctl"

John Ferlan (4):
  virnetdev: Document reasons for ignoring some SIOCETHTOOL errno values
  virnetdev: Fix function comments for virNetDevGetFeatures
  virnetdev: Check for root in virNetDevGetFeatures
  virnetdev: Use virNetDevSetupControl in virNetDevSendEthtoolIoctl

 src/util/virnetdev.c | 52 ++++++++++++++++++++++++++++++++++++++--------------
 1 file changed, 38 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]