On Thu, Aug 27, 2015 at 07:57:14PM +0000, Nithin Raju wrote:
> > This patch surpresses flow attribute probing in the windows datapath.
> > 
> > Signed-off-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
> 
> The change look good. Only thing I’d avoid is the OVS_LOG_ERROR()
> statement. The documentation for ‘OVS_FLOW_ATTR_PROBE’ explicitly
> mentions that logging should be suppressed.

When it's supported, it's supposed to suppress logging of *other*
errors.  Since it *isn't* supported, I think it's fair game to log that.

> Acked-by: Nithin Raju <nit...@vmware.com>

Thanks Alin and Nithin, I'll apply this to master and branch-2.4 in a
minute.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to