[PATCH libdrm] amdgpu: use EINVAL instead of EBADMSG in amdgpu_bo_cpu_unmap()
Christian König
deathsimple at vodafone.de
Tue Sep 1 06:07:21 PDT 2015
On 01.09.2015 13:37, Jonathan Gray wrote:
> EBADMSG is a streams errno. OpenBSD does not implement streams and does
> include the streams errnos, this commit fixes the build on OpenBSD.
>
> None of the callers of this function check the return value for -EBADMSG.
>
> Signed-off-by: Jonathan Gray <jsg at jsg.id.au>
Probably not intentional to use this error code here.
Patch is Reviewed-by: Christian König <christian.koenig at amd.com>
Regards,
Christian.
> ---
> amdgpu/amdgpu_bo.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c
> index 220422d..348da00 100644
> --- a/amdgpu/amdgpu_bo.c
> +++ b/amdgpu/amdgpu_bo.c
> @@ -482,7 +482,7 @@ int amdgpu_bo_cpu_unmap(amdgpu_bo_handle bo)
> if (bo->cpu_map_count == 0) {
> /* not mapped */
> pthread_mutex_unlock(&bo->cpu_access_mutex);
> - return -EBADMSG;
> + return -EINVAL;
> }
>
> bo->cpu_map_count--;
More information about the dri-devel
mailing list