Maciej Cencora wrote: > I'm sending updated patches. Updates: > - trailing whitespaces are cleaned up, > - multiply cmdpackets per state atom are correctly handled, > - W pos and fogcoord attributes are pushed in correct order in hw tcl path > (testing needed), > - added Nicolai's regression fix (combined with fix for non tcl hw), > - added few comments in commit log, > - out of texcoord case is handled now in hw tcl path. > > Nicolai Haehnle wrote: > >> By the way, have you considered stuffing wpos and fogcoord in the same >> register if both are needed? I'm just curious - if this is impossible for >> some reason or requires significantly more changes to the code, just forget >> about it. It's an optimization anyway, and it's more important to get >> things to work correctly in the first place. >> > > I don't know if it's worth the effort. I'll certainly look into it when > someone finds an app which uses all free texcoords. > > Markus Amsler wrote: > >> I tested it on r300/r500 with WoW: >> after 0004 i get gpu lockups but 0005 fixes the lockups. After 0006 part >> of the ingame screen is black. That's a regression from >> newest_fog_w5.patch, with w5 the game looks fine. >> > > patches 7 and 8 should fix the regression > > Thank you both for testing. > > Maciej > retested on r500: patch 7 fixes the regression introduced in patch 6. With patch 8 demos/arbfplight looks wrong, and I also get lockups in Wow.
Markus ------------------------------------------------------------------------------ Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD http://p.sf.net/sfu/XcvMzF8H _______________________________________________ Mesa3d-dev mailing list Mesa3d-dev@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
