diff options
author | Saurabh Sengar <saurabh.truth@gmail.com> | 2015-10-30 10:16:44 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-11-02 15:35:19 -0500 |
commit | d3ffaefa1bbcfba1b4ef985fd53406f31b98f277 (patch) | |
tree | 95d0e2c2f1f62bedcb281b499d0a57c5785fc6aa /net/rds | |
parent | 64cf370887ac497e74b2e4224b662bfd1d8db41a (diff) |
net: rds: changing the return type from int to void
as result of function rds_iw_flush_mr_pool is nowhere checked,
changing its return type from int to void.
also removing the unused variable rc as there is nothing to return
Signed-off-by: Saurabh Sengar <saurabh.truth@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/rds')
-rw-r--r-- | net/rds/iw_rdma.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/rds/iw_rdma.c b/net/rds/iw_rdma.c index 6a8fbd6e69e7..d3d4454ffc84 100644 --- a/net/rds/iw_rdma.c +++ b/net/rds/iw_rdma.c | |||
@@ -75,7 +75,7 @@ struct rds_iw_mr_pool { | |||
75 | int max_pages; | 75 | int max_pages; |
76 | }; | 76 | }; |
77 | 77 | ||
78 | static int rds_iw_flush_mr_pool(struct rds_iw_mr_pool *pool, int free_all); | 78 | static void rds_iw_flush_mr_pool(struct rds_iw_mr_pool *pool, int free_all); |
79 | static void rds_iw_mr_pool_flush_worker(struct work_struct *work); | 79 | static void rds_iw_mr_pool_flush_worker(struct work_struct *work); |
80 | static int rds_iw_init_fastreg(struct rds_iw_mr_pool *pool, struct rds_iw_mr *ibmr); | 80 | static int rds_iw_init_fastreg(struct rds_iw_mr_pool *pool, struct rds_iw_mr *ibmr); |
81 | static int rds_iw_map_fastreg(struct rds_iw_mr_pool *pool, | 81 | static int rds_iw_map_fastreg(struct rds_iw_mr_pool *pool, |
@@ -479,14 +479,13 @@ void rds_iw_sync_mr(void *trans_private, int direction) | |||
479 | * If the number of MRs allocated exceeds the limit, we also try | 479 | * If the number of MRs allocated exceeds the limit, we also try |
480 | * to free as many MRs as needed to get back to this limit. | 480 | * to free as many MRs as needed to get back to this limit. |
481 | */ | 481 | */ |
482 | static int rds_iw_flush_mr_pool(struct rds_iw_mr_pool *pool, int free_all) | 482 | static void rds_iw_flush_mr_pool(struct rds_iw_mr_pool *pool, int free_all) |
483 | { | 483 | { |
484 | struct rds_iw_mr *ibmr, *next; | 484 | struct rds_iw_mr *ibmr, *next; |
485 | LIST_HEAD(unmap_list); | 485 | LIST_HEAD(unmap_list); |
486 | LIST_HEAD(kill_list); | 486 | LIST_HEAD(kill_list); |
487 | unsigned long flags; | 487 | unsigned long flags; |
488 | unsigned int nfreed = 0, ncleaned = 0, unpinned = 0; | 488 | unsigned int nfreed = 0, ncleaned = 0, unpinned = 0; |
489 | int ret = 0; | ||
490 | 489 | ||
491 | rds_iw_stats_inc(s_iw_rdma_mr_pool_flush); | 490 | rds_iw_stats_inc(s_iw_rdma_mr_pool_flush); |
492 | 491 | ||
@@ -538,7 +537,6 @@ static int rds_iw_flush_mr_pool(struct rds_iw_mr_pool *pool, int free_all) | |||
538 | atomic_sub(nfreed, &pool->item_count); | 537 | atomic_sub(nfreed, &pool->item_count); |
539 | 538 | ||
540 | mutex_unlock(&pool->flush_lock); | 539 | mutex_unlock(&pool->flush_lock); |
541 | return ret; | ||
542 | } | 540 | } |
543 | 541 | ||
544 | static void rds_iw_mr_pool_flush_worker(struct work_struct *work) | 542 | static void rds_iw_mr_pool_flush_worker(struct work_struct *work) |