habanalabs: eliminate aggregate use warning

When doing sizeof() and giving as argument a dereference of
a pointer-to-a-pointer object, clang will issue a warning.

Eliminate the warning by passing struct <name>*

Signed-off-by: Oded Gabbay <ogabbay@kernel.org>

diff --git a/drivers/misc/habanalabs/common/command_submission.c b/drivers/misc/habanalabs/common/command_submission.c
index fbe5003191bf350beb3f947272beee14ee4f63f5..fa05770865c65930ef913029920cb62b227f5420 100644 (file)
--- a/drivers/misc/habanalabs/common/command_submission.c
+++ b/drivers/misc/habanalabs/common/command_submission.c
@@ -2571,7 +2571,7 @@ static int hl_cs_poll_fences(struct multi_cs_data *mcs_data, struct multi_cs_com
        ktime_t max_ktime, first_cs_time;
        enum hl_cs_wait_status status;
 
-       memset(fence_ptr, 0, arr_len * sizeof(*fence_ptr));
+       memset(fence_ptr, 0, arr_len * sizeof(struct hl_fence *));
 
        /* get all fences under the same lock */
        rc = hl_ctx_get_fences(mcs_data->ctx, seq_arr, fence_ptr, arr_len);
@@ -2873,7 +2873,7 @@ static int hl_multi_cs_wait_ioctl(struct hl_fpriv *hpriv, void *data)
        }
 
        /* allocate array for the fences */
-       fence_arr = kmalloc_array(seq_arr_len, sizeof(*fence_arr), GFP_KERNEL);
+       fence_arr = kmalloc_array(seq_arr_len, sizeof(struct hl_fence *), GFP_KERNEL);
        if (!fence_arr) {
                rc = -ENOMEM;
                goto free_seq_arr;

diff --git a/drivers/misc/habanalabs/common/device.c b/drivers/misc/habanalabs/common/device.c
index c6a00bb259fb0ef3297952f460e84006b5d83e5c..233d8b46c831fcc3e1ef8c6b28d13a5b41868cb6 100644 (file)
--- a/drivers/misc/habanalabs/common/device.c
+++ b/drivers/misc/habanalabs/common/device.c
@@ -719,7 +719,7 @@ static int device_early_init(struct hl_device *hdev)
 
        if (hdev->asic_prop.completion_queues_count) {
                hdev->cq_wq = kcalloc(hdev->asic_prop.completion_queues_count,
-                               sizeof(*hdev->cq_wq),
+                               sizeof(struct workqueue_struct *),
                                GFP_KERNEL);
                if (!hdev->cq_wq) {
                        rc = -ENOMEM;
@@ -1863,7 +1863,7 @@ int hl_device_init(struct hl_device *hdev, struct class *hclass)
        }
 
        hdev->shadow_cs_queue = kcalloc(hdev->asic_prop.max_pending_cs,
-                                       sizeof(*hdev->shadow_cs_queue), GFP_KERNEL);
+                                       sizeof(struct hl_cs *), GFP_KERNEL);
        if (!hdev->shadow_cs_queue) {
                rc = -ENOMEM;
                goto cq_fini;

diff --git a/drivers/misc/habanalabs/common/hw_queue.c b/drivers/misc/habanalabs/common/hw_queue.c
index 3f15ab9d827ff113a7efc18abae62ed75f4ddba5..d0087c0ec48c9f5758a2eb8614ae31efb27ce9fa 100644 (file)
--- a/drivers/misc/habanalabs/common/hw_queue.c
+++ b/drivers/misc/habanalabs/common/hw_queue.c
@@ -826,9 +826,7 @@ static int ext_and_cpu_queue_init(struct hl_device *hdev, struct hl_hw_queue *q,
 
        q->kernel_address = p;
 
-       q->shadow_queue = kmalloc_array(HL_QUEUE_LENGTH,
-                                       sizeof(*q->shadow_queue),
-                                       GFP_KERNEL);
+       q->shadow_queue = kmalloc_array(HL_QUEUE_LENGTH, sizeof(struct hl_cs_job *), GFP_KERNEL);
        if (!q->shadow_queue) {
                dev_err(hdev->dev,
                        "Failed to allocate shadow queue for H/W queue %d\n",

diff --git a/drivers/misc/habanalabs/common/hwmon.c b/drivers/misc/habanalabs/common/hwmon.c
index 8c262aeb425e6363ee4ceeb484670e29bf2dd389..55eb0203817f1761aab75d595a83e6078059105b 100644 (file)
--- a/drivers/misc/habanalabs/common/hwmon.c
+++ b/drivers/misc/habanalabs/common/hwmon.c
@@ -194,7 +194,8 @@ int hl_build_hwmon_channel_info(struct hl_device *hdev, struct cpucp_sensor *sen
                curr_arr[sensors_by_type_next_index[type]++] = flags;
        }
 
-       channels_info = kcalloc(num_active_sensor_types + 1, sizeof(*channels_info), GFP_KERNEL);
+       channels_info = kcalloc(num_active_sensor_types + 1, sizeof(struct hwmon_channel_info *),
+                               GFP_KERNEL);
        if (!channels_info) {
                rc = -ENOMEM;
                goto channels_info_array_err;

diff --git a/drivers/misc/habanalabs/common/memory.c b/drivers/misc/habanalabs/common/memory.c
index 096fa3c1ae9519b9ea9a7538ec0f79c7baae791a..ef28f3b37b93216bf9a7b50f296f21fcdff62261 100644 (file)
--- a/drivers/misc/habanalabs/common/memory.c
+++ b/drivers/misc/habanalabs/common/memory.c
@@ -2308,8 +2308,7 @@ static int get_user_memory(struct hl_device *hdev, u64 addr, u64 size,
                return -EFAULT;
        }
 
-       userptr->pages = kvmalloc_array(npages, sizeof(*userptr->pages),
-                                       GFP_KERNEL);
+       userptr->pages = kvmalloc_array(npages, sizeof(struct page *), GFP_KERNEL);
        if (!userptr->pages)
                return -ENOMEM;