diff options
| author | Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> | 2019-05-02 16:14:27 +0300 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2019-05-06 17:36:48 +0300 |
| commit | 570c91d51b337053a90ac91710b5fa5d2aacd311 (patch) | |
| tree | 34621926cd71493a891731a0673b8ca3f9513c0d /fs/file_table.c | |
| parent | b6818520edb0dc83d8de807cb40dff44995eab86 (diff) | |
| download | linux-570c91d51b337053a90ac91710b5fa5d2aacd311.tar.xz | |
drm/amd/display: Use long for signed error code checks in commit planes
[Why]
The type of 'r' is uint32_t and the return codes for both:
- reservation_object_wait_timeout_rcu
- amdgpu_bo_reserve
...are signed. While it works for the latter since the check is
done on != 0 it doesn't work for the former since we check <= 0.
[How]
Make 'r' a long in commit planes so we're not doing any unsigned/signed
conversion here in the first place.
v2: use long instead of int (Christian)
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'fs/file_table.c')
0 files changed, 0 insertions, 0 deletions