> -----Original Message----- > From: Sumit Semwal [mailto:sumit.semwal at linaro.org] > Sent: Friday, November 04, 2016 1:15 PM > To: Alex Deucher > Cc: amd-gfx at lists.freedesktop.org; DRI mailing list; Zhang, Jerry; Deucher, > Alexander > Subject: Re: [PATCH 2/3] drm/amdgpu: add the interface of waiting multiple > fences (v3) > > 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? Yes, Sorry, I messed up the order when I rebased the patches. It should be 2 1 3. Alex > > Best regards, > Sumit.