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. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
