Am 01.09.2018 um 06:24 schrieb Emily Deng:
The startx will have segmant fault if return success.

SWDEV-163962

Change-Id: I56b189fa26efdcd1d96e5100af3f3e0b1208b0c3
Signed-off-by: Emily Deng <emily.d...@amd.com>

Jerry already send a much better patch for this.

---
  amdgpu/amdgpu_bo.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c
index f25cacc..7e297fa 100644
--- a/amdgpu/amdgpu_bo.c
+++ b/amdgpu/amdgpu_bo.c
@@ -760,6 +760,7 @@ int amdgpu_find_bo_by_cpu_mapping(amdgpu_device_handle dev,
                                  uint64_t *offset_in_bo)
  {
        uint32_t i;
+       int r = 0;
        struct amdgpu_bo *bo;
if (cpu == NULL || size == 0)
@@ -787,10 +788,11 @@ int amdgpu_find_bo_by_cpu_mapping(amdgpu_device_handle 
dev,
        } else {
                *buf_handle = NULL;
                *offset_in_bo = 0;
+               r = -errno;

errno doesn't contain any error in this case.

        }
        pthread_mutex_unlock(&dev->bo_table_mutex);
- return 0;
+       return r;
  }
int amdgpu_create_bo_from_user_mem(amdgpu_device_handle dev,

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to