Jerome Glisse wrote: > On 2/26/07, Roland Scheidegger <[EMAIL PROTECTED]> wrote: >> Christoph Brill wrote: >>> Attached is a mini-patch to add the address of early Z to r300_reg.h >>> and use it. Jerome Glisse helped me with this patch. Thanks. :-) >> Not really related directly to the patch itself, but it seems to me that >> the conditions when to enable early-z are a bit wrong. First, I'd think >> you need to disable early-z when writing to the depth output (or use >> texkill) in fragment programs. Second, why disable early-z specifically >> if the depth function is gl_never? Is that some workaround because >> stencil updates don't work otherwise (in that case should certainly >> rather check for that) or something similar? >> >> Roland > > Yes we need to disable early z when the fragment program write to > z buffer, so far there isn't real support for depth writing in the driver > (it's in the todo list). I fail to see why we should disable early z when > there is texkill instruction (and stencil disabled), if we disable early > z then if the fragment doesn't pass the test in the early pass it > won't after texkill so will be killed anyway (and better kill early then > at the end).
If you don't disable early z, you can end up writing values to the depth buffer for fragments that are later killed. Keith ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel