Tapani Pälli tapani.pa...@intel.com writes:
Hi;
Here's additions to patches I sent earlier (reviewed by curro), these just
set EmitNoIndirectSampler on when there's no ARB_gpu_shader5 support liked
discussed.
Original mail thread:
This just seems super-hacky. Given that we set _MaintainTexEnvProgram
it seems reasonable to assume there will always be one there. IMHO the
core should be fixed to not allow this situation to occur in the first
place...
On Mon, Jun 29, 2015 at 10:08 AM, Marek Olšák mar...@gmail.com wrote:
Ping
From: Marek Olšák marek.ol...@amd.com
Absolute timeouts are used with the amdgpu kernel driver.
It also makes waiting for several variables and fences at the same time
easier (the timeout doesn't have to be recalculated after every wait call).
---
src/gallium/auxiliary/os/os_time.c | 41
From: Marek Olšák marek.ol...@amd.com
The reason might be that no commands have been submitted before the flush
and the GPU is idle.
---
src/mesa/state_tracker/st_cb_syncobj.c | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git
Instead of using symbol table, build mask by inspecting IR. This
change is required by further patches to move resource list creation
to happen later when symbol table does not exist anymore.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/glsl/linker.cpp | 11 ---
1 file
These changes move program resource list creation to happen later
after backend LinkShader hook has been called. So, there are no changes
in overall logic but functionality is called only after LinkShader that
is still able to remove some variables considered dead.
No Piglit regressions observed.
Resource list can be created properly only after LinkShader hook
has been called to make sure all dead variables have been removed.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
https://bugs.freedesktop.org/show_bug.cgi?id=90925
---
src/glsl/linker.cpp | 4
This is required so that we can move resource list creation
to happen later.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/glsl/linker.cpp | 2 +-
src/glsl/program.h | 4
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
On 29/06/15 15:35, Tapani Pälli wrote:
Instead of using symbol table, build mask by inspecting IR. This
change is required by further patches to move resource list creation
to happen later when symbol table does not exist anymore.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
I can imagine an app calling glBlitFramebuffer only. Why would the
driver generate a fixed-func fragment program then?
Marek
On Mon, Jun 29, 2015 at 4:10 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
This just seems super-hacky. Given that we set _MaintainTexEnvProgram
it seems reasonable to
On 29/06/15 13:26, Francisco Jerez wrote:
Davin McCall dav...@davmac.org writes:
On 29/06/15 10:40, Francisco Jerez wrote:
Davin McCall dav...@davmac.org writes:
On 26/06/15 14:53, Francisco Jerez wrote:
[...]
Your first approach seemed quite reasonable IMHO. Were you able to
measure
Ping
On Fri, Jun 26, 2015 at 11:01 AM, Marek Olšák mar...@gmail.com wrote:
From: Marek Olšák marek.ol...@amd.com
No states are initialized at that point yet.
Cc: 10.5 10.6 mesa-sta...@lists.freedesktop.org
---
src/mesa/state_tracker/st_atom_constbuf.c | 14 --
Davin McCall dav...@davmac.org writes:
On 26/06/15 14:53, Francisco Jerez wrote:
[...]
Your first approach seemed quite reasonable IMHO. Were you able to
measure any performance regression from it?
Thanks.
When I run an apitrace replay of a Dota 2 trace [1] with
Reviewed-by: Marek Olšák marek.ol...@amd.com
The fix for amdgpu should be in libdrm_amdgpu / amdgpu_device.c.
Marek
On Sun, Jun 28, 2015 at 7:49 AM, Mario Kleiner
mario.kleiner...@gmail.com wrote:
Thanks for the review Ilia. Just saw that the new amdgpu winsys driver will
need the same fix,
On 29/06/15 10:40, Francisco Jerez wrote:
Davin McCall dav...@davmac.org writes:
On 26/06/15 14:53, Francisco Jerez wrote:
[...]
Your first approach seemed quite reasonable IMHO. Were you able to
measure any performance regression from it?
Thanks.
When I run an apitrace replay of a Dota
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Mon, Jun 29, 2015 at 9:35 AM, Tapani Pälli tapani.pa...@intel.com wrote:
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/mesa/state_tracker/st_context.c | 5 +
1 file changed, 5 insertions(+)
diff --git
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Mon, Jun 29, 2015 at 9:21 AM, Dave Airlie airl...@gmail.com wrote:
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/gallium/auxiliary/tgsi/tgsi_info.c | 37
++
1 file changed, 37 insertions(+)
Davin McCall dav...@davmac.org writes:
On 29/06/15 10:40, Francisco Jerez wrote:
Davin McCall dav...@davmac.org writes:
On 26/06/15 14:53, Francisco Jerez wrote:
[...]
Your first approach seemed quite reasonable IMHO. Were you able to
measure any performance regression from it?
Don't worry about the AoS stuff. Only meant to do simple things.
Looks good overall, I guess it makes sense to not split execution too
(so you'd have native hw vector size there), llvm should handle that
pretty well these days (the sse intrinsics won't get used that way
probably (though there's a
ping?
2015-06-21 23:06 GMT+08:00 Boyan Ding boyan.j.d...@gmail.com:
This series of patches add support for EGL_KHR_create_context when
using swrast. The first patch moves context_attrib filling into a
separate function for reuse in swrast. The second one uses
createContextAttribs in swrast
In commit c61bc6ed8 (util: port _mesa_strto[df] to C) we renamed
strtod.cpp - strtod.c.
Applying this patch on top of a tree that has already built strtod.cpp
will yield the error in the subject. To fix, simply run
sed -i -e 's/strtod\.cpp/strtod.c/' src/util/.deps/libmesautil_la-strtod.Plo
in
On Mon, Jun 29, 2015 at 6:40 PM, Matt Turner matts...@gmail.com wrote:
Thanks!
Pushed, and sent a note describing how to fix the problems caused by
renaming strtod.cpp - strtod.c.
Thanks a lot :)
___
mesa-dev mailing list
Thanks!
Pushed, and sent a note describing how to fix the problems caused by
renaming strtod.cpp - strtod.c.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
It couldn't have worked anyway. There were calls to undefined functions.
---
src/glx/packrender.h | 46 --
src/glx/packsingle.h | 56
2 files changed, 102 deletions(-)
diff --git a/src/glx/packrender.h
On 29/06/15 13:26, Francisco Jerez wrote:
Davin McCall dav...@davmac.org writes:
On 29/06/15 10:40, Francisco Jerez wrote:
Davin McCall dav...@davmac.org writes:
On 26/06/15 14:53, Francisco Jerez wrote:
[...]
Your first approach seemed quite reasonable IMHO. Were you able to
measure
On Fri, Jun 26, 2015 at 01:23:41PM -0700, Anuj Phogat wrote:
On Mon, Jun 22, 2015 at 5:23 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
On Mon, Jun 22, 2015 at 2:53 PM, Ben Widawsky b...@bwidawsk.net wrote:
On Wed, Jun 10, 2015 at 03:30:47PM -0700, Anuj Phogat wrote:
Buffers with Yf/Ys
On Thu, Jun 25, 2015 at 5:45 AM, Topi Pohjolainen
topi.pohjolai...@intel.com wrote:
Current logic re-writes the same data when existing data is found.
Not that this actually matters at the moment in practice, the
contraint for finding matching data is too severe to ever allow
data to be shared
On Thu, Jun 25, 2015 at 5:45 AM, Topi Pohjolainen
topi.pohjolai...@intel.com wrote:
and simplify the interface to take directly the size and to return
the offset. The routine does nothing more than allocate, it doesn't
upload anything.
CC: Kenneth Graunke kenn...@whitecape.org
Signed-off-by:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
It basically allocates registers local to a function in a nir_locals map,
then emits all its control-flow blocks.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89580
---
src/mesa/drivers/dri/i965/brw_vec4.h
On Mon 29 Jun 2015, Matt Turner wrote:
---
src/mesa/main/context.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
Reviewed-by: Chad Versace chad.vers...@intel.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
On Thu, Jun 25, 2015 at 5:45 AM, Topi Pohjolainen
topi.pohjolai...@intel.com wrote:
Items in the program cache consist of three things: key, the data
representing the instructions and auxiliary data representing
uniform storage. The data consisting of instructions is stored into
a drm buffer
Kenneth Graunke kenn...@whitecape.org writes:
On Wednesday, June 03, 2015 09:21:11 PM Topi Pohjolainen wrote:
Shader programs are cached per stage (FS, VS, GS) using the
corresponding shader source identifier and compile time choices
as key. However, one not only stores the program binary but
Jason Ekstrand ja...@jlekstrand.net writes:
Reviewed-by: Iago Toral Quiroga ito...@igalia.com
Reviewed-by: Topi Pohjolainen topi.pohjolai...@intel.com
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
On 30 June 2015 at 00:58, Roland Scheidegger srol...@vmware.com wrote:
Don't worry about the AoS stuff. Only meant to do simple things.
Looks good overall, I guess it makes sense to not split execution too
(so you'd have native hw vector size there), llvm should handle that
pretty well these
https://bugs.freedesktop.org/show_bug.cgi?id=91149
Bug ID: 91149
Summary: make check optimization-test regression
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: All
Status: NEW
Keywords:
https://bugs.freedesktop.org/show_bug.cgi?id=91149
--- Comment #3 from Erik Faye-Lund kusmab...@gmail.com ---
Created attachment 116806
-- https://bugs.freedesktop.org/attachment.cgi?id=116806action=edit
Fix for test-regression
Yeah, seems like there were another user of the
On 06/29/2015 11:22 PM, Jason Ekstrand wrote:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
The NIR-vec4 pass will be activated if ALL the following conditions are met:
* INTEL_USE_NIR environment variable is defined and is positive (1 or true)
* The stage is
On Mon, Jun 29, 2015 at 2:49 PM, Eduardo Lima Mitev el...@igalia.com wrote:
On 06/29/2015 11:22 PM, Jason Ekstrand wrote:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
The NIR-vec4 pass will be activated if ALL the following conditions are
met:
* INTEL_USE_NIR
On Friday, June 26, 2015 10:06:18 AM Eduardo Lima Mitev wrote:
The NIR-vec4 pass will be activated if ALL the following conditions are met:
* INTEL_USE_NIR environment variable is defined and is positive (1 or true)
* The stage is vertex shader
* The HW generation is either SandyBridge
On 06/29/2015 10:20 PM, Kenneth Graunke wrote:
On Friday, June 26, 2015 10:06:18 AM Eduardo Lima Mitev wrote:
The NIR-vec4 pass will be activated if ALL the following
conditions are met:
* INTEL_USE_NIR environment variable is defined and is positive
(1 or true) * The stage is vertex shader
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
The NIR-vec4 pass will be activated if ALL the following conditions are met:
* INTEL_USE_NIR environment variable is defined and is positive (1 or true)
* The stage is vertex shader
* The HW generation is either
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
This implementation sets up a map of input variable offsets to source
registers
that are already initialized with the corresponding register offset.
This map will then be queried when processing load_input intrinsic
This is the third iteration of a patch to resolve a strict aliasing
problem in the exec_list structure.
At the suggestion of Francisco Jerez this is a return to the original
(v1) style of the patch, which replaces the three 'exec_node *' members
in the list with two 'struct exec_node'
https://bugs.freedesktop.org/show_bug.cgi?id=91149
--- Comment #1 from Erik Faye-Lund kusmab...@gmail.com ---
Without seeing what the actual problem is (it's lacking from the report), my
guess would be that Matt's work-around from
caedq38gjr6azkry4upepsfhttcneqvix0pndvmxae2nsr8h...@mail.gmail.com
https://bugs.freedesktop.org/show_bug.cgi?id=91149
--- Comment #2 from Vinson Lee v...@freedesktop.org ---
Reproduced with a clean checkout and build.
== Testing optimization passes ==
Testing ./lower_jumps/lower_returns_main_false.opt_test...FAIL
Traceback (most recent call last):
On Mon, 2015-06-29 at 15:21 +0900, Michel Dänzer wrote:
On 25.06.2015 21:42, Marek Olšák wrote:
Gallium should be alright. We'll let you know if we find a regression,
but I don't think there will be any.
Famous last words. :) This change broke the piglit test
On 25.06.2015 21:42, Marek Olšák wrote:
Gallium should be alright. We'll let you know if we find a regression,
but I don't think there will be any.
Famous last words. :) This change broke the piglit test
spec@ext_texture_integer@fbo_integer_readpixels_sint_uint for me with
the radeonsi driver:
Reviewed-by: Iago Toral Quiroga ito...@igalia.com
On Fri, 2015-06-26 at 13:15 -0700, Anuj Phogat wrote:
Currently used ctx-_ImageTransferState check is not sufficient
because it doesn't include the read color clamping enabled with
GL_CLAMP_READ_COLOR. So, use the helper function
On 2015-06-24 07:36:24, Iago Toral wrote:
On Wed, 2015-06-24 at 15:43 +0300, Francisco Jerez wrote:
AFAICT the reason why this (and many of the other changes in GLSL
optimization passes) is needed is because SSBO loads have been
implemented as ir_expression nodes instead of being lowered
This adds support for ARB_gpu_shader_fp64 and ARB_vertex_attrib_64bit to
llvmpipe.
Two things that don't mix well are SoA and doubles, see
emit_fetch_double, and emit_store_double_chan in this.
I've also had to split emit_data.chan, to add src_chan,
which can be different for doubles.
Open
From: Dave Airlie airl...@redhat.com
Signed-off-by: Dave Airlie airl...@redhat.com
---
docs/GL3.txt | 4 ++--
docs/relnotes/10.7.0.html | 2 ++
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 54e4574..ce3b134 100644
--- a/docs/GL3.txt
Before considering radeonsi, I felt llvmpipe support would
be a good stepping stone.
Things are messy with SoA, and I've no idea how test AoS,
so I left it alone.
but this set does work and does pass the tests and doesn't regress.
Dave.
___
mesa-dev
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/gallium/auxiliary/tgsi/tgsi_info.c | 37 ++
1 file changed, 37 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_info.c
b/src/gallium/auxiliary/tgsi/tgsi_info.c
index 9295311..4b16ef3 100644
---
These checks were in Mesa prior to commit fbba25bba, but they were
not necessary for the purpose that Mesa intended (check if we could
resolve ReadPixels via memcpy), so that commit took them away.
Unfortunately, it seems that some Gallium drivers rely on these
checks to make the decision of
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp
b/src/mesa/drivers/dri/i965/brw_shader.cpp
index 32c4013..3e3d78b 100644
---
Hi;
Here's additions to patches I sent earlier (reviewed by curro), these just
set EmitNoIndirectSampler on when there's no ARB_gpu_shader5 support liked
discussed.
Original mail thread:
http://lists.freedesktop.org/archives/mesa-dev/2015-June/086049.html
All patches applied on master:
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/mesa/state_tracker/st_context.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/state_tracker/st_context.c
b/src/mesa/state_tracker/st_context.c
index ed9ed0f..62a0fbe 100644
--- a/src/mesa/state_tracker/st_context.c
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/mesa/drivers/dri/i915/i915_context.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/drivers/dri/i915/i915_context.c
b/src/mesa/drivers/dri/i915/i915_context.c
index 42ea54e..57b033c 100644
---
From: Christian König christian.koe...@amd.com
We need to check what the 3D pipe is able to handle for the mixer, not what
the decoder is able to decode. This fixes output of resolutions like 720x1280.
Signed-off-by: Christian König christian.koe...@amd.com
CC: mesa-sta...@lists.freedesktop.org
Mesa 10.6.1 is now available.
This release includes core mesa and glsl patches (amonst which
a Dota2 Reborn bugfix) affecting all dri drivers, nouveau specific
fixes and a selection of shared-glapi commits - from build fixes, to
ones ensuring that the dri modules can be loaded.
Anuj Phogat (4):
From: Christian König christian.koe...@amd.com
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=90728
Signed-off-by: Christian König christian.koe...@amd.com
CC: mesa-sta...@lists.freedesktop.org
---
src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 24
On Mon, 2015-06-29 at 00:11 -0700, Jordan Justen wrote:
On 2015-06-24 07:36:24, Iago Toral wrote:
On Wed, 2015-06-24 at 15:43 +0300, Francisco Jerez wrote:
AFAICT the reason why this (and many of the other changes in GLSL
optimization passes) is needed is because SSBO loads have been
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
From: Alejandro Piñeiro apinhe...@igalia.com
Similar to other variable setups, system values will initialize the
corresponding register inside a 'nir_system_values' map, which will then
be queried later when
https://bugs.freedesktop.org/show_bug.cgi?id=84570
--- Comment #36 from Andreas Hartmetz ahartm...@gmail.com ---
Forgot to mention: Ubuntu kernel 3.19.
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
From: Iago Toral Quiroga ito...@igalia.com
This is based on similar code existing in vec4_visitor. It builds the
uniform register file iterating through each uniform variable. It
also stores the index of each register
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
New method brw_writemask_for_size() will return a writemask with the first
'size'
components activated.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89580
---
src/mesa/drivers/dri/i965/brw_reg.h | 9
https://bugs.freedesktop.org/show_bug.cgi?id=84570
Andreas Hartmetz ahartm...@gmail.com changed:
What|Removed |Added
Status|RESOLVED|REOPENED
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
From: Alejandro Piñeiro apinhe...@igalia.com
The new virtual method is more flexible, it has a signature:
dst_reg *make_reg_for_system_value(int location, const glsl_type *type);
so the current method will be
https://bugs.freedesktop.org/show_bug.cgi?id=77449
Bug 77449 depends on bug 84570, which changed state.
Bug 84570 Summary: Borderlands 2/Pre-Sequel: Constant frame rate drops while
playing; really bad with additionl lighting
https://bugs.freedesktop.org/show_bug.cgi?id=84570
What
This adds support for ARB_gpu_shader_fp64 and ARB_vertex_attrib_64bit to
llvmpipe.
Two things that don't mix well are SoA and doubles, see
emit_fetch_double, and emit_store_double_chan in this.
I've also had to split emit_data.chan, to add src_chan,
which can be different for doubles.
It
On 30 June 2015 at 09:36, Roland Scheidegger srol...@vmware.com wrote:
Am 29.06.2015 um 22:18 schrieb Dave Airlie:
On 30 June 2015 at 00:58, Roland Scheidegger srol...@vmware.com wrote:
Don't worry about the AoS stuff. Only meant to do simple things.
Looks good overall, I guess it makes sense
Am 29.06.2015 um 22:18 schrieb Dave Airlie:
On 30 June 2015 at 00:58, Roland Scheidegger srol...@vmware.com wrote:
Don't worry about the AoS stuff. Only meant to do simple things.
Looks good overall, I guess it makes sense to not split execution too
(so you'd have native hw vector size
On 06/30/2015 01:49 AM, Jason Ekstrand wrote:
As a general comment, please remove the Bugzilla tags from the
commits. This is a new feature, not a bugfix. If you want to leave a
reference to the bug, pick a commit and put it there but it doesn't
need to be in all of them.
--Jason
Ok,
On Jun 29, 2015 11:21 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
Reviewed-by: Iago Toral Quiroga ito...@igalia.com
Reviewed-by: Topi Pohjolainen topi.pohjolai...@intel.com
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 6 ++
1 file
Good work guys! I've started reviewing but review will probably take
a few days so please be patient.
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com wrote:
Hello,
This series adds a new vec4 backend for i965 based on NIR. It is the result
of working on
As a general comment, please remove the Bugzilla tags from the
commits. This is a new feature, not a bugfix. If you want to leave a
reference to the bug, pick a commit and put it there but it doesn't
need to be in all of them.
--Jason
On Mon, Jun 29, 2015 at 4:29 PM, Jason Ekstrand
Hello,
Since git 2.3, there have been a number of new fsck options added which
produce issues when I clone the repository
git://anongit.freedesktop.org/git/mesa/mesa
These new tests are enabled by default when using git fsck. I have been
testing with git version 2.4.4.409.g5b1d901 and thought
77 matches
Mail list logo