On 4 November 2016 at 11:14, Sumit Semwal <sumit.semwal at linaro.org> wrote: > Hi Alex, > > Thanks for the patch! > > On 3 November 2016 at 11:41, Alex Deucher <alexdeucher at gmail.com> wrote: >> From: Junwei Zhang <Jerry.Zhang at amd.com> >> >> v2: agd: rebase and squash in all the previous optimizations and >> changes so everything compiles. >> v3: squash in Slava's 32bit build fix >> >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h > <snip> >> + >> + r = fence_wait_any_timeout(array, fence_count, true, timeout); > > With the change proposed about returning index of signalled fence in > patch 1/3, I don't think this would build? > O, just noticed - perhaps just a reordering of patches so that this is 1/3 (or even a separate patch?) and dma-buf change for index + amdgpu change for the index are 2/3 and 3/3, should solve it. > Best regards, > Sumit.