Re: [ovs-dev] [PATCH] netdev-vport: Do not log empty warnings on success.

2017-01-12 Thread Daniele Di Proietto
On 12/01/2017 09:33, "Ben Pfaff" wrote: >On Thu, Jan 12, 2017 at 12:23:55AM -0800, Daniele Di Proietto wrote: >> set_tunnel_config() always logs a warning, even on success. This >> shouldn't happen. >> >> Without this, some unit tests fail. >> >> Fixes: 9fff138ec3a6("netdev:

Re: [ovs-dev] [PATCH] netdev-vport: Do not log empty warnings on success.

2017-01-12 Thread Fischetti, Antonio
l Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] On Behalf Of Daniele Di Proietto > Sent: Thursday, January 12, 2017 8:24 AM > To: d...@openvswitch.org > Cc: Daniele Di Proietto <diproiet...@vmware.com> > Subject: [ovs-dev] [PATC

[ovs-dev] [PATCH] netdev-vport: Do not log empty warnings on success.

2017-01-12 Thread Daniele Di Proietto
set_tunnel_config() always logs a warning, even on success. This shouldn't happen. Without this, some unit tests fail. Fixes: 9fff138ec3a6("netdev: Add 'errp' to set_config().") Signed-off-by: Daniele Di Proietto --- lib/netdev-vport.c | 10 ++ 1 file changed, 6