Re: [PATCH] drm/amdgpu: set sched_hw_submission higher for KIQ (v2)

2017-08-23 Thread Christian König
As noted in the otehr thread we could set it to 256 as well and still only use one page, but 16 is fine with me as well. Patch is Reviewed-by: Christian König Regards, Christian. Am 23.08.2017 um 04:25 schrieb Felix Kuehling: This change is Reviewed-by: Felix

Re: [PATCH] drm/amdgpu: set sched_hw_submission higher for KIQ

2017-08-23 Thread Christian König
Am 22.08.2017 um 23:34 schrieb Jay Cornwall: On Tue, Aug 22, 2017, at 16:17, Felix Kuehling wrote: Thanks Alex! Jay, do you think this is enough? This bumps the number of concurrent operations on KIQ to 4 by default. I'm not sure what the best number is. Up to 8 KFD processes is common

Re: [PATCH] drm/amdgpu: set sched_hw_submission higher for KIQ (v2)

2017-08-22 Thread Felix Kuehling
This change is Reviewed-by: Felix Kuehling On 2017-08-22 09:52 PM, Alex Deucher wrote: > KIQ doesn't really use the GPU scheduler. The base > drivers generally use the KIQ ring directly rather than > submitting IBs. However, amdgpu_sched_hw_submission > (which defaults

[PATCH] drm/amdgpu: set sched_hw_submission higher for KIQ (v2)

2017-08-22 Thread Alex Deucher
KIQ doesn't really use the GPU scheduler. The base drivers generally use the KIQ ring directly rather than submitting IBs. However, amdgpu_sched_hw_submission (which defaults to 2) limits the number of outstanding fences to 2. KFD uses the KIQ for TLB flushes and the 2 fence limit hurts

Re: [PATCH] drm/amdgpu: set sched_hw_submission higher for KIQ

2017-08-22 Thread Jay Cornwall
On Tue, Aug 22, 2017, at 16:17, Felix Kuehling wrote: > Thanks Alex! > > Jay, do you think this is enough? This bumps the number of concurrent > operations on KIQ to 4 by default. I'm not sure what the best number is. Up to 8 KFD processes is common (beyond that performance drops off due to VMID

Re: [PATCH] drm/amdgpu: set sched_hw_submission higher for KIQ

2017-08-22 Thread Xie, AlexBin
To: amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander Subject: [PATCH] drm/amdgpu: set sched_hw_submission higher for KIQ KIQ doesn't really use the GPU scheduler. The base drivers generally use the KIQ ring directly rather than submitting IBs. However, amdgpu_sched_hw_submission (which defaults

Re: [PATCH] drm/amdgpu: set sched_hw_submission higher for KIQ

2017-08-22 Thread Felix Kuehling
Thanks Alex! Jay, do you think this is enough? This bumps the number of concurrent operations on KIQ to 4 by default. Regards, Felix On 2017-08-22 04:49 PM, Alex Deucher wrote: > KIQ doesn't really use the GPU scheduler. The base > drivers generally use the KIQ ring directly rather than >

[PATCH] drm/amdgpu: set sched_hw_submission higher for KIQ

2017-08-22 Thread Alex Deucher
KIQ doesn't really use the GPU scheduler. The base drivers generally use the KIQ ring directly rather than submitting IBs. However, amdgpu_sched_hw_submission (which defaults to 2) limits the number of outstanding fences to 2. KFD uses the KIQ for TLB flushes and the 2 fence limit hurts