From: Wei Yongjun <[email protected]>

Upstream commit:
    commit 6dc14dc40a1d1dafd8491c349b5f3e15aabc4edb
    Author: Wei Yongjun <[email protected]>
    Date:   Tue Nov 14 06:27:12 2017 +0000

    openvswitch: Using kfree_rcu() to simplify the code

    The callback function of call_rcu() just calls a kfree(), so we
    can use kfree_rcu() instead of call_rcu() + callback function.

    Signed-off-by: Wei Yongjun <[email protected]>
    Acked-by: Pravin B Shelar <[email protected]>
    Signed-off-by: David S. Miller <[email protected]>

Cc: Wei Yongjun <[email protected]>
Signed-off-by: Greg Rose <[email protected]>
---
 datapath/meter.c | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/datapath/meter.c b/datapath/meter.c
index 2e58b6c..52ddd6c 100644
--- a/datapath/meter.c
+++ b/datapath/meter.c
@@ -42,19 +42,12 @@ static const struct nla_policy 
band_policy[OVS_BAND_ATTR_MAX + 1] = {
        [OVS_BAND_ATTR_STATS] = { .len = sizeof(struct ovs_flow_stats) },
 };
 
-static void rcu_free_ovs_meter_callback(struct rcu_head *rcu)
-{
-       struct dp_meter *meter = container_of(rcu, struct dp_meter, rcu);
-
-       kfree(meter);
-}
-
 static void ovs_meter_free(struct dp_meter *meter)
 {
        if (!meter)
                return;
 
-       call_rcu(&meter->rcu, rcu_free_ovs_meter_callback);
+       kfree_rcu(meter, rcu);
 }
 
 static struct hlist_head *meter_hash_bucket(const struct datapath *dp,
-- 
1.8.3.1

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to