On 27/04/17 07:04 PM, Christian König wrote:
> Am 27.04.2017 um 10:18 schrieb Michel Dänzer:
>> From: Michel Dänzer <michel.daen...@amd.com>
>>
>> This reverts commit cb341a319f7e66f879d69af929c3dadfc1a8f31e.
>>
>> The purpose of the refactor was for amdgpu_crtc_prepare/submit_flip to
>> be used by the DC code, but that's no longer the case.
>>
>> Signed-off-by: Michel Dänzer <michel.daen...@amd.com>
[...]
>> +pflip_cleanup:
>> + if (unlikely(amdgpu_bo_reserve(new_abo, false) != 0)) {
>
> Well here we explicitly shouldn't use an interruptible wait, otherwise
> we would keep the BO pinned.
Makes sense, but this patch just reverts another commit and moves this
code unchanged. I'll send a follow-up patch which fixes this (possibly
also in other error paths).
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
