Re: [PATCH 2/3] drm/amdgpu: add the interface of waiting multiple fences (v3)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4 November 2016 at 11:14, Sumit Semwal <sumit.semwal@xxxxxxxxxx> wrote:
> Hi Alex,
>
> Thanks for the patch!
>
> On 3 November 2016 at 11:41, Alex Deucher <alexdeucher@xxxxxxxxx> wrote:
>> From: Junwei Zhang <Jerry.Zhang@xxxxxxx>
>>
>> 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.
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux