[ovs-git] [openvswitch/ovs] bc7845: ofp-parse: Separate fields properly.

GitHub noreply at github.com
Wed Sep 9 16:49:02 UTC 2015


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: bc78455bdbabd5f16abde1d0b4d63d488286f2e3
      https://github.com/openvswitch/ovs/commit/bc78455bdbabd5f16abde1d0b4d63d488286f2e3
  Author: Jesse Gross <jesse at nicira.com>
  Date:   2015-09-09 (Wed, 09 Sep 2015)

  Changed paths:
    M lib/ofp-parse.c

  Log Message:
  -----------
  ofp-parse: Separate fields properly.

Currently, each token in an OpenFlow match field is treated separately -
whether this is a name, a value, or a single identifier. However, this
means that attempting to get a value may result in grabbing the next
token if no value exists. This avoids that problem by breaking the match
string down into its components and then individually separating it into
name/value pairs if appropriate.

Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


  Commit: 0eede6b600dcf946e1d5de0fab508d09ec27b86b
      https://github.com/openvswitch/ovs/commit/0eede6b600dcf946e1d5de0fab508d09ec27b86b
  Author: Jesse Gross <jesse at nicira.com>
  Date:   2015-09-09 (Wed, 09 Sep 2015)

  Changed paths:
    M lib/ofp-parse.c

  Log Message:
  -----------
  ofp-parse: Allow ofctl flow monitor filtering on field existence.

It is supposed to be possible to allow ovs-ofctl to filter flows
it is monitoring based on a match string. However, the parser will
reject expressions that match only on a field's existence (such as
Geneve options). This relaxes the restriction to bring it in line
with matches supported by other commands.

Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


  Commit: 4f7b100c2b52f39eb686888c181ed0e57a574e23
      https://github.com/openvswitch/ovs/commit/4f7b100c2b52f39eb686888c181ed0e57a574e23
  Author: Jesse Gross <jesse at nicira.com>
  Date:   2015-09-09 (Wed, 09 Sep 2015)

  Changed paths:
    M lib/meta-flow.c
    M lib/meta-flow.h
    M lib/nx-match.c
    M lib/ofp-parse.c
    M lib/tun-metadata.c
    M lib/tun-metadata.h
    M tests/tunnel.at

  Log Message:
  -----------
  tun-metadata: Provide error messages during auto-allocation.

In cases where we don't have a map of tunnel metadata options (such
as with ovs-ofctl) we dynamically allocate them as part of the match.
However, dynamic allocation brings the possibility of errors such as
duplicate entries or running out of space. Up until now, anything that
would cause an error was silently ignored. Since that is not very user
friendly, this adds a mechanism for reporting these types of errors.

Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


Compare: https://github.com/openvswitch/ovs/compare/9e3ddd45eb4b...4f7b100c2b52


More information about the git mailing list