accel/ivpu: Use struct_size()
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Mon, 29 May 2023 13:28:04 +0000 (15:28 +0200)
committerStanislaw Gruszka <stanislaw.gruszka@linux.intel.com>
Thu, 8 Jun 2023 05:46:51 +0000 (07:46 +0200)
Use struct_size() instead of hand-writing it. It is less verbose, more
robust and more informative.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: Marco Pagani <marpagan@redhat.com>
Signed-off-by: Stanislaw Gruszka <stanislaw.gruszka@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/0ae53be873c27c9a8740c4fe6d8e7cd1b1224994.1685366864.git.christophe.jaillet@wanadoo.fr
drivers/accel/ivpu/ivpu_job.c

index 3c6f1e1..0a09bba 100644 (file)
@@ -289,15 +289,13 @@ ivpu_create_job(struct ivpu_file_priv *file_priv, u32 engine_idx, u32 bo_count)
 {
        struct ivpu_device *vdev = file_priv->vdev;
        struct ivpu_job *job;
-       size_t buf_size;
        int ret;
 
        ret = ivpu_rpm_get(vdev);
        if (ret < 0)
                return NULL;
 
-       buf_size = sizeof(*job) + bo_count * sizeof(struct ivpu_bo *);
-       job = kzalloc(buf_size, GFP_KERNEL);
+       job = kzalloc(struct_size(job, bos, bo_count), GFP_KERNEL);
        if (!job)
                goto err_rpm_put;