On Mon, Jun 10, 2019 at 10:00:01PM -0400, Jonathan Marek wrote: > On 6/10/19 10:00 PM, Brian Masney wrote: > > On Mon, Jun 10, 2019 at 09:53:25PM -0400, Jonathan Marek wrote: > > > > > > This error doesn't happen on X11 using the mesa master branch. > > > > > > Instead, > > > > > > I get the following error on that branch: > > > > > > > > > > > > ../src/gallium/drivers/freedreno/freedreno_batch.c:424:fd_batch_add_dep: > > > > > > Assertion `!batch_depends_on(dep, batch)' failed. > > > > > > > > > > > > Full disclosure though: I rebuilt the mesa package using the > > > > > > postmarketOS packaging yesterday and it includes a few extra > > > > > > patches for > > > > > > musl libc. > > > > > > > > > > > > https://gitlab.com/postmarketOS/pmaports/tree/master/temp/mesa > > > > > > > > > > > > > > > I don't see anything obvious in those patches that would be related.. > > > > > but I suspect this type of error is going to be timing related. > > > > > (Which could ofc be due to musl or something else) > > > > > > > > > > but a bit surprised debug_assert() is enabled in debug builds.. it > > > > > would probably be a "harmless" situation if asserts were not enabled. > > > > > > > > > > (note that I do most of my testing with debug builds with asserts > > > > > enabled.. this is the type of thing that I want to see and fix.. but > > > > > probably shouldn't matter to end users) > > > > > > > > I recompiled the master branch of mesa in pmOS with '-Db_ndebug=true' > > > > and X11 is now working properly on the Nexus 5. glxgears averages about > > > > 59.5 FPS. I'll add a bug report with pmOS to have them add that flag to > > > > their mesa build. Fedora added that flag to their builds: > > > > https://bugzilla.redhat.com/show_bug.cgi?id=1692426 > > > > > > > > 19.1.0-rc5 still doesn't work for me due to the original error. > > > > > > > > Brian > > > > > > > > > > You probably want '--buildtype=release' instead of '-Db_ndebug=true' > > > > According to: > > https://gitlab.freedesktop.org/mesa/mesa/blob/master/docs/meson.html#L321 > > > > -Db_ndebug - This option controls assertions in meson projects. When > > set to false (the default) assertions are enabled, when set to true > > they are disabled. This is unrelated to the buildtype; setting the > > latter to release will not turn off assertions. > > > > Brian > > > > I always thought release == no assertions, I guess meson has different > ideas. You will want '--buildtype=release' anyway for optimizations, etc.
That's also missing from the pmOS build, so I added it to the bug report as well: https://gitlab.com/postmarketOS/pmaports/issues/296 Thanks, Brian _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
