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

[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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-amdgpu-Fix-mistake-in-initial-hole-size-calculation.patch
Type: text/x-patch
Size: 1178 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20180219/e526468b/attachment.bin>