Bug 29972

Summary:	d-r-t /r6xx+: use new style fencing (v2) causes very poor gl perf.
Product:	Mesa	Reporter:	Andy Furniss <adf.lists>
Component:	Drivers/DRI/R600	Assignee:	Default DRI bug account <dri-devel>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	medium	CC:	chainsawbike, marvin24
Version:	git
Hardware:	x86 (IA32)
OS:	Linux (All)
Whiteboard:
i915 platform:		i915 features:
Attachments:	properly enable eop int

Description Andy Furniss 2010-09-02 06:02:09 UTC

rv790 PCIE, current git ddx, mesa from yesterday.

Just tried today's updated d-r-t and 
/r6xx+: use new style fencing (v2)
is causing very poor gl perf eg. 1 fps in mesa demo tunnel, some demos appear OK at first but even though they reach refresh rate they glitch for a second after a while.

exa is also affected - it's slower drawing than the "instant" speed I am used to eg when switching between metacity workspaces. Seamonkey scrolling with a page with images is glitchy.

Tiling on/off makes no difference, with without last ddx commit makes no difference.

git reset --hard d-r-t to the commit before this one is working OK.

Comment 1 Alexandre Derumier 2010-09-03 03:24:39 UTC

Same problem for me, on rv620 chipset.

Comment 2 Alexandre Derumier 2010-09-03 07:56:44 UTC

i think the problem come from the writeback patch:

other archlinux users have the same problem

https://bbs.archlinux.org/viewtopic.php?pid=820584#p820584


with radeon.no_wb=1  in boot parameter, i don't have any slowdown.

Comment 3 Andy Furniss 2010-09-03 09:07:05 UTC

(In reply to comment #2)
> i think the problem come from the writeback patch:
> 
> other archlinux users have the same problem
> 
> https://bbs.archlinux.org/viewtopic.php?pid=820584#p820584
> 
> 
> with radeon.no_wb=1  in boot parameter, i don't have any slowdown.

no_wb=1 fixes for me also.
I have no problems with wb enabled + old fencing.

Comment 4 Alex Deucher 2010-09-04 01:53:41 UTC

Created attachment 38420 [details] [review]
properly enable eop int

This patch should fix the issue.  revert "drm/radeon/kms/r6xx+: use new style fencing (v2)" and apply this one instead.

Comment 5 Andy Furniss 2010-09-04 04:45:32 UTC

(In reply to comment #4)
> Created an attachment (id=38420) [details]
> properly enable eop int
> 
> This patch should fix the issue.  revert "drm/radeon/kms/r6xx+: use new style
> fencing (v2)" and apply this one instead.

Confirmed, this fixes it for me.

Comment 6 Alexandre Derumier 2010-09-04 05:57:13 UTC

for me too, no more problem with v3 patch

Comment 7 Andy Furniss 2010-09-05 05:33:47 UTC

*** Bug 30025 has been marked as a duplicate of this bug. ***

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.