[ovs-git] [openvswitch/ovs] 68dfc2: ofp-parse: Fix parsing, formatting of multiple fie...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 68dfc25b62e5edc2939bcae791a35fddfecb5d20
      https://github.com/openvswitch/ovs/commit/68dfc25b62e5edc2939bcae791a35fddfecb5d20
  Author: Ben Pfaff <blp at nicira.com>
  Date:   2015-11-03 (Tue, 03 Nov 2015)

  Changed paths:
    M lib/ofp-parse.c
    M lib/ofp-print.c
    M tests/ofp-print.at
    M tests/ofproto.at
    M utilities/ovs-ofctl.8.in

  Log Message:
  -----------
  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 at nicira.com>
Acked-by: Simon Horman <simon.horman at netronome.com>