Am 22.11.2017 um 09:42 schrieb Michel Dänzer:
On 2017-11-21 07:38 PM, Christian König wrote:
Am 21.11.2017 um 18:29 schrieb Michel Dänzer:
From: Michel Dänzer <[email protected]>
This matches the corresponding UAPI fields. Treating the ring index as
signed could result in accessing random unrelated memory if the MSB was
set.
Fixes: effd924d2f3b ("drm/amdgpu: untie user ring ids from kernel ring
ids v6")
Cc: [email protected]
Signed-off-by: Michel Dänzer <[email protected]>
Subject line is off, [...]
I assume you're referring to the prefix. Well, I'm actually trying to
set a trend there. :) Since there's no other amdgpu driver, the drm/ is
redundant. In the specific case of this patch, adding it would make the
shortlog too long, and IME it's quite often the case that those four
character cells could be put to better use than for the redundant prefix.
Yeah, making the prefix shorter is one thing I have thought about as well.
But currently we use "drm/amdgpu:" for the kernel and "amdgpu:" for
libdrm and I actually would like to keep that.
Christian.
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
