diff options
| author | Santosh Shilimkar <santosh.shilimkar@oracle.com> | 2012-02-03 11:09:23 -0500 |
|---|---|---|
| committer | Santosh Shilimkar <ssantosh@kernel.org> | 2015-09-30 12:43:24 -0400 |
| commit | 59fe4606748f7016e9e02bbd26da185f8620661a (patch) | |
| tree | 016296245d4f2e0129d2caecde0e07bacede448f /net/rds | |
| parent | 6ff33f3902c3b1c5d0db6b1e2c70b6d76fba357f (diff) | |
| download | op-kernel-dev-59fe4606748f7016e9e02bbd26da185f8620661a.zip op-kernel-dev-59fe4606748f7016e9e02bbd26da185f8620661a.tar.gz | |
RDS: use kfree_rcu in rds_ib_remove_ipaddr
synchronize_rcu() slowing down un-necessarily the socket shutdown
path. It is used just kfree() the ip addresses in rds_ib_remove_ipaddr()
which is perfect usecase for kfree_rcu();
So lets use that to gain some speedup.
Signed-off-by: Santosh Shilimkar <ssantosh@kernel.org>
Signed-off-by: Santosh Shilimkar <santosh.shilimkar@oracle.com>
Diffstat (limited to 'net/rds')
| -rw-r--r-- | net/rds/ib.h | 1 | ||||
| -rw-r--r-- | net/rds/ib_rdma.c | 6 |
2 files changed, 3 insertions, 4 deletions
diff --git a/net/rds/ib.h b/net/rds/ib.h index aae60fd..f1fd5ffec 100644 --- a/net/rds/ib.h +++ b/net/rds/ib.h @@ -164,6 +164,7 @@ struct rds_ib_connection { struct rds_ib_ipaddr { struct list_head list; __be32 ipaddr; + struct rcu_head rcu; }; struct rds_ib_device { diff --git a/net/rds/ib_rdma.c b/net/rds/ib_rdma.c index 251d1ce..872f523 100644 --- a/net/rds/ib_rdma.c +++ b/net/rds/ib_rdma.c @@ -159,10 +159,8 @@ static void rds_ib_remove_ipaddr(struct rds_ib_device *rds_ibdev, __be32 ipaddr) } spin_unlock_irq(&rds_ibdev->spinlock); - if (to_free) { - synchronize_rcu(); - kfree(to_free); - } + if (to_free) + kfree_rcu(to_free, rcu); } int rds_ib_update_ipaddr(struct rds_ib_device *rds_ibdev, __be32 ipaddr) |
