[ovs-dev] [PATCH 2/2] ofproto: Remove ofproto_refresh_rule().

Joe Stringer joestringer at nicira.com
Tue Jun 3 22:48:08 UTC 2014


Thanks, I pushed this series to master and branch-2.3.


On 4 June 2014 10:06, Alex Wang <alexw at nicira.com> wrote:

> Acked-by: Alex Wang <alexw at nicira.com>
>
>
> On Tue, Jun 3, 2014 at 2:59 AM, Joe Stringer <joestringer at nicira.com>
> wrote:
>
>> The only user of this function was removed in the previous patch, so
>> remove it.
>>
>> Signed-off-by: Joe Stringer <joestringer at nicira.com>
>> ---
>>  ofproto/ofproto-dpif.c     |   12 ------------
>>  ofproto/ofproto-provider.h |    2 --
>>  ofproto/ofproto.c          |   41
>> -----------------------------------------
>>  3 files changed, 55 deletions(-)
>>
>> diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
>> index 06be234..22ba14e 100644
>> --- a/ofproto/ofproto-dpif.c
>> +++ b/ofproto/ofproto-dpif.c
>> @@ -364,18 +364,6 @@ ofproto_dpif_flow_mod(struct ofproto_dpif *ofproto,
>>      ofproto_flow_mod(&ofproto->up, fm);
>>  }
>>
>> -/* Resets the modified time for 'rule' or an equivalent rule. If 'rule'
>> is not
>> - * in the classifier, but an equivalent rule is, unref 'rule' and ref
>> the new
>> - * rule. Otherwise if 'rule' is no longer installed in the classifier,
>> - * reinstall it.
>> - *
>> - * Returns the rule whose modified time has been reset. */
>> -struct rule_dpif *
>> -ofproto_dpif_refresh_rule(struct rule_dpif *rule)
>> -{
>> -    return rule_dpif_cast(ofproto_refresh_rule(&rule->up));
>> -}
>> -
>>  /* Appends 'pin' to the queue of "packet ins" to be sent to the
>> controller.
>>   * Takes ownership of 'pin' and pin->packet. */
>>  void
>> diff --git a/ofproto/ofproto-provider.h b/ofproto/ofproto-provider.h
>> index ff539b9..7741044 100644
>> --- a/ofproto/ofproto-provider.h
>> +++ b/ofproto/ofproto-provider.h
>> @@ -1716,8 +1716,6 @@ BUILD_ASSERT_DECL(OFPROTO_POSTPONE < OFPERR_OFS);
>>
>>  int ofproto_flow_mod(struct ofproto *, struct ofputil_flow_mod *)
>>      OVS_EXCLUDED(ofproto_mutex);
>> -struct rule *ofproto_refresh_rule(struct rule *rule)
>> -    OVS_EXCLUDED(ofproto_mutex);
>>  void ofproto_add_flow(struct ofproto *, const struct match *,
>>                        unsigned int priority,
>>                        const struct ofpact *ofpacts, size_t ofpacts_len)
>> diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
>> index 24a709b..f34cfde 100644
>> --- a/ofproto/ofproto.c
>> +++ b/ofproto/ofproto.c
>> @@ -1978,47 +1978,6 @@ ofproto_flow_mod(struct ofproto *ofproto, struct
>> ofputil_flow_mod *fm)
>>      return handle_flow_mod__(ofproto, NULL, fm, NULL);
>>  }
>>
>> -/* Resets the modified time for 'rule' or an equivalent rule. If 'rule'
>> is not
>> - * in the classifier, but an equivalent rule is, unref 'rule' and ref
>> the new
>> - * rule. Otherwise if 'rule' is no longer installed in the classifier,
>> - * reinstall it.
>> - *
>> - * Returns the rule whose modified time has been reset. */
>> -struct rule *
>> -ofproto_refresh_rule(struct rule *rule)
>> -{
>> -    const struct oftable *table = &rule->ofproto->tables[rule->table_id];
>> -    const struct cls_rule *cr = &rule->cr;
>> -    struct rule *r;
>> -
>> -    /* do_add_flow() requires that the rule is not installed. We lock the
>> -     * ofproto_mutex here so that another thread cannot add the flow
>> before
>> -     * we get a chance to add it.*/
>> -    ovs_mutex_lock(&ofproto_mutex);
>> -
>> -    fat_rwlock_rdlock(&table->cls.rwlock);
>> -    r = rule_from_cls_rule(classifier_find_rule_exactly(&table->cls,
>> cr));
>> -    if (r != rule) {
>> -        ofproto_rule_ref(r);
>> -    }
>> -    fat_rwlock_unlock(&table->cls.rwlock);
>> -
>> -    if (!r) {
>> -        do_add_flow(rule->ofproto, NULL, NULL, 0, rule);
>> -    } else if  (r != rule) {
>> -        ofproto_rule_unref(rule);
>> -        rule = r;
>> -    }
>> -    ovs_mutex_unlock(&ofproto_mutex);
>> -
>> -    /* Refresh the modified time for the rule. */
>> -    ovs_mutex_lock(&rule->mutex);
>> -    rule->modified = MAX(rule->modified, time_msec());
>> -    ovs_mutex_unlock(&rule->mutex);
>> -
>> -    return rule;
>> -}
>> -
>>  /* Searches for a rule with matching criteria exactly equal to 'target'
>> in
>>   * ofproto's table 0 and, if it finds one, deletes it.
>>   *
>> --
>> 1.7.10.4
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20140604/359f9c99/attachment-0005.html>


More information about the dev mailing list