I think the problem is confusion between MESA_FORMAT_Z24_S8 and
MESA_FORMAT_S8_Z24 in the radeon drivers.

Looking at the span code, it appears that R300, R200 use Z24_S8 format
while R600 and others use S8_Z24.

Here's a patch that attempts to fix things.  I don't have radeon
hardware to test so maybe someone else can start with this and finish
it up.

-Brian



2009/11/11 Maciej Cencora <[email protected]>:
> I've checked the other failing tests.
> Following were also passing before the texformat-rework merge:
> fdo23670-drawpix_stencil
> stencil-drawpixels
> fragProg1 (Z write test)
> readPixSanity
> stencil2
>
> Maciej
>
> Dnia środa, 11 listopada 2009 o 03:44:18 Brian Paul napisał(a):
>> It passes with swrast and the i965 driver here.
>>
>> Did this test pass prior to the texformat work?
>>
>> -Brian
>>
>> 2009/11/10 Maciej Cencora <[email protected]>:
>> > It doesn't assert anymore, but the test is still failing.
>> >
>> > @test: texturing/texdepth
>> > info: @@@Returncode: 1\n\nErrors:\nMesa: Mesa 7.7-devel DEBUG build Nov
>> >  8 2009 21:21:48\nMesa warning: couldn\'t open libtxc_dxtn.so, software
>> > DXTn compression/decompression unavailable\nMesa: Initializing x86-64
>> > optimizations\nMesa: 3Dnow! detected\n\n\nOutput:\nProbe at (80,16)\n
>> > Expected: 0.250000 0.250000 0.250000 1.000000\n  Observed: 0.000000
>> > 0.000000 0.000000 1.000000\nTest failed: \'Render textures GL_LUMINANCE
>> > (no shadow functionality)\'\nSee above for details.\n\n
>> > errors!
>> >
>> > Maciej
>> >
>> > Dnia środa, 11 listopada 2009 o 02:25:52 Brian Paul napisał(a):
>> >> Can you try again with Mesa git/master?  I've updated the assertions.
>> >>
>> >> -Brian
>> >>
>> >> On Tue, Nov 10, 2009 at 3:43 PM, Maciej Cencora <[email protected]>
> wrote:
>> >> > It is r300 driver (RV530 GPU).
>> >> >
>> >> > I'm sending the backtrace as attachement.
>> >> >
>> >> > Maciej
>> >> >
>> >> > Dnia wtorek, 10 listopada 2009 o 23:38:09 Brian Paul napisał(a):
>> >> >> Which driver?
>> >> >>
>> >> >> Could you provide a stack trace for the failed assertion?
>> >> >>
>> >> >> -Brian
>> >> >>
>> >> >> On Tue, Nov 10, 2009 at 3:30 PM, Maciej Cencora <[email protected]>
>> >
>> > wrote:
>> >> >> > Hi Brian,
>> >> >> >
>> >> >> > there's at least one more regression in the code.
>> >> >> > Piglit's texdepth test is failing because of following assertion:
>> >> >> > texdepth: swrast/s_readpix.c:122: read_depth_pixels: Assertion `rb-
>> >> >> >
>> >> >> >>InternalFormat == 0x81A6' failed.
>> >> >> >
>> >> >> > There are also other regressions (mostly related to depth/stencil
>> >> >> > buffer) but I'm not sure it's the texformat branch merge is to
>> >> >> > blame: fdo23670-drawpix_stencil
>> >> >> > stencil-drawpixels
>> >> >> > fragProg1 (Z write test)
>> >> >> > paths
>> >> >> > polygonOffset
>> >> >> > readPixSanity
>> >> >> > stencil2
>> >> >> >
>> >> >> > Regards,
>> >> >> > Maciej
>> >> >> >
>> >> >> > Dnia piątek, 23 października 2009 o 23:23:34 Brian Paul napisał(a):
>> >> >> >> Alex, Nicolai,
>> >> >> >>
>> >> >> >> Would you guys please test the texformat-rework branch again?
>> >> >> >>
>> >> >> >> If it looks OK, I'd like to merge to master soon, but probably not
>> >> >> >> until next week.
>> >> >> >>
>> >> >> >> Thanks.
>> >> >> >>
>> >> >> >> -Brian
>> >> >> >>
>> >> >> >>
>> >> >> >> ------------------------------------------------------------------
>> >> >> >>--- --- --- --- Come build with us! The BlackBerry(R) Developer
>> >> >> >> Conference in SF, CA is the only developer event you need to
>> >> >> >> attend this year. Jumpstart your developing skills, take
>> >> >> >> BlackBerry mobile applications to market and stay ahead of the
>> >> >> >> curve. Join us from November 9 - 12, 2009. Register now!
>> >> >> >> http://p.sf.net/sfu/devconference
>> >> >> >> _______________________________________________
>> >> >> >> Mesa3d-dev mailing list
>> >> >> >> [email protected]
>> >> >> >> https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
>> >> >> >
>> >> >> > -------------------------------------------------------------------
>> >> >> >--- --- ----- Let Crystal Reports handle the reporting - Free
>> >> >> > Crystal Reports 2008 30-Day trial. Simplify your report design,
>> >> >> > integration and deployment - and focus on what you do best, core
>> >> >> > application coding. Discover what's new with Crystal Reports now.
>> >> >> >  http://p.sf.net/sfu/bobj-july
>> >> >> > _______________________________________________
>> >> >> > Mesa3d-dev mailing list
>> >> >> > [email protected]
>> >> >> > https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
>>
>

Attachment: radeon_z_stencil.patch
Description: Binary data

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Mesa3d-dev mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to