[PATCH v2] drm/amd/display: Don't print error when bo_pin is interrupted

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

 



Hi Christian,

thanks for clarifying the semantics. I wasn't fully clear on that. What you say makes sense. I'll respin the patch.

Harry

On 2017-10-27 03:46 AM, Christian König wrote:
> Hi Harry,
>
> actually it's best practice that you only ignore ERESTARTSYS here, see other code in the driver as well.
>
> EINTR means that the IOCTL was interrupted and can't be restarted because of some problem.
>
> EAGAIN mean the we can't do this operation right now, but might be able to do it at some point in the future.
>
> Both sound like a good idea to note to the user.
>
> Regards,
> Christian.
>
> Am 26.10.2017 um 22:16 schrieb Andrey Grodzovsky:
>>
>>
>>
>> On 2017-10-26 04:08 PM, Harry Wentland wrote:
>>> v2: Also don't print for ERESTARTSYS or EAGAIN
>>>
>>> Signed-off-by: Harry Wentland <harry.wentland at amd.com>
>>
>> Reviewed-by: Andrey Grodzovsky <andrey.grodzovsky at amd.com>
>>
>> Thanks,
>> Andrey
>>
>>> ---
>>>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 ++-
>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>>> index cf15701f208d..4401f0fb3f02 100644
>>> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>>> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>>> @@ -2944,7 +2944,8 @@ static int dm_plane_helper_prepare_fb(struct drm_plane *plane,
>>>  	amdgpu_bo_unreserve(rbo);
>>>  
>>>  	if (unlikely(r != 0)) {
>>> -		DRM_ERROR("Failed to pin framebuffer\n");
>>> +		if (!(r == -EINTR || r == -ERESTARTSYS || r == EAGAIN))
>>> +			DRM_ERROR("Failed to pin framebuffer with error %d\n", r);
>>>  		return r;
>>>  	}
>>>  
>>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20171027/4d9b4a82/attachment-0001.html>


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux