xen-netback: using kfree_rcu() to simplify the code
authorWei Yongjun <weiyongjun1@huawei.com>
Mon, 22 Aug 2016 23:01:29 +0000 (23:01 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 24 Aug 2016 00:00:26 +0000 (17:00 -0700)
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 <weiyongjun1@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/xen-netback/hash.c

index 282b16d8093a54e142b717b2c56bbd21f95c711f..e8c5dddc54ba27ee43354ab1fae263e0c31c5f1e 100644 (file)
 #include <linux/vmalloc.h>
 #include <linux/rculist.h>
 
-static void xenvif_del_hash(struct rcu_head *rcu)
-{
-       struct xenvif_hash_cache_entry *entry;
-
-       entry = container_of(rcu, struct xenvif_hash_cache_entry, rcu);
-
-       kfree(entry);
-}
-
 static void xenvif_add_hash(struct xenvif *vif, const u8 *tag,
                            unsigned int len, u32 val)
 {
@@ -76,7 +67,7 @@ static void xenvif_add_hash(struct xenvif *vif, const u8 *tag,
                if (++vif->hash.cache.count > xenvif_hash_cache_size) {
                        list_del_rcu(&oldest->link);
                        vif->hash.cache.count--;
-                       call_rcu(&oldest->rcu, xenvif_del_hash);
+                       kfree_rcu(oldest, rcu);
                }
        }
 
@@ -114,7 +105,7 @@ static void xenvif_flush_hash(struct xenvif *vif)
        list_for_each_entry_rcu(entry, &vif->hash.cache.list, link) {
                list_del_rcu(&entry->link);
                vif->hash.cache.count--;
-               call_rcu(&entry->rcu, xenvif_del_hash);
+               kfree_rcu(entry, rcu);
        }
 
        spin_unlock_irqrestore(&vif->hash.cache.lock, flags);