答复: 答复: 答复: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error

2017-02-08 Thread Liu, Monk
don't block driver proceeding after ib test failure detected , doable ? 发件人: Christian König <deathsim...@vodafone.de> 发送时间: 2017年2月8日 23:59:10 收件人: Liu, Monk; Michel Dänzer 抄送: amd-gfx@lists.freedesktop.org 主题: Re: 答复: 答复: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new

Re: 答复: 答复: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error

2017-02-08 Thread Christian König
is really adding to much delay at bootup. *发件人:* Michel Dänzer <mic...@daenzer.net> *发送时间:* 2017年2月8日 23:52:02 *收件人:* Christian König; Liu, Monk *抄送:* amd-gfx@lists.freedesktop.org *主题:* Re: 答复: [PATCH] drm/amdgpu:fix

答复: 答复: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error

2017-02-08 Thread Liu, Monk
agreed, why not just use cpu to clear it ? is it because performance ? 发件人: Michel Dänzer <mic...@daenzer.net> 发送时间: 2017年2月8日 23:52:02 收件人: Christian König; Liu, Monk 抄送: amd-gfx@lists.freedesktop.org 主题: Re: 答复: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new

Re: 答复: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error

2017-02-08 Thread Michel Dänzer
On 09/02/17 12:30 AM, Christian König wrote: > The IB test make the decision if the hardware is working or not. > > So they should be the first commands (except for the ring tests) we send > to the hardware. > > When we allocate the fb before the test we send the clear command to the > hardware

Re: 答复: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error

2017-02-08 Thread Christian König
? *发件人:* Michel Dänzer <mic...@daenzer.net> *发送时间:* 2017年2月8日 23:19:23 *收件人:* Christian König; Liu, Monk *抄送:* amd-gfx@lists.freedesktop.org *主题:* Re: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error On 08/02/17 07:52 PM, Christian König wrote: > Am 08.02.2017 um 10:41 schrieb

答复: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error

2017-02-08 Thread Liu, Monk
yeah, make sense @Christian, why move fbdev_init() further down after ib test ? any consideration ? 发件人: Michel Dänzer <mic...@daenzer.net> 发送时间: 2017年2月8日 23:19:23 收件人: Christian König; Liu, Monk 抄送: amd-gfx@lists.freedesktop.org 主题: Re: [PATCH] drm/amdg

Re: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error

2017-02-08 Thread Michel Dänzer
On 08/02/17 07:52 PM, Christian König wrote: > Am 08.02.2017 um 10:41 schrieb Monk Liu: >> ib_pool init should prior to fbdev_init, otherwise >> there will be error from amdgpu_sa_bo_new >> (amdgpu_sa.c:323) >> >> fbdev_init will call ttm_validate which further call >> amdgpu_sa_bo_new. >> >>

Re: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error

2017-02-08 Thread Christian König
Am 08.02.2017 um 10:41 schrieb Monk Liu: ib_pool init should prior to fbdev_init, otherwise there will be error from amdgpu_sa_bo_new (amdgpu_sa.c:323) fbdev_init will call ttm_validate which further call amdgpu_sa_bo_new. Change-Id: I3a969570d443f61a44f67b0d76b3871ca5c3ea81 Signed-off-by:

[PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error

2017-02-08 Thread Monk Liu
ib_pool init should prior to fbdev_init, otherwise there will be error from amdgpu_sa_bo_new (amdgpu_sa.c:323) fbdev_init will call ttm_validate which further call amdgpu_sa_bo_new. Change-Id: I3a969570d443f61a44f67b0d76b3871ca5c3ea81 Signed-off-by: Monk Liu ---