Re: [PATCH 16/21] drm/amdgpu: implement lru amdgpu_queue_mgr policy for compute v3

2017-03-03 Thread Andres Rodriguez
On 3/3/2017 10:20 AM, Alex Deucher wrote: On Fri, Mar 3, 2017 at 8:23 AM, Christian König wrote: Am 02.03.2017 um 22:44 schrieb Andres Rodriguez: Use an LRU policy to map usermode rings to HW compute queues. Most compute clients use one queue, and usually the first queue available. This re

Re: [PATCH 16/21] drm/amdgpu: implement lru amdgpu_queue_mgr policy for compute v3

2017-03-03 Thread William Lewis
Minor spelling nitpick. >> +/** >> + * amdgpu_ring_lru_get - get the least recently used ring for a HW >> IP block >> + * >> + * @adev: amdgpu_device pointer >> + * @type: amdgpu_ring_type enum >> + * @ring: output ring >> + * >> + * Retreive the amdgpu_ring structure for the least recently used

Re: [PATCH 16/21] drm/amdgpu: implement lru amdgpu_queue_mgr policy for compute v3

2017-03-03 Thread Alex Deucher
On Fri, Mar 3, 2017 at 8:23 AM, Christian König wrote: > Am 02.03.2017 um 22:44 schrieb Andres Rodriguez: >> >> Use an LRU policy to map usermode rings to HW compute queues. >> >> Most compute clients use one queue, and usually the first queue >> available. This results in poor pipe/queue work dis

Re: [PATCH 16/21] drm/amdgpu: implement lru amdgpu_queue_mgr policy for compute v3

2017-03-03 Thread Christian König
Am 02.03.2017 um 22:44 schrieb Andres Rodriguez: Use an LRU policy to map usermode rings to HW compute queues. Most compute clients use one queue, and usually the first queue available. This results in poor pipe/queue work distribution when multiple compute apps are running. In most cases pipe 0

[PATCH 16/21] drm/amdgpu: implement lru amdgpu_queue_mgr policy for compute v3

2017-03-02 Thread Andres Rodriguez
Use an LRU policy to map usermode rings to HW compute queues. Most compute clients use one queue, and usually the first queue available. This results in poor pipe/queue work distribution when multiple compute apps are running. In most cases pipe 0 queue 0 is the only queue that gets used. In orde