ofp-parse: Fix parsing, formatting of multiple fields in NTR extension.
authorBen Pfaff <blp@nicira.com>
Thu, 15 Oct 2015 16:46:21 +0000 (09:46 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 4 Nov 2015 03:45:29 +0000 (19:45 -0800)
commit68dfc25b62e5edc2939bcae791a35fddfecb5d20
treea64413e9309dabf569e1141b5c1ddef9a6c244af
parent337c452854451ed1b32d414a44c62b78b4a057c9
ofp-parse: Fix parsing, formatting of multiple fields in NTR extension.

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)

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Simon Horman <simon.horman@netronome.com>
lib/ofp-parse.c
lib/ofp-print.c
tests/ofp-print.at
tests/ofproto.at
utilities/ovs-ofctl.8.in