Reviewed-by: Iago Toral Quiroga ito...@igalia.com
On Mon, 2015-08-03 at 16:36 -0700, Eric Anholt wrote:
I lazily generated some of these in VC4 NIR lowering.
---
src/glsl/nir/nir_opt_algebraic.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/glsl/nir/nir_opt_algebraic.py
Argument validation for glTexSubImageXD is missing a check of format and type
against texture object's internal format when profile is OpenGL-ES 3.0+.
This patch also groups together all format and type checks into a single
block of code for clarity.
Fixes 2 dEQP tests:
*
On Tue, 2015-08-04 at 08:24 +0300, Tapani Pälli wrote:
On 08/04/2015 01:27 AM, Timothy Arceri wrote:
On Mon, 2015-08-03 at 23:16 +1000, Timothy Arceri wrote:
On Mon, 2015-08-03 at 09:02 +0300, Tapani Pälli wrote:
Currently stage reference mask is built using the variable name
only.
This provides an i965 implementation of the
OpenGL ES 3.1 needed function, glMemoryBarrierByRegion.
Marta Lofstedt (4):
gles/es3.1: Enable dispatch of glMemoryBarrierByRegion
mesa/es3.1: Add driver interface for glMemoryBarrierByRegion
mesa/es3.1: Implement the entry point of
Ping. Some 10-20 other ARB_shader_image_load_store patches are blocking
on this patch I've been carrying around since I originally sent it for
review in Dec 2013.
Francisco Jerez curroje...@riseup.net writes:
This will be used to pass image meta-data to the shader when we cannot
use typed
Francisco Jerez curroje...@riseup.net writes:
This works as-is on SKL, only the assertion needs to be relaxed.
I'm going to push this straightforward series I sent back in May without
review if there have been no complaints by the end of the week.
---
src/mesa/drivers/dri/i965/brw_program.c
On Tue, 2015-08-04 at 14:54 +0300, Tapani Pälli wrote:
Hi;
I've tried to understand more about AoA to review the linker changes.
Now I'm testing your patches (and taking currently closer look at 9/20).
Overall it looks fine, calling itself recursively for each array level.
However, with
Francisco Jerez curroje...@riseup.net writes:
Ping. Some 10-20 other ARB_shader_image_load_store patches are blocking
on this patch I've been carrying around since I originally sent it for
review in Dec 2013.
Hint for reviewers: A quick comparison with i965/fs: Import image
memory offset
On 08/03/2015 02:47 PM, Martin Peres wrote:
On 03/08/15 14:35, Tapani Pälli wrote:
On 08/03/2015 01:57 PM, Martin Peres wrote:
On 03/08/15 13:50, Tapani Pälli wrote:
On 08/03/2015 01:48 PM, Martin Peres wrote:
On 03/08/15 11:10, Tapani Pälli wrote:
_mesa_get_program_resource_name
-Original Message-
From: Matt Turner [mailto:matts...@gmail.com]
Sent: Monday, August 3, 2015 5:20 PM
To: Lofstedt, Marta
Cc: Ilia Mirkin; mesa-dev@lists.freedesktop.org
Subject: Re: [Mesa-dev] [PATCH 4/4] i965/es3.1: Implement
glMemoryBarrierByRegion
On Mon, Aug 3, 2015 at 1:04
git://people.freedesktop.org/~jljusten/mesa i965-local-inv-id-v1
Note: These depend on 'i965/cs: Setup push constant data for uniforms'
which Ben look at, but it still doesn't have an r-b. I've included
that patch here due to the dependency, and since I made a minor change
to it since I last sent
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/builtin_variables.cpp | 2 ++
src/glsl/shader_enums.h| 9 +
2 files changed, 11 insertions(+)
diff --git a/src/glsl/builtin_variables.cpp b/src/glsl/builtin_variables.cpp
index 0ff3a3f..b2936a5 100644
---
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/drivers/dri/i965/brw_cs.cpp | 54 +---
1 file changed, 50 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_cs.cpp
b/src/mesa/drivers/dri/i965/brw_cs.cpp
index
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/drivers/dri/i965/brw_cs.cpp | 29 +
src/mesa/drivers/dri/i965/brw_cs.h | 5 +
src/mesa/drivers/dri/i965/brw_fs.cpp | 11 +++
src/mesa/drivers/dri/i965/brw_fs.h | 1 +
4 files changed,
brw_upload_cs_push_constants was based on gen6_upload_push_constants.
v2:
* Add FINISHME comments about more efficient ways to push uniforms
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
Cc: Ben Widawsky b...@bwidawsk.net
---
src/mesa/drivers/dri/i965/brw_context.h | 2 +-
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/drivers/dri/i965/brw_cs.cpp | 25 +++--
src/mesa/drivers/dri/i965/brw_fs.h | 1 +
2 files changed, 24 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_cs.cpp
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 16
1 file changed, 16 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 5549a2d..711ace6 100644
---
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/nir/nir_intrinsics.h | 1 +
src/glsl/nir/nir_lower_system_values.c | 3 +++
2 files changed, 4 insertions(+)
diff --git a/src/glsl/nir/nir_intrinsics.h b/src/glsl/nir/nir_intrinsics.h
index bc6e6b8..29b5e64 100644
---
On Fri, 2015-07-31 at 13:12 +0300, Francisco Jerez wrote:
Iago Toral ito...@igalia.com writes:
On Thu, 2015-07-30 at 17:08 +0300, Francisco Jerez wrote:
Iago Toral Quiroga ito...@igalia.com writes:
When we have code such as this:
mov vgrf1.0.x:F, vgrf2.:F
mov vgrf3.0.x:F,
I added this patch to the 7 patch 'i965/cs: gl_LocalInvocationID
support' series, so go review it there instead. :)
-Jordan
On 2015-07-31 10:44:13, Jordan Justen wrote:
brw_upload_cs_push_constants was based on gen6_upload_push_constants.
v2:
* Add FINISHME comments about more efficient
_mesa_get_program_resource_name has logic to append '[0]' in name
if variable is an array, this should be skipped for XFB varyings
that have array index already appended.
v2: fix comment, change also GL_NAME_LENGTH query to match
the behaviour
Fixes:
From: Marta Lofstedt marta.lofst...@intel.com
Signed-off-by: Marta Lofstedt marta.lofst...@intel.com
---
src/mapi/glapi/gen/gl_API.xml | 4
src/mesa/main/shaderimage.c | 40 +
src/mesa/main/shaderimage.h | 3 +++
On 03/08/15 13:05, Emil Velikov wrote:
On 31 July 2015 at 13:34, Frank Binns frank.bi...@imgtec.com wrote:
On 31/07/15 12:53, Emil Velikov wrote:
On 31 July 2015 at 11:02, Frank Binns frank.bi...@imgtec.com wrote:
Commit 4ed23fd590 introduced some calls to _eglError inappropriately
passing it
On 03/08/15 13:28, Emil Velikov wrote:
On 31 July 2015 at 11:02, Frank Binns frank.bi...@imgtec.com wrote:
Both eglCreatePixmapSurface and eglCreateWindowSurface were incorrectly
setting the EGL error to be EGL_BAD_ALLOC when an invalid native drawable
handle was being passed in. The EGL spec
On Tue, 2015-08-04 at 11:32 +0300, Tapani Pälli wrote:
Original purpose of these lines was to be more friendly against
GUI tools using the extension. However conformance suite explicitly
checks that buffers are not modified in error conditions.
Fixes:
Hi;
I've tried to understand more about AoA to review the linker changes.
Now I'm testing your patches (and taking currently closer look at 9/20).
Overall it looks fine, calling itself recursively for each array level.
However, with fs-initializer-const-index.shader_test it seems to set
Commit 4ed23fd590 introduced some calls to _eglError inappropriately
passing it EGL_BAD_NATIVE_WINDOW. This was actually harmless in two of the
cases as _eglError gets called later on with a more appropriate error code
but (just to be safe) switch these to _eglLog calls instead.
The final case is
When calling either eglCreateWindowSurface or eglCreatePixmapSurface it
was possible for an application to be aborted as a result of it failing
to create a DRI2 drawable on the server. This could happen due to an
application passing in an invalid native drawable handle, for example.
v2: Handle
Both eglCreatePixmapSurface and eglCreateWindowSurface were incorrectly
setting the EGL error to be EGL_BAD_ALLOC when an invalid native drawable
handle was being passed in. The EGL spec states the following for
eglCreatePixmapSurface:
If pixmap is not a valid native pixmap handle, then
Iago Toral ito...@igalia.com writes:
On Fri, 2015-07-31 at 13:12 +0300, Francisco Jerez wrote:
Iago Toral ito...@igalia.com writes:
On Thu, 2015-07-30 at 17:08 +0300, Francisco Jerez wrote:
Iago Toral Quiroga ito...@igalia.com writes:
When we have code such as this:
mov
Iago Toral ito...@igalia.com writes:
On Thu, 2015-07-30 at 17:08 +0300, Francisco Jerez wrote:
Iago Toral Quiroga ito...@igalia.com writes:
When we have code such as this:
mov vgrf1.0.x:F, vgrf2.:F
mov vgrf3.0.x:F, vgrf1.:F
...
mov vgrf3.0.x:F, vgrf1.:F
And vgrf1
Hi! Jason,
On lun, 2015-08-03 at 17:25 -0700, Jason Ekstrand wrote:
On Mon, Aug 3, 2015 at 9:51 AM, Jason Ekstrand ja...@jlekstrand.net wrote:
I also pushed a version to our CI system that switched the
INTEL_USE_NIR default to true and things look pretty good over-all.
On ILK (gen5) and
On Thu, 2015-07-30 at 17:08 +0300, Francisco Jerez wrote:
Iago Toral Quiroga ito...@igalia.com writes:
When we have code such as this:
mov vgrf1.0.x:F, vgrf2.:F
mov vgrf3.0.x:F, vgrf1.:F
...
mov vgrf3.0.x:F, vgrf1.:F
And vgrf1 is chosen for spilling, we can emit
On 03/08/15 13:33, Emil Velikov wrote:
On 31 July 2015 at 11:02, Frank Binns frank.bi...@imgtec.com wrote:
When calling either eglCreateWindowSurface or eglCreatePixmapSurface it
was possible for an application to be aborted as a result of it failing
to create a DRI2 drawable on the server.
---
.../drivers/dri/i965/brw_vec4_reg_allocate.cpp | 24 +++---
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp
b/src/mesa/drivers/dri/i965/brw_vec4_reg_allocate.cpp
index cd89edd..617c988 100644
---
The dst is always written, in this case the predicate is only used to select
the value to write, so if we are spilling the dst we always want to write
whatever value we selected to scratch.
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
If we have spilled/unspilled a register in the current instruction, avoid
emitting unspills for the same register in the same instruction or consecutive
instructions following the current one as long as they keep reading the spilled
register. This should allow us to avoid emitting costy unspills
---
src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vec4.cpp| 2 +-
src/mesa/drivers/dri/i965/intel_debug.c | 3 ++-
src/mesa/drivers/dri/i965/intel_debug.h | 5 +++--
4 files changed, 7 insertions(+), 5 deletions(-)
diff
Link to v1:
http://lists.freedesktop.org/archives/mesa-dev/2015-July/089766.html
Link to v2:
http://lists.freedesktop.org/archives/mesa-dev/2015-July/089958.html
Changes in v3 (Curro):
- Move the caching logic to a separate helper function and reuse it in both
spill_reg and
Original purpose of these lines was to be more friendly against
GUI tools using the extension. However conformance suite explicitly
checks that buffers are not modified in error conditions.
Fixes:
ES31-CTS.program_interface_query.buff-length
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
https://bugs.freedesktop.org/show_bug.cgi?id=90264
--- Comment #44 from Heiko lil_...@web.de ---
(In reply to Antoine Labour from comment #43)
I see, the XResizeWindow would cause a DRI2 Invalidate event on the server
side, and the XSync would make sure it's handled on the client before
On Tue, 2015-08-04 at 23:03 +1000, Timothy Arceri wrote:
On Fri, 2015-07-24 at 15:48 -0400, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
There seems to be a huge overlap between what initialize_context and
initialize_context_to_defaults do, wasn't sure where
On 08/04/2015 05:07 AM, Eduardo Lima Mitev wrote:
Argument validation for glTexSubImageXD is missing a check of format and type
against texture object's internal format when profile is OpenGL-ES 3.0+.
This patch also groups together all format and type checks into a single
block of code for
On Fri, 2015-07-24 at 15:48 -0400, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
There seems to be a huge overlap between what initialize_context and
initialize_context_to_defaults do, wasn't sure where to stick what.
src/glsl/main.cpp | 26
On Mon, Aug 03, 2015 at 05:29:47PM +0900, Michel Dänzer wrote:
On 01.08.2015 00:35, Marek Olšák wrote:
From: Marek Olšák marek.ol...@amd.com
[...]
@@ -1176,8 +1176,18 @@ static void emit_frac(
struct lp_build_emit_data * emit_data)
{
LLVMBuilderRef builder =
On Tue, Aug 4, 2015 at 5:09 PM, Jordan Justen jordan.l.jus...@intel.com wrote:
I checked the extension spec, and it appears that like image
load/store, ssbo is only required in fragment and compute stages. I
don't think the ssbo support in vec4 is quite as problematic as image
load/store, so
On 03.08.2015 21:42, Marek Olšák wrote:
This improves the HUD for RadeonSI.
The series is
Reviewed-by: Michel Dänzer michel.daen...@amd.com
Hopefully, an introductory message like this makes it easier for
Michel not to miss this.
FWIW, I don't remember having missed any of your patches.
On 03.08.2015 21:43, Marek Olšák wrote:
From: Marek Olšák marek.ol...@amd.com
Also use only one store if stride = 4.
All the fetches from and stores to temporaries can be removed now.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91461
Reviewed-by: Michel Dänzer
On 03.08.2015 21:43, Marek Olšák wrote:
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c
Adam Jackson a...@redhat.com writes:
In the DRI2 path this event is magically synthesized from the
corresponding DRI2 event, but with Present, the server sends us the
event itself. The DRI2 path fills in the serial number, send_event, and
display fields of the XEvent struct that the app sees,
Emil Velikov emil.l.veli...@gmail.com writes:
Strictly speaking the xcb_connection_has_error() might be sufficient,
yet the documentation does not mention what will happen if NULL is fed
to the function.
Keep things explicit, which will allow us to remove the dri2_dpy-conn
checking with the
On Tue, 2015-08-04 at 22:40 +0200, Zoltan Gilian wrote:
There is no MDOperand in llvm 3.5.
thanks, this fixes the build for me.
jan
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 19
++-
1 file changed, 10 insertions(+), 9 deletions(-)
diff --git
This avoids exceeding the size of the .index bitfield since it got
truncated, and should make our NIR look more like the NIR that the rest of
the NIR developers are working on.
---
The freedreno changes here are untested.
src/gallium/auxiliary/nir/tgsi_to_nir.c| 144 +++--
They're used by glsl_to_nir.cpp, and I want to use them in TGSI-to-NIR as
well (our use of the var-index slot to store slot properties no longer
works since it got truncated).
The *_MAX defines are left in mtypes.h, because they depend on config.h.
---
src/glsl/shader_enums.h | 98
On Tue, Aug 4, 2015 at 5:28 PM, Eric Anholt e...@anholt.net wrote:
This avoids exceeding the size of the .index bitfield since it got
truncated, and should make our NIR look more like the NIR that the rest of
the NIR developers are working on.
---
The freedreno changes here are untested.
On 2015-07-14 00:46:11, Iago Toral Quiroga wrote:
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
It also creates unop and triop expressions to tell the driver to
calculate the unsized array length.
It is needed two expressions to do the calculation:
Two expressions are needed to do
I checked the extension spec, and it appears that like image
load/store, ssbo is only required in fragment and compute stages. I
don't think the ssbo support in vec4 is quite as problematic as image
load/store, so I'm not sure this information will prove useful.
I did wonder what path we should
On 2015-08-04 15:12:06, Jordan Justen wrote:
On 2015-07-14 00:46:11, Iago Toral Quiroga wrote:
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
It also creates unop and triop expressions to tell the driver to
calculate the unsized array length.
It is needed two expressions to do
On 2015-07-14 00:46:10, Iago Toral Quiroga wrote:
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
They only can be defined in the last position of the shader
storage blocks.
When an unsized array is used in different shaders, it might be
converted in different sized arrays, avoid get
On Sun, Aug 2, 2015 at 1:04 AM, Timothy Arceri t_arc...@yahoo.com.au wrote:
The only values allowed are 0 and 1, and the value is checked before
assigning.
This is a copy of 8eeca7a56c that seems to have been made to the glsl
ir type after it was copied for use in nir but before nir landed.
On 2015-07-14 00:46:12, Iago Toral Quiroga wrote:
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
src/glsl/lower_ubo_reference.cpp | 179
+++
1 file changed, 179 insertions(+)
LGTM
Reviewed-by: Edward O'Callaghan eocallag...@alterapraxis.com
--
Edward O'Callaghan
edward.ocallag...@koparo.com
On Tue, Aug 4, 2015, at 03:51 AM, Alex Deucher wrote:
v2: fix tonga chip check
Reviewed-by: Marek Olšák marek.ol...@amd.com
Reviewed-by: Christian König
LGTM
Reviewed-by: Edward O'Callaghan eocallag...@alterapraxis.com
--
Edward O'Callaghan
edward.ocallag...@koparo.com
On Tue, Aug 4, 2015, at 03:51 AM, Alex Deucher wrote:
v2: incorporate comments from Marek
v3: add missing fiji case in winsys init
use tonga raster config (double
LGTM
Reviewed-by: Edward O'Callaghan eocallag...@alterapraxis.com
--
Edward O'Callaghan
edward.ocallag...@koparo.com
On Tue, Aug 4, 2015, at 03:51 AM, Alex Deucher wrote:
Reviewed-by: Marek Olšák marek.ol...@amd.com
Reviewed-by: Christian König christian.koe...@amd.com
Reviewed-by:
https://bugs.freedesktop.org/show_bug.cgi?id=91556
--- Comment #1 from Pavan Yalamanchili cont...@pavanky.com ---
This was tested on Fedora 22 with AMD A10 5800K.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
This patch fixes a bug in big-endian treatment, where the previous
swizzle info wasn't cleared before a new swizzle info was inserted into
the format field using a bitwise-OR operation.
v2: use MESA_ARRAY_FORMAT_SWIZZLE_*_MASK instead of numeric constants
Signed-off-by: Oded Gabbay
On Mon, Aug 3, 2015 at 5:44 PM, Eric Anholt e...@anholt.net wrote:
This is useful to increase the CSE opportunities for a scalar backend. It
avoids regressions when dropping vc4's custom CSE implementation.
---
src/glsl/Makefile.sources | 1 +
src/glsl/nir/nir.h
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 16
1 file changed, 16 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 711ace6..a0b20c4 100644
---
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/drivers/dri/i965/brw_cs.cpp | 19 +++
src/mesa/drivers/dri/i965/brw_fs.h | 1 +
2 files changed, 20 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_cs.cpp
b/src/mesa/drivers/dri/i965/brw_cs.cpp
index
git://people.freedesktop.org/~jljusten/mesa i965-workgroup-id-v1
Allows piglit's
tests/spec/arb_compute_shader/execution/basic-group-id-atomic.shader_test
to pass.
To test, I set MESA_GL_VERSION_OVERRIDE=4.3,
MESA_GLSL_VERSION_OVERRIDE=430 and
MESA_EXTENSION_OVERRIDE=GL_ARB_compute_shader.
On Mon, Aug 3, 2015 at 6:29 PM, Emil Velikov emil.l.veli...@gmail.com
wrote:
On 2 August 2015 at 11:37, Oded Gabbay oded.gab...@gmail.com wrote:
This patch fixes a bug in big-endian handling, where the previous
swizzle info wasn't cleared before a new swizzle info was inserted into
the
Argument validation for glTexSubImageXD is missing a check of format and type
against texture object's internal format when profile is OpenGL-ES 3.0+.
This patch also groups together all format and type checks on GLES into a
new function texture_format_error_check_gles(), to factorize similar
This patch is
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
On 08/04/2015 11:02 AM, Eduardo Lima Mitev wrote:
Argument validation for glTexSubImageXD is missing a check of format and type
against texture object's internal format when profile is OpenGL-ES 3.0+.
This patch also groups
https://bugs.freedesktop.org/show_bug.cgi?id=91556
Bug ID: 91556
Summary: clSetKernelArg from OpenCL is erroring out incorrectly
Product: Mesa
Version: 10.6
Hardware: Other
OS: All
Status: NEW
Severity:
On Tue, Aug 4, 2015 at 10:17 AM, Oded Gabbay oded.gab...@gmail.com wrote:
This patch fixes a bug in big-endian treatment, where the previous
swizzle info wasn't cleared before a new swizzle info was inserted into
the format field using a bitwise-OR operation.
v2: use
On 07/22/2015 07:52 PM, Ryan Houdek wrote:
All vendors I've tested allow token pasting in their ESSL shaders.
At least one application expects this to be supported
even though it is against spec.
I'm not very excited about deviating from the spec. Do you know if the
Khronos reference compiler
On Mon, Aug 3, 2015 at 6:40 PM, Emil Velikov emil.l.veli...@gmail.com
wrote:
Hi Oded,
On 2 August 2015 at 11:37, Oded Gabbay oded.gab...@gmail.com wrote:
This patch fixes a bug that is manifested in the read path of mesa when
running on big-endian machines. The effects can be seen when
https://bugs.freedesktop.org/show_bug.cgi?id=91556
Pavan Yalamanchili cont...@pavanky.com changed:
What|Removed |Added
CC|
On 08/04/2015 12:02 PM, Eduardo Lima Mitev wrote:
Argument validation for glTexSubImageXD is missing a check of format and type
against texture object's internal format when profile is OpenGL-ES 3.0+.
This patch also groups together all format and type checks on GLES into a
new function
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/nir/nir_intrinsics.h | 1 +
src/glsl/nir/nir_lower_system_values.c | 3 +++
2 files changed, 4 insertions(+)
diff --git a/src/glsl/nir/nir_intrinsics.h b/src/glsl/nir/nir_intrinsics.h
index 29b5e64..6987d42 100644
---
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/builtin_variables.cpp | 2 ++
src/glsl/shader_enums.h| 1 +
2 files changed, 3 insertions(+)
diff --git a/src/glsl/builtin_variables.cpp b/src/glsl/builtin_variables.cpp
index b2936a5..8fa7b19 100644
---
On Tue, Aug 4, 2015 at 8:24 PM, Matt Turner matts...@gmail.com wrote:
On Tue, Aug 4, 2015 at 10:17 AM, Oded Gabbay oded.gab...@gmail.com wrote:
This patch fixes a bug in big-endian treatment, where the previous
swizzle info wasn't cleared before a new swizzle info was inserted into
the
This patch fixes a bug in big-endian treatment, where the previous
swizzle info wasn't cleared before a new swizzle info was inserted into
the format field using a bitwise-OR operation.
v2: use MESA_ARRAY_FORMAT_SWIZZLE_*_MASK instead of numeric constants
v3: align according to coding style
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
On 2015-07-22 23:42:55, Samuel Iglesias Gonsalvez wrote:
v2:
- Add tessellation shader constants assignment
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
src/mesa/drivers/dri/i965/brw_context.c | 12
1
There is no MDOperand in llvm 3.5.
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 19 ++-
1 file changed, 10 insertions(+), 9 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
On Tue, Aug 4, 2015 at 2:42 AM, Antía Puentes apuen...@igalia.com wrote:
Hi! Jason,
On lun, 2015-08-03 at 17:25 -0700, Jason Ekstrand wrote:
On Mon, Aug 3, 2015 at 9:51 AM, Jason Ekstrand ja...@jlekstrand.net wrote:
I also pushed a version to our CI system that switched the
INTEL_USE_NIR
On Mon, Aug 03, 2015 at 12:53:32PM +0300, Francisco Jerez wrote:
EdB edb+m...@sigluy.net writes:
As a side effect, this fix clRetain/ReleaseEvent Piglit test
---
src/gallium/state_trackers/clover/api/transfer.cpp | 29
--
1 file changed, 27 insertions(+), 2
On Tue, 2015-08-04 at 18:15 -0400, Rob Clark wrote:
On Sun, Aug 2, 2015 at 1:04 AM, Timothy Arceri t_arc...@yahoo.com.au
wrote:
The only values allowed are 0 and 1, and the value is checked before
assigning.
This is a copy of 8eeca7a56c that seems to have been made to the glsl
ir
On 03.08.2015 21:36, Marek Olšák wrote:
From: Marek Olšák marek.ol...@amd.com
---
src/gallium/drivers/radeon/r600_buffer_common.c | 6 --
src/gallium/drivers/radeon/r600_pipe_common.c | 1 +
src/gallium/drivers/radeon/r600_pipe_common.h | 1 +
3 files changed, 6 insertions(+),
Hi Ian,
On 04.08.2015 03:11, Ian Romanick wrote:
Module: Mesa
Branch: master
Commit: 7ac946e546bba440f87ce95ef022745201744f9c
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=7ac946e546bba440f87ce95ef022745201744f9c
Author: Ian Romanick ian.d.roman...@intel.com
Date: Thu Jul
Hi All,
I know the false colors problems. Some Linux programs appear to be ABGR
instead of RGBA, thus green becomes purple, red becomes light blue etc.
I created a bug report on freedesktop.org. Bug report 72877:
https://bugs.freedesktop.org/show_bug.cgi?id=72877 and
the original code worked because it was called after
result = eglMakeCurrent(display, surface, surface, context);
I wanted to use OSMesa.
After I changed the call to OSMesaMakeCurrent.glGetString() returned NULL.
Please advise.
thanks
background:having android apps run on X11 on base
mesa OES lib: libgles1-mesa-dev libgles2-mesa-dev libosmesa6-dev
what to do:
render all surfaces to program memory.
problems:
1) glMapBuffer returns NULL.OES_mapbuffer seemed not work.
though the extension was available.
2) GL_READ_ONLY ,
93 matches
Mail list logo