[ovs-dev] [PATCH] ofp-parse: Fix parsing, formatting of multiple fields in NTR extension.

Simon Horman simon.horman at netronome.com
Mon Oct 19 02:17:16 UTC 2015


On Fri, Oct 16, 2015 at 08:21:06AM -0700, Ben Pfaff wrote:
> Until now, the only way to specify multiple fields in the "fields"
> parameter for the Netronome groups extension, was to specify "fields"
> more than once, e.g. fields=eth_dst,fields=ip_dst
> 
> However, this wasn't documented and the code in ofp-print didn't use it,
> generating output that couldn't be parsed.
> 
> This commit fixes the situation by introducing a more straightforward
> syntax, e.g. fields(eth_dst,ip_dst), documents it, and adjusts ofp-print
> code to use it when there is more than one field (it retains the previous
> format for backward compatibility when there is exactly one field)
> 
> CC: Simon Horman <simon.horman at netronome.com>
> Signed-off-by: Ben Pfaff <blp at nicira.com>

Acked-by: Simon Horman <simon.horman at netronome.com>




More information about the dev mailing list