On Thu, Feb 11, 2016 at 12:24:43PM +0530, [email protected] wrote:
> From: Niti Rohilla <[email protected]>
>
> "ovs-ofctl set-first-egress-table <SWITCH> <table_id>" can be used to set
> first egress
> table.
> This patch enhances TABLE_FEATURES_REQUEST message to set the first egress
> table and
> TABLE_FEATURES_REPLY to identify the table which is configured as first
> egress table.
>
> Signed-off-by: Niti Rohilla <[email protected]>
There's a major problem here: it stores the first egress table in a
global variable, in function ofputil_egress_table_id(). That's not
acceptable. It will not work properly when there is more than one
bridge.
The function ofpact_put_ofp15_write_actions() has a comment that doesn't
describe what it actually does, and it can be better written as just
f->nonmiss.write.ofpacts &= ~((UINT64_C(1) << OFPACT_OUTPUT) |
(UINT64_C(1) << OFPACT_GROUP));
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
