On Thu, May 17, 2012 at 05:59:02PM +0900, Simon Horman wrote:
> Cc: Kyle Mestery <kmest...@cisco.com>
> Signed-off-by: Simon Horman <ho...@verge.net.au>

I think that this is inconsistent with patch 1, which I read as
printing these attributes in the form "ipv4_tunnel(...)" not
"tun_key(...)".  The output form should be valid for input.

You should add some tests for the ipv4_tunnel() formatting and parsing
to tests/odp.at.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to