diff options
| author |   Dan Carpenter <dan.carpenter@linaro.org> | 2023-05-25 11:04:51 +0300 |
|---|---|---|
| committer |   Alex Deucher <alexander.deucher@amd.com> | 2023-06-09 11:10:04 -0400 |
| commit | 9c9d501b28a00f4365632260df6cae488a905af7 (patch) | |
| tree | c665772e9358278f1a42b89ccbe423dd6304d752 /drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c | |
| parent | drm/amd/display: remove unused definition (diff) | |
| download | wireguard-linux-9c9d501b28a00f4365632260df6cae488a905af7.tar.xz wireguard-linux-9c9d501b28a00f4365632260df6cae488a905af7.zip | |
drm/amd/amdgpu: Fix up locking etc in amdgpu_debugfs_gprwave_ioctl()
There are two bugs here.
1) Drop the lock if copy_from_user() fails.
2) If the copy fails then the correct error code is -EFAULT instead of
-EINVAL.
I also broke up the long line and changed "sizeof rd->id" to
"sizeof(rd->id)".
Fixes: 553f973a0d7b ("drm/amd/amdgpu: Update debugfs for XCC support (v3)")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c')
0 files changed, 0 insertions, 0 deletions