[Mesa-dev] [PATCH] r600g/sb: fix stack size computation on evergreen

2013-12-07 Thread Vadim Girlin
On evergreen we have to reserve 1 stack element in some additional cases besides the ones mentioned in the docs, but stack size computation was recently reimplemented exactly as described in the docs by the patch that added workarounds for stack issues on EG/CM, resulting in regressions with some

Re: [Mesa-dev] [PATCH 06/11] mesa: Add MESA_FORMAT_ABGR2101010.

2013-12-07 Thread Francisco Jerez
Paul Berry stereotype...@gmail.com writes: On 24 November 2013 21:00, Francisco Jerez curroje...@riseup.net wrote: Including pack/unpack and texstore code. This texture format is a requirement for ARB_shader_image_load_store. --- src/mesa/main/format_pack.c | 29

Re: [Mesa-dev] [PATCH 01/11] glapi: Update dispatch XML files for ARB_shader_image_load_store.

2013-12-07 Thread Francisco Jerez
Paul Berry stereotype...@gmail.com writes: On 24 November 2013 21:00, Francisco Jerez curroje...@riseup.net wrote: [...] +enum name=IMAGE_BINDING_LAYERED value=0x8F3C/ +enum name=IMAGE_BINDING_LAYERED_EXT value=0x8F3C/ I'm surprised to see IMAGE_BINDING_LAYERED_EXT here instead of

Re: [Mesa-dev] [Nouveau] [PATCH] nv50: enable MPEG-4 for NVA3+ (VP4.0)

2013-12-07 Thread Ilia Mirkin
On Sat, Dec 7, 2013 at 8:11 AM, Martin Peres martin.pe...@free.fr wrote: From: Martin Peres martin.pe...@labri.fr This patch is a follow-up from Ilia Mirkin's enable H.264 patch which solves the problem that prevented MPEG-4 videos to play correctly. Tested on an nva3. I might reword this

Re: [Mesa-dev] memory stomping in 95bd8a332d1 (dri: Move i965-specific context flag logic to dri common.)

2013-12-07 Thread Ilia Mirkin
ping? I'm pretty sure all gallium drivers are getting random bits corrupted in 10.0+... at least if driCreateContextAttribs is getting called. Kristian, you had a patch on IRC, but I didn't see it get posted... -ilia On Thu, Nov 28, 2013 at 11:27 PM, Ilia Mirkin imir...@alum.mit.edu wrote:

Re: [Mesa-dev] [PATCH 08/11] mesa: Implement the GL entry points defined by ARB_shader_image_load_store.

2013-12-07 Thread Fredrik Höglund
On Tuesday 26 November 2013, Francisco Jerez wrote: Fredrik Höglund fred...@kde.org writes: [...] +} + +void GLAPIENTRY +_mesa_MemoryBarrier(GLbitfield barriers) +{ + GET_CURRENT_CONTEXT(ctx); + + if (ctx-Driver.MemoryBarrier) + ctx-Driver.MemoryBarrier(ctx, barriers);

Re: [Mesa-dev] [PATCH 08/11] mesa: Implement the GL entry points defined by ARB_shader_image_load_store.

2013-12-07 Thread Francisco Jerez
Fredrik Höglund fred...@kde.org writes: On Tuesday 26 November 2013, Francisco Jerez wrote: Fredrik Höglund fred...@kde.org writes: [...] +} + +void GLAPIENTRY +_mesa_MemoryBarrier(GLbitfield barriers) +{ + GET_CURRENT_CONTEXT(ctx); + + if (ctx-Driver.MemoryBarrier) +

[Mesa-dev] [PATCH] r600g: Add missing break for callstack_push FC_PUSH_WQM case.

2013-12-07 Thread Vinson Lee
Fixes Missing break in switch reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/r600/r600_shader.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/drivers/r600/r600_shader.c index

[Mesa-dev] [PATCH] nvc0: Remove unused pointer value cfg.

2013-12-07 Thread Vinson Lee
cfg is only used inside for loop so also move declaration there. Fixes Unused pointer value defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/nouveau/nvc0/nvc0_query.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 08/11] mesa: Implement the GL entry points defined by ARB_shader_image_load_store.

2013-12-07 Thread Francisco Jerez
Paul Berry stereotype...@gmail.com writes: On 24 November 2013 21:00, Francisco Jerez curroje...@riseup.net wrote: [...] + + case GL_RGBA16UI: + return MESA_FORMAT_RGBA_UINT16; + + case GL_RGB10_A2UI: + return MESA_FORMAT_ABGR2101010_UINT; I don't understand the naming

Re: [Mesa-dev] [PATCH 03/11] mesa: Add state data structures requried for ARB_shader_image_load_store.

2013-12-07 Thread Francisco Jerez
Paul Berry stereotype...@gmail.com writes: On 24 November 2013 21:00, Francisco Jerez curroje...@riseup.net wrote: [...] @@ -175,6 +175,7 @@ #define MAX_COMBINED_ATOMIC_BUFFERS(MAX_UNIFORM_BUFFERS * 6) /* Size of an atomic counter in bytes according to ARB_shader_atomic_counters */

[Mesa-dev] [PATCH] mesa: Define helper function to get the number of texture layers.

2013-12-07 Thread Francisco Jerez
And to check if it can have layers at all. This will be used by the implementation of ARB_shader_image_load_store. --- src/mesa/main/teximage.c | 85 src/mesa/main/teximage.h | 6 2 files changed, 91 insertions(+) diff --git

Re: [Mesa-dev] [PATCH 00/11] Core mesa changes for ARB_shader_image_load_store.

2013-12-07 Thread Francisco Jerez
Paul Berry stereotype...@gmail.com writes: On 6 December 2013 02:54, Francisco Jerez curroje...@riseup.net wrote: Paul Berry stereotype...@gmail.com writes: [...] This order of patches introduces temporary build breakages, which I'd prefer to avoid if possible. Here's one order which

Re: [Mesa-dev] [PATCH] nv50: enable H.264 for NV98+ (VP3, VP4.0)

2013-12-07 Thread Ilia Mirkin
On Fri, Dec 6, 2013 at 11:43 PM, Ilia Mirkin imir...@alum.mit.edu wrote: Create the ref_bo without any storage type flags set for now. This can probably be split up somehow later on, but this seems to work. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Cc: 10.0

[Mesa-dev] [PATCH 2/3] nouveau/video: update h264 picparm field names based on usage

2013-12-07 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- src/gallium/drivers/nouveau/nouveau_vp3_video_vp.c | 28 ++ 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/src/gallium/drivers/nouveau/nouveau_vp3_video_vp.c

[Mesa-dev] [PATCH 3/3] nouveau/video: update a few more h264 picparm field names

2013-12-07 Thread Ilia Mirkin
Based on comments by Benjamin Morris bmor...@nvidia.com in http://lists.freedesktop.org/archives/nouveau/2013-December/015328.html This adds setting of is_long_term, and updates a few field names we were unclear about. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu ---

[Mesa-dev] [PATCH 1/3] nv50: enable h264 and mpeg4 for nv98+ (vp3, vp4.0)

2013-12-07 Thread Ilia Mirkin
Create the ref_bo without any storage type flags set for now. The issue probably arises from our use of the additional buffer space at the end of the ref_bo. It should probably be split up in the future. Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Tested-by: Martin Peres martin.pe...@labri.fr

[Mesa-dev] [PATCH] dri_util: Don't assume __DRIcontext-driverPrivate is a gl_context

2013-12-07 Thread Kristian Høgsberg
The driverPrivate pointer is opaque to the driver and we can't assume it's a struct gl_context in dri_util.c. Instead provide a helper function to set the struct gl_context flags from the incoming DRI context flags. Signed-off-by: Kristian Høgsberg k...@bitplanet.net Cc: 10.0

Re: [Mesa-dev] memory stomping in 95bd8a332d1 (dri: Move i965-specific context flag logic to dri common.)

2013-12-07 Thread Kristian Høgsberg
On Sat, Dec 7, 2013 at 8:17 AM, Ilia Mirkin imir...@alum.mit.edu wrote: ping? I'm pretty sure all gallium drivers are getting random bits corrupted in 10.0+... at least if driCreateContextAttribs is getting called. Kristian, you had a patch on IRC, but I didn't see it get posted... I sent

[Mesa-dev] [PATCH] mesa: fix texture view use of _mesa_get_tex_image()

2013-12-07 Thread Chris Forbes
The target parameter to _mesa_get_tex_image() is a target enum, not an index. When we're setting up faces for a cubemap, it should be CUBE_MAP_POSITIVE_X .. CUBE_MAP_NEGATIVE_Z; for all other targets it should be the same as the texobj's target. Fixes broken cubemaps [had only +X face but claimed

Re: [Mesa-dev] [PATCH] dri_util: Don't assume __DRIcontext-driverPrivate is a gl_context

2013-12-07 Thread Ilia Mirkin
On Sun, Dec 8, 2013 at 1:02 AM, Kristian Høgsberg k...@bitplanet.net wrote: The driverPrivate pointer is opaque to the driver and we can't assume it's a struct gl_context in dri_util.c. Instead provide a helper function to set the struct gl_context flags from the incoming DRI context flags.