Re: [PATCH] drm/amd/amdgpu: remove vram_page_split kernel option (v3)

2019-06-06 Thread Christian König
Am 06.06.19 um 13:51 schrieb StDenis, Tom: On 2019-06-06 7:49 a.m., Christian König wrote: Am 06.06.19 um 12:50 schrieb StDenis, Tom: This option is no longer needed.  The default code paths are now the only option. v2: Add HPAGE support and a default for non contiguous maps v3: Misread 512 pa

Re: [PATCH] drm/amd/amdgpu: remove vram_page_split kernel option (v3)

2019-06-06 Thread StDenis, Tom
On 2019-06-06 7:49 a.m., Christian König wrote: > Am 06.06.19 um 12:50 schrieb StDenis, Tom: >> This option is no longer needed.  The default code paths >> are now the only option. >> >> v2: Add HPAGE support and a default for non contiguous maps >> v3: Misread 512 pages as MiB ... >> >> Signed-of

Re: [PATCH] drm/amd/amdgpu: remove vram_page_split kernel option (v3)

2019-06-06 Thread Christian König
Am 06.06.19 um 12:50 schrieb StDenis, Tom: This option is no longer needed. The default code paths are now the only option. v2: Add HPAGE support and a default for non contiguous maps v3: Misread 512 pages as MiB ... Signed-off-by: Tom St Denis --- drivers/gpu/drm/amd/amdgpu/amdgpu.h

Re: [PATCH] drm/amd/amdgpu: remove vram_page_split kernel option (v2)

2019-06-06 Thread StDenis, Tom
Ah ya I misread the original default as MiB instead of pages. Tom On 2019-06-06 6:35 a.m., Christian König wrote: > Am 04.06.19 um 19:15 schrieb StDenis, Tom: >> This option is no longer needed.  The default code paths >> are now the only option. >> >> v2: Add HPAGE support and a default for non

[PATCH] drm/amd/amdgpu: remove vram_page_split kernel option (v3)

2019-06-06 Thread StDenis, Tom
This option is no longer needed. The default code paths are now the only option. v2: Add HPAGE support and a default for non contiguous maps v3: Misread 512 pages as MiB ... Signed-off-by: Tom St Denis --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 - drivers/gpu/drm/amd/amdgpu/amdgpu_d

Re: [PATCH] drm/amd/amdgpu: remove vram_page_split kernel option (v2)

2019-06-06 Thread Christian König
Am 04.06.19 um 19:15 schrieb StDenis, Tom: This option is no longer needed. The default code paths are now the only option. v2: Add HPAGE support and a default for non contiguous maps Signed-off-by: Tom St Denis --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 - drivers/gpu/drm/amd/a

Re: [PATCH] drm/amd/amdgpu: remove vram_page_split kernel option (v2)

2019-06-06 Thread Tom St Denis
ping? On Tue, Jun 4, 2019 at 1:15 PM StDenis, Tom wrote: > This option is no longer needed. The default code paths > are now the only option. > > v2: Add HPAGE support and a default for non contiguous maps > > Signed-off-by: Tom St Denis > --- > drivers/gpu/drm/amd/amdgpu/amdgpu.h |

[PATCH] drm/amd/amdgpu: remove vram_page_split kernel option (v2)

2019-06-04 Thread StDenis, Tom
This option is no longer needed. The default code paths are now the only option. v2: Add HPAGE support and a default for non contiguous maps Signed-off-by: Tom St Denis --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 - drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 7 --- drivers/g

Re: [PATCH] drm/amd/amdgpu: remove vram_page_split kernel option

2019-06-04 Thread Christian König
Am 04.06.19 um 15:49 schrieb StDenis, Tom: This option is no longer needed. The default code paths are now the only option. NAK, we still need the functionality. It just doesn't needs to be configurable, but rather use the CPU defaults. Christian. Signed-off-by: Tom St Denis --- drive

[PATCH] drm/amd/amdgpu: remove vram_page_split kernel option

2019-06-04 Thread StDenis, Tom
This option is no longer needed. The default code paths are now the only option. Signed-off-by: Tom St Denis --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 - drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 7 --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 8 drivers/gpu/d