diff mbox

IB/hfi1: use kvfree() in sdma.c

Message ID 9ab0afd682cc1b3919ef926aa9086eb09c9f90e9.1442864311.git.geliangtang@163.com (mailing list archive)
State Not Applicable
Headers show

Commit Message

Geliang Tang Sept. 21, 2015, 11:43 a.m. UTC
Use kvfree() instead of open-coding it.

Signed-off-by: Geliang Tang <geliangtang@163.com>
---
 drivers/staging/rdma/hfi1/sdma.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comments

Marciniszyn, Mike Sept. 21, 2015, 1:18 p.m. UTC | #1
> Subject: [PATCH] IB/hfi1: use kvfree() in sdma.c
> 
> Use kvfree() instead of open-coding it.
> 
> Signed-off-by: Geliang Tang <geliangtang@163.com>
> ---
>  drivers/staging/rdma/hfi1/sdma.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)

Thanks for the patch.

Acked-by: Mike Marciniszyn <mike.marciniszyn@intel.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/staging/rdma/hfi1/sdma.c b/drivers/staging/rdma/hfi1/sdma.c
index aecd1a7..9c02a3d 100644
--- a/drivers/staging/rdma/hfi1/sdma.c
+++ b/drivers/staging/rdma/hfi1/sdma.c
@@ -966,10 +966,7 @@  static void sdma_clean(struct hfi1_devdata *dd, size_t num_engines)
 			sde->descq = NULL;
 			sde->descq_phys = 0;
 		}
-		if (is_vmalloc_addr(sde->tx_ring))
-			vfree(sde->tx_ring);
-		else
-			kfree(sde->tx_ring);
+		kvfree(sde->tx_ring);
 		sde->tx_ring = NULL;
 	}
 	spin_lock_irq(&dd->sde_map_lock);