[Mesa-dev] [Bug 72895] Missing trees in flightgear 2.12.1 with r600 driver and mesa 10.0.1

2014-01-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=72895 --- Comment #7 from Barto --- does anyone need more informations about this bug ? because this bug is still here with the new mesa 10.0.2, I did a bisect, the bug begins with 59b01ca252bd6706f08cd80a864819d71dfe741c commit, I can do another tes

Re: [Mesa-dev] [PATCH] vdpau: flush the context before exporting the surface v2

2014-01-13 Thread Christian König
Yeah, probably because XBMC still (incorrectly) calls the map function only once. Putting the flush into vlVdpResolveDelayedRendering solves the problem because it's called the next time somebody starts rendering, but it's way to late at this point. Need to sync up with the XBMC devs on this

Re: [Mesa-dev] [PATCH] st/egl: Flush resources before presentation

2014-01-13 Thread Marek Olšák
Pushed. Marek On Tue, Jan 7, 2014 at 11:20 PM, Martin Andersson wrote: > Hi Marek, > > Since it seems no one else have any comments on this, maybe you could > commit it for me? > > //Martin > > On Thu, Dec 26, 2013 at 1:15 PM, Marek Olšák wrote: >> Reviewed-by: Marek Olšák >> >> Marek >> >> On

Re: [Mesa-dev] [PATCH] vdpau: flush the context before exporting the surface v2

2014-01-13 Thread Marek Olšák
This patch doesn't fix the bug. :( Marek On Mon, Jan 13, 2014 at 2:55 PM, Christian König wrote: > From: Marek Olšák > > Bugzilla: > https://bugs.freedesktop.org/show_bug.cgi?id=73191 > > When VL uploads vertex buffers, it uses PIPE_TRANSFER_DONTBLOCK, which always > flushes the context in the

Re: [Mesa-dev] [PATCH] Mark debug_print with __attribute__ ((format(__printf__, 1, 0)))

2014-01-13 Thread Keith Packard
Thierry Reding writes: > While at it, perhaps the drmMsg() and drmDebugPrint() functions should > be similarily annotated as well? I don't know; I'm just fixing X server warnings this week and this was the source of one of them. Additional warning fixes for drm would be a great idea! -- keith.

[Mesa-dev] [PATCH] vdpau: flush the context before exporting the surface v2

2014-01-13 Thread Christian König
From: Marek Olšák Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73191 When VL uploads vertex buffers, it uses PIPE_TRANSFER_DONTBLOCK, which always flushes the context in the winsys if the buffer being mapped is busy. Since I added handling of DISCARD_RANGE, DONTBLOCK has had no effect

[Mesa-dev] [PATCH] mesa-demos: Fixes a bug in demo2 application

2014-01-13 Thread Yasir Khan
From: Yasir-Khan Vertices array is being passed to glColorPointer whereas its supposed to pass the color array Signed-off-by: Yasir-Khan diff --git a/src/egl/opengl/demo2.c b/src/egl/opengl/demo2.c index 71a1a31..505b474 100644 --- a/src/egl/opengl/demo2.c +++ b/src/egl/opengl/demo2.c @@ -35,7

Re: [Mesa-dev] [PATCH 2/6] gallium/vl: use u_upload_mgr to upload vertices for vl_compositor

2014-01-13 Thread Christian König
Am 13.01.2014 14:13, schrieb Marek Olšák: From: Marek Olšák This patch is: Reviewed-by: Christian König This is the recommended way for streaming vertices. Always use this if you need to upload vertices every frame. --- src/gallium/auxiliary/vl/vl_compositor.c | 51 --

[Mesa-dev] [PATCH] glx: Add missing null check in glXCreateContextAttribsARB

2014-01-13 Thread Juha-Pekka Heikkila
Signed-off-by: Juha-Pekka Heikkila --- src/glx/create_context.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/glx/create_context.c b/src/glx/create_context.c index 38e949a..b15921f 100644 --- a/src/glx/create_context.c +++ b/src/glx/create_context.c @@ -90,6 +90,9 @@ glXCreateContext

[Mesa-dev] [PATCH 4/6] gallium/u_upload_mgr: don't expose u_upload_flush

2014-01-13 Thread Marek Olšák
From: Marek Olšák It's unused and shouldn't be used at all in my opinion. If some driver doesn't support the unsynchronized flag, u_upload_mgr should avoid the synchronization by other means, e.g. by using the DONTBLOCK flag. --- src/gallium/auxiliary/util/u_upload_mgr.c | 16

[Mesa-dev] [PATCH 2/6] gallium/vl: use u_upload_mgr to upload vertices for vl_compositor

2014-01-13 Thread Marek Olšák
From: Marek Olšák This is the recommended way for streaming vertices. Always use this if you need to upload vertices every frame. --- src/gallium/auxiliary/vl/vl_compositor.c | 51 src/gallium/auxiliary/vl/vl_compositor.h | 1 + 2 files changed, 20 insertions(+)

[Mesa-dev] [PATCH 3/6] gallium/hud: just unmap the upload vertex buffer instead of recreating it

2014-01-13 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/hud/hud_context.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/hud/hud_context.c b/src/gallium/auxiliary/hud/hud_context.c index c4a4f18..465013c 100644 --- a/src/gallium/auxiliary/hud/hud_context.c +++ b/s

[Mesa-dev] [PATCH 6/6] radeonsi: handle R600_CONTEXT_PS_PARTIAL_FLUSH in si_emit_cache_flush

2014-01-13 Thread Marek Olšák
From: Marek Olšák For consistency only, This unused by radeonsi currently. --- src/gallium/drivers/radeonsi/si_state_draw.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c b/src/gallium/drivers/radeonsi/si_state_draw.c index f64

[Mesa-dev] [PATCH 1/6] vdpau: flush the context after resolving delayed rendering

2014-01-13 Thread Marek Olšák
From: Marek Olšák Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73191 When VL uploads vertex buffers, it uses PIPE_TRANSFER_DONTBLOCK, which always flushes the context in the winsys if the buffer being mapped is busy. Since I added handling of DISCARD_RANGE, DONTBLOCK has had no effect

[Mesa-dev] [PATCH 5/6] r600g, radeonsi: if discarding the whole buffer range, discard the whole resource instead

2014-01-13 Thread Marek Olšák
From: Marek Olšák Also set the unsynchronized flag if the whole resource was discarded to avoid doing buffer-busy checks again. --- src/gallium/drivers/radeon/r600_buffer_common.c | 8 1 file changed, 8 insertions(+) diff --git a/src/gallium/drivers/radeon/r600_buffer_common.c b/src/g

Re: [Mesa-dev] Naming everything in src/gallium/drivers/radeonsi si_*

2014-01-13 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák Feel free to push this. Marek On Sat, Jan 11, 2014 at 4:20 PM, Andreas Hartmetz wrote: > Continuing here because the threads had diverged... > > I've updated the patch series under the same URL and applied all the > suggested improvements. The variable

Re: [Mesa-dev] [PATCH] Mark debug_print with __attribute__ ((format(__printf__, 1, 0)))

2014-01-13 Thread Thierry Reding
On Sun, Jan 12, 2014 at 10:34:19AM -0800, Keith Packard wrote: > the drmServerInfo member, debug_print, takes a printf format string > and varargs list. Tell the compiler about it. > > Signed-off-by: Keith Packard > --- > xf86drm.h | 8 +++- > 1 file changed, 7 insertions(+), 1 deletion(-) >

Re: [Mesa-dev] [PATCH] mesa: use signed temporary variable to store _ColorDrawBufferIndexes

2014-01-13 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Sun, Jan 12, 2014 at 11:52 PM, Emil Velikov wrote: > _ColorDrawBufferIndexes is defined as GLint* and using a GLuint* > will result in the first part of the conditional to be evaluated to > true always. > > Unintentionally introduced by the following commit, th

<    1   2