Re: [PATCH] Add support for high priority scheduling in amdgpu v8

2017-04-12 Thread Andres Rodriguez
Thanks for the comments. I'll get these and Alex's comments addressed and re-send the series out. Regards, Andres On 2017-04-11 06:31 PM, Felix Kuehling wrote: I replied to patch 5 with an observation. But I think that code was broken before your change. The order of patches 7 and 8 should be

Re: [PATCH] Add support for high priority scheduling in amdgpu v8

2017-04-11 Thread Felix Kuehling
I replied to patch 5 with an observation. But I think that code was broken before your change. The order of patches 7 and 8 should be reversed. Otherwise you break radeon before you fix it. See my comments on patch 9, 15 and 16 in separate emails. With those comments addressed: 1-4, 10, 13-14 ar

[PATCH] Add support for high priority scheduling in amdgpu v8

2017-04-05 Thread Andres Rodriguez
Includes fixes for Alex's comments on v7. Also includes a new patch to deal with an issue we discussed on #radeon: [PATCH 15/26] drm/amdgpu: avoid KIQ clashing with compute or KFD Patches missing review/ack: 10 14 15 18 20 21 23 24 ___ amd-gfx mailing