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.leichtzumerken at gmail.com>
>>>> Sent: Saturday, September 1, 2018 4:17 PM
>>>> To: Deng, Emily <Emily.Deng at amd.com>; amd-gfx at 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.Deng at 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 at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>>>
>
