Re: [PATCH] RDS: use rb_entry()

2016-12-21 Thread Doug Ledford
On 12/20/2016 9:02 AM, Geliang Tang wrote: > To make the code clearer, use rb_entry() instead of container_of() to > deal with rbtree. > > Signed-off-by: Geliang Tang > --- > net/rds/rdma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH] RDS: use rb_entry()

2016-12-20 Thread David Miller
From: Geliang Tang Date: Tue, 20 Dec 2016 22:02:18 +0800 > To make the code clearer, use rb_entry() instead of container_of() to > deal with rbtree. > > Signed-off-by: Geliang Tang Applied.

Re: [PATCH] RDS: use rb_entry()

2016-12-20 Thread Santosh Shilimkar
On 12/20/2016 6:02 AM, Geliang Tang wrote: To make the code clearer, use rb_entry() instead of container_of() to deal with rbtree. Signed-off-by: Geliang Tang --- Looks fine. Acked-by: Santosh Shilimkar

Re: [PATCH] RDS: use rb_entry()

2016-12-20 Thread Leon Romanovsky
On Tue, Dec 20, 2016 at 10:02:18PM +0800, Geliang Tang wrote: > To make the code clearer, use rb_entry() instead of container_of() to > deal with rbtree. > > Signed-off-by: Geliang Tang > --- > net/rds/rdma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Thanks,

[PATCH] RDS: use rb_entry()

2016-12-20 Thread Geliang Tang
To make the code clearer, use rb_entry() instead of container_of() to deal with rbtree. Signed-off-by: Geliang Tang --- net/rds/rdma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/rds/rdma.c b/net/rds/rdma.c index 4c93bad..ea96114 100644 ---