[PATCH] drm/amd/amdgpu: set gtt size according to system memory size only

[He, Roger]

-----Original Message-----
From: Michel Dänzer [mailto:michel at daenzer.net] 
Sent: Monday, December 11, 2017 7:29 PM
To: He, Roger <Hongbo.He at amd.com>; Koenig, Christian <Christian.Koenig at amd.com>
Cc: amd-gfx at lists.freedesktop.org
Subject: Re: [PATCH] drm/amd/amdgpu: set gtt size according to system memory size only

On 2017-12-11 03:49 AM, He, Roger wrote:
>> From: Michel Dänzer [mailto:michel at daenzer.net] On 2017-12-07 07:07 
>> PM, Christian König wrote:
>>> 
>>> I think I know what is going on here. The max-texture-size keeps  
>>> increasing the texture size as long as it doesn't fails to allocate 
>>> one.
>>> 
>>> So the "Out of kernel memory" message is actually the desired effect 
>>> (but we should probably remove the message).
>>> 
>>> The price question is what happens after that? Those code paths are 
>>> probably not very well tested.
>> 
>> I'm attaching all the related dmesg output I've captured.
>> Basically, best case is the OOM reaper killing piglit, worst case is 
>> no response to SSH => hard reboot.

BTW, even when the machine stays responsive via SSH, the GPU hangs.


>> Until this becomes more robust, this change should probably be 
>> reverted.
> 
> I have not got any valuable info from error log. Please revert it now 
> as quick solution, later let me check the reason.

	I'm busy with other stuff, can you revert it?

Sorry for until now noticing your mail. And thanks Grodzovsky.Andrey for doing that.

Could you tell me how to duplicate this issue?  Maybe now I can look into it.

Thanks
Roger(Hongbo.He)





-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer