Re: [PATCH] drm/amdgpu: potential shift wrapping bug

2017-08-10 Thread walter harms
Am 10.08.2017 15:02, schrieb Christian König: > Am 10.08.2017 um 14:53 schrieb Dan Carpenter: >> On Thu, Aug 10, 2017 at 02:30:15PM +0200, Christian König wrote: >>> Am 10.08.2017 um 14:16 schrieb Dan Carpenter: "frag_align" is a u64, so presumably we want to use the high bits as well

Re: [PATCH] drm/amdgpu: potential shift wrapping bug

2017-08-10 Thread Christian König
Am 10.08.2017 um 14:53 schrieb Dan Carpenter: On Thu, Aug 10, 2017 at 02:30:15PM +0200, Christian König wrote: Am 10.08.2017 um 14:16 schrieb Dan Carpenter: "frag_align" is a u64, so presumably we want to use the high bits as well instead of shift wrapping. Fixes: 6be7adb37d9b ("drm/amdgpu:

Re: [PATCH] drm/amdgpu: potential shift wrapping bug

2017-08-10 Thread Dan Carpenter
On Thu, Aug 10, 2017 at 03:02:53PM +0200, Christian König wrote: > Am 10.08.2017 um 14:53 schrieb Dan Carpenter: > > On Thu, Aug 10, 2017 at 02:30:15PM +0200, Christian König wrote: > > > Am 10.08.2017 um 14:16 schrieb Dan Carpenter: > > > > "frag_align" is a u64, so presumably we want to use the

Re: [PATCH] drm/amdgpu: potential shift wrapping bug

2017-08-10 Thread Dan Carpenter
On Thu, Aug 10, 2017 at 02:30:15PM +0200, Christian König wrote: > Am 10.08.2017 um 14:16 schrieb Dan Carpenter: > > "frag_align" is a u64, so presumably we want to use the high bits as > > well instead of shift wrapping. > > > > Fixes: 6be7adb37d9b ("drm/amdgpu: increase fragmentation size for

Re: [PATCH] drm/amdgpu: potential shift wrapping bug

2017-08-10 Thread Dan Carpenter
On Thu, Aug 10, 2017 at 02:30:15PM +0200, Christian König wrote: > Am 10.08.2017 um 14:16 schrieb Dan Carpenter: > > "frag_align" is a u64, so presumably we want to use the high bits as > > well instead of shift wrapping. > > > > Fixes: 6be7adb37d9b ("drm/amdgpu: increase fragmentation size for

Re: [PATCH] drm/amdgpu: potential shift wrapping bug

2017-08-10 Thread Christian König
Am 10.08.2017 um 14:16 schrieb Dan Carpenter: "frag_align" is a u64, so presumably we want to use the high bits as well instead of shift wrapping. Fixes: 6be7adb37d9b ("drm/amdgpu: increase fragmentation size for Vega10 v2") Signed-off-by: Dan Carpenter The fragment