https://bugs.freedesktop.org/show_bug.cgi?id=86690
Bug ID: 86690
Summary: glmark2-es2-wayland shortly freezes on some frames
with egl_dri2 backend (Nouveau/GK20A)
Product: Mesa
Version: git
Hardware: ARM
Hi Pekka,
On 11/20/2014 08:41 PM, Pekka Paalanen wrote:
On Thu, 20 Nov 2014 18:24:34 +0900
Alexandre Courbot acour...@nvidia.com wrote:
Hi Pekka,
On 11/19/2014 04:34 PM, Pekka Paalanen wrote:
On Wed, 19 Nov 2014 15:32:38 +0900
Alexandre Courbot acour...@nvidia.com wrote:
Some more
https://bugs.freedesktop.org/show_bug.cgi?id=86690
--- Comment #1 from Alexandre Courbot gnu...@gmail.com ---
Created attachment 109990
-- https://bugs.freedesktop.org/attachment.cgi?id=109990action=edit
Trace when running glmark2 with WAYLAND_DEBUG=client
Attached the trace suggested by
https://bugs.freedesktop.org/show_bug.cgi?id=78914
--- Comment #21 from Florian Link florianl...@gmail.com ---
It is ok for me if you close this bug as WONTFIX, since I worked around the
depth fighting and I agree that it is a hard problem to do this with high
precision and speed at the same
On Tue, 25 Nov 2014 17:04:36 +0900
Alexandre Courbot acour...@nvidia.com wrote:
Hi Pekka,
On 11/20/2014 08:41 PM, Pekka Paalanen wrote:
At this point, I think it would be best to open a bug report against
Mesa, and continue there. Such freezes obviously should not happen on
either EGL
https://bugs.freedesktop.org/show_bug.cgi?id=71199
--- Comment #8 from José Fonseca jfons...@vmware.com ---
Yes, I suspect that ideal MRD at near plane may never go below 2, because of
the the near_1_far_infinity perspective transformation applied.
We could experiment whether an orthogonal
Hi,
I sent exactly the same patch back in April. Despite getting a review-by
it was never merged. See:
http://lists.freedesktop.org/archives/mesa-dev/2014-April/058347.html
http://lists.freedesktop.org/archives/mesa-dev/2014-May/060432.html
Thanks
Frank
On 25/11/14 03:00, Mario Kleiner wrote:
On Monday, November 24, 2014 10:19:50 PM Jason Ekstrand wrote:
Previously, the hash_table API required the user to do all of the hashing
of keys as it passed them in. Since the hashing function is intrinsicly
tied to the comparison function, it makes sense for the hash table to know
about it.
On 21.11.2014 23:57, Samuel Iglesias Gonsálvez wrote:
On Wed, 2014-11-19 at 17:09 +0900, Michel Dänzer wrote:
On 18.11.2014 17:43, Iago Toral Quiroga wrote:
For software drivers we worked with a trimmed set of piglit tests (related to
format conversion), ~5700 tests selected with the
Fixes broken rendering in Windows-based QtQuick2 apps run through Wine.
This library sets all texture units' GL_COORD_REPLACE, leaves point
sprite mode enabled, and then draws a triangle fan.
Will need a slightly different fix for Gen4-5, but I don't have my old
machines in a usable state
On Tuesday, November 25, 2014 10:05:18 PM Chris Forbes wrote:
Fixes broken rendering in Windows-based QtQuick2 apps run through Wine.
This library sets all texture units' GL_COORD_REPLACE, leaves point
sprite mode enabled, and then draws a triangle fan.
Will need a slightly different fix for
On 25/11/14 09:59, Michel Dänzer wrote:
On 21.11.2014 23:57, Samuel Iglesias Gonsálvez wrote:
On Wed, 2014-11-19 at 17:09 +0900, Michel Dänzer wrote:
On 18.11.2014 17:43, Iago Toral Quiroga wrote:
For software drivers we worked with a trimmed set of piglit tests
(related to
format
Changes done and pushed.
On Tue, Nov 25, 2014 at 10:17 PM, Kenneth Graunke kenn...@whitecape.org wrote:
On Tuesday, November 25, 2014 10:05:18 PM Chris Forbes wrote:
Fixes broken rendering in Windows-based QtQuick2 apps run through Wine.
This library sets all texture units' GL_COORD_REPLACE,
On 25/11/14 09:43, Samuel Iglesias Gonsálvez wrote:
On 25/11/14 09:59, Michel Dänzer wrote:
On 21.11.2014 23:57, Samuel Iglesias Gonsálvez wrote:
On Wed, 2014-11-19 at 17:09 +0900, Michel Dänzer wrote:
On 18.11.2014 17:43, Iago Toral Quiroga wrote:
For software drivers we worked with a
Patch enables ES2 extension that utilizes existing ES3 functionality.
Changes make all the subtests to run and pass in WebGL conformance
test 'webgl-draw-buffers' when running Chrome on OpenGL ES.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/glsl/glcpp/glcpp-parse.y| 2 ++
Calling pipe-flush is unnecessary before get_query_result, because
get_query_result flushes automatically if it has to (at least on
radeon).
Marek
On Sun, Nov 23, 2014 at 11:40 PM, David Heidelberg da...@ixit.cz wrote:
From: Axel Davy axel.d...@ens.fr
From this moment we should handle errors
BTW, radeon flushes in get_query_result even if wait is FALSE,
therefore this loop will always finish:
while (!get_query_result(...));
I think it's a requirement for OpenGL.
Marek
On Tue, Nov 25, 2014 at 12:42 PM, Marek Olšák mar...@gmail.com wrote:
Calling pipe-flush is unnecessary before
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/glsl/linker.cpp | 19 ++-
1 file changed, 18 insertions(+), 1 deletion(-)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index de6b1fb..a3a43a0 100644
--- a/src/glsl/linker.cpp
+++ b/src/glsl/linker.cpp
@@ -732,8
https://bugs.freedesktop.org/show_bug.cgi?id=86690
--- Comment #2 from Pekka Paalanen ppaala...@gmail.com ---
(In reply to Alexandre Courbot from comment #1)
Created attachment 109990 [details]
Trace when running glmark2 with WAYLAND_DEBUG=client
Attached the trace suggested by Pekka. The
These always happen together; the extra atom just means another item to
iterate through, flags to check, and a call through a function pointer.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_cc.c | 9 -
On Gen4-5, unit state is specified as indirect state, rather than
commands. If any unit state changes, we upload it via brw_state_batch
and arrange for 3DSTATE_PIPELINED_POINTERS to be re-emitted, which
updates pointers to all unit state at once.
Since there's only one command and state atom
I put the BRW_NEW_*_PROG_DATA flags at the beginning so that
brw_state_cache.c can still continue using 1 brw_cache_id.
I also added a comment explaining the difference between
BRW_NEW_*_PROG_DATA and BRW_NEW_*_PROGRAM, as it took me a long time
to remember it.
Non-mechanical changes:
-
Marking brw_stage_state::sampler_count as CACHE_NEW_SAMPLER is wrong.
The number of samplers used by each program is actually computed at
draw time (brw_try_draw_prims), based purely on the currently bound
shader programs (gl_program::SamplersUsed).
CACHE_NEW_SAMPLER means that we've emitted a
This flag signifies that we've emitted a new SAMPLER_STATE table.
Given that we haven't cached those in years, CACHE_NEW_SAMPLER isn't
a great name. Putting it in the BRW_NEW_* hierarchy would make more
sense; BRW_NEW_SAMPLER_STATE_TABLE better reflects its actual purpose.
When this flag is
Now that we've moved a bunch of CACHE_NEW_* bits to BRW_NEW_*, the only
ones that are left are legitimately related to the program cache. Yet,
it seems a bit wasteful to have an entire bitfield for only 7 bits.
State upload is one of the hottest paths in the driver. For each atom
in the list,
We've been streaming these out for ages, so they basically have nothing
to do with brw_state_cache.c.
Saves 6 * sizeof(void *) bytes per context, as we won't have useless
aux_compare/aux_free functions for them.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
Most of the dirty flags were listed in some arbitrary order. Some used
bonus parenthesis. Some put multiple flags on one line, others put one
per line. Some used tabs instead of spaces...but only on some lines.
This patch settles on one flag per line, in alphabetical order, using
spaces
Hello,
This series does some longstanding cleaning I've been meaning to do
in the i965 state upload code. The distinction between BRW_NEW_* and
CACHE_NEW_* flags has been pretty arbitrary for a while - 10/17 of
them were for things we stopped caching years ago. So, I moved
those to be BRW_NEW_*
BRW_CACHE_VS_PROG is more easily associated with program caches than
plain BRW_VS_PROG.
While we're at it, rename BRW_WM_PROG to BRW_CACHE_FS_PROG, to move away
from the outdated Windowizer/Masker name.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
It's been merged into brw_state_flags::brw for simplicity and
efficiency.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_binding_tables.c | 2 --
src/mesa/drivers/dri/i965/brw_blorp.cpp| 1 -
src/mesa/drivers/dri/i965/brw_cc.c
https://bugs.freedesktop.org/show_bug.cgi?id=86701
Bug ID: 86701
Summary: [regression] weston-simple-egl not running anymore
inside qemu
Product: Mesa
Version: git
Hardware: Other
OS: All
https://bugs.freedesktop.org/show_bug.cgi?id=86701
Pekka Paalanen ppaala...@gmail.com changed:
What|Removed |Added
CC||ppaala...@gmail.com
https://bugs.freedesktop.org/show_bug.cgi?id=86701
--- Comment #1 from Pekka Paalanen ppaala...@gmail.com ---
egl_gallium.so was able to use llvmpipe and direct that rendering to wl_shm
based buffers, and therefore does not need any EGL support from the compositor.
egl_dri2 does not implement
On Tue, Nov 25, 2014 at 6:23 AM, Tapani Pälli tapani.pa...@intel.com wrote:
Patch enables ES2 extension that utilizes existing ES3 functionality.
Changes make all the subtests to run and pass in WebGL conformance
test 'webgl-draw-buffers' when running Chrome on OpenGL ES.
Signed-off-by:
https://bugs.freedesktop.org/show_bug.cgi?id=86701
Javier Jardón jjar...@gnome.org changed:
What|Removed |Added
CC||e...@anholt.net,
On 25/11/14 00:39, Matt Turner wrote:
I've got some thread-safety fixes queued up after this and thought I'd
be a good Mesa citizen and pull some code into src/util.
Thanks for going the extra mile!
I did some clean ups like replacing INLINE (MSVC knows about inline
these days, right?) and
On Tue, 4 Nov 2014 23:42:44 +0100
Marek Olšák mar...@gmail.com wrote:
From: Marek Olšák marek.ol...@amd.com
Btw. would have been *really* nice if the commit message here explained
why you do this and what all things it intentionally breaks, before
pushing it.
Thanks,
pq
---
configure.ac
Hi Pekka,
I explained in the introductory thread [PATCH 0/2] Disable the EGL
state tracker for Linux/DRI builds that it may break swrast
(llvmpipe) on Wayland. I also explained all the reasons why I was
about to do it. Does it break something not discussed before?
Marek
On Tue, Nov 25, 2014 at
https://bugs.freedesktop.org/show_bug.cgi?id=78914
José Fonseca jfons...@vmware.com changed:
What|Removed |Added
Status|NEW |RESOLVED
On Tue, Nov 25, 2014 at 4:43 AM, Kenneth Graunke kenn...@whitecape.org wrote:
Hello,
This series does some longstanding cleaning I've been meaning to do
in the i965 state upload code. The distinction between BRW_NEW_* and
CACHE_NEW_* flags has been pretty arbitrary for a while - 10/17 of
When restoring the current state in _mesa_meta_end it was previously trying to
copy the on-going sample count of the current occlusion query into the new
query after restarting it so that the driver will continue adding to the
previous value. This wouldn't work for two reasons. Firstly, the query
Series Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
On 2014-11-24 22:03:10, Kenneth Graunke wrote:
brw-wm.prog_data is covered by CACHE_NEW_WM_PROG, not
BRW_NEW_FRAGMENT_PROGRAM. So, we should listen to it.
However, I believe that BRW_NEW_FRAGMENT_PROGRAM is sufficient to cover
all
Whoops I edited the wrong patch before sending this out it was meant to
be RFC and have the following comments attached:
I came across this when I was writting arrays of arrays piglit tests.
This change fixes the new link error single dimension array
tests I sent to the piglit list [1].
I've
On Thu, Nov 13, 2014 at 6:38 PM, Connor Abbott cwabbo...@gmail.com wrote:
On Thu, Nov 13, 2014 at 7:28 PM, Kristian Høgsberg k...@bitplanet.net wrote:
This will be reused for the scalar VS pass.
Signed-off-by: Kristian Høgsberg k...@bitplanet.net
---
src/mesa/drivers/dri/i965/brw_fs.cpp |
On Thu, Nov 13, 2014 at 9:03 PM, Matt Turner matts...@gmail.com wrote:
On Thu, Nov 13, 2014 at 4:28 PM, Kristian Høgsberg k...@bitplanet.net wrote:
@@ -3148,6 +3150,9 @@ fs_visitor::dump_instruction(backend_instruction
*be_inst, FILE *file)
case UNIFORM:
fprintf(file, ***u%d***,
Patches 3-6 are
Reviewed-by: Matt Turner matts...@gmail.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Tue, Nov 25, 2014 at 6:48 AM, Jose Fonseca jfons...@vmware.com wrote:
On 25/11/14 00:39, Matt Turner wrote:
I've got some thread-safety fixes queued up after this and thought I'd
be a good Mesa citizen and pull some code into src/util.
Thanks for going the extra mile!
I did some clean
On 25/11/14 21:04, Matt Turner wrote:
On Tue, Nov 25, 2014 at 6:48 AM, Jose Fonseca jfons...@vmware.com wrote:
On 25/11/14 00:39, Matt Turner wrote:
I've got some thread-safety fixes queued up after this and thought I'd
be a good Mesa citizen and pull some code into src/util.
Thanks for
On Tue, Nov 25, 2014 at 1:42 PM, Jose Fonseca jfons...@vmware.com wrote:
On 25/11/14 21:04, Matt Turner wrote:
On Tue, Nov 25, 2014 at 6:48 AM, Jose Fonseca jfons...@vmware.com wrote:
On 25/11/14 00:39, Matt Turner wrote:
I've got some thread-safety fixes queued up after this and thought
On Tue, Nov 25, 2014 at 4:43 AM, Kenneth Graunke kenn...@whitecape.org wrote:
BRW_CACHE_VS_PROG is more easily associated with program caches than
plain BRW_VS_PROG.
While we're at it, rename BRW_WM_PROG to BRW_CACHE_FS_PROG, to move away
from the outdated Windowizer/Masker name.
On Tue, Nov 25, 2014 at 4:43 AM, Kenneth Graunke kenn...@whitecape.org wrote:
4:43 AM
If I didn't see you a couple of times a week I'd wonder what timezone
you were in!
Most of the dirty flags were listed in some arbitrary order. Some used
bonus parenthesis. Some put multiple flags on one
On Thu, Nov 13, 2014 at 8:49 PM, Matt Turner matts...@gmail.com wrote:
On Thu, Nov 13, 2014 at 4:28 PM, Kristian Høgsberg k...@bitplanet.net wrote:
The LOAD_PAYLOAD opcode can't saturate its sources, so skip
saturating MOVs. The register coalescing after lower_load_payload()
will clean up the
Timothy Arceri t_arc...@yahoo.com.au writes:
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
Reviewed-by: Eric Anholt e...@anholt.net
and I've pushed the equivalent change with your name to the upstream
repo.
signature.asc
Description: PGP signature
Jason Ekstrand ja...@jlekstrand.net writes:
Previously, the hash_table API required the user to do all of the hashing
of keys as it passed them in. Since the hashing function is intrinsicly
tied to the comparison function, it makes sense for the hash table to know
about it. Also, it makes
On Fri, Nov 14, 2014 at 4:08 PM, Kenneth Graunke kenn...@whitecape.org wrote:
On Thursday, November 13, 2014 04:28:20 PM Kristian Høgsberg wrote:
This patch uses the previous refactoring to add a new run_vs() method
that generates vertex shader code using the scalar visitor and
optimizer.
On Tue, Nov 25, 2014 at 3:40 PM, Eric Anholt e...@anholt.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
Previously, the hash_table API required the user to do all of the hashing
of keys as it passed them in. Since the hashing function is intrinsicly
tied to the comparison
like how C11's stdatomic.h provides generic functions. GCC's __sync_*
builtins already take a variety of types, so that's simple.
MSVC and Sun Studio don't, but we can implement it with something that
looks a little crazy but is actually quite readable.
---
v2: - Squash in Jose's MSVC fixes
-
On Mon, Nov 24, 2014 at 4:39 PM, Matt Turner matts...@gmail.com wrote:
I've got some thread-safety fixes queued up after this and thought I'd
be a good Mesa citizen and pull some code into src/util.
I did some clean ups like replacing INLINE (MSVC knows about inline
these days, right?) and
https://bugs.freedesktop.org/show_bug.cgi?id=85203
--- Comment #5 from Mario Kleiner mario.klei...@tuebingen.mpg.de ---
(In reply to Michel Dänzer from comment #4)
Mario, any ideas?
These are possible scenarios I can think of where this could still happen:
1. The pflip interrupt triggers
https://bugs.freedesktop.org/show_bug.cgi?id=85203
--- Comment #6 from Mario Kleiner mario.klei...@tuebingen.mpg.de ---
Also looking at the current ati-ddx, there have been changes to the logic since
i last touched that code a long time ago. Functions like
radeon_dri2_extrapolate_msc_delay() and
On Tue, 2014-11-25 at 01:56 -0500, Connor Abbott wrote:
On Tue, Nov 25, 2014 at 1:19 AM, Jason Ekstrand ja...@jlekstrand.net wrote:
Previously, the hash_table API required the user to do all of the hashing
of keys as it passed them in. Since the hashing function is intrinsicly
tied to the
On 11/25/2014 03:56 PM, Ilia Mirkin wrote:
On Tue, Nov 25, 2014 at 6:23 AM, Tapani Pälli tapani.pa...@intel.com wrote:
Patch enables ES2 extension that utilizes existing ES3 functionality.
Changes make all the subtests to run and pass in WebGL conformance
test 'webgl-draw-buffers' when running
Patch enables ES2 extension that utilizes existing ES3 functionality.
Changes make all the subtests to run and pass in WebGL conformance
test 'webgl-draw-buffers' when running Chrome on OpenGL ES.
v2: remove unused boolean (Ilia Mirkin)
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
On Tue, 25 Nov 2014 16:22:25 +0100
Marek Olšák mar...@gmail.com wrote:
Hi Pekka,
I explained in the introductory thread [PATCH 0/2] Disable the EGL
state tracker for Linux/DRI builds that it may break swrast
(llvmpipe) on Wayland. I also explained all the reasons why I was
about to do it.
64 matches
Mail list logo
