Commit b8b5cbd4 authored by Yunsheng Lin's avatar Yunsheng Lin Committed by ZhouJuan
Browse files

RDMA/hns: Remove redundant 'use_lowmem' argument from hns_roce_init_hem_table() 

mainline inclusion
from mainline-v6.1-rc1
commit 29dc0635
category: cleanup
bugzilla: https://gitee.com/openeuler/kernel/issues/I7A4LC
CVE: NA

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=29dc063596772368aa896f293f5c5aef06381712

----------------------------------------------------------------------

As hns_roce_init_hem_table() is always called with use_lowmem
being '1', and table->lowmem is set according to that argument,
so remove table->lowmem too.

Also, as the table->lowmem is used to indicate a dma buffer
is allocated with GFP_HIGHUSER or GFP_KERNEL, and calling
dma_alloc_coherent() with GFP_KERNEL seems like a common
pattern.

Link: https://lore.kernel.org/r/20220922123315.3732205-7-xuhaoyue1@hisilicon.com


Signed-off-by: default avatarYunsheng Lin <linyunsheng@huawei.com>
Signed-off-by: default avatarHaoyue Xu <xuhaoyue1@hisilicon.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@nvidia.com>
Signed-off-by: default avatarZhou Juan <nnuzj07170227@163.com>
parent 36771615

drivers/infiniband/hw/hns/hns_roce_device.h

+0 −1
Original line number Diff line number Diff line
@@ -293,7 +293,6 @@ struct hns_roce_hem_table { 
	/* Single obj size */

	unsigned long	obj_size;

	unsigned long	table_chunk_size;

	int		lowmem;

	struct mutex	mutex;

	struct hns_roce_hem **hem;

	u64		**bt_l1;

drivers/infiniband/hw/hns/hns_roce_hem.c

+3 −9
Original line number Diff line number Diff line
@@ -455,7 +455,7 @@ static int alloc_mhop_hem(struct hns_roce_dev *hr_dev, 
	 * alloc bt space chunk for MTT/CQE.

	 */

	size = table->type < HEM_TYPE_MTT ? mhop->buf_chunk_size : bt_size;

	flag = (table->lowmem ? GFP_KERNEL : GFP_HIGHUSER) | __GFP_NOWARN;

	flag = GFP_KERNEL | __GFP_NOWARN;

	table->hem[index->buf] = hns_roce_alloc_hem(hr_dev, size >> PAGE_SHIFT,

						    size, flag);

	if (!table->hem[index->buf]) {
@@ -588,8 +588,7 @@ int hns_roce_table_get(struct hns_roce_dev *hr_dev, 
	table->hem[i] = hns_roce_alloc_hem(hr_dev,

				       table->table_chunk_size >> PAGE_SHIFT,

				       table->table_chunk_size,

				       (table->lowmem ? GFP_KERNEL :

					GFP_HIGHUSER) | __GFP_NOWARN);

				       GFP_KERNEL | __GFP_NOWARN);

	if (!table->hem[i]) {

		ret = -ENOMEM;

		goto out;
@@ -738,9 +737,6 @@ void *hns_roce_table_find(struct hns_roce_dev *hr_dev, 
	int length;

	int i, j;



	if (!table->lowmem)

		return NULL;



	mutex_lock(&table->mutex);



	if (!hns_roce_check_whether_mhop(hr_dev, table->type)) {
@@ -796,8 +792,7 @@ void *hns_roce_table_find(struct hns_roce_dev *hr_dev, 


int hns_roce_init_hem_table(struct hns_roce_dev *hr_dev,

			    struct hns_roce_hem_table *table, u32 type,

			    unsigned long obj_size, unsigned long nobj,

			    int use_lowmem)

			    unsigned long obj_size, unsigned long nobj)

{

	unsigned long obj_per_chunk;

	unsigned long num_hem;
@@ -874,7 +869,6 @@ int hns_roce_init_hem_table(struct hns_roce_dev *hr_dev, 
	table->type = type;

	table->num_hem = num_hem;

	table->obj_size = obj_size;

	table->lowmem = use_lowmem;

	mutex_init(&table->mutex);



	return 0;

drivers/infiniband/hw/hns/hns_roce_hem.h

+1 −2
Original line number Diff line number Diff line
@@ -111,8 +111,7 @@ void *hns_roce_table_find(struct hns_roce_dev *hr_dev, 
			  dma_addr_t *dma_handle);

int hns_roce_init_hem_table(struct hns_roce_dev *hr_dev,

			    struct hns_roce_hem_table *table, u32 type,

			    unsigned long obj_size, unsigned long nobj,

			    int use_lowmem);

			    unsigned long obj_size, unsigned long nobj);

void hns_roce_cleanup_hem_table(struct hns_roce_dev *hr_dev,

				struct hns_roce_hem_table *table);

void hns_roce_cleanup_hem(struct hns_roce_dev *hr_dev);

drivers/infiniband/hw/hns/hns_roce_main.c

+10 −10
Original line number Diff line number Diff line
@@ -1105,7 +1105,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 


	ret = hns_roce_init_hem_table(hr_dev, &hr_dev->mr_table.mtpt_table,

				      HEM_TYPE_MTPT, hr_dev->caps.mtpt_entry_sz,

				      hr_dev->caps.num_mtpts, 1);

				      hr_dev->caps.num_mtpts);

	if (ret) {

		dev_err(dev, "Failed to init MTPT context memory, aborting.\n");

		return ret;
@@ -1113,7 +1113,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 


	ret = hns_roce_init_hem_table(hr_dev, &hr_dev->qp_table.qp_table,

				      HEM_TYPE_QPC, hr_dev->caps.qpc_sz,

				      hr_dev->caps.num_qps, 1);

				      hr_dev->caps.num_qps);

	if (ret) {

		dev_err(dev, "Failed to init QP context memory, aborting.\n");

		goto err_unmap_dmpt;
@@ -1123,7 +1123,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 
				      HEM_TYPE_IRRL,

				      hr_dev->caps.irrl_entry_sz *

				      hr_dev->caps.max_qp_init_rdma,

				      hr_dev->caps.num_qps, 1);

				      hr_dev->caps.num_qps);

	if (ret) {

		dev_err(dev, "Failed to init irrl_table memory, aborting.\n");

		goto err_unmap_qp;
@@ -1135,7 +1135,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 
					      HEM_TYPE_TRRL,

					      hr_dev->caps.trrl_entry_sz *

					      hr_dev->caps.max_qp_dest_rdma,

					      hr_dev->caps.num_qps, 1);

					      hr_dev->caps.num_qps);

		if (ret) {

			dev_err(dev,

				"Failed to init trrl_table memory, aborting.\n");
@@ -1145,7 +1145,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 


	ret = hns_roce_init_hem_table(hr_dev, &hr_dev->cq_table.table,

				      HEM_TYPE_CQC, hr_dev->caps.cqc_entry_sz,

				      hr_dev->caps.num_cqs, 1);

				      hr_dev->caps.num_cqs);

	if (ret) {

		dev_err(dev, "Failed to init CQ context memory, aborting.\n");

		goto err_unmap_trrl;
@@ -1155,7 +1155,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 
		ret = hns_roce_init_hem_table(hr_dev, &hr_dev->srq_table.table,

					      HEM_TYPE_SRQC,

					      hr_dev->caps.srqc_entry_sz,

					      hr_dev->caps.num_srqs, 1);

					      hr_dev->caps.num_srqs);

		if (ret) {

			dev_err(dev,

				"Failed to init SRQ context memory, aborting.\n");
@@ -1168,7 +1168,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 
					      &hr_dev->qp_table.sccc_table,

					      HEM_TYPE_SCCC,

					      hr_dev->caps.sccc_sz,

					      hr_dev->caps.num_qps, 1);

					      hr_dev->caps.num_qps);

		if (ret) {

			dev_err(dev,

				"Failed to init SCC context memory, aborting.\n");
@@ -1180,7 +1180,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 
		ret = hns_roce_init_hem_table(hr_dev, &hr_dev->qpc_timer_table,

					      HEM_TYPE_QPC_TIMER,

					      hr_dev->caps.qpc_timer_entry_sz,

					      hr_dev->caps.num_qpc_timer, 1);

					      hr_dev->caps.num_qpc_timer);

		if (ret) {

			dev_err(dev,

				"Failed to init QPC timer memory, aborting.\n");
@@ -1192,7 +1192,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 
		ret = hns_roce_init_hem_table(hr_dev, &hr_dev->cqc_timer_table,

					      HEM_TYPE_CQC_TIMER,

					      hr_dev->caps.cqc_timer_entry_sz,

					      hr_dev->caps.cqc_timer_bt_num, 1);

					      hr_dev->caps.cqc_timer_bt_num);

		if (ret) {

			dev_err(dev,

				"Failed to init CQC timer memory, aborting.\n");
@@ -1204,7 +1204,7 @@ static int hns_roce_init_hem(struct hns_roce_dev *hr_dev) 
		ret = hns_roce_init_hem_table(hr_dev, &hr_dev->gmv_table,

					      HEM_TYPE_GMV,

					      hr_dev->caps.gmv_entry_sz,

					      hr_dev->caps.gmv_entry_num, 1);

					      hr_dev->caps.gmv_entry_num);

		if (ret) {

			dev_err(dev,

				"failed to init gmv table memory, ret = %d\n",