[ovs-git] Open vSwitch: ofproto-dpif: Get rid of effectively unused 'check_special' flag. (next)

Wed May 11 19:18:53 UTC 2011

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, next has been updated
  discards  016ddb8b803daeac2280e726b8e55cdda1f28a0c (commit)
  discards  f2c99cd07a07c8ba04f58d49988ede2804dd7617 (commit)
  discards  72ae2ec6476862e91de4bfd433ead7f9bedd8b01 (commit)
  discards  59a62fffd1d503e8e66e097cf960b2d87caad933 (commit)
  discards  484e2fb7d07c9a1013253ee70d67f52ad9eee59d (commit)
  discards  ffe3f8d00d71cc8f151899e41df5663be92e7929 (commit)
  discards  5cccd9a7672995dfc4a60ebe4da69ddc0b66e385 (commit)
  discards  2776c90a0eaeba9cc53e12e9d14feac9c29a4dc2 (commit)
  discards  a6d199fcf08a882e1f17b5726d55f1094770b034 (commit)
  discards  27ab842867fa7921b8a8a2be4b0776256486b088 (commit)
  discards  6976b45570e26631a1912e2b6029ffafef50fea4 (commit)
  discards  ed1fc1fef541d5dd5896781d3862990223d82b1a (commit)
  discards  53c64b82e907b6720f9a1c8b04b9c2400211daa0 (commit)
       via  e8fe684bd0b67f5bacc142904999fccdbb2aad74 (commit)
       via  788ac766fe97a7a0460dedd51d48de90586b7b0e (commit)
       via  6bb8be1b5a562094eba0a3fab24d8e2d3d4121fb (commit)
       via  9b5349b612d78f9e57ed1d2469516ad1d7331dca (commit)
       via  a3cbcf28b0d408cfb76fe9fad2fc22e0a6f42f95 (commit)
       via  3b8b876a70ad56ac2e8c8411994ba142a12d71d6 (commit)
       via  8327af83ccaadbd889b216ec6fc3e294828639ec (commit)
       via  a8d3ad9d5136139691d8cf1a0e234a30538e6da4 (commit)
       via  0a2cd0cf99e86c5e172c18555c4e2c6cb97d1807 (commit)
       via  2e7ed36af2e43332d29c014c1a1c388b708f3b14 (commit)
       via  4dc8ba8a6b7d5358e29c6080c9a2eb265211779a (commit)
       via  c796d58290cdc027094a301cfab682429e566c13 (commit)
       via  8f559832e54ef4938309f8e0dff48011f3afbde7 (commit)
      from  016ddb8b803daeac2280e726b8e55cdda1f28a0c (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit e8fe684bd0b67f5bacc142904999fccdbb2aad74
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e8fe684bd0b67f5bacc142904999fccdbb2aad74
Author: Ben Pfaff <blp at nicira.com>

ofproto-dpif: Get rid of effectively unused 'check_special' flag.

Nothing ever sets this flag to false any longer, so there's no need to
store it or test its value.

Reported-by: Ethan Jackson <ethan at nicira.com>

commit 788ac766fe97a7a0460dedd51d48de90586b7b0e
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=788ac766fe97a7a0460dedd51d48de90586b7b0e
Author: Ben Pfaff <blp at nicira.com>

Implement basic multiple table support.

This implements basic multiple table support in ofproto and supporting
libraries and utilities. The design is the same as the one that has been
on the Open vSwitch "wdp" branch for a long time.  There is no support for
multiple tables in the software switch implementation (ofproto-dpif), only
a set of hooks for other switch implementations to use.

To allow controllers to add flows in a particular table, Open vSwitch adds
an OpenFlow 1.0 extension called NXT_FLOW_MOD_TABLE_ID.

commit 6bb8be1b5a562094eba0a3fab24d8e2d3d4121fb
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=6bb8be1b5a562094eba0a3fab24d8e2d3d4121fb
Author: Ben Pfaff <blp at nicira.com>

ofproto: Drop ofproto_rule_lookup().

There's no reason not to implement this trivial function in ofproto-dpif,
especially since it makes less sense once multiple table support is
implemented (which table should be searched?).

commit 9b5349b612d78f9e57ed1d2469516ad1d7331dca
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=9b5349b612d78f9e57ed1d2469516ad1d7331dca
Author: Ben Pfaff <blp at nicira.com>

PORTING: Describe usage of ovs_be<N>.

commit a3cbcf28b0d408cfb76fe9fad2fc22e0a6f42f95
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=a3cbcf28b0d408cfb76fe9fad2fc22e0a6f42f95
Author: Ben Pfaff <blp at nicira.com>

ofproto: Fix number of reported tables in OFPT_FEATURES_REPLY message.

This has been wrong for a long time.

commit 3b8b876a70ad56ac2e8c8411994ba142a12d71d6
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=3b8b876a70ad56ac2e8c8411994ba142a12d71d6
Author: Ben Pfaff <blp at nicira.com>

ofproto: Make rule construction and destruction more symmetric.

Before, ->rule_construct() both created the rule and inserted into the
flow table, but ->rule_destruct() only destroyed the rule.  This makes
->rule_destruct() also remove the rule from the flow table.

commit 8327af83ccaadbd889b216ec6fc3e294828639ec
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=8327af83ccaadbd889b216ec6fc3e294828639ec
Author: Ben Pfaff <blp at nicira.com>

classifier: Remove OF1.0 special case from classifier_find_rule_exactly().

This special case should never have actually triggered in practice, because
OpenFlow 1.0 cannot set up an exact-match rule as defined by
flow_wildcards_is_exact().  (OpenFlow 1.0 will always, for example,
wildcard all NXM registers.)

OVS implements this OF1.0 special case differently, by changing flow
priority to 65535 in cls_rule_from_match() if the flow is an exact match as
defined by OpenFlow 1.0.

commit a8d3ad9d5136139691d8cf1a0e234a30538e6da4
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=a8d3ad9d5136139691d8cf1a0e234a30538e6da4
Author: Ben Pfaff <blp at nicira.com>

ofproto: Remove unused coverage counters.

These were mostly moved into ofproto-dpif.c, but the definitions weren't
deleted along with them.

commit 0a2cd0cf99e86c5e172c18555c4e2c6cb97d1807
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=0a2cd0cf99e86c5e172c18555c4e2c6cb97d1807
Author: Ben Pfaff <blp at nicira.com>

ofproto: Update some comments.

commit 2e7ed36af2e43332d29c014c1a1c388b708f3b14
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=2e7ed36af2e43332d29c014c1a1c388b708f3b14
Author: Ben Pfaff <blp at nicira.com>

ofproto: Eliminate reference to dpif_upcall from ofproto.

The dpif_upcall structure is specific to the ofproto-dpif implementation.
The generic ofproto and connmgr interface have no business using it, so
this commit switches to using ofputil_packet_in instead.

commit 4dc8ba8a6b7d5358e29c6080c9a2eb265211779a
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=4dc8ba8a6b7d5358e29c6080c9a2eb265211779a
Author: Ben Pfaff <blp at nicira.com>

Remove unneeded #include directives.

commit c796d58290cdc027094a301cfab682429e566c13
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=c796d58290cdc027094a301cfab682429e566c13
Author: Ben Pfaff <blp at nicira.com>

ofproto: Better document the ofproto_class interface.

Also, make a few minor adjustments to the interface so that it makes a
little more sense.

commit 8f559832e54ef4938309f8e0dff48011f3afbde7
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=8f559832e54ef4938309f8e0dff48011f3afbde7
Author: Ben Pfaff <blp at nicira.com>

ofproto: Break apart into generic and hardware-specific parts.

In addition to the changes to ofproto, this commit changes all of the
instances of "struct flow" in the tree so that the "in_port" member is an
OpenFlow port number.  Previously, this member was an OpenFlow port number
in some cases and an ODP port number in other cases.

-----------------------------------------------------------------------

Summary of changes:
 ofproto/ofproto-dpif.c |   12 +++++++-----
 ofproto/ofproto.c      |   13 +++++++------
 ofproto/private.h      |   15 ++++++++-------
 3 files changed, 22 insertions(+), 18 deletions(-)

hooks/post-receive
-- 
Open vSwitch