[Bug 51658] New: r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[Bug 51658] New: r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

             Bug #: 51658
           Summary: r200 (& possibly radeon) DRI fixes for gnome shell on
                    Mesa 8.0.3
    Classification: Unclassified
           Product: Mesa
           Version: 8.0
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: major
          Priority: medium
         Component: Drivers/DRI/r200
        AssignedTo: dri-devel@lists.freedesktop.org
        ReportedBy: grimrc@yahoo.com


Created attachment 63711
  --> https://bugs.freedesktop.org/attachment.cgi?id=63711
Essential patch to disable texture formats that are reported unrenderable
elsewhere in driver.

Without r200-tex-format-fix.patch, gnome shell crashes. Texture formats are
selected by radeonChoose8888TexFormat but reported as unrenderable by
radeonIsFormatRenderable.

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #5 from Eugene St Leger <grimrc@yahoo.com> 2012-07-02 10:45:01 UTC ---
Created attachment 63716
  --> https://bugs.freedesktop.org/attachment.cgi?id=63716
Optional patch to warn (once) when a blit with 2048 pixel dimension occurs on
r200.

If 2048 pixel blits cause graphical glitches/problems on r200, this patch can
be used to provide a single warning.

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #1 from Eugene St Leger <grimrc@yahoo.com> 2012-07-02 10:29:44 PDT ---
Created attachment 63712
  --> https://bugs.freedesktop.org/attachment.cgi?id=63712
Essential patch to allow 2048 pixel blits.

Without this patch, gnome shell crashes. 2048 pixel blits are used.

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #2 from Eugene St Leger <grimrc@yahoo.com> 2012-07-02 10:34:41 PDT ---
Created attachment 63713
  --> https://bugs.freedesktop.org/attachment.cgi?id=63713
Essential patch to reapply dirtied texenv registers.

Without this patch, colour corruption happens with xv.  xf86-video-ati textured
xv dirties texenv registers and r200 DRI does not reapply them.

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #3 from Eugene St Leger <grimrc@yahoo.com> 2012-07-02 10:37:47 PDT ---
Created attachment 63714
  --> https://bugs.freedesktop.org/attachment.cgi?id=63714
Unessential fixes/enhancements patch.

Minor spelling fixes & enhancements.

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #4 from Eugene St Leger <grimrc@yahoo.com> 2012-07-02 10:40:51 PDT ---
Created attachment 63715
  --> https://bugs.freedesktop.org/attachment.cgi?id=63715
Proposed blit register dirtying patch.

It appears bliting dirties some registers without notifying. This proposed
patch notifies of register dirtying.

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #6 from Eugene St Leger <grimrc@yahoo.com> 2012-07-02 10:49:07 PDT ---
Created attachment 63717
  --> https://bugs.freedesktop.org/attachment.cgi?id=63717
Untested but probably essential patch to allow 2048 pixel blits on r100.

Without this patch, gnome shell is expected to crash. This patch is untested.

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #7 from Eugene St Leger <grimrc@yahoo.com> 2012-07-02 10:51:10 PDT ---
Created attachment 63718
  --> https://bugs.freedesktop.org/attachment.cgi?id=63718
Optional untested patch to warn (once) when a blit with 2048 pixel dimension
occurs on r100.

If 2048 pixel blits cause graphical glitches/problems on r100, this patch can
be used to provide a single warning. This patch is untested.

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #8 from Eugene St Leger <grimrc@yahoo.com> 2012-07-02 11:04:54 PDT ---
A summary of all of the patches follows.

r200 essential patches (1st 3 patches) for gnome shell:
"Essential patch to disable texture formats that are reported unrenderable
elsewhere in driver."
"Essential patch to allow 2048 pixel blits."
"Essential patch to reapply dirtied texenv registers."

suggested minor fixes patch (4th patch):
"Unessential fixes/enhancements patch."

proposed r200 blit patch (5th patch) - unknown importance:
"Proposed blit register dirtying patch."

untested but probably essential r100/radeon patch (7th patch) for gnome shell:
"Untested but probably essential patch to allow 2048 pixel blits on r100."

optional unrecommended patches to supplement blit patches already mentioned
above (6th & 8th patches):
"Optional patch to warn (once) when a blit with 2048 pixel dimension occurs on
r200."
"Optional untested patch to warn (once) when a blit with 2048 pixel dimension
occurs on r100."

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #9 from Roland Scheidegger <sroland@vmware.com> 2012-07-03 05:20:47 PDT ---
(In reply to comment #8)
> A summary of all of the patches follows.
> 
> r200 essential patches (1st 3 patches) for gnome shell:
> "Essential patch to disable texture formats that are reported unrenderable
> elsewhere in driver."
I'm not really happy with that. I guess the problem is if we try to attach a
texture to a fbo it is too late to notice we cannot render to that format. But
this sort of sucks for ordinary textures. Ideally we'd probably be able to
really determine our format only at the first upload of data to the texture
(which presumably won't happen) so we could change it if we attach it to a fbo.
Though if there is indeed already data uploaded we're screwed.

> "Essential patch to allow 2048 pixel blits."
This one looks good as far as I can tell. The programmed values are
width/height -1 so this clearly should work.

> "Essential patch to reapply dirtied texenv registers."
I don't understand this. If the state isn't used anyway why need to reuppload
it, dirty or not? I think this is just hiding the root cause of another bug.

> 
> suggested minor fixes patch (4th patch):
> "Unessential fixes/enhancements patch."
Look good.

> 
> proposed r200 blit patch (5th patch) - unknown importance:
> "Proposed blit register dirtying patch."
Makes sense to me.

> 
> untested but probably essential r100/radeon patch (7th patch) for gnome shell:
> "Untested but probably essential patch to allow 2048 pixel blits on r100."
Yes that should be same as for r200.

> 
> optional unrecommended patches to supplement blit patches already mentioned
> above (6th & 8th patches):
> "Optional patch to warn (once) when a blit with 2048 pixel dimension occurs on
> r200."
> "Optional untested patch to warn (once) when a blit with 2048 pixel dimension
> occurs on r100."
I don't think this is necessary - at least not there. texture src width/height
of 2048 should clearly work. I am however not so sure about destination.
width/height are fed into RADEON_RE_WIDTH_HEIGHT, and those have just 11 bit,
which isn't enough for 2048. Furthermore, docs say the values are inclusive.
Not sure what's up with that, the ddx also just uses width/height and not
width/height -1 but clamps them to 2047 in some places. dri driver though
otherwise seems to use x2/y2 (i.e. width -1, height -1). So I think this should
be fixed everywhere to really use width/height -1.

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

[Bug 51658] r200 (& possibly radeon) DRI fixes for gnome shell on Mesa 8.0.3

[bugzilla-daemon]

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

--- Comment #10 from Roland Scheidegger <sroland@vmware.com> 2012-08-01 13:07:09 UTC ---
With 5b88a2a22daae4d09596804d8edc6b8796d05150 attachment 63712, 63716, 63717,
63718 are obsolete. Still unsure what to do with the others.

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