| Message ID | 2c42065049bb2b99bededdc423a9babf4a98adee.1635093628.git.christophe.jaillet@wanadoo.fr (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Jason Gunthorpe |
| Headers | show |
| Series | [1/2] RDMA/rxe: Save a few bytes from struct rxe_pool | expand |
On Sun, Oct 24, 2021 at 06:43:30PM +0200, Christophe JAILLET wrote: > 'table_size' is never read, it can be removed. > > In fact, the only place that uses something that could be 'table_size' is > 'alloc_index()'. In this function, it is re-computed from 'min_index' and > 'max_index'. > > Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> > --- > drivers/infiniband/sw/rxe/rxe_pool.c | 1 - > drivers/infiniband/sw/rxe/rxe_pool.h | 1 - > 2 files changed, 2 deletions(-) This and the next applied to for-next, thanks Jason
diff --git a/drivers/infiniband/sw/rxe/rxe_pool.c b/drivers/infiniband/sw/rxe/rxe_pool.c index 7b4cb46edfd9..271d4ac0e0aa 100644 --- a/drivers/infiniband/sw/rxe/rxe_pool.c +++ b/drivers/infiniband/sw/rxe/rxe_pool.c @@ -114,7 +114,6 @@ static int rxe_pool_init_index(struct rxe_pool *pool, u32 max, u32 min) goto out; } - pool->index.table_size = size; bitmap_zero(pool->index.table, max - min + 1); out: diff --git a/drivers/infiniband/sw/rxe/rxe_pool.h b/drivers/infiniband/sw/rxe/rxe_pool.h index 1feca1bffced..1ff2250edf6d 100644 --- a/drivers/infiniband/sw/rxe/rxe_pool.h +++ b/drivers/infiniband/sw/rxe/rxe_pool.h @@ -74,7 +74,6 @@ struct rxe_pool { struct { struct rb_root tree; unsigned long *table; - size_t table_size; u32 last; u32 max_index; u32 min_index;
'table_size' is never read, it can be removed. In fact, the only place that uses something that could be 'table_size' is 'alloc_index()'. In this function, it is re-computed from 'min_index' and 'max_index'. Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> --- drivers/infiniband/sw/rxe/rxe_pool.c | 1 - drivers/infiniband/sw/rxe/rxe_pool.h | 1 - 2 files changed, 2 deletions(-)