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 > > Signed-off-by: Junwei Zhang <Jerry.Zhang at amd.com> > Reviewed-by: Monk Liu <monk.liu at amd.com> > Reviewed-by: Jammy Zhou <Jammy.Zhou at amd.com> > Signed-off-by: Alex Deucher <alexander.deucher at amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 + > drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 171 ++++++++++++++++++++++++++++++++ > drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 1 + > include/uapi/drm/amdgpu_drm.h | 27 +++++ > 4 files changed, 201 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h > index 8d0c689..c2050c0 100644 > --- 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? Best regards, Sumit.