<div class="gmail_quote">On Fri, Jul 23, 2010 at 7:01 PM, Patrice Mandin <<a href="mailto:mandin.patrice@orange.fr" target="_blank">mandin.patrice@orange.fr</a>> wrote: <blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> Le Fri, 18 Jun 2010 18:43:27 +0200 <div><div></div><div>Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>> a écrit: > On Fri, Jun 18, 2010 at 6:05 PM, Patrice Mandin <<a href="mailto:mandin.patrice@orange.fr" target="_blank">mandin.patrice@orange.fr</a>>wrote: > > > Le Thu, 17 Jun 2010 03:35:19 +0200 > > Marek Olšák <<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>> a écrit: > > > > > On Fri, Jun 11, 2010 at 3:37 PM, Xavier Chantry < > > <a href="mailto:chantry.xavier@gmail.com" target="_blank">chantry.xavier@gmail.com</a>>wrote: > > > > > > > Hi Marek > > > > > > > > Thanks a lot for your rebasing work. > > > > Here is my report : > > > > > > > > - all my games that broke with temporaries patch (they were either > > > > completely black or lot of black screen flash every frame) behave > > > > badly, but in different ways : > > > > * etracer is very slow and often crash in ttm code [1] (I think this > > > > is an old bug that just resurrected, no idea why) > > > > * foobillard is very slow and still flash a bit > > > > * strangely, neverball seems to work, I get similar results than with > > > > old nvfx-next-6b branch with temporaries reverted. no black flash > > > > while playing. > > > > * glest segfault [2] > > > > > > > > I also compared with piglit the old nvfx branch with the new merged one > > : > > > > 114/174 vs 113/174 > > > > That looks quite good with 3 new pass, but 4 new fail : > > > > * fbo-copypix > > > > Returncode was -6 > > > > * glean pbo > > > > > > ../../../../src/gallium/auxiliary/util/u_inlines.h:77:pipe_reference: > > > > Assertion `pipe_is_referenced(reference)\\\' failed. > > > > * texCombine4: FAIL rgba8, db, z24, s8, win+pmap, id 33 > > > > * fp-long-alu > > > > > > > > > > Hi Xavier, > > > > > > Sorry for the late reply. > > > > > > The assertion in pipe_reference can be fixed quite easily I think. There > > is > > > pipe_*_reference missing somewhere. > > > > > > Concerning fp-long-alu, there are new CAPs for shader limits which should > > be > > > filled out but I don't know what values to put there. If the two get > > fixed, > > > it will hopefully be just 2 new failures along with 3 that pass. > > > > > > > > > > There is just fp-long-alu that is for sure a regression caused by new > > > > master code (some gallium changes). I don't know about the 3 others. > > > > > > > > It might be worth to re-test everything on your new branch with this > > > > patch reverted : > > > > nvfx: rewrite render temporaries code, also affecting 2D and resource > > code > > > > > > > > > > Here is the tree with the commit reverted: > > > </div></div>> > > git://<a href="http://anongit.freedesktop.org/%7Emareko/mesa" target="_blank">anongit.freedesktop.org/~mareko/mesa</a><<a href="http://anongit.freedesktop.org/%7Emareko/mesa" target="_blank">http://anongit.freedesktop.org/%7Emareko/mesa</a>>nvfx-next-6b-notemps <div>> > > > > > It is compile-tested so if it does not work, there is nothing I can do > > about > > > it. > > > > > > -Marek > > > > I just tested the new tree nvfx-next-6b-notemps, I think we should go > > some commits before, because Luca removed the check in fbo setting > > between difference in bits between colour and depth/stencil buffer, and > > nv30 hw does not support that (they must be equal) and doing that > > simply hang the gpu. > > > > Commit 4787059263755fb92b2bb09ac71658d9b4cc9368 'nvfx: new 2D: add > > support for render temporaries' removed this check and fbo tests > > trigger the bug. Maybe Luca was planning to use temporaries to avoid > > this check, but unfortunately we do not know if he finished it or not. > > I do not want to guess here. I would like to hear something from Luca before > doing anything else with his code. </div>It has been 3 months since latest Luca's sign, and I'm afraid it will be a while before it changes :-(. Marek, if you are still interested in merging stuff from nvfx-next-6b, I would like to propose up to commit 9a4c66b0d1963c4d90fccac41e7aa48105835857 included (as I said before, the following ones bring back nasty regression regarding fbo stuff, hanging the gpu hard). </blockquote><div> 9a4c66b0d1963c4d90fccac41e7aa48105835857 doesn't compile, so I went one commit back. It's here for testing: git://<a href="http://anongit.freedesktop.org/~mareko/mesa">anongit.freedesktop.org/~mareko/mesa</a> nvfx-new Please let me know if it works. -Marek </div></div>