On Wed, Aug 5, 2015 at 9:36 AM, Christian Zigotzky
wrote:
> Hi Oded,
>
> Is the r600g driver ready for the Gallium3D infrastructure of big endian
> hosts or did you only solve the wrong colors problem?
>
> Cheers,
>
> Christian
>
Hi Christian,
I'm only working on software rendering at the moment
On Tue, 2015-08-04 at 14:08 -0700, Jordan Justen wrote:
> On 2015-07-14 00:46:10, Iago Toral Quiroga wrote:
> > From: Samuel Iglesias Gonsalvez
> >
> > They only can be defined in the last position of the shader
> > storage blocks.
> >
> > When an unsized array is used in different shaders, it m
On Tue, Aug 4, 2015 at 11:18 PM, Matt Turner wrote:
> Regression since commit 3a31876600, when tiling modes were moved into
> layout_flags.
>
> The relevant enum values are
>
>MIPTREE_LAYOUT_ALLOC_YTILED = 1 << 5
>MIPTREE_LAYOUT_ALLOC_XTILED = 1 << 6
>MIPTREE_LAYOUT_ALLOC_ANY_TILED = M
On Tue, 2015-08-04 at 17:54 -0700, Jason Ekstrand wrote:
> On Tue, Aug 4, 2015 at 5:09 PM, Jordan Justen
> 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 qu
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.h
b/src/mesa/drivers/dri/i965/intel_mipmap_tree.h
index 89fdccb..540c655 100644
--- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.
Regression since commit 3a31876600, when tiling modes were moved into
layout_flags.
The relevant enum values are
MIPTREE_LAYOUT_ALLOC_YTILED = 1 << 5
MIPTREE_LAYOUT_ALLOC_XTILED = 1 << 6
MIPTREE_LAYOUT_ALLOC_ANY_TILED = MIPTREE_LAYOUT_ALLOC_YTILED |
MI
On Tue, 2015-08-04 at 21:20 +1000, Timothy Arceri wrote:
> 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:
> >
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 a/s
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
http://lists.freedesktop
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 ,
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
h___
Emil Velikov 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 next commit.
>
>
Adam Jackson 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, but the Pre
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
> Date: Thu Jul 30 06:00:56 2015
On 03.08.2015 21:42, Marek Olšák wrote:
> This improves the HUD for RadeonSI.
The series is
Reviewed-by: Michel Dänzer
> 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. Last week I
was swampe
On 03.08.2015 21:43, Marek Olšák wrote:
> From: Marek Olšák
>
> 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
--
Earthling Michel Dänzer
On 03.08.2015 21:43, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> 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
> b/src/gallium/winsys/radeon/drm/radeon_drm_
On 03.08.2015 21:36, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> 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(+), 2 deletions(
On Tue, Aug 4, 2015 at 5:09 PM, Jordan Justen 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 I'm not sure this infor
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 ta
LGTM
Reviewed-by: Edward O'Callaghan
--
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 check this)
> v4: rebase on h
LGTM
Reviewed-by: Edward O'Callaghan
--
Edward O'Callaghan
edward.ocallag...@koparo.com
On Tue, Aug 4, 2015, at 03:51 AM, Alex Deucher wrote:
> Reviewed-by: Marek Olšák
> Reviewed-by: Christian König
> Reviewed-by: David Zhang
> Signed-off-by: Alex Deucher
> ---
> include/pci_ids/rade
LGTM
Reviewed-by: Edward O'Callaghan
--
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
> Reviewed-by: Christian König
> Reviewed-by: David Zhang
> Signed-off-by: Alex Deucher
On Tue, 2015-08-04 at 18:15 -0400, Rob Clark wrote:
> On Sun, Aug 2, 2015 at 1:04 AM, Timothy Arceri
> 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
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
> >
> > 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 cal
On 2015-07-14 00:46:12, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
>
> Signed-off-by: Samuel Iglesias Gonsalvez
> ---
> src/glsl/lower_ubo_reference.cpp | 179
> +++
> 1 file changed, 179 insertions(+)
>
> diff --git a/src/glsl/lower_ubo_re
On Sun, Aug 2, 2015 at 1:04 AM, Timothy Arceri 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.
> ---
> src/glsl/
On 2015-07-14 00:46:11, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
>
> 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 the calculatio
On Tue, Aug 4, 2015 at 5:28 PM, Eric Anholt 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.
>
> src/galliu
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 +
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 +++--
On 2015-07-14 00:46:10, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
>
> 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 a linker erro
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
index
On Tue, Aug 4, 2015 at 8:24 PM, Matt Turner wrote:
>
> On Tue, Aug 4, 2015 at 10:17 AM, Oded Gabbay 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 ope
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
Signe
On Tue, Aug 4, 2015 at 2:42 AM, Antía Puentes 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 wrote:
>
>> > I also pushed a version to our CI system that switched the
>> > INTEL_USE_NIR default to true and things l
Reviewed-by: Jordan Justen
On 2015-07-22 23:42:55, Samuel Iglesias Gonsalvez wrote:
> v2:
> - Add tessellation shader constants assignment
>
> Signed-off-by: Samuel Iglesias Gonsalvez
> ---
> src/mesa/drivers/dri/i965/brw_context.c | 12
> 1 file changed, 12 insertions(+)
>
> dif
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 compil
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.
Jorda
Signed-off-by: Jordan Justen
---
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 3e68327..14c69a2 100644
-
Signed-off-by: Jordan Justen
---
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
--- a/src/mesa/drivers/dri/i965/brw_fs
Signed-off-by: Jordan Justen
---
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
--- a/src/glsl/nir/nir_intrinsi
Signed-off-by: Jordan Justen
---
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
--- a/src/glsl/builtin_variables.cpp
+++ b/src
This patch is
Reviewed-by: Ian Romanick
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 together all format and
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 texture_
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
code
On Mon, Aug 3, 2015 at 5:44 PM, Eric Anholt 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
On Tue, Aug 4, 2015 at 10:17 AM, Oded Gabbay 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 MESA_ARRAY_FORMAT_SWIZZLE_*_MASK instead o
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
CC: "10.5 10.
On Mon, Aug 3, 2015 at 6:29 PM, Emil Velikov
wrote:
> On 2 August 2015 at 11:37, Oded Gabbay 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 format field using a bitwise-OR operation.
https://bugs.freedesktop.org/show_bug.cgi?id=91556
--- Comment #1 from Pavan Yalamanchili ---
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.
__
On Mon, Aug 3, 2015 at 6:40 PM, Emil Velikov
wrote:
> Hi Oded,
>
> On 2 August 2015 at 11:37, Oded Gabbay 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 running
> > piglit sanity test and/or taki
https://bugs.freedesktop.org/show_bug.cgi?id=91556
Pavan Yalamanchili changed:
What|Removed |Added
CC||cont...@pavanky.com
--
You are rec
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: no
On Mon, Aug 03, 2015 at 12:53:32PM +0300, Francisco Jerez wrote:
> EdB 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 deletions(-)
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
>
> [...]
>
> > @@ -1176,8 +1176,18 @@ static void emit_frac(
> > struct lp_build_emit_data * emit_data)
> > {
> > LLVMBuilderRef builder = bld_base->ba
Francisco Jerez 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 calculation code."
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
Francisco Jerez 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 | 2 +-
> 1 file c
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 writes:
> This will be used to pass image meta-data to the shader when we cannot
> use typed surface reads and write
Iago Toral writes:
> On Thu, 2015-07-30 at 17:08 +0300, Francisco Jerez wrote:
>> Iago Toral Quiroga 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 chos
Iago Toral writes:
> On Fri, 2015-07-31 at 13:12 +0300, Francisco Jerez wrote:
>> Iago Toral writes:
>>
>> > On Thu, 2015-07-30 at 17:08 +0300, Francisco Jerez wrote:
>> >> Iago Toral Quiroga writes:
>> >>
>> >> > When we have code such as this:
>> >> >
>> >> > mov vgrf1.0.x:F, vgrf2.:F
>
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 a
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 the
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 clar
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
> > ---
> >
> > There seems to be a huge overlap between what initialize_context and
> > initialize_context_to_defaults do, wasn't sure where to stick wh
On Fri, 2015-07-24 at 15:48 -0400, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
>
> 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 +-
> 1 file
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
bindi
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:
>ES31-CTS.program_interface_quer
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
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:
* dEQP-GLES3.functional.ne
Reviewed-by: Iago Toral Quiroga
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
> b/src/glsl/nir/ni
---
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 --g
---
.../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
--- a/src/mes
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 th
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 evaluate_spill_c
On Thu, 2015-07-30 at 17:08 +0300, Francisco Jerez wrote:
> Iago Toral Quiroga 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 this
On 03/08/15 13:33, Emil Velikov wrote:
> On 31 July 2015 at 11:02, Frank Binns 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. This could happen
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 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 965 (gen 4), th
On 03/08/15 13:28, Emil Velikov wrote:
> On 31 July 2015 at 11:02, Frank Binns 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 states the follo
On 03/08/15 13:05, Emil Velikov wrote:
> On 31 July 2015 at 13:34, Frank Binns wrote:
>> On 31/07/15 12:53, Emil Velikov wrote:
>>> On 31 July 2015 at 11:02, Frank Binns wrote:
Commit 4ed23fd590 introduced some calls to _eglError inappropriately
passing it EGL_BAD_NATIVE_WINDOW. This wa
https://bugs.freedesktop.org/show_bug.cgi?id=90264
--- Comment #44 from Heiko ---
(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
> anything else happ
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
---
src/mesa/main/prog
> 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 Me
From: Marta Lofstedt
Signed-off-by: Marta Lofstedt
---
src/mapi/glapi/gen/gl_API.xml | 4
src/mesa/main/shaderimage.c | 40 +
src/mesa/main/shaderimage.h | 3 +++
src/mesa/main/tests/dispatch_sanity.cpp | 3 +--
4 files c
_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:
ES31-CTS.program_interface_query.transf
> -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
On Fri, 2015-07-31 at 13:12 +0300, Francisco Jerez wrote:
> Iago Toral writes:
>
> > On Thu, 2015-07-30 at 17:08 +0300, Francisco Jerez wrote:
> >> Iago Toral Quiroga writes:
> >>
> >> > When we have code such as this:
> >> >
> >> > mov vgrf1.0.x:F, vgrf2.:F
> >> > mov vgrf3.0.x:F, vgrf1.xx
90 matches
Mail list logo
