<html> <head> <meta content="text/html; charset=utf-8" http-equiv="Content-Type"> </head> <body text="#000000" bgcolor="#FFFFFF"> <div class="moz-cite-prefix">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 without knowing if the hardware really works or not. Not a big issue, but I think the order makes more sense that way. Regards, Christian. Am 08.02.2017 um 16:21 schrieb Liu, Monk: </div> <blockquote cite="mid:DM5PR12MB161067C1383FC5829F4FCD8984420@DM5PR12MB1610.namprd12.prod.outlook.com" type="cite"> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Exchange Server">  <style></style> <meta content="text/html; charset=UTF-8"> <style type="text/css" style="">  </style> <div dir="ltr"> <div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Arial,Helvetica,sans-serif"> yeah, make sense @Christian, why move fbdev_init() further down after ib test ? any consideration ? </div> <hr tabindex="-1" style="display:inline-block; width:98%"> <div id="x_divRplyFwdMsg" dir="ltr">发件人: Michel Dänzer <a class="moz-txt-link-rfc2396E" href="mailto:michel@daenzer.net"><michel@daenzer.net></a> 发送时间: 2017年2月8日 23:19:23 收件人: Christian König; Liu, Monk 抄送: <a class="moz-txt-link-abbreviated" href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> 主题: Re: [PATCH] drm/amdgpu:fix amdgpu_sa_bo_new error <div> </div> </div> </div> <div class="PlainText">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. >> >> Change-Id: I3a969570d443f61a44f67b0d76b3871ca5c3ea81 >> Signed-off-by: Monk Liu <a class="moz-txt-link-rfc2396E" href="mailto:Monk.Liu@amd.com"><Monk.Liu@amd.com></a> >> --- >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> index afcae15..4169bb1 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> @@ -1918,14 +1918,14 @@ int amdgpu_device_init(struct amdgpu_device >> *adev, >> /* Get a log2 for easy divisions. */ >> adev->mm_stats.log2_max_MBps = ilog2(max(1u, max_MBps)); >> - amdgpu_fbdev_init(adev); >> - >> r = amdgpu_ib_pool_init(adev); >> if (r) { >> dev_err(adev->dev, "IB initialization failed (%d).\n", r); >> goto failed; >> } >> + amdgpu_fbdev_init(adev); >> + > > As noted internally as well, please more that one more further down > behind the call to amdgpu_ib_ring_tests(). If I understand correctly, that could result in no console output in the worst case if there's trouble with the ring & IB tests? That doesn't seem very good. -- Earthling Michel Dänzer | <a moz-do-not-send="true" href="http://www.amd.com"> http://www.amd.com</a> Libre software enthusiast | Mesa and X developer </div> <fieldset class="mimeAttachmentHeader"></fieldset> <pre wrap="">_______________________________________________ amd-gfx mailing list <a class="moz-txt-link-abbreviated" href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> <a class="moz-txt-link-freetext" href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> </pre> </blockquote> </body> </html>