On 09/03/2018 03:11 PM, Christian König wrote:
About master branch, needs someone's help with correct permission.
I've already took care of that on the weekend.

Thank you again.
BTW, how to apply that permission?

Regards,
Jerry


Regards,
Christian.

Am 03.09.2018 um 03:42 schrieb Zhang, Jerry (Junwei):
On 09/01/2018 04:58 PM, Deng, Emily wrote:
Ok, then just ignore this patch. But seems didn't saw the patch on branch 
amd-staging-hybrid-master20180315.

Thanks to take care of this as well.

I'm waiting some verification, and now push the patch to internal staging branch
mainline will be pushed later for another verification.

About master branch, needs someone's help with correct permission.

Regards,
Jerry

Best wishes
Emily Deng

-----Original Message-----
From: Christian König <ckoenig.leichtzumer...@gmail.com>
Sent: Saturday, September 1, 2018 4:17 PM
To: Deng, Emily <emily.d...@amd.com>; amd-gfx@lists.freedesktop.org
Subject: Re: [PATCH libdrm] amdgpu: When couldn't find bo, need to return
error.

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


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

Reply via email to