* [Bug 45921] [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl
2012-02-11 10:08 [Bug 45921] New: [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl bugzilla-daemon
@ 2012-02-11 3:59 ` bugzilla-daemon
2012-02-11 11:21 ` bugzilla-daemon
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2012-02-11 3:59 UTC (permalink / raw)
To: dri-devel
https://bugs.freedesktop.org/show_bug.cgi?id=45921
--- Comment #2 from Pavel Ondračka <pavel.ondracka@email.cz> 2012-02-11 03:59:26 UTC ---
(In reply to comment #1)
> I am sure the commit in question is not the cause. I prematurely added the code for the new ioctl and disabled it later on, because my kernel code the commit
> was supposed to interact with wasn't accepted.
>
> The problem when bisecting is that now it tries to call a non-existing ioctl if the DRM minor version is >=12, which makes some tests fail. Could you please
> check dmesg to see if that's the case? There should be some error messages from DRM.
None of the tests produce any output in dmesg.
>
> I think that some of those piglit failures are caused by the glsl-to-tgsi translator. You can disable glsl-to-tgsi by commenting out:
>
> functions->LinkShader = st_link_shader;
>
> at the end of src/mesa/state_tracker/st_cb_program.c.
From all the mentioned tests only glsl-fs-discard-02 pass when I disable glsl-to-tgsi.
BTW in which commit was the new ioctl disabled? I mean, I can understand that if I don't have kernel with support for the new ioctl it breaks the tests,
however if the ioctl was disabled later shouldn't be current git working unless another regression happened?
> Anyway, sorry, I won't probably help you with this bug anytime soon.
Well, I don't care much for piglit, I just want to make sure r300g don't get too rusty now when the development has moved to new drivers.
--
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] 8+ messages in thread* [Bug 45921] [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl
2012-02-11 10:08 [Bug 45921] New: [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl bugzilla-daemon
2012-02-11 3:59 ` [Bug 45921] " bugzilla-daemon
@ 2012-02-11 11:21 ` bugzilla-daemon
2012-02-11 12:19 ` bugzilla-daemon
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2012-02-11 11:21 UTC (permalink / raw)
To: dri-devel
https://bugs.freedesktop.org/show_bug.cgi?id=45921
--- Comment #1 from Marek Olšák <maraeo@gmail.com> 2012-02-11 03:21:46 PST ---
I am sure the commit in question is not the cause. I prematurely added the code for the new ioctl and disabled it later on, because my kernel code the commit
was supposed to interact with wasn't accepted.
The problem when bisecting is that now it tries to call a non-existing ioctl if the DRM minor version is >=12, which makes some tests fail. Could you please
check dmesg to see if that's the case? There should be some error messages from DRM.
I think that some of those piglit failures are caused by the glsl-to-tgsi translator. You can disable glsl-to-tgsi by commenting out:
functions->LinkShader = st_link_shader;
at the end of src/mesa/state_tracker/st_cb_program.c.
Anyway, sorry, I won't probably help you with this bug anytime soon.
--
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] 8+ messages in thread* [Bug 45921] [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl
2012-02-11 10:08 [Bug 45921] New: [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl bugzilla-daemon
2012-02-11 3:59 ` [Bug 45921] " bugzilla-daemon
2012-02-11 11:21 ` bugzilla-daemon
@ 2012-02-11 12:19 ` bugzilla-daemon
2012-02-11 12:51 ` bugzilla-daemon
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2012-02-11 12:19 UTC (permalink / raw)
To: dri-devel
https://bugs.freedesktop.org/show_bug.cgi?id=45921
--- Comment #3 from Marek Olšák <maraeo@gmail.com> 2012-02-11 04:19:27 PST ---
(In reply to comment #2)
> (In reply to comment #1)
> > I am sure the commit in question is not the cause. I prematurely added the code for the new ioctl and disabled it later on, because my kernel code the commit
> > was supposed to interact with wasn't accepted.
> >
> > The problem when bisecting is that now it tries to call a non-existing ioctl if the DRM minor version is >=12, which makes some tests fail. Could you please
> > check dmesg to see if that's the case? There should be some error messages from DRM.
>
> None of the tests produce any output in dmesg.
Okay.
> >
> > I think that some of those piglit failures are caused by the glsl-to-tgsi translator. You can disable glsl-to-tgsi by commenting out:
> >
> > functions->LinkShader = st_link_shader;
> >
> > at the end of src/mesa/state_tracker/st_cb_program.c.
>
> From all the mentioned tests only glsl-fs-discard-02 pass when I disable glsl-to-tgsi.
>
> BTW in which commit was the new ioctl disabled?
In this one: ef64da8f013691c66744064769db379e57ef95de
> I mean, I can understand that if I don't have kernel with support for the new ioctl it breaks the tests,
> however if the ioctl was disabled later shouldn't be current git working unless another regression happened?
Yes, it should.
--
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] 8+ messages in thread* [Bug 45921] [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl
2012-02-11 10:08 [Bug 45921] New: [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl bugzilla-daemon
` (2 preceding siblings ...)
2012-02-11 12:19 ` bugzilla-daemon
@ 2012-02-11 12:51 ` bugzilla-daemon
2012-02-11 13:59 ` bugzilla-daemon
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2012-02-11 12:51 UTC (permalink / raw)
To: dri-devel
https://bugs.freedesktop.org/show_bug.cgi?id=45921
--- Comment #4 from Pavel Ondračka <pavel.ondracka@email.cz> 2012-02-11 04:51:23 PST ---
OK, I can finally see whats going on. ef64da8f013691c66744064769db379e57ef95de fails however 1e3c81a068c4ae04cd1c6b18c687d5be69b7b8c4 + manually applied
ef64da8f013691c66744064769db379e57ef95de pass. So indeed a second regression happened :-( Running second bisect right now.
--
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] 8+ messages in thread* [Bug 45921] [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl
2012-02-11 10:08 [Bug 45921] New: [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl bugzilla-daemon
` (3 preceding siblings ...)
2012-02-11 12:51 ` bugzilla-daemon
@ 2012-02-11 13:59 ` bugzilla-daemon
2012-02-11 14:20 ` [Bug 45921] [r300g, bisected] Multiple piglit regressions after glsl_to_tgsi changes bugzilla-daemon
2012-04-14 15:16 ` bugzilla-daemon
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2012-02-11 13:59 UTC (permalink / raw)
To: dri-devel
https://bugs.freedesktop.org/show_bug.cgi?id=45921
Pavel Ondračka <pavel.ondracka@email.cz> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |bryancain3+fdo@gmail.com
--- Comment #5 from Pavel Ondračka <pavel.ondracka@email.cz> 2012-02-11 05:59:42 PST ---
Marek you were right with the glsl_to_tgsi cause. It indeed seems majority of the commits fails due to some glsl_to_tgsi changes.
fs-any-bvec2-using-if
fs-op-ne-mat2-mat2-using-if
fs-op-ne-mat2x3-mat2x3-using-if
fs-op-ne-mat2x4-mat2x4-using-if
regressed with:
a43f68810a347f3e952a0bc401be6edb91e1baea is the first bad commit
commit a43f68810a347f3e952a0bc401be6edb91e1baea
Author: Bryan Cain <bryancain3@gmail.com>
Date: Sat Aug 20 13:26:12 2011 -0500
glsl_to_tgsi: implement ir_unop_any using DP4 w/saturate or DP4 w/SLT
This is a port of commit 92ca560d68e8 to glsl_to_tgsi, with integer support
added.
glsl1-inequality (vec2, pass)
fs-op-ne-bvec2-bvec2-using-if
fs-op-ne-ivec2-ivec2-using-if
fs-op-ne-vec2-vec2-using-if
regressed with:
f3dce133f0422c42ca61f07f488237107efc30e6 is the first bad commit
commit f3dce133f0422c42ca61f07f488237107efc30e6
Author: Bryan Cain <bryancain3@gmail.com>
Date: Sat Aug 20 13:56:06 2011 -0500
glsl_to_tgsi: implement ir_binop_any_nequal using DP4 w/saturate or DP4 w/SLT
Implement the any() part of the operation the same way regular ir_unop_any
is implemented.
This is a port of commit e7bf096e8b04 to glsl_to_tgsi, with added integer
support.
The rest of the tests (glsl-fs-discard-02, glsl-max-varyings and glsl-vs-loop-redundant-condition) regressed in another (probably 3 different) places. I'll get
to them later. BTW this may be the worst bisect I've ever done.
--
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] 8+ messages in thread* [Bug 45921] [r300g, bisected] Multiple piglit regressions after glsl_to_tgsi changes
2012-02-11 10:08 [Bug 45921] New: [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl bugzilla-daemon
` (4 preceding siblings ...)
2012-02-11 13:59 ` bugzilla-daemon
@ 2012-02-11 14:20 ` bugzilla-daemon
2012-04-14 15:16 ` bugzilla-daemon
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2012-02-11 14:20 UTC (permalink / raw)
To: dri-devel
https://bugs.freedesktop.org/show_bug.cgi?id=45921
Pavel Ondračka <pavel.ondracka@email.cz> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|[r300g, bisected] Multiple |[r300g, bisected] Multiple
|piglit regression after |piglit regressions after
|winsys/radeon: hook up the |glsl_to_tgsi changes
|new DRM_RADEON_GEM_WAIT |
|ioctl |
--
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] 8+ messages in thread* [Bug 45921] [r300g, bisected] Multiple piglit regressions after glsl_to_tgsi changes
2012-02-11 10:08 [Bug 45921] New: [r300g, bisected] Multiple piglit regression after winsys/radeon: hook up the new DRM_RADEON_GEM_WAIT ioctl bugzilla-daemon
` (5 preceding siblings ...)
2012-02-11 14:20 ` [Bug 45921] [r300g, bisected] Multiple piglit regressions after glsl_to_tgsi changes bugzilla-daemon
@ 2012-04-14 15:16 ` bugzilla-daemon
6 siblings, 0 replies; 8+ messages in thread
From: bugzilla-daemon @ 2012-04-14 15:16 UTC (permalink / raw)
To: dri-devel
https://bugs.freedesktop.org/show_bug.cgi?id=45921
Tom Stellard <tstellar@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #6 from Tom Stellard <tstellar@gmail.com> 2012-04-14 08:16:15 PDT ---
(In reply to comment #0)
>
> Regressed tests:
> glsl1-inequality (vec2, pass)
Fixed by commit 73249239cf71e3595ee19f3c1a02b8b0f58994cd
> glsl-fs-discard-02
Fixed by commit b2df031a959f36743527b9abc89913ce4f895de3
> glsl-max-varyings
Duplicate: https://bugs.freedesktop.org/show_bug.cgi?id=46006
> glsl-vs-loop-redundant-condition
Fixed by commit b2df031a959f36743527b9abc89913ce4f895de3
> fs-any-bvec2-using-if
> fs-op-ne-bvec2-bvec2-using-if
> fs-op-ne-ivec2-ivec2-using-if
> fs-op-ne-mat2-mat2-using-if
> fs-op-ne-vec2-vec2-using-if
> fs-op-ne-mat2x3-mat2x3-using-if
> fs-op-ne-mat2x4-mat2x4-using-if
Fixed by commit 73249239cf71e3595ee19f3c1a02b8b0f58994cd
--
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] 8+ messages in thread