[libvirt] [PATCH 3/3] util: clean up and expand 802.1QbX negotiation logging
jferlan at redhat.com
Fri Feb 12 21:53:56 UTC 2016
On 02/11/2016 04:37 PM, Laine Stump wrote:
> The existing log messages for this have several problems; there are
> two lines of log when one will suffice, they duplicate the function
> name in log message (when it's already included by VIR_DEBUG), they're
> missing some useful bits, they get logged even when the call is a NOP.
> This patch cleans up the problems with those existing logs, and also
> adds a new VIR_INFO-level log down at the function that is actually
> creating and sending the netlink message that logs *everything* going
> into the netlink message (which turns out to be much more useful in
> practice for me; I didn't want to eliminate the logs at the existing
> location though, in case they are useful in some scenario I'm
> unfamiliar with; anyway those logs are remaining at debug level, so it
> shouldn't be a bother to anyone).
> src/util/virnetdevvportprofile.c | 60 ++++++++++++++++++++++++++++++++++------
> 1 file changed, 51 insertions(+), 9 deletions(-)
Less noise than pesky VIR_LOG_DEBUG that's for sure.... Hey if they're
useful, then they're useful. Duplicitous for those wanting to wade
through numerous _DEBUG level messages...
More information about the libvir-list