[ovs-dev] [PATCH 13/16] ofproto: Additional simplifications.
Ethan Jackson
ethan at nicira.com
Thu Jun 12 18:55:45 UTC 2014
I don't think anyone (myself included) understands this code. That said:
Acked-by: Ethan Jackson <ethan at nicira.com>
On Mon, Jun 9, 2014 at 4:06 PM, Thomas Graf <tgraf at suug.ch> wrote:
> On 06/05/14 at 10:02pm, Ben Pfaff wrote:
>> This commit finishes the removal of asynchronous flow table operations
>> begun in the previous commit, by removing ofoperation and ofopgroup
>> entirely and all of the code that depended on them. Following this commit,
>> all the internal documentation and comments should again be consistent and
>> correct.
>>
>> Signed-off-by: Ben Pfaff <blp at nicira.com>
>
> Looks sane, minor comment below. I suggest someone else with better
> understanding of this code does a review as well.
>
>> ---
>> ofproto/connmgr.c | 4 +
>> ofproto/ofproto-provider.h | 124 ++------
>> ofproto/ofproto.c | 690 ++++++++++-----------------------------------
>> 3 files changed, 170 insertions(+), 648 deletions(-)
>
>> @@ -1263,17 +1180,9 @@ void
>> ofproto_rule_delete(struct ofproto *ofproto, struct rule *rule)
>> OVS_EXCLUDED(ofproto_mutex)
>> {
>> - struct ofopgroup *group;
>> - struct ofoperation *op;
>> -
>> ovs_mutex_lock(&ofproto_mutex);
>> - group = ofopgroup_create_unattached(ofproto);
>> - op = ofoperation_create(group, rule, OFOPERATION_DELETE, OFPRR_DELETE);
>> oftable_remove_rule__(ofproto, rule);
>> ofproto->ofproto_class->rule_delete(rule);
>> - ofoperation_complete(op, 0);
>> - ofopgroup_submit(group);
>> -
>> ovs_mutex_unlock(&ofproto_mutex);
>
> Does this need a ofmonitor_flush() as in ofproto_rule_delete__()?
>
>> }
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
More information about the dev
mailing list