* [Bug 85596] SB is used only after GPR check / GPR max is not dynamic
2014-10-29 11:37 [Bug 85596] New: SB is used only after GPR check / GPR max is not dynamic bugzilla-daemon
@ 2014-10-29 11:37 ` bugzilla-daemon
2014-10-29 13:31 ` bugzilla-daemon
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2014-10-29 11:37 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 376 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=85596
Lauri Kasanen <curaga@operamail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Depends on| |74868, 81683, 69623
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 1079 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug 85596] SB is used only after GPR check / GPR max is not dynamic
2014-10-29 11:37 [Bug 85596] New: SB is used only after GPR check / GPR max is not dynamic bugzilla-daemon
2014-10-29 11:37 ` [Bug 85596] " bugzilla-daemon
@ 2014-10-29 13:31 ` bugzilla-daemon
2014-10-29 21:36 ` bugzilla-daemon
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2014-10-29 13:31 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 770 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=85596
--- Comment #1 from Alex Deucher <agd5f@yahoo.com> ---
(In reply to Lauri Kasanen from comment #0)
> Many big shaders that currently fail with
> r600_shader_select - Failed to build shader variant (type=1) -12
> r600_shader_from_tgsi - GPR limit exceeded - shader requires foo registers
>
> would actually work if the GPR check was moved to after SB, as SB reduces
> the GPR usage quite nicely.
>
> Another thing is that 128 is the lowest common denominator, and many cards
> have 192 or 256, which could be checked with the radeon_info ioctl.
>From an ISA perspective, there are only 128 GPRs. The higher limits are hw
internal details.
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 1639 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug 85596] SB is used only after GPR check / GPR max is not dynamic
2014-10-29 11:37 [Bug 85596] New: SB is used only after GPR check / GPR max is not dynamic bugzilla-daemon
2014-10-29 11:37 ` [Bug 85596] " bugzilla-daemon
2014-10-29 13:31 ` bugzilla-daemon
@ 2014-10-29 21:36 ` bugzilla-daemon
2014-10-30 12:34 ` bugzilla-daemon
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2014-10-29 21:36 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 1402 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=85596
--- Comment #2 from Vadim Girlin <ptpzz@yandex.ru> ---
(In reply to Lauri Kasanen from comment #0)
> Many big shaders that currently fail with
> r600_shader_select - Failed to build shader variant (type=1) -12
> r600_shader_from_tgsi - GPR limit exceeded - shader requires foo registers
>
> would actually work if the GPR check was moved to after SB, as SB reduces
> the GPR usage quite nicely.
>
> Another thing is that 128 is the lowest common denominator, and many cards
> have 192 or 256, which could be checked with the radeon_info ioctl.
As Alex said, ISA encoding doesn't allow to address more than 128 registers in
the instructions. IIRC we also by default reserve 4 GPRs as temporary (they are
not preserved between ALU clauses), so the actual limit is 124 (or even 120?).
It's also the reason why we can't simply move the GPR check, the shader is
passed from TGSI translator to SB in the ISA encoding which can't represent the
code that uses more than 128 registers.
If anyone would like to revive a direct TGSI->SB translator that solves the
problem, here is the branch:
http://cgit.freedesktop.org/~vadimg/mesa/log/?h=wip-sb-tgsi
There were no piglit regressions with that branch on evergreen when it was
implemented, but now I suspect it's a bit outdated.
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 2348 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug 85596] SB is used only after GPR check / GPR max is not dynamic
2014-10-29 11:37 [Bug 85596] New: SB is used only after GPR check / GPR max is not dynamic bugzilla-daemon
` (2 preceding siblings ...)
2014-10-29 21:36 ` bugzilla-daemon
@ 2014-10-30 12:34 ` bugzilla-daemon
2016-04-19 9:54 ` bugzilla-daemon
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2014-10-30 12:34 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 237 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=85596
--- Comment #3 from Lauri Kasanen <curaga@operamail.com> ---
Thanks, at least that info is now in one place.
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 1018 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug 85596] SB is used only after GPR check / GPR max is not dynamic
2014-10-29 11:37 [Bug 85596] New: SB is used only after GPR check / GPR max is not dynamic bugzilla-daemon
` (3 preceding siblings ...)
2014-10-30 12:34 ` bugzilla-daemon
@ 2016-04-19 9:54 ` bugzilla-daemon
2019-09-18 19:16 ` bugzilla-daemon
2019-09-18 19:17 ` bugzilla-daemon
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2016-04-19 9:54 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 551 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=85596
Bug 85596 depends on bug 69623, which changed state.
Bug 69623 Summary: Pink Pony misrenders - - shader requires 126 registers
https://bugs.freedesktop.org/show_bug.cgi?id=69623
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 1298 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug 85596] SB is used only after GPR check / GPR max is not dynamic
2014-10-29 11:37 [Bug 85596] New: SB is used only after GPR check / GPR max is not dynamic bugzilla-daemon
` (4 preceding siblings ...)
2016-04-19 9:54 ` bugzilla-daemon
@ 2019-09-18 19:16 ` bugzilla-daemon
2019-09-18 19:17 ` bugzilla-daemon
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2019-09-18 19:16 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 569 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=85596
Bug 85596 depends on bug 81683, which changed state.
Bug 81683 Summary: [r600g] graphic glitches in Skyrim - failed to build shader
https://bugs.freedesktop.org/show_bug.cgi?id=81683
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |MOVED
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 1333 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread* [Bug 85596] SB is used only after GPR check / GPR max is not dynamic
2014-10-29 11:37 [Bug 85596] New: SB is used only after GPR check / GPR max is not dynamic bugzilla-daemon
` (5 preceding siblings ...)
2019-09-18 19:16 ` bugzilla-daemon
@ 2019-09-18 19:17 ` bugzilla-daemon
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2019-09-18 19:17 UTC (permalink / raw)
To: dri-devel
[-- Attachment #1.1: Type: text/plain, Size: 840 bytes --]
https://bugs.freedesktop.org/show_bug.cgi?id=85596
GitLab Migration User <gitlab-migration@fdo.invalid> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |MOVED
--- Comment #4 from GitLab Migration User <gitlab-migration@fdo.invalid> ---
-- GitLab Migration Automatic Message --
This bug has been migrated to freedesktop.org's GitLab instance and has been
closed from further activity.
You can subscribe and participate further through the new bug through this link
to our GitLab instance: https://gitlab.freedesktop.org/mesa/mesa/issues/530.
--
You are receiving this mail because:
You are the assignee for the bug.
[-- Attachment #1.2: Type: text/html, Size: 2459 bytes --]
[-- Attachment #2: Type: text/plain, Size: 159 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 8+ messages in thread