Re: [Mesa-dev] [PATCH 08/18] meta/blit: Don't pollute the sampler object namespace in _mesa_meta_setup_sampler

2016-01-12 Thread Erik Faye-Lund
On Sat, Jan 9, 2016 at 3:59 AM, Ian Romanick wrote: > From: Ian Romanick > > tl;dr: For many types of GL object, we can *NEVER* use the Gen function. > > In OpenGL ES (all versions!) and OpenGL compatibility profile, > applications don't have to

Re: [Mesa-dev] [PATCH 1/2] nir: Add a fquantize2f16 opcode

2016-01-12 Thread Erik Faye-Lund
On Wed, Jan 13, 2016 at 1:10 AM, Jason Ekstrand wrote: > On Tue, Jan 12, 2016 at 3:52 PM, Matt Turner wrote: >> >> On Tue, Jan 12, 2016 at 3:35 PM, Jason Ekstrand >> wrote: >> > This opcode simply takes a 32-bit floating-point

Re: [Mesa-dev] [PATCH V2] meta: Fix indentation in shader code

2016-06-02 Thread Erik Faye-Lund
On Thu, Jun 2, 2016 at 8:25 PM, Anuj Phogat wrote: > Signed-off-by: Anuj Phogat > --- > src/mesa/drivers/common/meta_blit.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/common/meta_blit.c >

Re: [Mesa-dev] [PATCH 2/2] svga: fix coverity MIXED_ENUMS warning

2016-05-31 Thread Erik Faye-Lund
On Tue, May 31, 2016 at 6:14 PM, Roland Scheidegger <srol...@vmware.com> wrote: > Am 31.05.2016 um 16:33 schrieb Erik Faye-Lund: >> On Tue, May 31, 2016 at 3:46 PM, Rob Clark <robdcl...@gmail.com> wrote: >>> On Tue, May 31, 2016 at 9:29 AM, Brian Paul <bri...@vmwa

Re: [Mesa-dev] [PATCH 11/29] mesa: Use bitmask/ffs to iterate enabled lights for building ff shader keys.

2016-05-27 Thread Erik Faye-Lund
On Fri, May 27, 2016 at 11:59 AM, Mathias Fröhlich wrote: > > Hi, > > On Wednesday, May 25, 2016 12:06:02 you wrote: >> On Tue, May 24, 2016 at 8:42 AM, wrote: >> > From: Mathias Fröhlich >> > >> > Replaces a loop

Re: [Mesa-dev] Discussion: C++11 std::future in Mesa

2016-06-01 Thread Erik Faye-Lund
On Wed, Jun 1, 2016 at 3:02 PM, ⚛ <0xe2.0x9a.0...@gmail.com> wrote: > On Wed, Jun 1, 2016 at 2:19 PM, Marek Olšák wrote: >> I'll let you figure it out by yourself. > > Why would you withhold information if you already have it? Are you a > "bad person" or something? The problem

Re: [Mesa-dev] [PATCH 2/2] svga: fix coverity MIXED_ENUMS warning

2016-05-31 Thread Erik Faye-Lund
On Tue, May 31, 2016 at 3:46 PM, Rob Clark wrote: > On Tue, May 31, 2016 at 9:29 AM, Brian Paul wrote: >> On 05/31/2016 07:10 AM, Brian Paul wrote: >>> >>> On 05/29/2016 10:32 AM, Rob Clark wrote: From: Rob Clark

Re: [Mesa-dev] [PATCH 11/29] mesa: Use bitmask/ffs to iterate enabled lights for building ff shader keys.

2016-05-31 Thread Erik Faye-Lund
On Sat, May 28, 2016 at 12:31 PM, Mathias Fröhlich wrote: > So the xor mask handing is non local to the place where we do the > computation of i. > > Brian pointed me to the u_bit_scan function heavily used in gallium. That > definitely > > helps for the basic problem I

Re: [Mesa-dev] [PATCH] st/mesa: Remove tautological check (v2)

2016-06-17 Thread Erik Faye-Lund
Now the subject no longer makes any sense. On Thu, Jun 16, 2016 at 7:02 AM, wrote: > From: Francesco Ansanelli > > --- > src/mesa/state_tracker/st_cb_fbo.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [Mesa-dev] [PATCH 0/2] trivial cleanups

2016-01-11 Thread Erik Faye-Lund
Ping? On Wed, Dec 16, 2015 at 5:09 PM, Erik Faye-Lund <kusmab...@gmail.com> wrote: > Here's some trivial cleanups I found while diving into something else. > > Instead of them collecting dust in my tree, perhaps we could apply them to > the central tree? > > Erik Faye-Lu

Re: [Mesa-dev] [PATCH] mesa: check that internalformat of CopyTexImage*D is not 1, 2, 3, 4

2016-01-11 Thread Erik Faye-Lund
On Thu, Jan 7, 2016 at 11:49 PM, Ian Romanick wrote: > On 01/07/2016 12:31 PM, Nicolai Hähnle wrote: >> From: Nicolai Hähnle >> >> The piglit copyteximage check has recently been augmented to test this, but >> apparently it hasn't been fixed in Mesa

Re: [Mesa-dev] [PATCH 0/7] mesa: Enable -fstrict-aliasing

2016-06-28 Thread Erik Faye-Lund
On Mon, Jun 27, 2016 at 11:42 PM, Matt Turner wrote: > Based on work by Davin McCall from last summer. > > The biggest change is to exec_list. Previously, the head and tail sentinels > overlapped, saving the size of a pointer. Unfortunately this is not

Re: [Mesa-dev] [PATCH 2/5] mesa: implement GL_NVX_gpu_memory_info

2016-02-02 Thread Erik Faye-Lund
On Tue, Feb 2, 2016 at 4:02 PM, Roland Scheidegger wrote: > Does anyone use these extensions? > I suppose maybe to get the total amount of video memory? I've used them in a commercial data-visualization application. The purpose was to get the total amount of video memory, so

Re: [Mesa-dev] [PATCH 1/2] st/mesa: move dereference after null check

2016-03-01 Thread Erik Faye-Lund
set_shader_images) >return; > > + c = >ctx->Const.Program[shader->Stage]; > + > for (i = 0; i < shader->NumImages; i++) { >struct gl_image_unit *u = >ctx->ImageUnits[shader->ImageUnits[i]]; >struct st_te

Re: [Mesa-dev] [PATCH 03/65] glsl: Add initial functions to implement an on-disk cache

2016-04-30 Thread Erik Faye-Lund
On Fri, Apr 29, 2016 at 3:33 PM, Timothy Arceri wrote: > +/* Create a directory named 'path' if it does not already exist. > + * > + * Returns: 0 if path already exists as a directory or if created. > + * -1 in all other cases. > + */ > +static int >

Re: [Mesa-dev] [PATCH] egl: Add OpenGL_ES to API string regardless of GLES version

2016-05-18 Thread Erik Faye-Lund
On Wed, May 18, 2016 at 1:01 PM, Emil Velikov <emil.l.veli...@gmail.com> wrote: > On 18 May 2016 at 09:27, Erik Faye-Lund <kusmab...@gmail.com> wrote: >> On Wed, May 18, 2016 at 10:12 AM, Daniel Stone <dan...@fooishbar.org> wrote: >>> Hi, >>>

Re: [Mesa-dev] [PATCH] egl: Add OpenGL_ES to API string regardless of GLES version

2016-05-18 Thread Erik Faye-Lund
On Wed, May 18, 2016 at 10:12 AM, Daniel Stone wrote: > Hi, > > On 18 May 2016 at 00:00, Ian Romanick wrote: >> On 05/17/2016 09:59 AM, Ben Widawsky wrote: >>> I think you misstated this. It's not invalid to have any other value. It's >>> invalid to

[Mesa-dev] [PATCH] docs: correct name for GL_OES_primitive_bounding_box

2016-04-18 Thread Erik Faye-Lund
When this extension was added, an underscore were mistakenly replaced by a space. Let's correct this, so it's a tad easier to grep for this extension. Signed-off-by: Erik Faye-Lund <kusmab...@gmail.com> --- Just a tiny nit I noticed while reading docs... docs/GL3.txt | 2 +- 1 file chan

Re: [Mesa-dev] [PATCH 02/25] anv: remove unused dev_icd.json

2016-04-21 Thread Erik Faye-Lund
On Thu, Apr 21, 2016 at 3:16 PM, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov > --- > src/intel/vulkan/.gitignore | 1 - > src/intel/vulkan/Makefile.am | 1 - >

Re: [Mesa-dev] [PATCH 11/29] mesa: Use bitmask/ffs to iterate enabled lights for building ff shader keys.

2016-05-25 Thread Erik Faye-Lund
On Tue, May 24, 2016 at 8:42 AM, wrote: > From: Mathias Fröhlich > > Replaces a loop that iterates all lights and test > which of them is enabled by a loop only iterating over > the bits set in the enabled bitmask. This takes the code from

Re: [Mesa-dev] [PATCH 2/6] nir: Turn bcsel of +/- 1.0 and 0.0 into b2f sequences.

2016-08-10 Thread Erik Faye-Lund
On Wed, Aug 10, 2016 at 5:48 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote: > On Aug 10, 2016 1:02 AM, "Erik Faye-Lund" <kusmab...@gmail.com> wrote: >> >> On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke <kenn...@whitecape.org> >> wrote: >&

Re: [Mesa-dev] [PATCH 1/6] nir: Convert ineg(b2i(a)) to a if it's a boolean.

2016-08-10 Thread Erik Faye-Lund
On Wed, Aug 10, 2016 at 5:40 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote: > On Aug 10, 2016 1:00 AM, "Erik Faye-Lund" <kusmab...@gmail.com> wrote: >> >> On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke <kenn...@whitecape.org> >> wrote: >&

Re: [Mesa-dev] [PATCH 1/6] nir: Convert ineg(b2i(a)) to a if it's a boolean.

2016-08-10 Thread Erik Faye-Lund
On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke wrote: > On Haswell (GL 3.3): > > total instructions in shared programs: 6211678 -> 6211584 (-0.00%) > instructions in affected programs: 18968 -> 18874 (-0.50%) > helped: 62 > HURT: 0 > LOST: 0 > GAINED: 4 > > On Broadwell

Re: [Mesa-dev] [PATCH 2/6] nir: Turn bcsel of +/- 1.0 and 0.0 into b2f sequences.

2016-08-10 Thread Erik Faye-Lund
On Wed, Aug 10, 2016 at 4:30 AM, Kenneth Graunke wrote: > On Haswell (GL 3.3): > > total instructions in shared programs: 6208759 -> 6203860 (-0.08%) > instructions in affected programs: 856541 -> 851642 (-0.57%) > helped: 3157 > HURT: 113 > LOST: 7 > GAINED: 15 > > On

Re: [Mesa-dev] [PATCH 1/2] nir: Add optimization for (a || True == True)

2016-07-07 Thread Erik Faye-Lund
On Thu, Jul 7, 2016 at 2:12 AM, Eric Anholt wrote: > This was appearing in vc4 VS/CS in mupen64, due to vertex attrib lowering > producing some constants that were getting compared. > > total instructions in shared programs: 112276 -> 112198 (-0.07%) > instructions in affected

Re: [Mesa-dev] [PATCH 1/2] nir: Add optimization for (a || True == True)

2016-07-07 Thread Erik Faye-Lund
On Thu, Jul 7, 2016 at 7:01 PM, Connor Abbott <cwabbo...@gmail.com> wrote: > On Thu, Jul 7, 2016 at 12:57 PM, Erik Faye-Lund <kusmab...@gmail.com> wrote: >> On Thu, Jul 7, 2016 at 2:12 AM, Eric Anholt <e...@anholt.net> wrote: >>> This was appearing in vc4 VS

Re: [Mesa-dev] Moving amdgpu/addrlib into a git submodule

2016-08-09 Thread Erik Faye-Lund
On Tue, Aug 9, 2016 at 4:59 PM, Nicolai Hähnle wrote: > On 09.08.2016 15:58, Rob Clark wrote: >> >> tbh, git submodules are more annoying than they need to be, and I'm >> not really terribly excited to use that for something that is a build >> dependency. Maybe just move it

Re: [Mesa-dev] Moving amdgpu/addrlib into a git submodule

2016-08-09 Thread Erik Faye-Lund
On Tue, Aug 9, 2016 at 7:24 PM, Nicolai Hähnle <nhaeh...@gmail.com> wrote: > > On 09.08.2016 18:22, Erik Faye-Lund wrote: >> >> On Tue, Aug 9, 2016 at 4:59 PM, Nicolai Hähnle <nhaeh...@gmail.com> wrote: >>> >>> On 09.08.2016 15:58, Rob Clark wr

Re: [Mesa-dev] Question about handling RGBA/BGRA in etnaviv driver

2017-02-03 Thread Erik Faye-Lund
On Fri, Feb 3, 2017 at 11:06 AM, Wladimir wrote: > Yes, but it seems suboptimal, incurring overhead on every rendered pixel. Why would this incur overhead? Can't the etnaviv-hardware perform swizzles for free? I'd assume you could just remap writes to gl_FragColor-compoents...

Re: [Mesa-dev] [PATCH] vbo: Correctly handle attribute offsets in dlist draw.

2016-08-19 Thread Erik Faye-Lund
On Thu, Aug 18, 2016 at 7:50 PM, wrote: > From: Mathias Fröhlich > > Hi, > > I found the below while fixing a similar problem lately in > the immediate mode glBegin/glEnd code path. > > Please review > Thanks > > Mathias > > > > When

Re: [Mesa-dev] [PATCH] vbo: Correctly handle attribute offsets in dlist draw.

2016-08-19 Thread Erik Faye-Lund
On Fri, Aug 19, 2016 at 10:38 AM, Mathias Fröhlich <mathias.froehl...@gmx.net> wrote: > On Friday, 19 August 2016 09:20:41 CEST Erik Faye-Lund wrote: >> On Thu, Aug 18, 2016 at 7:50 PM, <mathias.froehl...@gmx.net> wrote: >> > From: Mathias Fröhlich <mathi

Re: [Mesa-dev] [PATCH] Revert "foo"

2016-09-01 Thread Erik Faye-Lund
On Thu, Sep 1, 2016 at 11:36 AM, Emil Velikov wrote: > This reverts commit 917e906379bc261c65e2cc4942d7bd3d8366f030. Huh? I can't see this commit in mesa.git... ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH 1/7] EGL: Update eglext.h

2016-09-12 Thread Erik Faye-Lund
An easy-ish way of upgrading a single source file from an upstream and keeping the changes that were done on top, is to branch out from the last update-commit, and update there. Then when merged, git will understand that the changes on top should still be applied when merging. Unfortunately, it

Re: [Mesa-dev] [PATCH] st/mesa: only enable MSAA coverage options when we have a MSAA buffer

2016-09-18 Thread Erik Faye-Lund
On Fri, Sep 16, 2016 at 4:42 PM, Brian Paul wrote: > On 09/16/2016 08:07 AM, Marek Olšák wrote: >> >> On Thu, Sep 15, 2016 at 11:20 PM, Brian Paul wrote: >>> >>> Regardless of whether GL_MULTISAMPLE is enabled (it's enabled by default) >>> we should not set

Re: [Mesa-dev] Expose ES 3.2 symbols in libGLESv2.so?

2016-09-22 Thread Erik Faye-Lund
On Wed, Sep 21, 2016 at 8:57 PM, Kenneth Graunke wrote: > Commit 5921f372c89a68fac6ddefc009442721d9df4db2 exposed GLES 3.1 symbols > in libGLESv2.so. Are we supposed to do the same thing for GLES 3.2? > > I imagine we're supposed to, but I'm not certain what spec actually

Re: [Mesa-dev] Expose ES 3.2 symbols in libGLESv2.so?

2016-09-22 Thread Erik Faye-Lund
On Thu, Sep 22, 2016 at 5:45 PM, Emil Velikov <emil.l.veli...@gmail.com> wrote: > On 22 September 2016 at 08:10, Erik Faye-Lund <kusmab...@gmail.com> wrote: >> On Wed, Sep 21, 2016 at 8:57 PM, Kenneth Graunke <kenn...@whitecape.org> >> wrote: >>> Commi

Re: [Mesa-dev] [PATCH 03/10] nir: add helpers to check if we can unroll loops

2016-09-16 Thread Erik Faye-Lund
On Thu, Sep 15, 2016 at 9:03 AM, Timothy Arceri wrote: > This will be used by the loop unroll and lcssa passes. > > V2: > - Check instruction count is not too large for unrolling > - Add helper for complex loop unrolling > --- > src/compiler/nir/nir.h | 31

Re: [Mesa-dev] [PATCH 02/10] nir: Add a loop analysis pass

2016-09-16 Thread Erik Faye-Lund
On Thu, Sep 15, 2016 at 9:03 AM, Timothy Arceri wrote: > + const int bias[] = { -1, 1, 1 }; > + > + for (unsigned i = 0; i < ARRAY_SIZE(bias); i++) { > + iter_int = iter_int + bias[i]; > + > + switch (cond_op) { > + case nir_op_ige: > + case

[Mesa-dev] [PATCH] compiler: avoid warning about redefinition of PYTHON_GEN

2016-10-26 Thread Erik Faye-Lund
PYTHON_GEN is defined to the exact same thing in both Makefile.glsl.am and Makefile.nir.am. This makes automake complain, so let's lift the definition up to Makefile.am, the same way as MKDIR_GEN. Signed-off-by: Erik Faye-Lund <kusmab...@gmail.com> --- src/compiler/Makefile.am | 1

Re: [Mesa-dev] Required-for-rendering internal formats

2016-10-13 Thread Erik Faye-Lund
On Thu, Oct 13, 2016 at 5:35 PM, Nicolai Hähnle wrote: > On 13.10.2016 17:15, Ilia Mirkin wrote: >> >> Anyone else have opinions on this? Should st/mesa be adding >> PIPE_BIND_RENDER_TARGET for all textures of the internal formats >> listed in table 8.12? (Which would, in this

Re: [Mesa-dev] [PATCH 2/2] spirv: Use a simpler and more correct implementaiton of tanh()

2016-12-10 Thread Erik Faye-Lund
nir_fadd(nb, exp2x, nir_imm_float(nb, > 1))); >return; > + } > > case GLSLstd450Asinh: >val->ssa->def = nir_fmul(nb, nir_fsign(nb, src[0]), Otherwise, looks good! Reviewed-by: Erik Faye-Lund <kusmab...@gmail.com> ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] glsl: always do sqrt(abs()) and inversesqrt(abs())

2017-01-11 Thread Erik Faye-Lund
On Wed, Jan 11, 2017 at 5:32 PM, Marek Olšák <mar...@gmail.com> wrote: > On Wed, Jan 11, 2017 at 4:33 PM, Erik Faye-Lund <kusmab...@gmail.com> wrote: >> On Wed, Jan 11, 2017 at 4:14 PM, Nicolai Hähnle <nhaeh...@gmail.com> wrote: >>> On 11.01.2017 13:17, Marek O

Re: [Mesa-dev] [PATCH] glsl: always do sqrt(abs()) and inversesqrt(abs())

2017-01-11 Thread Erik Faye-Lund
On Wed, Jan 11, 2017 at 4:14 PM, Nicolai Hähnle wrote: > On 11.01.2017 13:17, Marek Olšák wrote: >> >> On Tue, Jan 10, 2017 at 6:48 PM, Jason Ekstrand >> wrote: >>> >>> I'll be honest, I'm not a fan... Given that D3D10 has one defined >>> behavior, >>>

Re: [Mesa-dev] [PATCH] glsl: always do sqrt(abs()) and inversesqrt(abs())

2017-01-11 Thread Erik Faye-Lund
; >>> >>> On 01/11/2017 05:32 PM, Marek Olšák wrote: >>>> >>>> On Wed, Jan 11, 2017 at 4:33 PM, Erik Faye-Lund <kusmab...@gmail.com> >>>> wrote: >>>>> >>>>> On Wed, Jan 11, 2017 at 4:14 PM, Nicolai Hähnle &l

Re: [Mesa-dev] [PATCH] glsl: always do sqrt(abs()) and inversesqrt(abs())

2017-01-11 Thread Erik Faye-Lund
On Wed, Jan 11, 2017 at 7:33 PM, Jason Ekstrand <ja...@jlekstrand.net> wrote: > On Wed, Jan 11, 2017 at 10:31 AM, Erik Faye-Lund <kusmab...@gmail.com> > wrote: >> >> On Wed, Jan 11, 2017 at 7:22 PM, Marek Olšák <mar...@gmail.com> wrote: >> > On Wed

Re: [Mesa-dev] [PATCH] glsl: always do sqrt(abs()) and inversesqrt(abs())

2017-01-11 Thread Erik Faye-Lund
On Wed, Jan 11, 2017 at 9:22 PM, Samuel Pitoiset <samuel.pitoi...@gmail.com> wrote: > > > On 01/11/2017 07:34 PM, Erik Faye-Lund wrote: >> >> On Wed, Jan 11, 2017 at 7:33 PM, Jason Ekstrand <ja...@jlekstrand.net> >> wrote: >>> >>>

Re: [Mesa-dev] [PATCH] glsl: always do sqrt(abs()) and inversesqrt(abs())

2017-01-11 Thread Erik Faye-Lund
On Wed, Jan 11, 2017 at 9:42 PM, Erik Faye-Lund <kusmab...@gmail.com> wrote: > On Wed, Jan 11, 2017 at 9:22 PM, Samuel Pitoiset > <samuel.pitoi...@gmail.com> wrote: >> >> >> On 01/11/2017 07:34 PM, Erik Faye-Lund wrote: >>> >>> On Wed, Jan 11,

Re: [Mesa-dev] [PATCH] glsl: always do sqrt(abs()) and inversesqrt(abs())

2017-01-11 Thread Erik Faye-Lund
On Wed, Jan 11, 2017 at 9:49 PM, Erik Faye-Lund <kusmab...@gmail.com> wrote: > On Wed, Jan 11, 2017 at 9:42 PM, Erik Faye-Lund <kusmab...@gmail.com> wrote: >> On Wed, Jan 11, 2017 at 9:22 PM, Samuel Pitoiset >> <samuel.pitoi...@gmail.com> wrote: >>> >

Re: [Mesa-dev] [PATCH 3/7] gallium: add FBFETCH opcode to retrieve the current sample value

2017-01-04 Thread Erik Faye-Lund
Ah, right. Thanks for the explanation! On Jan 4, 2017 14:02, "Ilia Mirkin" <imir...@alum.mit.edu> wrote: > On Wed, Jan 4, 2017 at 4:24 AM, Erik Faye-Lund <kusmab...@gmail.com> > wrote: > > On Jan 2, 2017 06:03, "Ilia Mirkin" <imir...@alum.mit.edu&g

Re: [Mesa-dev] [PATCH v2 17/25] spirv: add support for doubles to OpSpecConstant

2017-01-04 Thread Erik Faye-Lund
On Jan 4, 2017 14:45, "Jason Ekstrand" <ja...@jlekstrand.net> wrote: On Jan 4, 2017 4:54 AM, "Erik Faye-Lund" <kusmab...@gmail.com> wrote: On Jan 3, 2017 16:34, "Jason Ekstrand" <ja...@jlekstrand.net> wrote: On Fri, Dec 16, 2016 at 6:49 AM, J

Re: [Mesa-dev] [PATCH v3 06/22] spirv: fix SpvOpSpecConstantOp with SpvOpVectorShuffle working with double-based vecs

2017-01-05 Thread Erik Faye-Lund
On Jan 5, 2017 10:20, "Samuel Iglesias Gonsálvez" wrote: We need to pick two 32-bit values per component to perform the right shuffle operation. v2 (Jason): - Add assert to check matching bit sizes (Jason) - Simplify the code to pick components (Jason) Signed-off-by:

Re: [Mesa-dev] [PATCH 3/7] gallium: add FBFETCH opcode to retrieve the current sample value

2017-01-04 Thread Erik Faye-Lund
On Jan 2, 2017 06:03, "Ilia Mirkin" wrote: Signed-off-by: Ilia Mirkin --- src/gallium/auxiliary/tgsi/tgsi_info.c | 2 +- src/gallium/docs/source/tgsi.rst | 11 +++ src/gallium/include/pipe/p_shader_tokens.h | 2 +- 3 files

[Mesa-dev] [PATCH 0/3] misc cleanups

2017-03-31 Thread Erik Faye-Lund
Here's a few cleanup-patches I've had in my tree for a while, that I thought I should send upstream soon. Erik Faye-Lund (3): st/mesa: avoid aliasing violation in st_cb_perfmon.c gallium/docs: remove documentation of removed arg glsl: ir_explog_to_explog2 is no more src/gallium/docs

[Mesa-dev] [PATCH 1/3] st/mesa: avoid aliasing violation in st_cb_perfmon.c

2017-03-31 Thread Erik Faye-Lund
Signed-off-by: Erik Faye-Lund <kusmab...@gmail.com> --- src/mesa/state_tracker/st_cb_perfmon.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mesa/state_tracker/st_cb_perfmon.c b/src/mesa/state_tracker/st_cb_perfmon.c index cd7fdc0..23d32dc 100644 --- a/sr

[Mesa-dev] [PATCH 3/3] glsl: ir_explog_to_explog2 is no more

2017-03-31 Thread Erik Faye-Lund
rrent tree if they encounter it. Signed-off-by: Erik Faye-Lund <kusmab...@gmail.com> --- src/mesa/program/ir_to_mesa.cpp| 4 +++- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/mesa/program/ir_to_mesa.cpp b/

[Mesa-dev] [PATCH 2/3] gallium/docs: remove documentation of removed arg

2017-03-31 Thread Erik Faye-Lund
geom was removed in e968975 ("gallium: remove the geom_flags param from is_format_supported", Tue Mar 8 00:01:58 2011 +0100), but the documentation of it was left over. Let's bring the documentation up to date. Signed-off-by: Erik Faye-Lund <kusmab...@gmail.com> --- src/gal

Re: [Mesa-dev] [PATCH 0/3] misc cleanups

2017-04-01 Thread Erik Faye-Lund
Thanks, pushed. On Fri, Mar 31, 2017 at 4:42 PM, Nicolai Hähnle <nhaeh...@gmail.com> wrote: > Series is > > Reviewed-by: Nicolai Hähnle <nicolai.haeh...@amd.com> > > On 31.03.2017 12:57, Erik Faye-Lund wrote: >> >> Here's a few cleanup-patches I've had in m

Re: [Mesa-dev] [PATCH 2/2] vc4: Use runtime CPU detection for whether NEON is available.

2017-04-17 Thread Erik Faye-Lund
On Apr 13, 2017 20:59, "Eric Anholt" wrote: This will allow Raspbian's ARMv6 builds to take advantage of the new NEON code, and could prevent problems if vc4 ends up getting used on a v7 CPU without NEON. --- src/gallium/drivers/vc4/vc4_screen.c | 3 +++

Re: [Mesa-dev] [PATCH 03/22] glformats: Add a mapping for MESA_FORMAT_S_UINT8

2017-08-10 Thread Erik Faye-Lund
On Sat, Aug 5, 2017 at 11:39 AM, Chris Wilson wrote: > Map the user format of GL_DEPTH_COMPONENT, GL_UNSIGNED_BYTE to the > internal format of MESA_FORMAT_S_UINT8. > --- > src/mesa/main/glformats.c | 4 > 1 file changed, 4 insertions(+) > > diff --git

Re: [Mesa-dev] [RFC PATCH 2/2] main/format: skip format conversion if src and dst format are equal

2017-08-17 Thread Erik Faye-Lund
On Wed, Aug 16, 2017 at 8:32 PM, Karol Herbst wrote: > Fixes 'KHR-GL45.copy_image.functional' on Nouveau > > Signed-off-by: Karol Herbst > --- > src/mesa/main/format_utils.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git

Re: [Mesa-dev] [PATCH 1/3] vbo: simplify vbo_save_NotifyBegin()

2017-07-10 Thread Erik Faye-Lund
Reviewed-by: Erik Faye-Lund <kusmab...@gmail.com> On Fri, Jul 7, 2017 at 4:11 PM, Brian Paul <bri...@vmware.com> wrote: > This function always returned GL_TRUE. Just make it a void function. > Remove unreachable code following the call to vbo_save_NotifyBegin() > in sa

Re: [Mesa-dev] [PATCH 7/9] spirv: Add vtn_fail and vtn_assert helpers

2017-08-17 Thread Erik Faye-Lund
On Thu, Aug 17, 2017 at 7:22 PM, Jason Ekstrand wrote: > These helpers are much nicer than just using assert because they don't > kill your process. Instead, it longjmps back to spirv_to_nir(), cleans > up all the temporary memory, and nicely returns NULL. While crashing

Re: [Mesa-dev] [PATCH 2/5] glsl: Ban #undefining __LINE__ and friends on GLES2.

2017-05-02 Thread Erik Faye-Lund
On Tue, May 2, 2017 at 7:36 PM, Eric Anholt wrote: > Fixes deqp_gles2 undefine_invalid_object_* failures. > --- > src/compiler/glsl/glcpp/glcpp-parse.y | 7 ++- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/src/compiler/glsl/glcpp/glcpp-parse.y >

Re: [Mesa-dev] [PATCH 2/5] glsl: Ban #undefining __LINE__ and friends on GLES2.

2017-05-02 Thread Erik Faye-Lund
On Wed, May 3, 2017 at 1:23 AM, Eric Anholt <e...@anholt.net> wrote: > Erik Faye-Lund <kusmab...@gmail.com> writes: > >> On Tue, May 2, 2017 at 7:36 PM, Eric Anholt <e...@anholt.net> wrote: >>> Fixes deqp_gles2 undefine_invalid_object_* failures. >>>

Re: [Mesa-dev] [PATCH 0/9] glsl: cleanup and fix handling of unnamed struct types

2017-05-15 Thread Erik Faye-Lund
On Mon, May 15, 2017 at 11:27 AM, Nicolai Hähnle wrote: > Hi all, > > This series aims to simplify how we handle unnamed struct types and fix some > bugs on the way. Some of the patches should be uncontroversial, and all of > them align with my understanding of the GLSL spec,

Re: [Mesa-dev] [PATCH 4/9] glcpp: Avoid unnecessary strcmp()

2017-05-22 Thread Erik Faye-Lund
On Sun, May 21, 2017 at 10:49 PM, Thomas Helland wrote: > From: Vladislav Egorov > > strcmp() is slow. Initiate comparison with "__LINE__" or "__FILE__" > only if the identifier starts with '_', which is rare. > > Reviewed-by: Ian Romanick > ---

Re: [Mesa-dev] [PATCH 2/4] gallium: Enable ARM NEON CPU detection.

2017-05-01 Thread Erik Faye-Lund
On Mon, Apr 24, 2017 at 11:59 PM, Eric Anholt wrote: > I wrote this code with reference to pixman, though I've only decided to > cover Linux (what I'm testing) and Android (seems obvious enough). Linux > has getauxval() as a cleaner interface to the /proc entry, but it's more >

Re: [Mesa-dev] [PATCH 01/21] mesa: add vertex_array_vertex_buffers_err() helper

2017-06-01 Thread Erik Faye-Lund
On Jun 1, 2017 15:06, "Samuel Pitoiset" wrote: Signed-off-by: Samuel Pitoiset --- src/mesa/main/varray.c | 108 -- --- 1 file changed, 61 insertions(+), 47 deletions(-) diff --git

Re: [Mesa-dev] [PATCH] Replace byte-swapping code with builtins in pack.c

2017-10-06 Thread Erik Faye-Lund
On Thu, Oct 5, 2017 at 8:59 PM, Jochen Rollwagen wrote: > Am 04.10.2017 um 05:59 schrieb Matt Turner: >> >> On Tue, Oct 3, 2017 at 11:01 AM, Jochen Rollwagen >> wrote: >>> >>> From 4cebe50a9bade6717923e104c954f3fad75f71bb Mon Sep 17 00:00:00 2001

Re: [Mesa-dev] [PATCH 00/20] meson, gallium edition

2017-10-12 Thread Erik Faye-Lund
On Wed, Oct 11, 2017 at 7:41 PM, Dylan Baker wrote: > This series is somewhat larger than the one to enable classic drivers. I've > made > an effort to split the patches up better this go around, so hopefully that > will > make review easier. > > I have enabled 3 drivers

Re: [Mesa-dev] [PATCH 01/15] mesa: Import libtxc_dxtn sources

2017-10-03 Thread Erik Faye-Lund
On Mon, Oct 2, 2017 at 2:57 PM, Emil Velikov wrote: > On 2 October 2017 at 07:59, Matt Turner wrote: >> Straight import of the libtxc_dxtn sources. Will be squashed with next >> commit that reorganizes them into a more familiar form > > The code

Re: [Mesa-dev] TGSI 16-bit support

2017-08-24 Thread Erik Faye-Lund
On Wed, Aug 23, 2017 at 6:19 PM, Marek Olšák wrote: > On Wed, Aug 23, 2017 at 3:08 PM, Nicolai Hähnle wrote: >> >> Here's another question: What does "low precision" mean on a texture >> instruction? Are the offsets low precision or is it the output? Maybe

Re: [Mesa-dev] [PATCH 1/2] gallium: Add PIPE_CAP_HALFS

2017-09-04 Thread Erik Faye-Lund
On Mon, Sep 4, 2017 at 4:13 PM, Jan Vesely <jan.ves...@rutgers.edu> wrote: > On Sat, 2017-09-02 at 22:21 +0200, Erik Faye-Lund wrote: >> On Sat, Sep 2, 2017 at 2:55 AM, Jan Vesely <jan.ves...@rutgers.edu> wrote: >> > Denotes native half precision float operations c

Re: [Mesa-dev] [PATCH 1/2] gallium: Add PIPE_CAP_HALFS

2017-09-02 Thread Erik Faye-Lund
On Sat, Sep 2, 2017 at 2:55 AM, Jan Vesely wrote: > Denotes native half precision float operations capability > > Signed-off-by: Jan Vesely > --- > I can change the spelling to HALVES, but simplified english sounded more > appropriate. > >

Re: [Mesa-dev] [PATCH] st/st_glsl_to_nir: call nir_lower_64bit_pack

2017-12-16 Thread Erik Faye-Lund
On Thu, Dec 14, 2017 at 6:02 AM, Timothy Arceri wrote: > Fixes 56 crashes in radeonsi. 56 crashes with what? Piglit? ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] etnaviv: Fix point sprite issue on HALTI0

2017-11-20 Thread Erik Faye-Lund
On Thu, Nov 16, 2017 at 6:26 PM, Ilia Mirkin wrote: > On Thu, Nov 16, 2017 at 7:15 AM, Wladimir wrote: >>> I think it would be reasonable to re-emit the shader state (or maybe >>> just the varyings) when the primitive type changes from points to >>>

Re: [Mesa-dev] [PATCH 2/4] st/mesa: use enum types instead of int/unsigned (v3)

2017-11-08 Thread Erik Faye-Lund
On Wed, Nov 8, 2017 at 1:07 AM, Brian Paul wrote: > Use the proper enum types for various variables. Makes life in gdb > a little nicer. Note that the size of enum bitfields must be one > larger so the high bit is always zero (for MSVC). You *could* also do something like

Re: [Mesa-dev] [PATCH mesa] meson: be explicit about the version required

2017-10-25 Thread Erik Faye-Lund
On Tue, Oct 24, 2017 at 11:48 PM, Dylan Baker wrote: > Quoting Eric Engestrom (2017-10-24 10:32:36) >> On Tuesday, 2017-10-24 09:40:22 -0700, Dylan Baker wrote: >> > This seems reasonable, could you wrap the hanging indent like >> > meson_options with >> > the closing brace

[Mesa-dev] [PATCH] meson: add opt-out of libunwind

2017-10-24 Thread Erik Faye-Lund
Libunwind has some issues on some platforms, so let's allow people who have issues to opt-out. This is similar to what we do in automake, and the implementation is modelled after our opt-out for valgrind. Signed-off-by: Erik Faye-Lund <kusmab...@gmail.com> --- This fixes a build-problem

Re: [Mesa-dev] [PATCH] meson: add opt-out of libunwind

2017-10-25 Thread Erik Faye-Lund
On Wed, Oct 25, 2017 at 8:10 AM, Gert Wollny <gw.foss...@gmail.com> wrote: > Am Dienstag, den 24.10.2017, 16:44 +0200 schrieb Erik Faye-Lund: >> >> dep_unwind = dependency('libunwind', required : false) >> -if dep_unwind.found() >> +if dep_unwind.found()

[Mesa-dev] [PATCH] meson: do not search for needless deps

2017-10-25 Thread Erik Faye-Lund
If we don't want to use these deps, there's no good reason to search for them in the first place. This should shave a bit of time for the initial build. --- This would be a way of dealing with Gert's suggestion. Goes on top of the previous patch. Thoughts? meson.build | 20 ++--

[Mesa-dev] [PATCH] meson: use dep_m in libgallium

2017-10-30 Thread Erik Faye-Lund
The u_format_other.c users sqrtf, which on some systems require a math-library. So let's make sure we link with it. Signed-off-by: Erik Faye-Lund <kusmab...@gmail.com> --- I noticed this while debugging something else, thought I'd just send it upstream directly. src/gallium/aux

Re: [Mesa-dev] [PATCH 1/2] meson: do not search for needless deps

2017-10-31 Thread Erik Faye-Lund
On Tue, Oct 31, 2017 at 12:04 PM, Eric Engestrom <eric.engest...@imgtec.com> wrote: > On Tuesday, 2017-10-31 11:37:25 +0100, Erik Faye-Lund wrote: >> On Tue, Oct 31, 2017 at 11:24 AM, Eric Engestrom >> <eric.engest...@imgtec.com> wrote: >> > On Tuesday, 2017-

[Mesa-dev] [PATCH 2/2] meson: prefer 'auto', 'true', 'false'

2017-10-31 Thread Erik Faye-Lund
This way users don't have to care if these options are boolean or not, as they take the same values (apart from 'auto'). Signed-off-by: Erik Faye-Lund <kusmab...@gmail.com> --- I'm not quite sure about this patch. Yes, it cleans up the semantics, but at the same time, it breaks bac

[Mesa-dev] [PATCH 1/2] meson: do not search for needless deps

2017-10-31 Thread Erik Faye-Lund
If we don't want to use these deps, there's no good reason to search for them in the first place. This should shave a bit of time for the initial build. Signed-off-by: Erik Faye-Lund <kusmab...@gmail.com> --- meson.build | 20 ++-- meson_options.txt | 14 ---

Re: [Mesa-dev] [PATCH] meson: use dep_m in libgallium

2017-10-31 Thread Erik Faye-Lund
On Tue, Oct 31, 2017 at 10:31 AM, Eric Engestrom <eric.engest...@imgtec.com> wrote: > On Monday, 2017-10-30 21:06:25 +0100, Erik Faye-Lund wrote: >> The u_format_other.c users sqrtf, which on some systems require >> a math-library. So let's make sure we link with it. >>

Re: [Mesa-dev] [PATCH] meson: use dep_m in libgallium

2017-10-31 Thread Erik Faye-Lund
On Tue, Oct 31, 2017 at 11:33 AM, Erik Faye-Lund <kusmab...@gmail.com> wrote: > On Tue, Oct 31, 2017 at 10:31 AM, Eric Engestrom > <eric.engest...@imgtec.com> wrote: >> On Monday, 2017-10-30 21:06:25 +0100, Erik Faye-Lund wrote: >>> The u_format_other.c users sqr

Re: [Mesa-dev] [PATCH 1/2] meson: do not search for needless deps

2017-10-31 Thread Erik Faye-Lund
On Tue, Oct 31, 2017 at 11:24 AM, Eric Engestrom <eric.engest...@imgtec.com> wrote: > On Tuesday, 2017-10-31 08:29:28 +0100, Erik Faye-Lund wrote: >> If we don't want to use these deps, there's no good reason to search >> for them in the first place. This shoul

Re: [Mesa-dev] [PATCH] Don't set swap interval for PBuffer surface. This fixes crash due to NULL window.

2018-04-28 Thread Erik Faye-Lund
On Fri, Apr 27, 2018 at 11:17 AM, samiuddi wrote: > Test: CtsDisplayTestCases pass > > Signed-off-by: samiuddi > --- > src/egl/drivers/dri2/platform_android.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git

Re: [Mesa-dev] [PATCH 00/16] Move the Mesa Website to Sphinx

2018-06-12 Thread Erik Faye-Lund
On Tue, Jun 12, 2018 at 1:59 PM Ilia Mirkin wrote: > > On Tue, Jun 12, 2018 at 7:04 AM, Erik Faye-Lund wrote: > > I've updated my attempt here, to now have the original colors, and > > centered the M-corners on the text a bit more properly: > > > > https://codepen.

Re: [Mesa-dev] [PATCH 00/16] Move the Mesa Website to Sphinx

2018-06-12 Thread Erik Faye-Lund
On Tue, Jun 12, 2018 at 2:12 PM Erik Faye-Lund wrote: > > On Tue, Jun 12, 2018 at 1:59 PM Ilia Mirkin wrote: > > > > On Tue, Jun 12, 2018 at 7:04 AM, Erik Faye-Lund wrote: > > > I've updated my attempt here, to now have the original colors, and > > > cent

Re: [Mesa-dev] [PATCH 00/16] Move the Mesa Website to Sphinx

2018-06-12 Thread Erik Faye-Lund
I've updated my attempt here, to now have the original colors, and centered the M-corners on the text a bit more properly: https://codepen.io/kusma/pen/vrXppL On Tue, Jun 12, 2018 at 2:51 AM Rob Clark wrote: > > On Mon, Jun 11, 2018 at 6:24 PM, Laura Ekstrand wrote: > > I really like the

Re: [Mesa-dev] [PATCH 00/16] Move the Mesa Website to Sphinx

2018-06-08 Thread Erik Faye-Lund
On Fri, Jun 8, 2018 at 4:09 PM Rhys Perry wrote: > > Might be good to do something like this: https://codepen.io/anon/pen/ERNdYJ > So that those with NoScript or something won't have gears constantly > rotating on their screen. > Yeah, good point. ___

Re: [Mesa-dev] [PATCH 00/16] Move the Mesa Website to Sphinx

2018-06-08 Thread Erik Faye-Lund
On Fri, Jun 8, 2018 at 2:06 PM, Rob Clark wrote: > On Fri, Jun 8, 2018 at 3:02 AM, Jordan Justen wrote: >> On Thu, Jun 7, 2018 at 2:56 AM Eero Tamminen >> wrote: >>> On 07.06.2018 12:01, Erik Faye-Lund wrote: >>> > Just as a fun toy, I decide

Re: [Mesa-dev] [PATCH 24/48] docs/meson.html: fix numerous issues spotted by xmllint

2018-06-17 Thread Erik Faye-Lund
On Tue, Jun 12, 2018 at 1:03 AM Dylan Baker wrote: > > --- > docs/meson.html | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/docs/meson.html b/docs/meson.html > index 6697dbb34ef..7541e7a71b0 100644 > --- a/docs/meson.html > +++ b/docs/meson.html > @@ -1,9 +1,9 @@ >

Re: [Mesa-dev] [PATCH 40/48] meosn: don't build gallium trivial tests on windows

2018-06-17 Thread Erik Faye-Lund
Typo in the subject, should be "meson", not "meosn". On Tue, Jun 12, 2018 at 1:03 AM Dylan Baker wrote: > > They require the pipe-loaders, which require xmlconfig, which doesn't > build with msvc. > --- > src/gallium/tests/meson.build | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-)

Re: [Mesa-dev] [PATCH 00/16] Move the Mesa Website to Sphinx

2018-06-07 Thread Erik Faye-Lund
Just as a fun toy, I decided to give an animated SVG "variation" of this a go myself: https://codepen.io/kusma/pen/vrXppL The actual SVG can be found here: https://gitlab.freedesktop.org/snippets/492 The gears were generated by this python script, based on the glxgears source code:

Re: [Mesa-dev] [PATCH 00/16] Move the Mesa Website to Sphinx

2018-06-07 Thread Erik Faye-Lund
ers > > Mike > > On Thu, 7 Jun 2018 at 10:02 Erik Faye-Lund wrote: >> >> Just as a fun toy, I decided to give an animated SVG "variation" of >> this a go myself: >> >> https://codepen.io/kusma/pen/vrXppL >> >> The actual SVG

Re: [Mesa-dev] GitLab migration update

2018-06-07 Thread Erik Faye-Lund
On Thu, Jun 7, 2018 at 9:25 AM, Daniel Stone wrote: > Hi, > > On 2 June 2018 at 16:45, Jason Ekstrand wrote: >> When the migration happens, there will be a very brief (less than 30 >> minutes?) period during which no one will be able to push to either repo. >> I'll fire off an e-mail as well as

Re: [Mesa-dev] [PATCH 2/2] freedreno: a2xx: fix crash when freeing context

2018-06-26 Thread Erik Faye-Lund
On Wed, Jun 20, 2018 at 3:03 AM Jonathan Marek wrote: > > Signed-off-by: Jonathan Marek > --- > src/gallium/drivers/freedreno/a2xx/fd2_program.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/gallium/drivers/freedreno/a2xx/fd2_program.c >

<    1   2   3   4   5   6   7   8   >