[PATCH libdrm 2/2] libdrm: clean up non list code path for vamgr

[Chunming Zhou]

Thanks for fix.


Regards,

David Zhou


On 2018年02月19日 15:50, Christian König wrote:
> Patch is Acked-by: Christian König <christian.koenig at amd.com>
>
> Regards,
> Christian.
>
> Am 19.02.2018 um 08:25 schrieb Andrey Grodzovsky:
>> Found time to do some debugging and found error in assigning   of 
>> initial VA hole size, attached patch makes this issue go away, 
>> otherwise amdgpu_vamgr_find_va would always fail.
>>
>> Thanks,
>>
>> Andrey
>>
>>
>> On 02/18/2018 03:41 AM, Andrey Grodzovsky wrote:
>>> Hi David, I have an issue on Vega 10 where sudo xinit fails with
>>>
>>> [  5801.227] Failed to allocate front buffer memory
>>> [  5801.227] (EE) AMDGPU(0): amdgpu_setup_kernel_mem failed
>>> [  5801.227] (EE)
>>> Fatal server error:
>>> [  5801.227] (EE) AddScreen/ScreenInit failed for driver 0
>>> [  5801.227] (EE)
>>> [  5801.227] (EE)
>>>
>>> Also  in MESA code 
>>> si_create_context->si_common_context_init->pipe_buffer_create 
>>> returns NULL.
>>>
>>> Ellsmire works fine, on Vega 10 reverting this patch makes it work.
>>>
>>> I am during my test system upgrade right now with some more issues 
>>> after this one, so I can spend time on debugging this once my system 
>>> is in good shape, unless you have an answer to this problem right away.
>>>
>>> Thanks,
>>>
>>> Andrey
>>>
>>>
>>> On 02/09/2018 03:58 AM, Michel Dänzer wrote:
>>>> On 2018-02-09 07:01 AM, Chunming Zhou wrote:
>>>>> On 2018年02月08日 17:13, Chunming Zhou wrote:
>>>>>> On 2018年02月08日 17:01, Michel Dänzer wrote:
>>>>>>> this change completely broke radeonsi due to memory management 
>>>>>>> errors
>>>>>>> (see valgrind output for glxgears below), so I had to revert it.
>>>>>> ok, I will look into what happens. Sorry for broken.
>>>>> The patch lost a "goto out" in free_va when applying patch from 
>>>>> hybrid
>>>>> to opensource.
>>>>> The attached fixes that, I tested successfully in my local side.
>>>>>   please review or test, if no problem, I will submit it again.
>>>> Please add:
>>>>
>>>> v2: Add missing "goto out"
>>>> Tested-by: Michel Dänzer <michel.daenzer at amd.com>
>>>>
>>>> to the commit log before pushing.
>>>>
>>>>
>>>
>>> _______________________________________________
>>> amd-gfx mailing list
>>> amd-gfx at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>>
>>
>>
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20180226/03c6d7bd/attachment-0001.html>