diff options
| author | Tom Rix <trix@redhat.com> | 2023-01-07 21:48:27 +0300 |
|---|---|---|
| committer | Oded Gabbay <ogabbay@kernel.org> | 2023-01-26 12:52:11 +0300 |
| commit | c7d7b9aca2423d243748361cfb72e7fbdb8b37b9 (patch) | |
| tree | 618204921f0d34836f6a1c345a736e9d22afa692 /drivers/accel/habanalabs | |
| parent | 78baccbdc343b0d4c3e3fd05644dadc3fc3f94c0 (diff) | |
| download | linux-c7d7b9aca2423d243748361cfb72e7fbdb8b37b9.tar.xz | |
habanalabs: remove redundant memset
From reviewing the code, the line
memset(kdata, 0, usize);
is not needed because kdata is either zeroed by
kdata = kzalloc(asize, GFP_KERNEL);
when allocated at runtime or by
char stack_kdata[128] = {0};
at compile time.
Signed-off-by: Tom Rix <trix@redhat.com>
Reviewed-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Oded Gabbay <ogabbay@kernel.org>
Diffstat (limited to 'drivers/accel/habanalabs')
| -rw-r--r-- | drivers/accel/habanalabs/common/habanalabs_ioctl.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/accel/habanalabs/common/habanalabs_ioctl.c b/drivers/accel/habanalabs/common/habanalabs_ioctl.c index 619d56c40b30..949d38527160 100644 --- a/drivers/accel/habanalabs/common/habanalabs_ioctl.c +++ b/drivers/accel/habanalabs/common/habanalabs_ioctl.c @@ -1123,8 +1123,6 @@ static long _hl_ioctl(struct file *filep, unsigned int cmd, unsigned long arg, retcode = -EFAULT; goto out_err; } - } else if (cmd & IOC_OUT) { - memset(kdata, 0, usize); } retcode = func(hpriv, kdata); |