Re: [Mesa-dev] [PATCH v2] glsl: add packed varyings to program resource list

On Fri, Sep 11, 2015 at 2:18 AM, Tapani Pälli wrote: > On 09/11/2015 09:15 AM, Ilia Mirkin wrote: >> >> On Fri, Sep 11, 2015 at 2:12 AM, Tapani Pälli >> wrote: >>> >>> This makes sure that user is still able to query properties about >>> variables that have gotten packed by lower_packed_varyings

Re: [Mesa-dev] [PATCH v2] glsl: add packed varyings to program resource list

On 09/11/2015 09:15 AM, Ilia Mirkin wrote: On Fri, Sep 11, 2015 at 2:12 AM, Tapani Pälli wrote: This makes sure that user is still able to query properties about variables that have gotten packed by lower_packed_varyings pass. Fixes following OpenGL ES 3.1 test: ES31-CTS.program_interfac

Re: [Mesa-dev] [PATCH v2] glsl: add packed varyings to program resource list

On Fri, Sep 11, 2015 at 2:12 AM, Tapani Pälli wrote: > This makes sure that user is still able to query properties about > variables that have gotten packed by lower_packed_varyings pass. > > Fixes following OpenGL ES 3.1 test: >ES31-CTS.program_interface_query.separate-programs-vertex > > v2:

[Mesa-dev] [PATCH v2] glsl: add packed varyings to program resource list

This makes sure that user is still able to query properties about variables that have gotten packed by lower_packed_varyings pass. Fixes following OpenGL ES 3.1 test: ES31-CTS.program_interface_query.separate-programs-vertex v2: fix 'name included in packed list' check (Ilia Mirkin) Signed-of

Re: [Mesa-dev] [PATCH v5 01/70] mesa: set MAX_SHADER_STORAGE_BUFFERS to 15.

On Thu, 2015-09-10 at 15:17 -0400, Ilia Mirkin wrote: > On Thu, Sep 10, 2015 at 2:52 PM, Ian Romanick wrote: > > On 09/10/2015 10:45 AM, Ilia Mirkin wrote: > >> On Thu, Sep 10, 2015 at 9:35 AM, Iago Toral Quiroga > >> wrote: > >>> From: Samuel Iglesias Gonsalvez > >>> > >>> This patch sets the

Re: [Mesa-dev] [PATCH 07/11] nir/lower_vec_to_movs: Handle partially SSA shaders

On 09/10/2015 08:38 PM, Jason Ekstrand wrote: > On Thu, Sep 10, 2015 at 9:08 AM, Eduardo Lima Mitev wrote: >> On 09/10/2015 02:50 AM, Jason Ekstrand wrote: >>> --- >>> src/glsl/nir/nir_lower_vec_to_movs.c | 18 -- >>> 1 file changed, 16 insertions(+), 2 deletions(-) >>> >>> diff -

Re: [Mesa-dev] [RFC 1/2] nir: Add a new lowering pass nir_lower_vec_and_coalesce

On 09/10/2015 08:45 PM, Jason Ekstrand wrote: > On Wed, Sep 9, 2015 at 11:51 PM, Eduardo Lima Mitev wrote: >> On 09/09/2015 07:10 PM, Jason Ekstrand wrote: >>> >>> On Sep 8, 2015 23:27, "Eduardo Lima Mitev" >> > wrote: This pass will propagate the destination com

[Mesa-dev] [Bug 91898] src/util/mesa-sha1.c:250:25: fatal error: openssl/sha.h: No such file or directory

https://bugs.freedesktop.org/show_bug.cgi?id=91898 Vinson Lee changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH 2/2] glsl: add packed varyings to program resource list

On 09/11/2015 08:48 AM, Ilia Mirkin wrote: On Fri, Sep 11, 2015 at 1:43 AM, Tapani Pälli wrote: This makes sure that user is still able to query properties about variables that have gotten packed by lower_packed_varyings pass. Fixes following OpenGL ES 3.1 test: ES31-CTS.program_interfac

Re: [Mesa-dev] [PATCH v5 25/70] glsl: Add std430 related member functions to glsl_type class

On 10/09/15 20:13, Jordan Justen wrote: > On 2015-09-10 06:35:41, Iago Toral Quiroga wrote: >> From: Samuel Iglesias Gonsalvez >> >> They are used to calculate size, base alignment and array stride values >> for a glsl_type following std430 rules. >> >> Signed-off-by: Samuel Iglesias Gonsalvez

Re: [Mesa-dev] [PATCH 2/2] glsl: add packed varyings to program resource list

On Fri, Sep 11, 2015 at 1:43 AM, Tapani Pälli wrote: > This makes sure that user is still able to query properties about > variables that have gotten packed by lower_packed_varyings pass. > > Fixes following OpenGL ES 3.1 test: >ES31-CTS.program_interface_query.separate-programs-vertex > > Sig

[Mesa-dev] [Bug 91898] src/util/mesa-sha1.c:250:25: fatal error: openssl/sha.h: No such file or directory

https://bugs.freedesktop.org/show_bug.cgi?id=91898 --- Comment #3 from Tapani Pälli --- Vinson, is this still a problem? -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the bug. ___ mesa-dev mailing

[Mesa-dev] [PATCH 1/2] mesa: add packed_varyings list to gl_shader

This is required to store information about packed varyings, currently these variables get lost and cannot be retrieved later in sensible way for program interface queries. List will be utilized by next patch. Signed-off-by: Tapani Pälli --- src/glsl/lower_packed_varyings.cpp | 16 --

[Mesa-dev] [PATCH 2/2] glsl: add packed varyings to program resource list

This makes sure that user is still able to query properties about variables that have gotten packed by lower_packed_varyings pass. Fixes following OpenGL ES 3.1 test: ES31-CTS.program_interface_query.separate-programs-vertex Signed-off-by: Tapani Pälli --- src/glsl/linker.cpp | 60 ++

Re: [Mesa-dev] [RFC] nir: add lowering stage for user-clip-planes

On Thu, Sep 10, 2015 at 10:54 PM, Rob Clark wrote: > > On Sep 10, 2015 7:39 PM, "Jason Ekstrand" wrote: >> >> On Thu, Sep 10, 2015 at 2:39 PM, Rob Clark wrote: >> > From: Rob Clark >> > >> > So this is basically working as a lowering pass for handling user-clip- >> > planes, and frag-shader emu

[Mesa-dev] [PATCH 2/2] r600g: lower number of driver const buffers

From: Dave Airlie I'm going to want a driver constant buffer for tess to coordinate LDS storage, so before I go tackling that I decided to merge the clip/samplepos and texture info buffers into one. So I can steal the spare one. This creates a single constant buffer between the two, with clip/sa

[Mesa-dev] [PATCH 1/2] r600: define some values for the fetch constant offsets.

From: Dave Airlie This just puts these in one place and #defines them. Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/evergreen_state.c | 30 +- src/gallium/drivers/r600/evergreend.h | 8 src/gallium/drivers/r600/r600_state.c | 25 ++

Re: [Mesa-dev] [PATCH V3 1/6] glsl: order indices for samplers inside a struct array

On Thu, Sep 10, 2015 at 7:43 PM, Timothy Arceri wrote: > On Tue, 2015-09-08 at 14:14 -0700, Jason Ekstrand wrote: >> On Mon, Sep 7, 2015 at 3:09 PM, Timothy Arceri > > wrote: >> > On Mon, 2015-09-07 at 11:24 -0700, Jason Ekstrand wrote: >> > > On Tue, Sep 1, 2015 at 7:44 PM, Timothy Arceri < >> >

Re: [Mesa-dev] [WIP 0/2] packed varyings problem

I guess there are no strong opinions on this (?) If not I'll just fix the part building stage_ref mask in patch 2 and resend the patches. On 09/04/2015 12:39 PM, Tapani Pälli wrote: Hi; Here's a solution to a problem with lower_packed_varyings pass. I wanted to ask if this seems OK approach fo

Re: [Mesa-dev] [PATCH V3 5/6] nir: support indirect indexing samplers in struct arrays

On Fri, 2015-09-11 at 12:55 +1000, Timothy Arceri wrote: > On Thu, 2015-09-10 at 18:04 -0700, Jason Ekstrand wrote: > > On Tue, Sep 1, 2015 at 7:44 PM, Timothy Arceri < > > t_arc...@yahoo.com.au > > > wrote: > > > As a bonus we get indirect support for arrays of arrays for free. > > > > > > V3: Us

[Mesa-dev] [PATCH 2/4] nv50/ir: add support for TXQS tgsi opcode

Signed-off-by: Ilia Mirkin --- .../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 17 - .../drivers/nouveau/codegen/nv50_ir_lowering_nv50.cpp | 19 +++ .../drivers/nouveau/codegen/nv50_ir_lowering_nvc0.cpp | 2 ++ 3 files changed, 33 insertions(+), 5 deletion

[Mesa-dev] [PATCH 4/4] st/mesa: emit TXQS, support ARB_shader_texture_image_samples

This is enabled for all drivers that support MS textures. The image component of the ext is a no-op since there is no image support in gallium (yet). Signed-off-by: Ilia Mirkin --- src/mesa/state_tracker/st_extensions.c | 1 + src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 6 +- 2 files ch

[Mesa-dev] [PATCH 3/4] r600g: add support for TXQS tgsi opcode

Signed-off-by: Ilia Mirkin --- src/gallium/drivers/r600/r600_shader.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/drivers/r600/r600_shader.c index f2c9e16..8132b6a 100644 --- a/src/gallium/drivers/r600

[Mesa-dev] [PATCH 0/4] gallium: add support for retrieving number of texture samples

My hope was (as you can see in the last patch) to enable this for all drivers that support MS textures. I've got nv50/nvc0/r600g covered. RadeonSI will have to read the data from the texture descriptor. I'm totally unfamiliar with LLVM, the descriptor format, etc. Hopefully someone will be able to

[Mesa-dev] [PATCH 1/4] tgsi: add a TXQS opcode to retrieve the number of texture samples

Signed-off-by: Ilia Mirkin --- src/gallium/auxiliary/tgsi/tgsi_info.c | 3 ++- src/gallium/docs/source/tgsi.rst | 12 +++- src/gallium/include/pipe/p_shader_tokens.h | 1 + 3 files changed, 14 insertions(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_info

Re: [Mesa-dev] [PATCH 1/2] r600g: Support I2D/U2D/D2I/D2U

On 11 September 2015 at 12:37, Roland Scheidegger wrote: > Just on a very quick glance, seems somewhat odd this function is named > "cypress" whereas the other 64bit ones are named "cayman". > In any case, saying enabling support for chips which have hative support > isn'quite correct, since there

Re: [Mesa-dev] [PATCH 00/17] RadeonSI: Handling buffer allocation failures properly

On 11.09.2015 02:56, Marek Olšák wrote: > Hi, > > I have realized that if we fail to allocate a shader, a scratch buffer, or a > GS or tess ring, we still execute the draw call. This can cause hangs, > because state registers are not set properly at that point. > > This series tries to handle a

Re: [Mesa-dev] [PATCH V3 5/6] nir: support indirect indexing samplers in struct arrays

On Thu, 2015-09-10 at 18:04 -0700, Jason Ekstrand wrote: > On Tue, Sep 1, 2015 at 7:44 PM, Timothy Arceri > wrote: > > As a bonus we get indirect support for arrays of arrays for free. > > > > V3: Use nir_instr_rewrite_src() with empty src rather then clearing > > the use_link list directly for t

Re: [Mesa-dev] [RFC] nir: add lowering stage for user-clip-planes

On Sep 10, 2015 7:39 PM, "Jason Ekstrand" wrote: > > On Thu, Sep 10, 2015 at 2:39 PM, Rob Clark wrote: > > From: Rob Clark > > > > So this is basically working as a lowering pass for handling user-clip- > > planes, and frag-shader emulation of CLIPDIST for hardware that needs > > to handle this

Re: [Mesa-dev] [PATCH V3 1/6] glsl: order indices for samplers inside a struct array

On Tue, 2015-09-08 at 14:14 -0700, Jason Ekstrand wrote: > On Mon, Sep 7, 2015 at 3:09 PM, Timothy Arceri > wrote: > > On Mon, 2015-09-07 at 11:24 -0700, Jason Ekstrand wrote: > > > On Tue, Sep 1, 2015 at 7:44 PM, Timothy Arceri < > > > t_arc...@yahoo.com.au> > > > wrote: > > > > This allows the c

Re: [Mesa-dev] [PATCH 1/2] r600g: Support I2D/U2D/D2I/D2U

Just on a very quick glance, seems somewhat odd this function is named "cypress" whereas the other 64bit ones are named "cayman". In any case, saying enabling support for chips which have hative support isn'quite correct, since there's a couple more which should be able to do it (rv670, rv770, mayb

Re: [Mesa-dev] [PATCH] tgsi, softpipe: capitalize the tgsi_sampler_control enum values

Am 10.09.2015 um 20:28 schrieb Brian Paul: > We use capitalized enum values everywhere else. > This improves understanding a bit too. > --- > src/gallium/auxiliary/tgsi/tgsi_exec.c | 34 ++-- > src/gallium/auxiliary/tgsi/tgsi_exec.h | 15 + > src/gallium/drivers

Re: [Mesa-dev] [PATCH 2.1/2] egl/dri2: don't leak the fd on dri2_terminate

2015-09-10 21:49 GMT+08:00 Emil Velikov : > Currently the check was incorrect as it did not consider the (unlikely) > case of fd == 0. In order to fix this we should first correctly > initialize it to -1, as the swrast implementations leave it set to zero > (props to calloc()). Reviewed-by: Boyan

Re: [Mesa-dev] [PATCH V3 5/6] nir: support indirect indexing samplers in struct arrays

On Tue, Sep 1, 2015 at 7:44 PM, Timothy Arceri wrote: > As a bonus we get indirect support for arrays of arrays for free. > > V3: Use nir_instr_rewrite_src() with empty src rather then clearing > the use_link list directly for the old indirects as suggested by Jason > > V2: Fixed validation error

Re: [Mesa-dev] [PATCH] i965: add ARB_texture_barrier support

ping On Thu, Aug 13, 2015 at 3:42 PM, Ilia Mirkin wrote: > Signed-off-by: Ilia Mirkin > --- > > The blending-in-shader piglit test passed even without the flush, > which doesn't inspire me with confidence, but those piglit_draw_rect > things are pretty heavy so perhaps cause a flush on their own

[Mesa-dev] [libdrm] small fix for atomic_inc_return macro.

Hi. There is a typo in macro atomic_inc_return in the file xf86atomic.h Output of format-patch: Date: Fri, 11 Sep 2015 01:36:31 +0300 Subject: [PATCH] Fix atomic_inc_return. nouveau.c: In function 'nouveau_bo_wrap_locked': nouveau.c:413: error: 'x' undeclared (first use in this function) --- xf8

Re: [Mesa-dev] [PATCH 1/2] r600g: Support I2D/U2D/D2I/D2U

On 11 September 2015 at 06:00, Glenn Kennard wrote: > int <-> float <-> double conversion, matches what the blob does > > Signed-off-by: Glenn Kennard > --- > src/gallium/drivers/r600/r600_shader.c | 95 > +++--- > 1 file changed, 87 insertions(+), 8 deletions(-) > >

Re: [Mesa-dev] [PATCH] nir: add nir_swizzle

On Thu, Sep 10, 2015 at 10:08 PM, Rob Clark wrote: > From: Rob Clark > > Rather than make yet another copy of channel(), let's move it into nir. > > Signed-off-by: Rob Clark > --- > src/glsl/nir/nir_builder.h | 6 ++ > src/glsl/nir/nir_lower_tex_projector.c | 24 +

[Mesa-dev] [PATCH] i965/skl: Use larger URB size where available.

All SKL SKUs except the lowest one which has half the L3 size actually have 384K of URB per slice. For once, I can explain how this mistake was made and how it was missed in review... Historically when we enable a platform and put the production sizes, you can simply look at the "smallest" SKU an

Re: [Mesa-dev] [PATCH v2 2/2] i965/vec4_nir: Load constants as integers

On Thu, Sep 10, 2015 at 3:27 PM, Jason Ekstrand wrote: > Please add the shader-db reults to the commit message. With that, > > Reviewed-by: Jason Ekstrand > > Also, please CC stable on this one as well. I think there's some bugs > without it. In particular it fixes https://bugs.freedesktop.org

Re: [Mesa-dev] [PATCH] ralloc: Use __attribute__((destructor)) instead of atexit(3)

On Thu, Sep 10, 2015 at 7:58 PM, Ian Romanick wrote: > On 09/10/2015 05:29 AM, Jose Fonseca wrote: >> On 10/09/15 02:54, Ian Romanick wrote: >> >> It's not necessary to do it in several places, at least for the >> destructor -- we could have a single C++ module inside src/util, which >> provided a

[Mesa-dev] [PATCH] nir: Move system value -> intrinsic mapping into nir.c

This way they're right next to the map going the other direction. --- src/glsl/nir/nir.c | 34 + src/glsl/nir/nir.h | 1 + src/glsl/nir/nir_lower_system_values.c | 35 +- 3 files changed, 36 in

Re: [Mesa-dev] [RFC] nir: add lowering stage for user-clip-planes

On Thu, Sep 10, 2015 at 2:39 PM, Rob Clark wrote: > From: Rob Clark > > So this is basically working as a lowering pass for handling user-clip- > planes, and frag-shader emulation of CLIPDIST for hardware that needs > to handle this in the shader. > > For user-clip-planes, instructions are insert

Re: [Mesa-dev] [PATCH] ralloc: Use __attribute__((destructor)) instead of atexit(3)

On Mon, Sep 7, 2015 at 3:54 PM, Jose Fonseca wrote: > On 07/09/15 10:17, Jean-Sébastien Pédron wrote: >> >> On 04.09.2015 01:37, Matt Turner wrote: >>> >>> You need to test for this support in configure.ac. It's as simple as >>> adding a call to AX_GCC_FUNC_ATTRIBUTE in the existing alphabetized >

[Mesa-dev] [PATCH 2/2] i965/vec4: Don't reswizzle hardware registers

Cc: "11.0 10.6" Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91719 --- src/mesa/drivers/dri/i965/brw_vec4.cpp | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp b/src/mesa/drivers/dri/i965/brw_vec4.cpp index 78ee8fe..01c1259 100644 --- a/s

[Mesa-dev] [PATCH 1/2] i965/emit: Add assertions for accumulator restrictions

--- src/mesa/drivers/dri/i965/brw_eu_emit.c | 16 1 file changed, 16 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c index 637fd07..b2f7409 100644 --- a/src/mesa/drivers/dri/i965/brw_eu_emit.c +++ b/src/mesa/drivers/dr

Re: [Mesa-dev] [PATCH 00/17] RadeonSI: Handling buffer allocation failures properly

On Thu, Sep 10, 2015 at 8:11 PM, Christian König wrote: > Indeed nice cleanup, I wonder if that helps with our current problem as > well. But allocation failures are printed in dmesg as well, aren't they? Yes, they are. Marek ___ mesa-dev mailing list

Re: [Mesa-dev] [PATCH v2 2/2] i965/vec4_nir: Load constants as integers

Please add the shader-db reults to the commit message. With that, Reviewed-by: Jason Ekstrand Also, please CC stable on this one as well. I think there's some bugs without it. --Jason On Mon, Aug 24, 2015 at 4:51 AM, Antia Puentes wrote: > Loads constants using integer as their register type

Re: [Mesa-dev] [PATCH 1/2] i965/vec4: Fix saturation errors when coalescing registers

On Fri, Aug 14, 2015 at 4:56 AM, Antia Puentes wrote: > If the register types do not match and the instruction > that contains the final destination is saturated, register > coalescing generated non-equivalent code. > > This did not happen when using IR because types usually > matched, but it is v

[Mesa-dev] [RFC] nir: add lowering stage for user-clip-planes

From: Rob Clark So this is basically working as a lowering pass for handling user-clip- planes, and frag-shader emulation of CLIPDIST for hardware that needs to handle this in the shader. For user-clip-planes, instructions are inserted to calculate CLIPDIST in the VS. And in both cases, discard

Re: [Mesa-dev] [PATCH V2 1/8] i965: Add a helper function intel_get_tile_dims()

On Thu, Sep 10, 2015 at 12:20:10PM -0700, Chad Versace wrote: > On Wed 19 Aug 2015, Anuj Phogat wrote: > > V2: > > - Do the tile width/height computations in the new helper > > function and use it later in intel_miptree_get_tile_masks(). > > - Change the name to intel_get_tile_dims(). > > > > Cc

Re: [Mesa-dev] [PATCH] nir: add nir_swizzle

On Thu, Sep 10, 2015 at 1:08 PM, Rob Clark wrote: > From: Rob Clark > > Rather than make yet another copy of channel(), let's move it into nir. > > Signed-off-by: Rob Clark Reviewed-by: Jason Ekstrand > --- > src/glsl/nir/nir_builder.h | 6 ++ > src/glsl/nir/nir_lower_t

[Mesa-dev] [PATCH] nir/lower_outputs_to_temporaries: Reparent the output name

We copy the output, make the old output the temporary, and give the temporary a new name. The copy keeps the pointer to the old name. This works just fine up until the point where we lower things to SSA and delete the old variable and, with it, the name. Instead, we should re-parent to the copy.

[Mesa-dev] XDC2015: Tentative schedule posted

Hi all, XDC2015, is less than a week a way! I have posted a tentative schedule on the wiki[1]. If you are a speaker and have questions about the schedule or want to request a new time slot, please let me know. I will do my best, but I may not be able to accommodate everyone's request. Each pre

[Mesa-dev] [PATCH] nir: add nir_swizzle

From: Rob Clark Rather than make yet another copy of channel(), let's move it into nir. Signed-off-by: Rob Clark --- src/glsl/nir/nir_builder.h | 6 ++ src/glsl/nir/nir_lower_tex_projector.c | 24 +--- src/glsl/nir/nir_normalize_cubemap_coords.c |

[Mesa-dev] [PATCH 1/2] r600g: Support I2D/U2D/D2I/D2U

int <-> float <-> double conversion, matches what the blob does Signed-off-by: Glenn Kennard --- src/gallium/drivers/r600/r600_shader.c | 95 +++--- 1 file changed, 87 insertions(+), 8 deletions(-) diff --git a/src/gallium/drivers/r600/r600_shader.c b/src/gallium/dr

[Mesa-dev] [PATCH 2/2] r600: Enable fp64 on chips with native support

Signed-off-by: Glenn Kennard --- docs/GL3.txt | 4 ++-- docs/relnotes/11.1.0.html| 2 +- src/gallium/drivers/r600/r600_pipe.c | 3 +++ 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/docs/GL3.txt b/docs/GL3.txt index 8ad1aac..7247eb6 100644 --- a

[Mesa-dev] [Bug 91889] Planetary Anihilation: Titans display content of other processes buffers

https://bugs.freedesktop.org/show_bug.cgi?id=91889 --- Comment #12 from Krzysztof A. Sobiecki --- (In reply to Albert Freeman from comment #11) > Does the screenshot you posted look [around about]/exactly the same as > actually playing the game does? Yes it does I have made two apitraces: 1. Sh

Re: [Mesa-dev] [PATCH 4/9] i965: Make intel_miptree_map_raw static

On 09/10/2015 11:52 AM, Anuj Phogat wrote: > > > On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick > wrote: > > From: Ian Romanick > > > This hasn't been used outside intel_mipmap_tree.c since d5d4ba9 started > using met

Re: [Mesa-dev] [PATCH 01/10] mesa: add infra for ARB_shader_texture_image_samples

On 09/10/2015 12:35 PM, Ilia Mirkin wrote: > On Thu, Sep 10, 2015 at 3:31 PM, Ian Romanick wrote: >> On 09/08/2015 10:20 PM, Ilia Mirkin wrote: >>> On Tue, Sep 8, 2015 at 11:46 PM, Ilia Mirkin wrote: On Tue, Sep 8, 2015 at 11:38 PM, Ian Romanick wrote: > On 09/08/2015 08:17 PM, Ian Roma

Re: [Mesa-dev] [PATCH 01/10] mesa: add infra for ARB_shader_texture_image_samples

On Thu, Sep 10, 2015 at 3:31 PM, Ian Romanick wrote: > On 09/08/2015 10:20 PM, Ilia Mirkin wrote: >> On Tue, Sep 8, 2015 at 11:46 PM, Ilia Mirkin wrote: >>> On Tue, Sep 8, 2015 at 11:38 PM, Ian Romanick wrote: On 09/08/2015 08:17 PM, Ian Romanick wrote: > On 09/08/2015 08:10 PM, Ilia Mi

Re: [Mesa-dev] [PATCH 01/10] mesa: add infra for ARB_shader_texture_image_samples

On 09/08/2015 10:20 PM, Ilia Mirkin wrote: > On Tue, Sep 8, 2015 at 11:46 PM, Ilia Mirkin wrote: >> On Tue, Sep 8, 2015 at 11:38 PM, Ian Romanick wrote: >>> On 09/08/2015 08:17 PM, Ian Romanick wrote: On 09/08/2015 08:10 PM, Ilia Mirkin wrote: > I'll give it a shot on NVIDIA and see what

Re: [Mesa-dev] [Mesa-dev, 4/4] i965/nir: Support gl_WorkGroupID variable

Jordan Justen writes: > 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 10064

Re: [Mesa-dev] [PATCH 9/9] i915, i965: Silence unused parameter warnings in intel_batchbuffer_advance

On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick wrote: > From: Ian Romanick > > These only occurred in release builds, but they occurred in every file > that included intel_batchbuffer.h. Lots of spam. :( > > intel_batchbuffer.h: In function 'intel_batchbuffer_advance': > intel_batchbuffer.h:153:4

Re: [Mesa-dev] [Mesa-dev, 3/4] i965/cs: Initialize gl_WorkGroupID variable from payload

Jordan Justen writes: > Signed-off-by: Jordan Justen I double checked the GPGPU payload, this looks corrent. Reviewed-by: Kristian Høgsberg > > --- > src/mesa/drivers/dri/i965/brw_cs.cpp | 19 +++ > src/mesa/drivers/dri/i965/brw_fs.h | 1 + > 2 files changed, 20 insertions

Re: [Mesa-dev] [PATCH V2 1/8] i965: Add a helper function intel_get_tile_dims()

On Wed 19 Aug 2015, Anuj Phogat wrote: > V2: > - Do the tile width/height computations in the new helper > function and use it later in intel_miptree_get_tile_masks(). > - Change the name to intel_get_tile_dims(). > > Cc: Ben Widawsky > Signed-off-by: Anuj Phogat > --- > src/mesa/drivers/dri/

Re: [Mesa-dev] [Mesa-dev,1/4] glsl/cs: Add gl_WorkGroupID variable

Jordan Justen writes: > Signed-off-by: Jordan Justen Reviewed-by: Kristian Høgsberg > > --- > 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 > in

Re: [Mesa-dev] [Mesa-dev, 2/4] nir: Add gl_WorkGroupID system variable

Jordan Justen writes: > 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..6987d4

Re: [Mesa-dev] [PATCH v5 01/70] mesa: set MAX_SHADER_STORAGE_BUFFERS to 15.

On Thu, Sep 10, 2015 at 2:52 PM, Ian Romanick wrote: > On 09/10/2015 10:45 AM, Ilia Mirkin wrote: >> On Thu, Sep 10, 2015 at 9:35 AM, Iago Toral Quiroga >> wrote: >>> From: Samuel Iglesias Gonsalvez >>> >>> This patch sets the same value used for uniform buffers. >>> >>> Signed-off-by: Samuel I

Re: [Mesa-dev] [PATCH 8/9] i915: Silence unused parameter warning in intel_miptree_create_layout

On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick wrote: > From: Ian Romanick > > The for_bo parameter of intel_miptree_create_layout appears to be unused > since 27eedca when Eric removed some Gen5 code (after the i915 and i965 > drivers parted ways). > > intel_mipmap_tree.c: In function 'old_intel_

Re: [Mesa-dev] [PATCH 7/9] i915, i965: Silence unused parameter warnings in intel_miptree_unmap_gtt

On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick wrote: > From: Ian Romanick > > intel_mipmap_tree.c: In function 'intel_miptree_unmap_gtt': > intel_mipmap_tree.c:777:34: warning: unused parameter 'map' > [-Wunused-parameter] > struct intel_miptree_map *map, > ^

Re: [Mesa-dev] [PATCH 6/9] i915: Silence unused parameter warnings

On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick wrote: > From: Ian Romanick > > intel_mipmap_tree.c: In function 'old_intel_miptree_unmap_raw': > intel_mipmap_tree.c:726:51: warning: unused parameter 'intel' > [-Wunused-parameter] > intel_miptree_unmap_raw(struct intel_context *intel, >

Re: [Mesa-dev] [PATCH 1/9] i965: Silence unused parameter warnings in intel_fbo.c

On Thu, Sep 10, 2015 at 11:47 AM, Ian Romanick wrote: > On 09/10/2015 11:27 AM, Anuj Phogat wrote: > > > > > > On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick > > wrote: > > > > From: Ian Romanick > > > > > > intel_fbo.c: In

Re: [Mesa-dev] [PATCH 5/9] i915: Remove prototype for nonexistent brw_miptree_layout

On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick wrote: > From: Ian Romanick > > Hasn't existed in the i915 source since the i915 and i965 drivers parted > ways. > > Signed-off-by: Ian Romanick > --- > src/mesa/drivers/dri/i915/intel_mipmap_tree.h | 2 -- > 1 file changed, 2 deletions(-) > > diff

Re: [Mesa-dev] [PATCH v5 01/70] mesa: set MAX_SHADER_STORAGE_BUFFERS to 15.

On 09/10/2015 10:45 AM, Ilia Mirkin wrote: > On Thu, Sep 10, 2015 at 9:35 AM, Iago Toral Quiroga wrote: >> From: Samuel Iglesias Gonsalvez >> >> This patch sets the same value used for uniform buffers. >> >> Signed-off-by: Samuel Iglesias Gonsalvez >> --- >> src/mesa/main/config.h | 2 +- >> 1

Re: [Mesa-dev] [PATCH 4/9] i965: Make intel_miptree_map_raw static

On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick wrote: > From: Ian Romanick > > This hasn't been used outside intel_mipmap_tree.c since d5d4ba9 started > using meta instead of the blitter for PBO TexSubImage. While we're here, > > 1. s/intel_miptree_\([^_]\)_raw/miptree_\1_raw/g because we don't >

Re: [Mesa-dev] [PATCH 1/9] i965: Silence unused parameter warnings in intel_fbo.c

On 09/10/2015 11:27 AM, Anuj Phogat wrote: > > > On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick > wrote: > > From: Ian Romanick > > > intel_fbo.c: In function 'intel_alloc_window_storage': > intel_fbo.c:415:48: warnin

Re: [Mesa-dev] [RFC 1/2] nir: Add a new lowering pass nir_lower_vec_and_coalesce

On Wed, Sep 9, 2015 at 11:51 PM, Eduardo Lima Mitev wrote: > On 09/09/2015 07:10 PM, Jason Ekstrand wrote: >> >> On Sep 8, 2015 23:27, "Eduardo Lima Mitev" > > wrote: >>> >>> This pass will propagate the destination components of a vecN >> instructions, >>> as destination

Re: [Mesa-dev] [Mesa-dev, 7/7] i965/nir: Support gl_LocalInvocationID variable

Jordan Justen writes: > Signed-off-by: Jordan Justen Reviewed-by: Kristian Høgsberg > > --- > 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.

Re: [Mesa-dev] [PATCH 3/9] i915, i965: Silence unused parameter warnings in intel_mipmap_tree.h

On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick wrote: > From: Ian Romanick > > These only occurred in release builds, but they occurred in every file > that included intel_mipmap_tree.h. Lots of spam. :( > > intel_mipmap_tree.h: In function 'intel_miptree_check_level_layer': > intel_mipmap_tree.h

Re: [Mesa-dev] [PATCH] r600g: update num_dw in scissor_enable workaround

On Thu, Sep 10, 2015 at 5:35 AM, Michel Dänzer wrote: > On 10.09.2015 01:33, Grazvydas Ignotas wrote: >> "r600g: apply disable workaround on all scissors" forgot to update >> num_dw, fix it. >> >> Fixes: fbb423b433 "r600g: apply disable workaround on all scissors" >> Reported-and-tested-by: Markus

Re: [Mesa-dev] [PATCH 07/11] nir/lower_vec_to_movs: Handle partially SSA shaders

On Thu, Sep 10, 2015 at 9:08 AM, Eduardo Lima Mitev wrote: > On 09/10/2015 02:50 AM, Jason Ekstrand wrote: >> --- >> src/glsl/nir/nir_lower_vec_to_movs.c | 18 -- >> 1 file changed, 16 insertions(+), 2 deletions(-) >> >> diff --git a/src/glsl/nir/nir_lower_vec_to_movs.c >> b/src/

Re: [Mesa-dev] [PATCH 2/9] i965: Silence unused parameter warnings in intel_mipmap_tree.c

On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick wrote: > From: Ian Romanick > > The target parameter of compute_msaa_layout appears to be unused since > 83b83fb when support for CMS textures was added for Gen7. > > The brw parameter of intel_get_non_msrt_mcs_alignment appears to be > unused since e

Re: [Mesa-dev] [Mesa-dev, 6/7] i965/cs: Initialize gl_LocalInvocationID from payload

Jordan Justen writes: > Signed-off-by: Jordan Justen Reviewed-by: Kristian Høgsberg > > --- > 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/drive

Re: [Mesa-dev] [Mesa-dev, 5/7] i965/cs: Initialize gl_LocalInvocationID in push constant data

Jordan Justen writes: > Signed-off-by: Jordan Justen > > --- > 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 > inde

Re: [Mesa-dev] [PATCH 04/11] nir: Add a function for rewriting instruction destinations

On Thu, Sep 10, 2015 at 3:01 AM, Michael Schellenberger Costa wrote: > Hi, > Am 10/09/2015 um 02:50 schrieb Jason Ekstrand: >> --- >> src/glsl/nir/nir.c | 24 >> src/glsl/nir/nir.h | 2 ++ >> 2 files changed, 26 insertions(+) >> >> diff --git a/src/glsl/nir/nir.c b/src/g

Re: [Mesa-dev] [Mesa-stable] [PATCH] meta: Always bind the texture

On 09/10/2015 11:12 AM, Neil Roberts wrote: > Make sense. > > Is this the only use of the currentTexUnitSave variable? Could be good > to remove it if so. It is. Good catch. I'll remove that before pushing. Thanks. > Reviewed-by: Neil Roberts > > - Neil > > Ian Romanick writes: > >> From

Re: [Mesa-dev] [PATCH 1/9] i965: Silence unused parameter warnings in intel_fbo.c

On Wed, Sep 9, 2015 at 7:18 PM, Ian Romanick wrote: > From: Ian Romanick > > intel_fbo.c: In function 'intel_alloc_window_storage': > intel_fbo.c:415:48: warning: unused parameter 'ctx' [-Wunused-parameter] > intel_alloc_window_storage(struct gl_context * ctx, struct > gl_renderbuffer *rb, >

[Mesa-dev] [PATCH] tgsi, softpipe: capitalize the tgsi_sampler_control enum values

We use capitalized enum values everywhere else. This improves understanding a bit too. --- src/gallium/auxiliary/tgsi/tgsi_exec.c | 34 ++-- src/gallium/auxiliary/tgsi/tgsi_exec.h | 15 + src/gallium/drivers/softpipe/sp_tex_sample.c | 48 ++--

[Mesa-dev] Mesa 10.6.7

Mesa 10.6.7 is now available. This is a emergency release which reverts an erroneous nomination and fixes issues in core mesa and glsl in certain workloads. Dave Airlie (1): mesa/teximage: use correct extension for accept stencil texture. Emil Velikov (4): docs: add sha256 checksums

Re: [Mesa-dev] [PATCH] mesa/prog: relative offsets into constbufs are not constant

On 07/01/2015 11:00 PM, Jose Fonseca wrote: > On 02/07/15 06:55, Matt Turner wrote: >> On Wed, Jul 1, 2015 at 3:22 PM, Ilia Mirkin wrote: >>> The optimization logic relies on being able to read out constbuf values >>> from program parameters. However that only works if there's no relative >>> addr

Re: [Mesa-dev] [PATCH 4/4] svga: clean up the compile_vs/gs/fs() functions

For the series, Reviewed-by: Charmaine Lee From: Brian Paul Sent: Thursday, September 10, 2015 8:04 AM To: mesa-dev@lists.freedesktop.org Cc: Charmaine Lee; Brian Paul Subject: [PATCH 4/4] svga: clean up the compile_vs/gs/fs() functions Simplify structur

Re: [Mesa-dev] [PATCH v5 25/70] glsl: Add std430 related member functions to glsl_type class

On 2015-09-10 06:35:41, Iago Toral Quiroga wrote: > From: Samuel Iglesias Gonsalvez > > They are used to calculate size, base alignment and array stride values > for a glsl_type following std430 rules. > > Signed-off-by: Samuel Iglesias Gonsalvez > --- > src/glsl/glsl_types.cpp | 209 > ++

Re: [Mesa-dev] [PATCH] meta: Always bind the texture

Make sense. Is this the only use of the currentTexUnitSave variable? Could be good to remove it if so. Reviewed-by: Neil Roberts - Neil Ian Romanick writes: > From: Ian Romanick > > We may have been called from glGenerateTextureMipmap with CurrentUnit > still set to 0, so we don't know when

Re: [Mesa-dev] [PATCH 00/17] RadeonSI: Handling buffer allocation failures properly

Indeed nice cleanup, I wonder if that helps with our current problem as well. But allocation failures are printed in dmesg as well, aren't they? Anyway patches are Acked-by: Christian König Regards, Christian. On 10.09.2015 19:56, Marek Olšák wrote: Hi, I have realized that if we fail to al

Re: [Mesa-dev] [Mesa-stable] [PATCH] meta: Always bind the texture

On Wed, Sep 9, 2015 at 1:04 PM, Ian Romanick wrote: > From: Ian Romanick > > We may have been called from glGenerateTextureMipmap with CurrentUnit > still set to 0, so we don't know when we can skip binding the texture. > Assume that _mesa_BindTexture will be fast if we're rebinding the same > t

Re: [Mesa-dev] [PATCH] ralloc: Use __attribute__((destructor)) instead of atexit(3)

On 09/10/2015 05:29 AM, Jose Fonseca wrote: > On 10/09/15 02:54, Ian Romanick wrote: >> On 09/07/2015 06:54 AM, Jose Fonseca wrote: >>> On 07/09/15 10:17, Jean-Sébastien Pédron wrote: On 04.09.2015 01:37, Matt Turner wrote: > You need to test for this support in configure.ac. It's as simpl

[Mesa-dev] [PATCH 10/17] radeonsi: handle shader precompile failures

From: Marek Olšák Cc: 11.0 --- src/gallium/drivers/radeonsi/si_state_shaders.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c b/src/gallium/drivers/radeonsi/si_state_shaders.c index bc7fdb3..74e1136 100644 --- a/src/gal

[Mesa-dev] [PATCH 04/17] gallium/radeon: handle buffer alloc failures in r600_draw_rectangle

From: Marek Olšák Cc: 11.0 --- src/gallium/drivers/radeon/r600_pipe_common.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c b/src/gallium/drivers/radeon/r600_pipe_common.c index 836da76..d97b721 100644 --- a/src/gallium/drivers/radeon/r600_

[Mesa-dev] [PATCH 12/17] radeonsi: skip drawing if VS, TCS, TES, GS fail to compile or upload

From: Marek Olšák Cc: 11.0 --- src/gallium/drivers/radeonsi/si_state_shaders.c | 30 +++-- 1 file changed, 23 insertions(+), 7 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c b/src/gallium/drivers/radeonsi/si_state_shaders.c index 43d2e88..aabd499

  1   2   3   >