Use kzalloc_flex to allocate cid_array to allocate cid_array as part of
the overall struct.
Add __counted_by for extra runtime analysis.
Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
drivers/scsi/be2iscsi/be_main.c | 17 ++---------------
drivers/scsi/be2iscsi/be_main.h | 2 +-
2 files changed, 3 insertions(+), 16 deletions(-)
diff --git a/drivers/scsi/be2iscsi/be_main.c b/drivers/scsi/be2iscsi/be_main.c
index fd18d4d3d219..aa5320535c1f 100644
--- a/drivers/scsi/be2iscsi/be_main.c
+++ b/drivers/scsi/be2iscsi/be_main.c
@@ -3992,25 +3992,14 @@ static int hba_setup_cid_tbls(struct beiscsi_hba *phba)
for (ulp_num = 0; ulp_num < BEISCSI_ULP_COUNT; ulp_num++) {
if (test_bit(ulp_num, (void *)&phba->fw_config.ulp_supported)) {
- ptr_cid_info = kzalloc_obj(struct ulp_cid_info);
+ ptr_cid_info = kzalloc_flex(*ptr_cid_info, cid_array,
+ BEISCSI_GET_CID_COUNT(phba, ulp_num));
if (!ptr_cid_info) {
ret = -ENOMEM;
goto free_memory;
}
- /* Allocate memory for CID array */
- ptr_cid_info->cid_array =
- kcalloc(BEISCSI_GET_CID_COUNT(phba, ulp_num),
- sizeof(*ptr_cid_info->cid_array),
- GFP_KERNEL);
- if (!ptr_cid_info->cid_array) {
- kfree(ptr_cid_info);
- ptr_cid_info = NULL;
- ret = -ENOMEM;
-
- goto free_memory;
- }
ptr_cid_info->avlbl_cids = BEISCSI_GET_CID_COUNT(
phba, ulp_num);
@@ -4061,7 +4050,6 @@ static int hba_setup_cid_tbls(struct beiscsi_hba *phba)
ptr_cid_info = phba->cid_array_info[ulp_num];
if (ptr_cid_info) {
- kfree(ptr_cid_info->cid_array);
kfree(ptr_cid_info);
phba->cid_array_info[ulp_num] = NULL;
}
@@ -4175,7 +4163,6 @@ static void beiscsi_cleanup_port(struct beiscsi_hba *phba)
ptr_cid_info = phba->cid_array_info[ulp_num];
if (ptr_cid_info) {
- kfree(ptr_cid_info->cid_array);
kfree(ptr_cid_info);
phba->cid_array_info[ulp_num] = NULL;
}
diff --git a/drivers/scsi/be2iscsi/be_main.h b/drivers/scsi/be2iscsi/be_main.h
index 71c95d144560..b5f8e746deab 100644
--- a/drivers/scsi/be2iscsi/be_main.h
+++ b/drivers/scsi/be2iscsi/be_main.h
@@ -241,10 +241,10 @@ struct hwi_wrb_context {
};
struct ulp_cid_info {
- unsigned short *cid_array;
unsigned short avlbl_cids;
unsigned short cid_alloc;
unsigned short cid_free;
+ unsigned short cid_array[] __counted_by(avlbl_cids);
};
#include "be.h"
--
2.53.0