habanalabs: bugs fixes in timestamps buff alloc
authorfarah kassabri <fkassabri@habana.ai>
Sun, 8 Jan 2023 15:33:44 +0000 (17:33 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 10 Mar 2023 08:33:16 +0000 (09:33 +0100)
[ Upstream commit 1693fef9e95dbe8ab767d208a02328fff13fbb94 ]

use argument instead of fixed GFP value for allocation
in Timestamps buffers alloc function.
change data type of size to size_t.

Fixes: 9158bf69e74f ("habanalabs: Timestamps buffers registration")
Signed-off-by: farah kassabri <fkassabri@habana.ai>
Reviewed-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/misc/habanalabs/common/memory.c

index ef28f3b..a49038d 100644 (file)
@@ -2089,12 +2089,13 @@ static int hl_ts_mmap(struct hl_mmap_mem_buf *buf, struct vm_area_struct *vma, v
 static int hl_ts_alloc_buf(struct hl_mmap_mem_buf *buf, gfp_t gfp, void *args)
 {
        struct hl_ts_buff *ts_buff = NULL;
-       u32 size, num_elements;
+       u32 num_elements;
+       size_t size;
        void *p;
 
        num_elements = *(u32 *)args;
 
-       ts_buff = kzalloc(sizeof(*ts_buff), GFP_KERNEL);
+       ts_buff = kzalloc(sizeof(*ts_buff), gfp);
        if (!ts_buff)
                return -ENOMEM;