https://bugs.freedesktop.org/show_bug.cgi?id=91263
Michel Dänzer mic...@daenzer.net changed:
What|Removed |Added
Component|Drivers/Gallium/r600|Mesa core
On Thu, 2015-07-09 at 17:50 -0700, Eric Anholt wrote:
Timothy Arceri t_arc...@yahoo.com.au writes:
---
Some of the AoA tests currently get stuck in an optimisation
loop for a long time and this was taking a large amount of time.
Its a symptom not a cause but I don't see any issue
This implementation should be faster and there was no
need to store a data field.
---
src/glsl/ir_validate.cpp | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/src/glsl/ir_validate.cpp b/src/glsl/ir_validate.cpp
index cfe0df3..684bef2 100644
---
Reviewed-by: Iago Toral Quiroga ito...@igalia.com
On Thu, 2015-07-09 at 18:59 -0700, Chad Versace wrote:
Don't assume that $(top_srcdir)/.git is a directory. It may be a
gitlink file [1] if $(top_srcdir) is a submodule checkout or a linked
worktree [2].
[1] A gitlink is a text file that
Perf data?
On Fri, Jul 10, 2015 at 6:41 PM, Timothy Arceri t_arc...@yahoo.com.au wrote:
This implementation should be faster and there was no
need to store a data field.
---
src/glsl/ir_validate.cpp | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff
On Fri, 2015-07-10 at 19:07 +1200, Chris Forbes wrote:
Perf data?
I can create some if you like, but wasn't program/hash_table.c meant to die
along time ago [1] anyway.
[1] http://lists.freedesktop.org/archives/mesa-dev/2013-December/050524.html
On Fri, Jul 10, 2015 at 6:41 PM, Timothy
From: Kristian Høgsberg k...@bitplanet.net
This will be used to identify buffer variables inside shader storage
buffer objects, which are very similar to uniforms except for a few
differences, most important of which is that they are writable.
Since buffer variables are so similar to uniforms,
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/nir/glsl_to_nir.cpp | 4
src/glsl/nir/nir.h | 1 +
src/glsl/nir/nir_lower_atomics.c | 3 ++-
src/glsl/nir/nir_lower_io.c | 9 ++---
src/glsl/nir/nir_print.c | 5 +++--
Since this now checks if a variable is inside a uniform or a shader
storage block.
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/ast_to_hir.cpp| 2 +-
src/glsl/ir.h | 5 +++--
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
Section 4.3.7 Buffer Variables of the GLSL 4.30 spec:
Buffer variables cannot have initializers.
v2:
- Rewrite error message (Jordan)
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
Reviewed-by: Jordan Justen
Shouldn't this stuff be in src/util?
Marek
On Wed, Jul 8, 2015 at 11:07 PM, Ben Widawsky
benjamin.widaw...@intel.com wrote:
WARNING: No perf data, please keep reading though)
This implements the suggestion provided by the paper, Fast USWC to WB Memory
Copy
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
Section 4.3.7 Buffer Variables, GLSL 4.30 spec:
Buffer variables may only be declared inside interface blocks
(section 4.3.9 “Interface Blocks”), which are then referred to as
shader storage blocks. It is a compile-time error to declare buffer
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
v2:
- Set MaxShaderStorageBlocks to 8.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/main/context.c | 2 ++
src/mesa/main/mtypes.h | 3 +++
2 files changed, 5
Since the backing storage for these is shared we cannot ensure that
the value won't change by writes from other threads. Normally SSBO
accesses are not guaranteed to be syncronized with other threads,
except when memoryBarrier is used. So, we might be able to optimize
some SSBO accesses, but for
Since the backing storage for these is shared we cannot ensure that
the value won't change by writes from other threads. Normally SSBO
accesses are not guaranteed to be syncronized with other threads,
except when memoryBarrier is used. So, we might be able to optimize
some SSBO accesses, but for
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/link_uniform_block_active_visitor.cpp | 1 +
src/glsl/link_uniform_block_active_visitor.h | 1 +
src/glsl/link_uniform_blocks.cpp | 4
src/mesa/main/mtypes.h | 5 +
4 files changed,
If we kill dead assignments we lose the buffer writes.
Also, we never kill UBO declarations even if they are never referenced
by the shader, they are always considered active. Although the spec
does not seem say this specifically for SSBOs, it is probably implied
since SSBOs are pretty much the
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/main/bufferobj.c | 56 +++
1 file changed, 56 insertions(+)
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index 0a9ffe4..c3548b5 100644
---
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/main/bufferobj.c | 37 +
1 file changed, 37 insertions(+)
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index c3548b5..4e25a72 100644
--- a/src/mesa/main/bufferobj.c
+++
v2:
- Add space before const (Jordan)
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/main/bufferobj.c | 142 ++
src/mesa/main/mtypes.h| 7 +++
2 files changed, 149 insertions(+)
diff --git a/src/mesa/main/bufferobj.c
From: Kristian Høgsberg k...@bitplanet.net
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/link_interface_blocks.cpp | 15 ---
src/glsl/link_uniform_initializers.cpp | 3 ++-
src/glsl/link_uniforms.cpp | 8 +---
src/glsl/linker.cpp
Jason Ekstrand ja...@jlekstrand.net writes:
On Jul 9, 2015 7:57 AM, Francisco Jerez curroje...@riseup.net wrote:
We were passing src0 alpha and oMask in reverse order. There seems to
be no good way to pass them in the correct order to the new-style
LOAD_PAYLOAD (how surprising) because src0
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
Extend the existing lower_ubo_reference pass to also detect SSBO writes
and lower them to __intrinsic_store_ssbo intrinsics.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
src/glsl/lower_ubo_reference.cpp | 441
Since the backing storage for these is shared we cannot ensure that
the value won't change by writes from other threads. Normally SSBO
accesses are not guaranteed to be syncronized with other threads,
except when memoryBarrier is used. So, we might be able to optimize
some SSBO accesses, but for
Am 10.07.2015 um 05:44 schrieb Michel Dänzer:
On 10.07.2015 05:13, Emil Velikov wrote:
The original code only half considered hyperz as an option. As per
previous commit major != 2 cannot occur we can simply things, and
allow users to set the option if they choose to do so.
Signed-off-by:
well, freedreno and vmware ddx still use XA.. they both could probably
be ported to use glamor instead, but that hasn't been done yet..
At least for freedreno with upstream drm/kms driver, one can just use
-modesetting ddx instead. But that doesn't work w/ android fbdev
driver. I need to check
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
Extend the existing lower_ubo_reference pass to also detect SSBO loads
and lower them to __intrinsic_load_ssbo intrinsics.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
---
src/glsl/lower_ubo_reference.cpp | 73
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Wed, Jul 8, 2015 at 10:34 PM, Rob Clark robdcl...@gmail.com wrote:
From: Rob Clark robcl...@freedesktop.org
No where was it spelled out that the state tracker may expect the pipe
driver to unref the old fence.
Signed-off-by: Rob Clark
As discussed with Jordan, this v3-part1 series contains a good part of the
frontend stuff (most of which has already been reviewed). The idea is to get
this landed ahead, since some of the remaining patches (specifically the i965
backend stuff) depends on ther patches from Curro that have not
v2:
- Fix error message (Jordan)
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/main/bufferobj.c | 110 ++
1 file changed, 110 insertions(+)
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index 27638ca..0a9ffe4
Otherwise we can lose writes into the buffers backing the variables.
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/opt_tree_grafting.cpp | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/glsl/opt_tree_grafting.cpp
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/glcpp/glcpp-parse.y| 3 ++
src/glsl/glsl_parser_extras.cpp | 63 +
v2:
- Remove the extra spaces (Jordan)
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/main/bufferobj.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index 2d70f7b..ee920a9 100644
---
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
Due to GL_ARB_shader_storage_buffer_object extension, shader storage blocks
have the same limitations as uniform blocks.
This patch fixes the corresponding error messages.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
This includes the array of bindings, the current buffer bound to the
GL_SHADER_STORAGE_BUFFER target and a set of general limits and default
values for shader storage buffers.
v2:
- Use spec values for the new defined constants (Jordan)
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
v2:
- Fix indention, used tabs instead of whitespaces. (Jordan)
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/mesa/main/bufferobj.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index c5d4ada..2d70f7b
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
See GLSL 4.30 spec, section 4.4.5 Uniform and Shader Storage Block
Layout Qualifiers.
v2:
- Add whitespace in an error message. Delete period '.' at the end of that
error message (Jordan).
Signed-off-by: Samuel Iglesias Gonsalvez
From: Kristian Høgsberg k...@bitplanet.net
This is used to identify shader storage buffer interface blocks where
buffer variables are declared.
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/ast.h | 1 +
src/glsl/ast_to_hir.cpp | 14 ++
On Fri, 2015-07-10 at 12:13 +0200, Iago Toral Quiroga wrote:
As discussed with Jordan, this v3-part1 series contains a good part of the
frontend stuff (most of which has already been reviewed). The idea is to get
this landed ahead, since some of the remaining patches (specifically the i965
I wonder... do we still need XA considering that everybody can just
use glamor instead?
Marek
On Wed, Jul 8, 2015 at 7:39 PM, Rob Clark robdcl...@gmail.com wrote:
From: Rob Clark robcl...@freedesktop.org
XA was never unref'ing last_fence in the various call paths to
pipe-flush(). Add this
Events can be added to an OpenCL command queue concurrently from multiple
threads, but pipe_context bjects are not threadsafe. The threadsafe
wrappers protect all pipe_context function calls with a mutex, so we
can safely use them with multiple threads.
v2:
- Don't use wrapper for pipe_screen.
On Wed, Jul 8, 2015 at 5:04 PM, Ben Widawsky
benjamin.widaw...@intel.com wrote:
Certain compressed formats require this setting. The docs don't go into much
detail as to why it's needed exactly.
This patch introduces no piglit regressions on gen9 (bsw is untested). Note
that
the SKL
On Fri, Apr 3, 2015 at 10:04 AM, Jason Ekstrand ja...@jlekstrand.net wrote:
On Fri, Apr 3, 2015 at 9:46 AM, Matt Turner matts...@gmail.com wrote:
On Fri, Apr 3, 2015 at 1:07 AM, Jordan Justen jordan.l.jus...@intel.com
wrote:
On 2015-04-02 20:56:15, Jason Ekstrand wrote:
---
The util/hash_table was intended to be a fast hash table
replacement for the program/hash_table see 35fd61bd99c1 and 72e55bb6888ff.
This replaces some more uses of the old hash table and also
destroys the interface_types hash table when _mesa_glsl_release_types()
is called which wasn't previously
These wrappers can be used by state trackers to ensure threadsafe access
to pipe_context objects.
v2:
- Don't add wrappers for pipe_screen.
- Build system cleanups
CC: 10.6 mesa-sta...@lists.freedesktop.org
---
configure.ac | 1 +
pipe_context::flush() can return a NULL fence if the queue is already
empty, so we should not assume that an event with a NULL fence
has the status of CL_QUEUED.
CC: 10.6 mesa-sta...@lists.freedesktop.org
---
src/gallium/state_trackers/clover/core/event.cpp | 7 ---
On 06/30/2015 06:58 PM, Jason Ekstrand wrote:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
This patch makes public the is_scalar_shader_stage() method in brw_shader,
and
renames it to brw_compiler_is_scalar_shader_stage(). The plan is to later
reuse it
in
https://bugs.freedesktop.org/show_bug.cgi?id=91292
Neil Roberts n...@linux.intel.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
For Evergreen (the first DX11 radeon), which only supports 12 writable
resources, we'll only expose 8 images, 8 shader storage buffers and 8
atomic counter buffers. There will be a CAP saying that only a total
of 12 is allowed to be used by a shader and all bindings slots will
thus be considered
This looks all pretty reasonable, though I can't really figure out if we
could translate from d3d11 to that. In particular, it's UAVs for all of
buffers and images, and I'm not entirely sure if it's actually possible
to figure out which UAV entries are a shader buffer and which ones are a
shader
https://bugs.freedesktop.org/show_bug.cgi?id=91292
Neil Roberts n...@linux.intel.com changed:
What|Removed |Added
CC||n...@linux.intel.com
On Thu, Jul 9, 2015 at 4:11 PM, Francisco Jerez curroje...@riseup.net wrote:
Ilia Mirkin imir...@alum.mit.edu writes:
FYI there's already a lowering pass that does this in the GLSL IR
(CARRY_TO_ARITH in lower_instructions). Perhaps the right place to do
this is NIR though, just wanted to let
https://bugs.freedesktop.org/show_bug.cgi?id=91292
Bug ID: 91292
Summary: glVertexAttribDivisor not working Linux 10.5.8 Intel
HD 5500?
Product: Mesa
Version: 10.5
Hardware: x86-64 (AMD64)
OS: Linux (All)
https://bugs.freedesktop.org/show_bug.cgi?id=91292
--- Comment #1 from Hugh Fisher lara...@ozemail.com.au ---
Created attachment 117030
-- https://bugs.freedesktop.org/attachment.cgi?id=117030action=edit
Expected output from running program
--
You are receiving this mail because:
You are the
On Thu, Jul 9, 2015 at 8:43 AM, Dave Airlie airl...@gmail.com wrote:
From: Glenn Kennard glenn.kenn...@gmail.com
This adds the main chunk of the geometry shader multiple stream
support to the r600 driver.
Glenn wrote the original pass, and I took his code and hacked
it into a working state.
On 10 July 2015 at 13:18, Roland Scheidegger srol...@vmware.com wrote:
Am 10.07.2015 um 05:44 schrieb Michel Dänzer:
On 10.07.2015 05:13, Emil Velikov wrote:
The original code only half considered hyperz as an option. As per
previous commit major != 2 cannot occur we can simply things, and
All of these are already defined in the headers provided.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/main/eglapi.c | 36 +---
src/egl/main/eglapi.h | 24
src/egl/main/eglfallbacks.c | 6 --
3 files
Cc: Axel Davy axel.d...@ens.fr
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
configure.ac| 2 +-
src/egl/drivers/dri2/platform_wayland.c | 20 ++--
2 files changed, 3 insertions(+), 19 deletions(-)
diff --git a/configure.ac b/configure.ac
Matt Turner matts...@gmail.com writes:
On Fri, Jul 10, 2015 at 10:06 AM, Francisco Jerez curroje...@riseup.net
wrote:
Booleans are represented as 0/-1 on modern hardware which means we can
just negate them to convert them into a numeric type. Negation has
the benefit that it can be
On Fri, Jul 10, 2015 at 10:06 AM, Francisco Jerez curroje...@riseup.net wrote:
Booleans are represented as 0/-1 on modern hardware which means we can
just negate them to convert them into a numeric type. Negation has
the benefit that it can be implemented using a source modifier which
can
On Fri, Jul 10, 2015 at 10:49 AM, Emil Velikov emil.l.veli...@gmail.com wrote:
The macro HAVE_MKOSTEMP was never defined.
Cc: Axel Davy axel.d...@ens.fr
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/drivers/dri2/platform_wayland.c | 6 --
1 file changed, 6
On Fri, Jul 10, 2015 at 4:20 AM, Marek Olšák mar...@gmail.com wrote:
Shouldn't this stuff be in src/util?
I guess? I'm not opposed to moving it, if other drivers can make use
of it, but I don't want this patch to do the move.
I wrote it in November 2013 and src/util only came into being in
I'm sure this doesn't build. dri2_dpy is not declared and screen is int.
Marek
On Fri, Jul 10, 2015 at 7:46 PM, Emil Velikov emil.l.veli...@gmail.com wrote:
Inspired (copied) from Marek's commit for egl/x11
commit 0b56e23e7f3(egl/dri2: use the correct screen index)
Cc: 10.6
BEGIN_BATCH() and ADVANCE_BATCH() will contain do { and } while (0)
respectively to allow declaring local variables used by intervening
OUT_BATCH macros. As such, BEGIN_BATCH() and ADVANCE_BATCH() need to be
in the same control flow.
---
src/mesa/drivers/dri/i965/brw_draw.c | 4 ++--
1 file
Previously OUT_BATCH was just a macro around an inline function which
does
brw-batch.map[brw-batch.used++] = dword;
When making consecutive calls to intel_batchbuffer_emit_dword() the
compiler isn't able to recognize that we're writing consecutive memory
locations or that it doesn't need to
Inspired (copied) from Marek's commit for egl/x11
commit 0b56e23e7f3(egl/dri2: use the correct screen index)
Cc: 10.6 mesa-sta...@lists.freedesktop.org
Cc: Marek Olšák marek.ol...@amd.com
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/gallium/auxiliary/vl/vl_winsys_dri.c | 18
The macro HAVE_MKOSTEMP was never defined.
Cc: Axel Davy axel.d...@ens.fr
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/drivers/dri2/platform_wayland.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/src/egl/drivers/dri2/platform_wayland.c
Prompt at configure time if it's missing otherwise we'll fail later on
in the build. Remove ambiguous HAVE_LIBDRM guard.
Cc: 10.6 mesa-sta...@lists.freedesktop.org
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
configure.ac| 3 +++
src/egl/drivers/dri2/egl_dri2.c |
Move the code around rather than having it scattered. No functional
change.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/drivers/dri2/egl_dri2.c | 210 +++-
1 file changed, 102 insertions(+), 108 deletions(-)
diff --git
... and update the documentation to reflect reality.
null and gdi are gone, and surfaceless is a recent addition.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
configure.ac | 3 ---
docs/egl.html | 6 +++---
2 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/configure.ac
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/drivers/dri2/platform_wayland.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/src/egl/drivers/dri2/platform_wayland.c
b/src/egl/drivers/dri2/platform_wayland.c
index
https://bugs.freedesktop.org/show_bug.cgi?id=66346
--- Comment #21 from Emil Velikov emil.l.veli...@gmail.com ---
(In reply to Vinson Lee from comment #20)
(In reply to José Fonseca from comment #19)
(In reply to Vinson Lee from comment #18)
mesa: 0166b4c165271bd7525a91049e58e390cb596c60
Its uses of OUT_BATCH will need a local variable defined by BEGIN_BATCH.
Increases .text size by 528 bytes.
---
src/mesa/drivers/dri/i965/intel_blit.c | 55 --
1 file changed, 25 insertions(+), 30 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_blit.c
Its uses of OUT_BATCH will need a local variable defined by BEGIN_BATCH.
Increases .text size by 8 bytes.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c | 79 +
1 file changed, 36 insertions(+), 43 deletions(-)
diff --git
So that everything writing to the batch between BEGIN_BATCH() and
ADVANCE_BATCH() goes through OUT_BATCH.
---
src/mesa/drivers/dri/i965/intel_batchbuffer.c | 30 ++-
src/mesa/drivers/dri/i965/intel_batchbuffer.h | 34 ++-
2 files changed, 30
... and update the documentation to reflect reality.
null and gdi are gone, and surfaceless is a recent addition.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
configure.ac | 3 ---
docs/egl.html | 6 +++---
2 files changed, 3 insertions(+), 6 deletions(-)
diff --git a/configure.ac
Hi folks,
I've pushed the latest version of my shader-cache work to a branch named
shader-cache at:
git://people.freedesktop.org/~cworth/mesa
I've rebased this against the latest master branch and verified that it
at least compiles and works with at least some trivial test programs.
Zoltan Gilian zoltan.gil...@gmail.com writes:
Read-only and write-only image arguments are recognized and
distinguished.
Attributes of the image arguments are passed to the kernel as implicit
arguments.
Thanks, this looks much better. One thing that still seems kind of
unfortunate is the
On Fri, Jul 10, 2015 at 5:25 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Jul 9, 2015 7:57 AM, Francisco Jerez curroje...@riseup.net wrote:
We were passing src0 alpha and oMask in reverse order. There seems to
be no good way to pass them in
This gets rid of two no16() fall-backs and should allow better
scheduling of the generated IR. There are no uses of usubBorrow() or
uaddCarry() in shader-db so no changes are expected. However the
arb_gpu_shader5/execution/built-in-functions/fs-usubBorrow and
Booleans are represented as 0/-1 on modern hardware which means we can
just negate them to convert them into a numeric type. Negation has
the benefit that it can be implemented using a source modifier which
can easily be propagated into some other instruction. shader-db
results on HSW:
total
Jason Ekstrand ja...@jlekstrand.net writes:
On Fri, Jul 10, 2015 at 5:25 AM, Francisco Jerez curroje...@riseup.net
wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Jul 9, 2015 7:57 AM, Francisco Jerez curroje...@riseup.net wrote:
We were passing src0 alpha and oMask in reverse order.
When the edge flag element is enabled then the elements are slightly
reordered so that the edge flag is always the last one. This was
confusing the code to upload the 3DSTATE_VF_INSTANCING state because
that is uploaded with a separate loop which has an instruction for
each element. The indices
On 10/07/15 19:22, Matt Turner wrote:
On Fri, Jul 10, 2015 at 10:49 AM, Emil Velikov emil.l.veli...@gmail.com
wrote:
The macro HAVE_MKOSTEMP was never defined.
Cc: Axel Davy axel.d...@ens.fr
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/drivers/dri2/platform_wayland.c
On Thu, Jul 9, 2015 at 11:00 AM, Ben Widawsky
benjamin.widaw...@intel.com wrote:
While implementing the workaround in the previous patch I noticed things were
starting to get a bit messy. Since gen8 works differently enough from gen7, I
thought splitting it out with be good.
While here, get
https://bugs.freedesktop.org/show_bug.cgi?id=91290
Vinson Lee v...@freedesktop.org changed:
What|Removed |Added
Keywords||regression
On Wed, Jul 8, 2015 at 10:58 PM, Zhenyu Wang zhen...@linux.intel.com wrote:
NOP actually has no compact version, but we use it for instruction
alignment for compact kernel. Although it seems working on HW, it is
illegal and might not be valid for any future one.
This trys to get a temporary
On Fri, Jul 10, 2015 at 9:53 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Fri, Jul 10, 2015 at 5:25 AM, Francisco Jerez curroje...@riseup.net
wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Jul 9, 2015 7:57 AM, Francisco Jerez
On Fri, Jul 10, 2015 at 12:03:54PM -0700, Matt Turner wrote:
On Thu, Jul 9, 2015 at 11:00 AM, Ben Widawsky
benjamin.widaw...@intel.com wrote:
While implementing the workaround in the previous patch I noticed things
were
starting to get a bit messy. Since gen8 works differently enough from
On Wed, Jul 1, 2015 at 7:28 PM, Kenneth Graunke kenn...@whitecape.org wrote:
Paul's original code had emit_control_data_bits() skip the URB write if
vertex_count was 0. This meant wrapping every control data write in a
conditional write.
We accumulate control data bits in a single UD
On 10 July 2015 at 19:33, Marek Olšák mar...@gmail.com wrote:
I'm sure this doesn't build. dri2_dpy is not declared and screen is int.
In all honesty I wrote it at 3am, but that is no excuse for such a
goof-up. A correct version should be out.
Thanks,
Emil
Fixes the following piglits:
fbo-alphatest-nocolor
fbo-alphatest-nocolor-ff
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
Cc: mesa-sta...@lists.freedesktop.org
---
The nv50 bits need testing, only have a GK208 on-hand. Will be sure to test
before pushing.
On 2015-07-10 03:13:44, Iago Toral Quiroga wrote:
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
Extend the existing lower_ubo_reference pass to also detect SSBO writes
and lower them to __intrinsic_store_ssbo intrinsics.
Signed-off-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
On 2015-07-10 03:13:45, Iago Toral Quiroga wrote:
From: Samuel Iglesias Gonsalvez sigles...@igalia.com
Extend the existing lower_ubo_reference pass to also detect SSBO loads
and lower them to __intrinsic_load_ssbo intrinsics.
Am 10.07.2015 um 19:41 schrieb Emil Velikov:
On 10 July 2015 at 13:18, Roland Scheidegger srol...@vmware.com wrote:
Am 10.07.2015 um 05:44 schrieb Michel Dänzer:
On 10.07.2015 05:13, Emil Velikov wrote:
The original code only half considered hyperz as an option. As per
previous commit major
On Wed, Jul 1, 2015 at 11:51 AM, Connor Abbott cwabbo...@gmail.com wrote:
sources with file == HW_REG get all their information from the
fixed_hw_reg field, so we need to get the stride and type from there
when computing the size.
Signed-off-by: Connor Abbott connor.w.abb...@intel.com
---
Inspired (copied) from Marek's commit for egl/x11
commit 0b56e23e7f3(egl/dri2: use the correct screen index)
v2: Fix copy/pasta errors.
Cc: 10.6 mesa-sta...@lists.freedesktop.org
Cc: Marek Olšák marek.ol...@amd.com
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
On 2015-06-03 00:01:23, Iago Toral Quiroga wrote:
Since these are a special kind of UBOs we emit them together reusing the
same infrastructure, however, we use a RAW surface so we can reuse
existing untyped read/write/atomic messages which include a pixel mask
header that we need to set to
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
On 2015-06-03 00:01:24, Iago Toral Quiroga wrote:
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
Rob Clark robdcl...@gmail.com writes:
well, freedreno and vmware ddx still use XA.. they both could probably
be ported to use glamor instead, but that hasn't been done yet..
At least for freedreno with upstream drm/kms driver, one can just use
-modesetting ddx instead. But that doesn't work
1 - 100 of 105 matches
Mail list logo