All of lore.kernel.org
 help / color / mirror / Atom feed
* [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows
@ 2010-11-22  7:56 bugzilla-daemon
  2010-11-22  7:57 ` [Bug 31830] " bugzilla-daemon
                   ` (20 more replies)
  0 siblings, 21 replies; 22+ messages in thread
From: bugzilla-daemon @ 2010-11-22  7:56 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

           Summary: [bisected] Unigine Sanctuary v2.2: broken shadows
           Product: Mesa
           Version: git
          Platform: Other
        OS/Version: All
            Status: NEW
          Keywords: regression
          Severity: normal
          Priority: medium
         Component: Drivers/Gallium/r300
        AssignedTo: dri-devel@lists.freedesktop.org
        ReportedBy: drakkk@centrum.cz


Created an attachment (id=40467)
 --> (https://bugs.freedesktop.org/attachment.cgi?id=40467)
screenshot

With latest mesa git shadows in Unigine are broken again. This is with shaders
set to low or medium (high works fine).

commit 8833f53e659e079e7ab74bb9197f9b44b1eeefe0
Author: Tom Stellard <tstellar@gmail.com>
Date:   Wed Nov 10 21:34:18 2010 -0800

    r300/compiler: Enable rename_reg pass for r500 cards

    In addition, the rename_reg pass has been rewritten to use
    rc_get_readers().


Workaround is RADEON_DEBUG=noopt.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] Unigine Sanctuary v2.2: broken shadows
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
@ 2010-11-22  7:57 ` bugzilla-daemon
  2011-01-05 10:54 ` [Bug 31830] [bisected] Unigine Sanctuary: " bugzilla-daemon
                   ` (19 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2010-11-22  7:57 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #1 from Pavel Ondračka <drakkk@centrum.cz> 2010-11-21 23:57:50 PST ---
Created an attachment (id=40468)
 --> (https://bugs.freedesktop.org/attachment.cgi?id=40468)
output with RADEON_DEBUG=fp and RADEON_DEBUG=fp,noopt

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] Unigine Sanctuary: broken shadows
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
  2010-11-22  7:57 ` [Bug 31830] " bugzilla-daemon
@ 2011-01-05 10:54 ` bugzilla-daemon
  2011-01-21 12:19 ` [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark bugzilla-daemon
                   ` (18 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-05 10:54 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

Pavel Ondračka <drakkk@centrum.cz> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[bisected] Unigine          |[bisected] Unigine
                   |Sanctuary v2.2: broken      |Sanctuary: broken shadows
                   |shadows                     |
                URL|                            |http://www.unigine.com/down
                   |                            |load/#sanctuary

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
  2010-11-22  7:57 ` [Bug 31830] " bugzilla-daemon
  2011-01-05 10:54 ` [Bug 31830] [bisected] Unigine Sanctuary: " bugzilla-daemon
@ 2011-01-21 12:19 ` bugzilla-daemon
  2011-01-23  8:33 ` bugzilla-daemon
                   ` (17 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-21 12:19 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

Pavel Ondračka <drakkk@centrum.cz> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[bisected] Unigine          |[bisected] broken shadows
                   |Sanctuary: broken shadows   |in Unigine Sanctuary and
                   |                            |Lightsmark

--- Comment #2 from Pavel Ondračka <drakkk@centrum.cz> 2011-01-21 04:19:37 PST ---
Penumbra shadows in Lightsmark 2008 2.0 are broken with the same commit. It
works fine with RADEON_DEBUG=noopt. 
BTW during bisecting I run into some interesting performance issues:

Before 8833f53e659e079e7ab74bb9197f9b44b1eeefe0: 5,66 +- 0,01 fps, shadows are
OK
mesa 8833f53e659e079e7ab74bb9197f9b44b1eeefe0: 3,69 +- 0,07 fps, broken shadows

with current mesa master (af4e2f46653cbc7ceaf1291ba22087ec5758d07f):
stock settings: 10,23 +- 0,08 fps, broken shadows
with RADEON_DEBUG=noopt: 11,73 +- 0,16 fps, shadows are OK
with register rename pass disabled: 14,35 +- 0,04 fps shadows are OK

I disabled register rename by changing r3xx_fragprog.c line 127,
{"register rename", 1, !is_r500 || opt, rc_rename_regs,    NULL},
to {"register rename", 1, !is_r500, rc_rename_regs,NULL},
I hope this is correct.

Mesa: af4e2f46653cbc7ceaf1291ba22087ec5758d07f
Kernel: 2.6.37
GPU: RV530

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (2 preceding siblings ...)
  2011-01-21 12:19 ` [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark bugzilla-daemon
@ 2011-01-23  8:33 ` bugzilla-daemon
  2011-01-23  8:53 ` bugzilla-daemon
                   ` (16 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-23  8:33 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #3 from Tom Stellard <tstellar@gmail.com> 2011-01-23 00:33:24 PST ---
What penumbra quality number are you using in Lightsmark? Are you seeing any
error messages like this:

r300: ERROR: FS input generic 18 unassigned, not enough hardware slots.
r300: ERROR: FS input generic 19 unassigned, not enough hardware slots.
r300: ERROR: FS input generic 20 unassigned, not enough hardware slots.
r300: ERROR: FS input WPOS unassigned, not enough hardware slots.

If you are, were they also present before commit
8833f53e659e079e7ab74bb9197f9b44b1eeefe0?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (3 preceding siblings ...)
  2011-01-23  8:33 ` bugzilla-daemon
@ 2011-01-23  8:53 ` bugzilla-daemon
  2011-01-24  9:13 ` bugzilla-daemon
                   ` (15 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-23  8:53 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #4 from Pavel Ondračka <drakkk@centrum.cz> 2011-01-23 00:53:39 PST ---
(In reply to comment #3)
> What penumbra quality number are you using in Lightsmark? Are you seeing any
> error messages like this:
> 
> r300: ERROR: FS input generic 18 unassigned, not enough hardware slots.
> r300: ERROR: FS input generic 19 unassigned, not enough hardware slots.
> r300: ERROR: FS input generic 20 unassigned, not enough hardware slots.
> r300: ERROR: FS input WPOS unassigned, not enough hardware slots.
> 
> If you are, were they also present before commit
> 8833f53e659e079e7ab74bb9197f9b44b1eeefe0?

Yes, I'm seeing identical messages, and they also were present before
8833f53e659e079e7ab74bb9197f9b44b1eeefe0.

Penumbra quality: 8/8 on Gallium 0.4 on ATI RV530.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (4 preceding siblings ...)
  2011-01-23  8:53 ` bugzilla-daemon
@ 2011-01-24  9:13 ` bugzilla-daemon
  2011-01-25 13:24 ` bugzilla-daemon
                   ` (14 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-24  9:13 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #5 from Tom Stellard <tstellar@gmail.com> 2011-01-24 01:13:49 PST ---
Created an attachment (id=42355)
 View: https://bugs.freedesktop.org/attachment.cgi?id=42355
 Review: https://bugs.freedesktop.org/review?bug=31830&attachment=42355

Possible Fix

I think this patch fixes Lightsmark, can you confirm this?  Does this patch fix
Sanctuary as well?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (5 preceding siblings ...)
  2011-01-24  9:13 ` bugzilla-daemon
@ 2011-01-25 13:24 ` bugzilla-daemon
  2011-01-26  9:11 ` bugzilla-daemon
                   ` (13 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-25 13:24 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #6 from Pavel Ondračka <drakkk@centrum.cz> 2011-01-25 05:24:00 PST ---
(In reply to comment #5)
> Created an attachment (id=42355)
 View: https://bugs.freedesktop.org/attachment.cgi?id=42355
 Review: https://bugs.freedesktop.org/review?bug=31830&attachment=42355

> Possible Fix
> 
> I think this patch fixes Lightsmark, can you confirm this?  Does this patch fix
> Sanctuary as well?

I'm sorry to report this, but your patch doesn't fix neither Lightsmark nor
Sanctuary here.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (6 preceding siblings ...)
  2011-01-25 13:24 ` bugzilla-daemon
@ 2011-01-26  9:11 ` bugzilla-daemon
  2011-01-26 10:17 ` bugzilla-daemon
                   ` (12 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-26  9:11 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

Tom Stellard <tstellar@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #42355|0                           |1
        is obsolete|                            |

--- Comment #7 from Tom Stellard <tstellar@gmail.com> 2011-01-26 01:11:35 PST ---
Created an attachment (id=42514)
 View: https://bugs.freedesktop.org/attachment.cgi?id=42514
 Review: https://bugs.freedesktop.org/review?bug=31830&attachment=42514

Disable instruction rewriting in the scheduler

I think this bug is actually in the scheduler, and it is just being exposed by
the reg rename pass.  I have been able to track down a few instructions that
the scheduler is converting from vector to scalar that change how the shadows
are rendered.  However, these conversions look valid too me, so I'm really not
sure what the problem is.  I think this patch fixes Lightsmark, but I'm not
sure what the shadows are supposed to look like.  Can you try this patch on
Lightsmark and Sanctuary.  Thanks.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (7 preceding siblings ...)
  2011-01-26  9:11 ` bugzilla-daemon
@ 2011-01-26 10:17 ` bugzilla-daemon
  2011-01-26 17:05 ` bugzilla-daemon
                   ` (11 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-26 10:17 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #8 from Pavel Ondračka <drakkk@centrum.cz> 2011-01-26 02:17:15 PST ---
Created an attachment (id=42516)
 --> (https://bugs.freedesktop.org/attachment.cgi?id=42516)
screenshot of the problem

(In reply to comment #7)
> Created an attachment (id=42514)
 View: https://bugs.freedesktop.org/attachment.cgi?id=42514
 Review: https://bugs.freedesktop.org/review?bug=31830&attachment=42514

> Disable instruction rewriting in the scheduler
> 
> I think this bug is actually in the scheduler, and it is just being exposed by
> the reg rename pass.  I have been able to track down a few instructions that
> the scheduler is converting from vector to scalar that change how the shadows
> are rendered.  However, these conversions look valid too me, so I'm really not
> sure what the problem is.  I think this patch fixes Lightsmark, but I'm not
> sure what the shadows are supposed to look like.  Can you try this patch on
> Lightsmark and Sanctuary.  Thanks.

Hi, this patch doesn't help neither. I tested it alone and also with the first
patch. Concerning how the penumbra shadows should look, you don't get the right
behavior with RADEON_DEBUG=noopt? I've attached screenshot comparing bad and
good shadows. Don't know how to help you more, I still have the debug patches
you've sent me when we were debugging sanctuary before the "enable rename_reg"
patch was committed, maybe I could use them again to identify the failing
shader?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (8 preceding siblings ...)
  2011-01-26 10:17 ` bugzilla-daemon
@ 2011-01-26 17:05 ` bugzilla-daemon
  2011-01-27  8:48 ` bugzilla-daemon
                   ` (10 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-26 17:05 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #9 from Tom Stellard <tstellar@gmail.com> 2011-01-26 09:05:17 PST ---
(In reply to comment #8)
> Created an attachment (id=42516)
 --> (https://bugs.freedesktop.org/attachment.cgi?id=42516)
> screenshot of the problem
> 
> Hi, this patch doesn't help neither. I tested it alone and also with the first
> patch. Concerning how the penumbra shadows should look, you don't get the right
> behavior with RADEON_DEBUG=noopt? I've attached screenshot comparing bad and
> good shadows. Don't know how to help you more, I still have the debug patches
> you've sent me when we were debugging sanctuary before the "enable rename_reg"
> patch was committed, maybe I could use them again to identify the failing
> shader?

That's OK, I already know which shader is failing.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (9 preceding siblings ...)
  2011-01-26 17:05 ` bugzilla-daemon
@ 2011-01-27  8:48 ` bugzilla-daemon
  2011-01-27  9:18 ` bugzilla-daemon
                   ` (9 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-27  8:48 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #10 from Tom Stellard <tstellar@gmail.com> 2011-01-27 00:48:55 PST ---
(In reply to comment #3)
> What penumbra quality number are you using in Lightsmark? Are you seeing any
> error messages like this:
> 
> r300: ERROR: FS input generic 18 unassigned, not enough hardware slots.
> r300: ERROR: FS input generic 19 unassigned, not enough hardware slots.
> r300: ERROR: FS input generic 20 unassigned, not enough hardware slots.
> r300: ERROR: FS input WPOS unassigned, not enough hardware slots.
> 

In Lightsmark, I've been able to identify the failing shader and a few
instruction transformations that change the way the shadows are rendered.  I've
spent a lot of time looking at this, and all the transformations in question
appear to be correct.  It's possible that above errors are the reason why the
shadows are misrendering.  The last error about WPOS also appears in sanctuary. 

My hypothesis is that with these kinds of errors, the behavior of a shader is
undefined.  This means that prior to the commit in question, the scenes were by
luck being rendered correctly, or close enough so that it wasn't noticeable. 
Then, the modifications made to the shader by the reg_rename pass changed the
state of the GPU in such a way that the misrendering became more obvious.

Maybe Marek can comment on whether or not this is a plausible hypothesis.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (10 preceding siblings ...)
  2011-01-27  8:48 ` bugzilla-daemon
@ 2011-01-27  9:18 ` bugzilla-daemon
  2011-01-27 10:20 ` bugzilla-daemon
                   ` (8 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-27  9:18 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #11 from Pavel Ondračka <drakkk@centrum.cz> 2011-01-27 01:18:41 PST ---
(In reply to comment #10)
> (In reply to comment #3)
> > What penumbra quality number are you using in Lightsmark? Are you seeing any
> > error messages like this:
> > 
> > r300: ERROR: FS input generic 18 unassigned, not enough hardware slots.
> > r300: ERROR: FS input generic 19 unassigned, not enough hardware slots.
> > r300: ERROR: FS input generic 20 unassigned, not enough hardware slots.
> > r300: ERROR: FS input WPOS unassigned, not enough hardware slots.
> > 
> 
> In Lightsmark, I've been able to identify the failing shader and a few
> instruction transformations that change the way the shadows are rendered.  I've
> spent a lot of time looking at this, and all the transformations in question
> appear to be correct.  It's possible that above errors are the reason why the
> shadows are misrendering.  The last error about WPOS also appears in sanctuary. 
> 
> My hypothesis is that with these kinds of errors, the behavior of a shader is
> undefined.  This means that prior to the commit in question, the scenes were by
> luck being rendered correctly, or close enough so that it wasn't noticeable. 
> Then, the modifications made to the shader by the reg_rename pass changed the
> state of the GPU in such a way that the misrendering became more obvious.
> 
> Maybe Marek can comment on whether or not this is a plausible hypothesis.

Does this also explain the slowdown mentioned in comment 2? Because it is in
all scenes, not only the "penumbra shadows" one and it is really weird that
RADEON_DEBUG=noopt actually makes things faster.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (11 preceding siblings ...)
  2011-01-27  9:18 ` bugzilla-daemon
@ 2011-01-27 10:20 ` bugzilla-daemon
  2011-01-28  8:20 ` bugzilla-daemon
                   ` (7 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-27 10:20 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #12 from Marek Olšák <maraeo@gmail.com> 2011-01-27 02:20:19 PST ---
Well, if I disable the register rename pass, the Penumbra shadows are rendered
more or less correctly, so it must be a bug somewhere in the pair passes (or
maybe even rename_regs).

My hypothesis is that a few unitialized shadow-map coordinates shouldn't
influence the results computed from other shadow maps and those undefined
shadow map samples should not contribute much to the final color, considering
that the majority of samples are still computed correctly.

Also, disabling rename_regs speeds up Lightsmark by about 50%, so we should
really reconsider whether having such a pass is worth the cost.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (12 preceding siblings ...)
  2011-01-27 10:20 ` bugzilla-daemon
@ 2011-01-28  8:20 ` bugzilla-daemon
  2011-01-28  8:21 ` bugzilla-daemon
                   ` (6 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-28  8:20 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #13 from Tom Stellard <tstellar@gmail.com> 2011-01-28 00:20:29 PST ---
I'm pretty sure this is a bug in the scheduler.  I'm attaching screenshots
comparing running with this patch:
https://bugs.freedesktop.org/attachment.cgi?id=42514 to running with
RADEON_DEBUG=noopt.  The screenshots look very similar, but I think the shadow
with RADEON_DEBUG=noopt is darker.

As for performance difference between RADEON_DEBUG=noopt and with
optimizations.  I think we are seeing this because Lightsmark compiles new
shaders while running the benchmark and the longer compile time for optimized
shaders is lowering the FPS.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (13 preceding siblings ...)
  2011-01-28  8:20 ` bugzilla-daemon
@ 2011-01-28  8:21 ` bugzilla-daemon
  2011-01-28  8:22 ` bugzilla-daemon
                   ` (5 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-28  8:21 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #14 from Tom Stellard <tstellar@gmail.com> 2011-01-28 00:21:18 PST ---
Created an attachment (id=42626)
 --> (https://bugs.freedesktop.org/attachment.cgi?id=42626)
Screenshot with RADEON_DEBUG=noopt

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (14 preceding siblings ...)
  2011-01-28  8:21 ` bugzilla-daemon
@ 2011-01-28  8:22 ` bugzilla-daemon
  2011-01-28  8:53 ` bugzilla-daemon
                   ` (4 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-28  8:22 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #15 from Tom Stellard <tstellar@gmail.com> 2011-01-28 00:22:12 PST ---
Created an attachment (id=42627)
 --> (https://bugs.freedesktop.org/attachment.cgi?id=42627)
Screenshot with scheduler patch

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (15 preceding siblings ...)
  2011-01-28  8:22 ` bugzilla-daemon
@ 2011-01-28  8:53 ` bugzilla-daemon
  2011-01-28  9:36 ` bugzilla-daemon
                   ` (3 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-28  8:53 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #16 from Pavel Ondračka <drakkk@centrum.cz> 2011-01-28 00:53:06 PST ---
I'm puzzled, because here with mesa master + disable scheduler rewrite patch, I
get the wrong behavior (right part of
https://bugs.freedesktop.org/attachment.cgi?id=42516 ). However your screenshot
in Comment 15 looks almost completely good. To be sure I did once again git
clean -fdx, git reset --hard origin/master, patch again and recompile, but
nothing changed. Some hardware difference or my setup is broken? Maybe someone
with the same card as mine (RV530) can test your patch to confirm if it indeed
fixes the shadows? Any chance you have also some other patches applied?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (16 preceding siblings ...)
  2011-01-28  8:53 ` bugzilla-daemon
@ 2011-01-28  9:36 ` bugzilla-daemon
  2011-01-28 11:42 ` bugzilla-daemon
                   ` (2 subsequent siblings)
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-28  9:36 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #17 from Tom Stellard <tstellar@gmail.com> 2011-01-28 01:36:08 PST ---
The right side of this screenshot that you posted:
https://bugs.freedesktop.org/attachment.cgi?id=42516
Looks a lot like what I see when I disable the reg rename pass.

I've double checked the results with the scheduler patch and they are the same
as the screenshot I posted.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (17 preceding siblings ...)
  2011-01-28  9:36 ` bugzilla-daemon
@ 2011-01-28 11:42 ` bugzilla-daemon
  2011-01-28 18:56 ` bugzilla-daemon
  2011-02-06  7:54 ` bugzilla-daemon
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-28 11:42 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #18 from Pavel Ondračka <drakkk@centrum.cz> 2011-01-28 03:42:50 PST ---
(In reply to comment #17)
> The right side of this screenshot that you posted:
> https://bugs.freedesktop.org/attachment.cgi?id=42516
> Looks a lot like what I see when I disable the reg rename pass.
> 
> I've double checked the results with the scheduler patch and they are the same
> as the screenshot I posted.

This is getting weird, because Marek in comment 12 mentioned that the penumbra
shadows are rendered correctly with register rename pass disabled. I guess he
also have RV530?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (18 preceding siblings ...)
  2011-01-28 11:42 ` bugzilla-daemon
@ 2011-01-28 18:56 ` bugzilla-daemon
  2011-02-06  7:54 ` bugzilla-daemon
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-01-28 18:56 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

--- Comment #19 from Marek Olšák <maraeo@gmail.com> 2011-01-28 10:56:20 PST ---
I think I know the reason why it's slow with the rename_regs pass.

The statistics of the most complex shader are:

~~~~~~~~ FRAGMENT PROGRAM ~~~~~~~
~ 108 Instructions
~  71 Vector Instructions (RGB)
~  35 Scalar Instructions (Alpha)
~   0 Flow Control Instructions
~  36 Texture Instructions
~   2 Presub Operations
~  36 Temporary Registers
~~~~~~~~~~~~~~ END ~~~~~~~~~~~~~~

If you disable rename_regs, you will get:

~~~~~~~~ FRAGMENT PROGRAM ~~~~~~~
~ 143 Instructions
~  71 Vector Instructions (RGB)
~  35 Scalar Instructions (Alpha)
~   0 Flow Control Instructions
~  36 Texture Instructions
~   2 Presub Operations
~   4 Temporary Registers
~~~~~~~~~~~~~~ END ~~~~~~~~~~~~~~

Wow, 4 temps! It definitely appears to be a register allocator issue. Even
though the rename_regs pass has helped with instruction scheduling a lot, it
made regalloc perform much worse.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark
  2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
                   ` (19 preceding siblings ...)
  2011-01-28 18:56 ` bugzilla-daemon
@ 2011-02-06  7:54 ` bugzilla-daemon
  20 siblings, 0 replies; 22+ messages in thread
From: bugzilla-daemon @ 2011-02-06  7:54 UTC (permalink / raw)
  To: dri-devel

https://bugs.freedesktop.org/show_bug.cgi?id=31830

Tom Stellard <tstellar@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #20 from Tom Stellard <tstellar@gmail.com> 2011-02-05 23:54:34 PST ---
This is fixed in master by commit 68b701f5ded5f6b73a6b07cf37d862fab4836607. 
I'm still not sure what exactly was causing the misrenderings, but it is clear
that the register rename pass and the scheduler were hurting the performance of
Lightsmark, so, I'm disabling the register rename pass until we have a better
scheduler.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

^ permalink raw reply	[flat|nested] 22+ messages in thread

end of thread, other threads:[~2011-02-06  7:54 UTC | newest]

Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-11-22  7:56 [Bug 31830] New: [bisected] Unigine Sanctuary v2.2: broken shadows bugzilla-daemon
2010-11-22  7:57 ` [Bug 31830] " bugzilla-daemon
2011-01-05 10:54 ` [Bug 31830] [bisected] Unigine Sanctuary: " bugzilla-daemon
2011-01-21 12:19 ` [Bug 31830] [bisected] broken shadows in Unigine Sanctuary and Lightsmark bugzilla-daemon
2011-01-23  8:33 ` bugzilla-daemon
2011-01-23  8:53 ` bugzilla-daemon
2011-01-24  9:13 ` bugzilla-daemon
2011-01-25 13:24 ` bugzilla-daemon
2011-01-26  9:11 ` bugzilla-daemon
2011-01-26 10:17 ` bugzilla-daemon
2011-01-26 17:05 ` bugzilla-daemon
2011-01-27  8:48 ` bugzilla-daemon
2011-01-27  9:18 ` bugzilla-daemon
2011-01-27 10:20 ` bugzilla-daemon
2011-01-28  8:20 ` bugzilla-daemon
2011-01-28  8:21 ` bugzilla-daemon
2011-01-28  8:22 ` bugzilla-daemon
2011-01-28  8:53 ` bugzilla-daemon
2011-01-28  9:36 ` bugzilla-daemon
2011-01-28 11:42 ` bugzilla-daemon
2011-01-28 18:56 ` bugzilla-daemon
2011-02-06  7:54 ` bugzilla-daemon

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.