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

[GitHub]

  Branch: refs/heads/branch-2.4
  Home:   https://github.com/openvswitch/ovs
  Commit: 4312f70e78f190264baf0022f04d1d67ae0766c9
      https://github.com/openvswitch/ovs/commit/4312f70e78f190264baf0022f04d1d67ae0766c9
  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>