> About master branch, needs someone's help with correct permission.
I've already took care of that on the weekend.
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
>>
