From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 95017] [drm:radeon_ib_ring_tests] *ERROR* radeon: failed testing IB on GFX ring (-35). Date: Mon, 02 May 2016 15:00:12 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1915304522==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 9A6BF6E076 for ; Mon, 2 May 2016 15:00:12 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1915304522== Content-Type: multipart/alternative; boundary="14622012120.dB0e.30483"; charset="UTF-8" --14622012120.dB0e.30483 Date: Mon, 2 May 2016 15:00:12 +0000 MIME-Version: 1.0 Content-Type: text/plain https://bugs.freedesktop.org/show_bug.cgi?id=95017 --- Comment #9 from Ilia Mirkin --- (In reply to Mathieu Malaterre from comment #8) > (In reply to Ilia Mirkin from comment #7) > > FWIW on nouveau we just default agpmode=0 in a #ifdef __powerpc__. Perhaps > > the same thing can work for radeon. > > Well radeon_agpmode is already set to 0 already[*]. My patch force to > PCI(-1) when user did not specify any value. > > [*]https://github.com/torvalds/linux/blob/master/drivers/gpu/drm/radeon/ > radeon_drv.c#L173 Oh, the logic is flipped from nouveau... on nouveau, -1 == autodetect and 0 == no agp. So in your case, just set it to -1 instead of 0 for powerpc. i.e. #ifdef __powerpc__ int radeon_agpmode = -1; #else int radeon_agpmode = 0; #endif or something similar. -- You are receiving this mail because: You are the assignee for the bug. --14622012120.dB0e.30483 Date: Mon, 2 May 2016 15:00:12 +0000 MIME-Version: 1.0 Content-Type: text/html

Comment # 9 on bug 95017 from 
(In reply to Mathieu Malaterre from comment #8)
> (In reply to Ilia Mirkin from comment #7)
> > FWIW on nouveau we just default agpmode=0 in a #ifdef __powerpc__. Perhaps
> > the same thing can work for radeon.
> 
> Well radeon_agpmode is already set to 0 already[*]. My patch force to
> PCI(-1) when user did not specify any value.
> 
> [*]https://github.com/torvalds/linux/blob/master/drivers/gpu/drm/radeon/
> radeon_drv.c#L173

Oh, the logic is flipped from nouveau... on nouveau, -1 == autodetect and 0 ==
no agp. So in your case, just set it to -1 instead of 0 for powerpc. i.e.

#ifdef __powerpc__
int radeon_agpmode = -1;
#else
int radeon_agpmode = 0;
#endif

or something similar.

You are receiving this mail because:
  • You are the assignee for the bug.
--14622012120.dB0e.30483-- --===============1915304522== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============1915304522==--