[Nouveau] nvfx

[Marek Olšák]

On Fri, Jul 23, 2010 at 7:01 PM, Patrice Mandin <mandin.patrice at orange.fr>wrote:

> Le Fri, 18 Jun 2010 18:43:27 +0200
> Marek Olšák <maraeo at gmail.com> a écrit:
>
> > On Fri, Jun 18, 2010 at 6:05 PM, Patrice Mandin <
> mandin.patrice at orange.fr>wrote:
> >
> > > Le Thu, 17 Jun 2010 03:35:19 +0200
> > > Marek Olšák <maraeo at gmail.com> a écrit:
> > >
> > > > On Fri, Jun 11, 2010 at 3:37 PM, Xavier Chantry <
> > > chantry.xavier at gmail.com>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:
> > > >
> > > > git://anongit.freedesktop.org/~mareko/mesa<http://anongit.freedesktop.org/%7Emareko/mesa>
> <http://anongit.freedesktop.org/%7Emareko/mesa>nvfx-next-6b-notemps
> > > >
> > > > 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.
>
> 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).
>

9a4c66b0d1963c4d90fccac41e7aa48105835857 doesn't compile, so I went one
commit back.

It's here for testing:
git://anongit.freedesktop.org/~mareko/mesa nvfx-new

Please let me know if it works.

-Marek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/nouveau/attachments/20100724/fc15158a/attachment.htm>