Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Dylan Baker
Quoting Rob Clark (2018-03-16 14:25:09) > On Fri, Mar 16, 2018 at 4:30 PM, Dylan Baker wrote: > > Quoting Rob Clark (2018-03-16 12:20:10) > >> On Fri, Mar 16, 2018 at 3:13 PM, Jason Ekstrand > >> wrote: > >> > On Fri, Mar 16, 2018 at 11:53 AM, Dylan Baker > >> > wrote: > >> >> > >> >> Quoting

[Mesa-dev] [Bug 105551] Unable to compile on FreeBSD 9.x and therefore derivatives... undefined reference to `loader_get_extensions_name'

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105551 --- Comment #3 from Michelle Sullivan --- It would appear that the -shared flag is being added by configure... # FreeBSD 3 and greater uses gcc -shared to do shared libraries. freebsd* | dragonfly*) archive_cmds='$CC -shared $pic_

Re: [Mesa-dev] [FLAG-DAY-PREP 0/9] nir: accessors for deref vars

2018-03-16 Thread Jason Ekstrand
On March 16, 2018 18:04:26 Rob Clark wrote: In ir3, and some of the lowering passes, pretty much 100% of the spots we reach into deref chain is just to find the nir_variable.. in amd, it is less (nir->llvm does a lot more with deref chains).. i965 was kinda in the middle. And from what I could

[Mesa-dev] [PATCH] virgl: Implement seamless cube maps

2018-03-16 Thread Stéphane Marchesin
This was previously ignored. Along with the virglrenderer patch, this fixes ~100 dEQP tests: dEQP-GLES3.functional.texture.filtering.cube.* Signed-off-by: Stéphane Marchesin --- src/gallium/drivers/virgl/virgl_encode.c | 3 ++- src/gallium/drivers/virgl/virgl_protocol.h | 1 + 2 files changed

[Mesa-dev] [PATCH] nir/dead_cf: also remove useless ifs

2018-03-16 Thread Caio Marcelo de Oliveira Filho
Generalize the code for remove dead loops to also remove dead if nodes. The conditions are the same in both cases, if the node (and it's children) don't have side-effects AND the nodes after it don't use the values produced by the node. The only difference is when evaluating side effects: loops co

[Mesa-dev] [Bug 105171] performance regression (3x slower) running glamor with PutImage workload (radeonsi)

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105171 --- Comment #18 from Dieter Nützel --- (In reply to Dieter Nützel from comment #17) > (In reply to Clemens Eisserer from comment #16) > > some here, on my Kaveri 7650k results with the patch are basically unchanged > > : > > > > amdgpu: > > 855

[Mesa-dev] [Bug 104926] swrast: Mesa 17.3.3 produces: HW cursor for format 875713089 not supported

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104926 --- Comment #5 from Wayne Blaszczyk --- The above patch has fixed the issue for me. Thanks! -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the bug._

Re: [Mesa-dev] [FLAG-DAY-PREP 0/9] nir: accessors for deref vars

2018-03-16 Thread Rob Clark
In ir3, and some of the lowering passes, pretty much 100% of the spots we reach into deref chain is just to find the nir_variable.. in amd, it is less (nir->llvm does a lot more with deref chains).. i965 was kinda in the middle. And from what I could tell, vc4/vc5 didn't really care (tbf, most of

[Mesa-dev] [Bug 105171] performance regression (3x slower) running glamor with PutImage workload (radeonsi)

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105171 --- Comment #17 from Dieter Nützel --- (In reply to Clemens Eisserer from comment #16) > some here, on my Kaveri 7650k results with the patch are basically unchanged > : > > amdgpu: > 8557.942992 Ops/s; put composition (!); 32x32 > > should I

Re: [Mesa-dev] [FLAG-DAY-PREP 0/9] nir: accessors for deref vars

2018-03-16 Thread Jason Ekstrand
I'm honestly not sure how helpful this is. At least in all the places where this series affects our drivers, more serious changes are needed. On Fri, Mar 16, 2018 at 3:45 PM, Bas Nieuwenhuizen wrote: > FWIW > > Reviewed-by: Bas Nieuwenhuizen > > for the series. > > On Thu, Mar 15, 2018 at 3:27

[Mesa-dev] [PATCH] i965: Add INTEL_DEBUG stages support for disk shader cache

2018-03-16 Thread Jordan Justen
Cc: Kenneth Graunke Cc: Matt Turner Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_disk_cache.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_disk_cache.c b/src/mesa/drivers/dri/i965/brw_disk_cache.c index 41f742e8

[Mesa-dev] [Bug 105567] meson/ninja: 1. mesa/vdpau incorrect symlinks in DESTDIR and 2. Ddri-drivers-path Dvdpau-libs-path overrides DESTDIR

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105567 Bug ID: 105567 Summary: meson/ninja: 1. mesa/vdpau incorrect symlinks in DESTDIR and 2. Ddri-drivers-path Dvdpau-libs-path overrides DESTDIR Product: Mesa Versio

Re: [Mesa-dev] [FLAG-DAY-PREP 0/9] nir: accessors for deref vars

2018-03-16 Thread Bas Nieuwenhuizen
FWIW Reviewed-by: Bas Nieuwenhuizen for the series. On Thu, Mar 15, 2018 at 3:27 PM, Rob Clark wrote: > Since a fair bit of access to nir_deref_var's is just to chase down the > nir_variable for an intrinsic (or texture/sampler vars for tex > instructions), we can make the flag day slightly le

[Mesa-dev] [PATCH shaderdb] run: shader program file created via GetProgramBinary (v4)

2018-03-16 Thread Dongwon Kim
With optin '-b', shader-db now generates a shader program binary file using GetProgramBinary(). This shader program binary can be loaded via ProgramBinary() to be executed by an application later. v2: 1. define MAX_LOG_LEN and use it as the size of gl log 2. define MAX_PROG_SIZE and use it as

Re: [Mesa-dev] [PATCH 2/2] gk110/ir: always use limm form for log ops with immediates

2018-03-16 Thread Karol Herbst
On Fri, Mar 16, 2018 at 11:10 PM, Ilia Mirkin wrote: > You got the subjects backwards. > > On Fri, Mar 16, 2018 at 6:05 PM, Karol Herbst wrote: >> in the short imm form 0x8 was sign-extended to 0xfff8 which leads to >> wrong results. >> >> Signed-off-by: Karol Herbst >> --- >> src/galli

[Mesa-dev] [PATCH] intel/blorp: Fix compiler warning about num_layers.

2018-03-16 Thread Eric Anholt
The compiler doesn't notice that the condition for num_layers to be undefined already defined it above (as our assert checked in a debug build). v2: Move the pair of assignments to one outside of the block. Cc: Jason Ekstrand Cc: Lionel Landwerlin --- src/mesa/drivers/dri/i965/brw_blorp.c | 2

Re: [Mesa-dev] [PATCH 1/4] intel/blorp: Fix compiler warning about num_layers.

2018-03-16 Thread Eric Anholt
Lionel Landwerlin writes: > On 13/03/18 17:38, Eric Anholt wrote: >> The compiler doesn't notice that the condition for num_layers to be >> undefined already defined it above (as our assert checked in a debug >> build). >> >> Cc: Jason Ekstrand >> --- >> src/mesa/drivers/dri/i965/brw_blorp.c |

Re: [Mesa-dev] [PATCH 2/2] gk110/ir: always use limm form for log ops with immediates

2018-03-16 Thread Ilia Mirkin
You got the subjects backwards. On Fri, Mar 16, 2018 at 6:05 PM, Karol Herbst wrote: > in the short imm form 0x8 was sign-extended to 0xfff8 which leads to > wrong results. > > Signed-off-by: Karol Herbst > --- > src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp | 2 +- > 1 file

[Mesa-dev] [PATCH 1/2] nvc0/ir: always use limm form for log ops with immediates

2018-03-16 Thread Karol Herbst
in the short imm form 0x8 was sign-extended to 0xfff8 which leads to wrong results. Signed-off-by: Karol Herbst --- src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_e

[Mesa-dev] [PATCH 2/2] gk110/ir: always use limm form for log ops with immediates

2018-03-16 Thread Karol Herbst
in the short imm form 0x8 was sign-extended to 0xfff8 which leads to wrong results. Signed-off-by: Karol Herbst --- src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_em

Re: [Mesa-dev] [PATCH] nir: add some comparison simplifications

2018-03-16 Thread Timothy Arceri
Ok feel free to push your patch with : Reviewed-by: Timothy Arceri On 17/03/18 02:19, Ian Romanick wrote: On 03/16/2018 07:37 AM, Ian Romanick wrote: I have basically the same patch setting in my simple-range-analysis tree. https://cgit.freedesktop.org/~idr/mesa/commit/?h=simple-range-analys

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Rob Clark
On Fri, Mar 16, 2018 at 4:30 PM, Dylan Baker wrote: > Quoting Rob Clark (2018-03-16 12:20:10) >> On Fri, Mar 16, 2018 at 3:13 PM, Jason Ekstrand wrote: >> > On Fri, Mar 16, 2018 at 11:53 AM, Dylan Baker wrote: >> >> >> >> Quoting Jason Ekstrand (2018-03-16 11:38:47) >> >> > On Fri, Mar 16, 2018

[Mesa-dev] [Bug 105551] Unable to compile on FreeBSD 9.x and therefore derivatives... undefined reference to `loader_get_extensions_name'

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105551 --- Comment #2 from Michelle Sullivan --- Same thing happens without LLVM (first thing I did was strip it all back to the bare minimum.) This is the only one out of 1200 that fails, so there is something specific to do with the setup of this on

Re: [Mesa-dev] [PATCH 2/2] intel/compiler: Use gen_get_device_info() in test_eu_validate

2018-03-16 Thread Dylan Baker
Quoting Rafael Antognolli (2018-03-16 13:19:11) > On Fri, Mar 16, 2018 at 10:56:24AM -0700, Matt Turner wrote: > > Previously the unit test filled out a minimal devinfo struct. A previous > > patch caused the test to begin assert failing because the devinfo was > > not complete. Avoid this by using

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Dylan Baker
Quoting Rob Clark (2018-03-16 12:20:10) > On Fri, Mar 16, 2018 at 3:13 PM, Jason Ekstrand wrote: > > On Fri, Mar 16, 2018 at 11:53 AM, Dylan Baker wrote: > >> > >> Quoting Jason Ekstrand (2018-03-16 11:38:47) > >> > On Fri, Mar 16, 2018 at 11:28 AM, Dylan Baker > >> > wrote: > >> > > >> > in

Re: [Mesa-dev] [PATCH 2/2] intel/compiler: Use gen_get_device_info() in test_eu_validate

2018-03-16 Thread Rafael Antognolli
On Fri, Mar 16, 2018 at 10:56:24AM -0700, Matt Turner wrote: > Previously the unit test filled out a minimal devinfo struct. A previous > patch caused the test to begin assert failing because the devinfo was > not complete. Avoid this by using the real mechanism to create devinfo. > > Note that we

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Rob Clark
On Fri, Mar 16, 2018 at 3:13 PM, Jason Ekstrand wrote: > On Fri, Mar 16, 2018 at 11:53 AM, Dylan Baker wrote: >> >> Quoting Jason Ekstrand (2018-03-16 11:38:47) >> > On Fri, Mar 16, 2018 at 11:28 AM, Dylan Baker >> > wrote: >> > >> > intr_opcodes = { >> > 'nop': Intrinsic('nop', flag

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Jason Ekstrand
On Fri, Mar 16, 2018 at 11:53 AM, Dylan Baker wrote: > Quoting Jason Ekstrand (2018-03-16 11:38:47) > > On Fri, Mar 16, 2018 at 11:28 AM, Dylan Baker > wrote: > > > > intr_opcodes = { > > 'nop': Intrinsic('nop', flags=[CAN_ELIMINATE]), > > ... > > } > > > > I prefer t

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Dylan Baker
Quoting Rob Clark (2018-03-16 11:45:56) > On Fri, Mar 16, 2018 at 2:28 PM, Dylan Baker wrote: > > Quoting Rob Clark (2018-03-15 18:43:28) > >> I threatened to do this a long time ago.. I probably *should* have done > >> it a long time ago when there where many fewer intrinsics. But the > >> syste

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Dylan Baker
Quoting Jason Ekstrand (2018-03-16 11:38:47) > On Fri, Mar 16, 2018 at 11:28 AM, Dylan Baker wrote: > > intr_opcodes = { >     'nop': Intrinsic('nop', flags=[CAN_ELIMINATE]), >     ... > } > > I prefer this since each dictionary is clearly created without a function > obs

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Rob Clark
On Fri, Mar 16, 2018 at 2:28 PM, Dylan Baker wrote: > Quoting Rob Clark (2018-03-15 18:43:28) >> I threatened to do this a long time ago.. I probably *should* have done >> it a long time ago when there where many fewer intrinsics. But the >> system of macro/#include magic for dealing with intrins

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Jason Ekstrand
On Fri, Mar 16, 2018 at 11:28 AM, Dylan Baker wrote: > Quoting Rob Clark (2018-03-15 18:43:28) > > > diff --git a/src/compiler/nir/nir_intrinsics.py b/src/compiler/nir/nir_ > intrinsics.py > > new file mode 100644 > > index 000..6bb6603586e > > --- /dev/null > > +++ b/src/compiler/nir/nir

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 17:33 +0100, Gert Wollny wrote: > Am Freitag, den 16.03.2018, 12:42 +0100 schrieb Juan A. Suarez Romero: > > > > Gert Wollny (1): > > c7cadcbda4 r600: Take ALU_EXTENDED into account when evaluating > > jump offsets > > Reason: The commit requires earlier commit a03d456f

Re: [Mesa-dev] [PATCH mesa 1/3] egl: add Ozone section to eglplatform.h

2018-03-16 Thread Eric Engestrom
On March 16, 2018 5:49:12 PM UTC, Emil Velikov wrote: > On 16 March 2018 at 15:58, Eric Engestrom > wrote: > > This pulls in commit a93f559e9c11fa53fb5f1cc255b8f75433f85d2a "Add > Ozone > > section to eglplatform.h" from Khronos [1] added by Brian Anderson > [2] > > a few months ago. > > > > [1

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Dylan Baker
Quoting Rob Clark (2018-03-15 18:43:28) > I threatened to do this a long time ago.. I probably *should* have done > it a long time ago when there where many fewer intrinsics. But the > system of macro/#include magic for dealing with intrinsics is a bit > annoying, and python has the nice property

Re: [Mesa-dev] [PATCH v2 2/2] anv/pipeline: set active_stages early

2018-03-16 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 15/03/18 20:09, Caio Marcelo de Oliveira Filho wrote: Since the intermediate states of active_stages are not used, i.e. active_stages is read only after all stages were set into it, just set its value before compiling the shaders. This will allow to conditiona

[Mesa-dev] [Bug 105551] Unable to compile on FreeBSD 9.x and therefore derivatives... undefined reference to `loader_get_extensions_name'

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105551 --- Comment #1 from Emil Velikov --- Something really funky is happening on your system. A normal thing like: /bin/sh ../../libtool --tag=CC --mode=link gcc48 -O2 -pipe $(includes) $(warings) $(compiler_f_flags) $(some_rpaths) -o libloader.

Re: [Mesa-dev] [PATCH 2/2] anv/pipeline: set active_stages early

2018-03-16 Thread Caio Marcelo de Oliveira Filho
On Thu, Mar 15, 2018 at 10:31:58AM +, Lionel Landwerlin wrote: > Looks good, just one tiny nit below. Fixed that in v2. > I assume allowing to conditionally run certain passes isn't implemented yet? That is correct. Thanks, Caio ___ mesa-dev mai

[Mesa-dev] [PATCH v5 1/2] gallium/winsys/kms: Fix possible leak in map/unmap.

2018-03-16 Thread Lepton Wu
If user calls map twice for kms_sw_displaytarget, the first mapped buffer could get leaked. Instead of calling mmap every time, just reuse previous mapping. Since user could map same displaytarget with different flags, we have to keep two different pointers, one for rw mapping and one for ro mappin

[Mesa-dev] [PATCH v5 2/2] gallium/winsys/kms: Add support for multi-planes

2018-03-16 Thread Lepton Wu
Add a new struct kms_sw_plane which delegate a plane and use it in place of sw_displaytarget. Multiple planes share same underlying kms_sw_displaytarget. Change-Id: I0e9ca1d0ba0aa78c27dfdb50c30dc0c424fec172 Signed-off-by: Lepton Wu --- .../winsys/sw/kms-dri/kms_dri_sw_winsys.c | 152

[Mesa-dev] [PATCH 2/2] intel/compiler: Use gen_get_device_info() in test_eu_validate

2018-03-16 Thread Matt Turner
Previously the unit test filled out a minimal devinfo struct. A previous patch caused the test to begin assert failing because the devinfo was not complete. Avoid this by using the real mechanism to create devinfo. Note that we have to drop icl from the table, since we now rely on the name -> PCI

[Mesa-dev] [PATCH 1/2] intel: Add cfl to gen_device_name_to_pci_device_id()

2018-03-16 Thread Matt Turner
--- src/intel/dev/gen_device_info.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/intel/dev/gen_device_info.c b/src/intel/dev/gen_device_info.c index 1773009d33c..3365bdd4dd6 100644 --- a/src/intel/dev/gen_device_info.c +++ b/src/intel/dev/gen_device_info.c @@ -55,6 +55,7 @@ gen_device_n

Re: [Mesa-dev] [PATCH mesa 1/3] egl: add Ozone section to eglplatform.h

2018-03-16 Thread Emil Velikov
On 16 March 2018 at 15:58, Eric Engestrom wrote: > This pulls in commit a93f559e9c11fa53fb5f1cc255b8f75433f85d2a "Add Ozone > section to eglplatform.h" from Khronos [1] added by Brian Anderson [2] > a few months ago. > > [1] > https://github.com/KhronosGroup/EGL-Registry/commit/a93f559e9c11fa53fb

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 17:39 +, Emil Velikov wrote: > On 16 March 2018 at 16:56, Mark Janes wrote: > > "Juan A. Suarez Romero" writes: > > > > > On Fri, 2018-03-16 at 12:03 -0400, Ilia Mirkin wrote: > > > > On Fri, Mar 16, 2018 at 11:52 AM, Juan A. Suarez Romero > > > > wrote: > > > > > On F

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Emil Velikov
On 16 March 2018 at 16:56, Mark Janes wrote: > "Juan A. Suarez Romero" writes: > >> On Fri, 2018-03-16 at 12:03 -0400, Ilia Mirkin wrote: >>> On Fri, Mar 16, 2018 at 11:52 AM, Juan A. Suarez Romero >>> wrote: >>> > On Fri, 2018-03-16 at 11:38 -0400, Ilia Mirkin wrote: >>> > > On Fri, Mar 16, 201

Re: [Mesa-dev] [PATCH 0/6] Shader cache; transform feedback; i965 program-binary

2018-03-16 Thread Jordan Justen
On 2018-03-16 04:52:55, Tapani Pälli wrote: > > > On 03/15/2018 11:07 AM, Tapani Pälli wrote: > > Patches 1-3: > > Reviewed-by: Tapani Pälli > > Make that the whole series! Thanks! > I tested also briefly with app that uses glProgramBinary to implement > cache and verified result of GL_LINK_

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Mark Janes
"Juan A. Suarez Romero" writes: > On Fri, 2018-03-16 at 12:03 -0400, Ilia Mirkin wrote: >> On Fri, Mar 16, 2018 at 11:52 AM, Juan A. Suarez Romero >> wrote: >> > On Fri, 2018-03-16 at 11:38 -0400, Ilia Mirkin wrote: >> > > On Fri, Mar 16, 2018 at 11:27 AM, Juan A. Suarez Romero >> > > wrote: >>

[Mesa-dev] [Bug 105208] [regression] (e5ff036c67 st/dri: Add support for BGR[A/X]1010102 formats) broke the mouse in mutter/shell

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105208 Yanko Kaneti changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH v2 3/7] nir: lower 64bit subgroup shuffle intrinsics

2018-03-16 Thread Jason Ekstrand
On Fri, Mar 16, 2018 at 2:50 AM, Daniel Schürmann < daniel.schuerm...@campus.tu-berlin.de> wrote: > Signed-off-by: Daniel Schürmann > --- > src/compiler/nir/nir.h | 1 + > src/compiler/nir/nir_lower_subgroups.c | 83 > +++--- > 2 files changed, 67 ins

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Gert Wollny
Am Freitag, den 16.03.2018, 12:42 +0100 schrieb Juan A. Suarez Romero: > > Gert Wollny (1): > c7cadcbda4 r600: Take ALU_EXTENDED into account when evaluating > jump offsets > Reason: The commit requires earlier commit a03d456f5a41 which did not > land in branch. This seems to be odd, a03d456

[Mesa-dev] [Bug 105232] piglit regressions and crashes from gbm alpha masks

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105232 --- Comment #2 from Daniel Stone --- (In reply to Ilia Mirkin from comment #1) > Theory: The tests didn't run prior to this commit because of the format > matching fail, since the original commit that broke argb8 formats, > 2ed344645d65. They ha

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 12:03 -0400, Ilia Mirkin wrote: > On Fri, Mar 16, 2018 at 11:52 AM, Juan A. Suarez Romero > wrote: > > On Fri, 2018-03-16 at 11:38 -0400, Ilia Mirkin wrote: > > > On Fri, Mar 16, 2018 at 11:27 AM, Juan A. Suarez Romero > > > wrote: > > > > On Fri, 2018-03-16 at 09:51 -0400,

Re: [Mesa-dev] [PATCH v2 2/7] nir/spirv: propagate constants of GroupNonUniformQuad instructions, eliminate warning and fix breaks

2018-03-16 Thread Jason Ekstrand
On Fri, Mar 16, 2018 at 2:50 AM, Daniel Schürmann < daniel.schuerm...@campus.tu-berlin.de> wrote: > Signed-off-by: Daniel Schürmann > --- > src/compiler/spirv/spirv_to_nir.c | 2 ++ > src/compiler/spirv/vtn_subgroup.c | 8 ++-- > 2 files changed, 8 insertions(+), 2 deletions(-) > > diff --gi

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Ilia Mirkin
On Fri, Mar 16, 2018 at 11:52 AM, Juan A. Suarez Romero wrote: > On Fri, 2018-03-16 at 11:38 -0400, Ilia Mirkin wrote: >> On Fri, Mar 16, 2018 at 11:27 AM, Juan A. Suarez Romero >> wrote: >> > On Fri, 2018-03-16 at 09:51 -0400, Ilia Mirkin wrote: >> > > On Fri, Mar 16, 2018 at 8:40 AM, Juan A. Su

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 09:51 -0400, Ilia Mirkin wrote: > On Fri, Mar 16, 2018 at 8:40 AM, Juan A. Suarez Romero > wrote: > > Nominated means that these patches does not enter in this release due they > > arrived a bit late, but they are proposed to cherry-pick them for the next > > release (in 1 or

[Mesa-dev] [PATCH mesa 3/3] egl: pull update from Khronos and drop local define

2018-03-16 Thread Eric Engestrom
Added in Khronos in 2b6bb4ee45cc46c89d4a "EGL_MESA_drm_image: add EGL_DRM_BUFFER_USE_CURSOR_MESA to egl.xml" [1] as part of PR #36 [2]. [1] https://github.com/KhronosGroup/EGL-Registry/commit/2b6bb4ee45cc46c89d4a4349f2ca94e80d77cd97 [2] https://github.com/KhronosGroup/EGL-Registry/pull/36 Signed

[Mesa-dev] [PATCH mesa 1/3] egl: add Ozone section to eglplatform.h

2018-03-16 Thread Eric Engestrom
This pulls in commit a93f559e9c11fa53fb5f1cc255b8f75433f85d2a "Add Ozone section to eglplatform.h" from Khronos [1] added by Brian Anderson [2] a few months ago. [1] https://github.com/KhronosGroup/EGL-Registry/commit/a93f559e9c11fa53fb5f1cc255b8f75433f85d2a [2] https://github.com/KhronosGroup/EG

[Mesa-dev] [PATCH mesa 2/3] egl: align the formatting of Haiku section of eglplatform.h with Khronos'

2018-03-16 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- include/EGL/eglplatform.h | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/include/EGL/eglplatform.h b/include/EGL/eglplatform.h index 8a78a9eaddd2f47ff2c0..b0541d52aed6584c63f2 100644 --- a/include/EGL/eglplatform.h +++ b/include/EG

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 11:38 -0400, Ilia Mirkin wrote: > On Fri, Mar 16, 2018 at 11:27 AM, Juan A. Suarez Romero > wrote: > > On Fri, 2018-03-16 at 09:51 -0400, Ilia Mirkin wrote: > > > On Fri, Mar 16, 2018 at 8:40 AM, Juan A. Suarez Romero > > > wrote: > > > > Nominated means that these patches d

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Rob Clark
On Fri, Mar 16, 2018 at 11:18 AM, Emil Velikov wrote: > On 16 March 2018 at 12:41, Rob Clark wrote: >> Ok, I came up with: >> >> https://hastebin.com/bulilojupo.cpp >> >> if you want to double check my methodology. >> >> It spotted a couple small mistakes, which I've fixed up locally. Will >>

Re: [Mesa-dev] [PATCH 0/3] Meson patches for FreeBSD

2018-03-16 Thread Daniel Stone
Hi, On 16 March 2018 at 15:23, Matt Turner wrote: > On Fri, Mar 16, 2018 at 8:07 AM, Emil Velikov > wrote: >> On 16 March 2018 at 02:39, Matt Turner wrote: >>> All of your messages are going to spam on gmail: >>> >> Setting up a filter for 'list:"mesa-dev.lists.freedesktop.org"' will help >>

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Ilia Mirkin
On Fri, Mar 16, 2018 at 11:27 AM, Juan A. Suarez Romero wrote: > On Fri, 2018-03-16 at 09:51 -0400, Ilia Mirkin wrote: >> On Fri, Mar 16, 2018 at 8:40 AM, Juan A. Suarez Romero >> wrote: >> > Nominated means that these patches does not enter in this release due they >> > arrived a bit late, but t

Re: [Mesa-dev] [PATCH v2 8/8] i965: add performance query support on CNL

2018-03-16 Thread Emil Velikov
On 14 March 2018 at 17:19, Lionel Landwerlin wrote: > v2: Add brw_oa_cnl.xml to EXTRA_DIST (Emil) > > Signed-off-by: Lionel Landwerlin > --- This and 4/8 are Reviewed-by: Emil Velikov It will cause some trivial conflicts with your other patch in the area. FWIW I'd recommend landing that work fi

Re: [Mesa-dev] [PATCH 0/3] Meson patches for FreeBSD

2018-03-16 Thread Matt Turner
On Fri, Mar 16, 2018 at 8:07 AM, Emil Velikov wrote: > On 16 March 2018 at 02:39, Matt Turner wrote: >> On Tue, Mar 6, 2018 at 11:16 AM, Greg V wrote: >>> Hi! Here's a few more patches that let me successfully build, package >>> and install fully working Meson-built Mesa-git on my FreeBSD box. >

Re: [Mesa-dev] [PATCH] nir: add some comparison simplifications

2018-03-16 Thread Ian Romanick
On 03/16/2018 07:37 AM, Ian Romanick wrote: > I have basically the same patch setting in my simple-range-analysis tree. > > https://cgit.freedesktop.org/~idr/mesa/commit/?h=simple-range-analysis&id=e9d29d71304347f8ff904294450a398e5838f49a > > As Jason mentions, you don't need to explicitly mentio

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 13:27 +, Alex Smith wrote: > On 16 March 2018 at 12:46, Juan A. Suarez Romero wrote: > > On Fri, 2018-03-16 at 13:40 +0100, Juan A. Suarez Romero wrote: > > > On Fri, 2018-03-16 at 12:17 +, Alex Smith wrote: > > > > Hi Juan, > > > > > > > > On 16 March 2018 at 11:42,

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Emil Velikov
On 16 March 2018 at 12:41, Rob Clark wrote: > Ok, I came up with: > > https://hastebin.com/bulilojupo.cpp > > if you want to double check my methodology. > > It spotted a couple small mistakes, which I've fixed up locally. Will > resend once I finish autotools build support.. hmm, and I guess t

[Mesa-dev] [Bug 96955] SSH Account Request for Mesa related development

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96955 Daniel Stone changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH 0/3] Meson patches for FreeBSD

2018-03-16 Thread Emil Velikov
On 16 March 2018 at 02:39, Matt Turner wrote: > On Tue, Mar 6, 2018 at 11:16 AM, Greg V wrote: >> Hi! Here's a few more patches that let me successfully build, package >> and install fully working Meson-built Mesa-git on my FreeBSD box. >> >> Greg V (3): >> meson: Use system_has_kms_drm in defa

Re: [Mesa-dev] [PATCH 2/2] i965: perf: reduce i965 binary size

2018-03-16 Thread Lionel Landwerlin
On 16/03/18 15:03, Lionel Landwerlin wrote: Here are the savings in a meson build : Before : $ du -h ./build/src/mesa/drivers/dri/libmesa_dri_drivers.so ./build/src/mesa/drivers/dri/libmesa_dri_drivers.so.old 43M ./build/src/mesa/drivers/dri/libmesa_dri_drivers.so 47M ./build/src/me

[Mesa-dev] [PATCH 2/2] i965: perf: reduce i965 binary size

2018-03-16 Thread Lionel Landwerlin
Performance metric numbers are calculated the following way : - out of the 256 bytes long OA reports, we accumulate the deltas into an array of uint64_t - the equations' generated code reads the accumulated uint64_t deltas and normalizes them for a particular platforms Our hardwa

[Mesa-dev] [PATCH 0/2] i965: reduce driver binary size through metrics code reuse

2018-03-16 Thread Lionel Landwerlin
Hi, Here is a v2 with a fixup commit from Emil. I've also fix another issue related to hashing of normalization equations. Cheers, Lionel Landwerlin (2): i965: perf: fix a counter return type on hsw i965: perf: reduce i965 binary size src/mesa/drivers/dri/i965/Android.mk | 11

[Mesa-dev] [PATCH 1/2] i965: perf: fix a counter return type on hsw

2018-03-16 Thread Lionel Landwerlin
The equation code computes a float (percentage) yet the return type was an uint64_t. Signed-off-by: Lionel Landwerlin --- src/mesa/drivers/dri/i965/brw_oa_hsw.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_oa_hsw.xml b/src/mesa/drivers/dri/

Re: [Mesa-dev] [PATCH] loader: Include include/drm-uapi in the autotools build

2018-03-16 Thread Emil Velikov
On 15 March 2018 at 21:29, Jason Ekstrand wrote: > We're already including it in the meson build. This fixes build issues > on systems which have a drm_fourcc.h that doesn't have modifiers. > --- > src/loader/Makefile.am | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/loader/Makefil

Re: [Mesa-dev] [PATCH] st/dri: don't set queryDmaBufFormats/queryDmaBufModifiers if the driver does not implement it

2018-03-16 Thread Emil Velikov
Cc-ing the author of the 'offending' commit - Nicolai On 15 March 2018 at 16:27, Michal Srb wrote: > This is equivalent to commit a65db0ad1c3, but for dri_kms_init_screen. Without > this gbm_dri_is_format_supported always returns false. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=1

[Mesa-dev] [Bug 96955] SSH Account Request for Mesa related development

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96955 --- Comment #3 from Brian Paul --- It looks like Gwan-gyeong has authored nearly 30 patches in Mesa in 2016-17. OK be me, Daniel. -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug._

Re: [Mesa-dev] [PATCH] nir: add some comparison simplifications

2018-03-16 Thread Ian Romanick
I have basically the same patch setting in my simple-range-analysis tree. https://cgit.freedesktop.org/~idr/mesa/commit/?h=simple-range-analysis&id=e9d29d71304347f8ff904294450a398e5838f49a As Jason mentions, you don't need to explicitly mention the compares. You don't need the @bool either... the

Re: [Mesa-dev] [PATCH v2] nir: add a comparison simplification

2018-03-16 Thread Ian Romanick
On 03/16/2018 01:58 AM, Timothy Arceri wrote: > > > On 16/03/18 19:34, Samuel Pitoiset wrote: >> >> >> On 03/16/2018 09:15 AM, Timothy Arceri wrote: >>> On 16/03/18 18:45, Samuel Pitoiset wrote: shader stats? >>> >>> I didn't bother posting them because they are misleading. There are a >>> f

Re: [Mesa-dev] [PATCH v2] egl/android: Implement the eglSwapinterval for Android.

2018-03-16 Thread Emil Velikov
On 16 March 2018 at 08:51, Wu, Zhongmin wrote: > Thanks very much Tomasz. > > So, who can help to commit such patch. > Fixed with s/dpy/disp/ (otherwise it won't even build) and pushed. Please go through the patches list and update it. I believe that all of those are superseded, do check them ove

[Mesa-dev] [Bug 105506] Vulkan MSAA is broken on SI

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105506 Turo Lamminen changed: What|Removed |Added Resolution|--- |NOTOURBUG Status|REOPENED

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Ilia Mirkin
On Fri, Mar 16, 2018 at 8:40 AM, Juan A. Suarez Romero wrote: > Nominated means that these patches does not enter in this release due they > arrived a bit late, but they are proposed to cherry-pick them for the next > release (in 1 or 2 weeks). > > The reason is that some days before this pre-anno

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Alex Smith
On 16 March 2018 at 12:46, Juan A. Suarez Romero wrote: > On Fri, 2018-03-16 at 13:40 +0100, Juan A. Suarez Romero wrote: > > On Fri, 2018-03-16 at 12:17 +, Alex Smith wrote: > > > Hi Juan, > > > > > > On 16 March 2018 at 11:42, Juan A. Suarez Romero > wrote: > > > > Hello list, > > > > > >

[Mesa-dev] [Bug 104926] swrast: Mesa 17.3.3 produces: HW cursor for format 875713089 not supported

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104926 --- Comment #4 from Stefan Dirsch --- https://lists.freedesktop.org/archives/mesa-dev/2018-March/189030.html -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug._

Re: [Mesa-dev] [PATCH 04/11] nir/spirv: add gl_spirv_validation method

2018-03-16 Thread Rob Clark
On Fri, Mar 16, 2018 at 7:48 AM, Alejandro Piñeiro wrote: > On 16/03/18 12:39, Rob Clark wrote: >> On Fri, Mar 16, 2018 at 7:27 AM, Alejandro Piñeiro >> wrote: >>> On 16/03/18 12:11, Rob Clark wrote: Just curious, how much different is this from what spirv-val does? >>> Well, spirv-val does

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 13:40 +0100, Juan A. Suarez Romero wrote: > On Fri, 2018-03-16 at 12:17 +, Alex Smith wrote: > > Hi Juan, > > > > On 16 March 2018 at 11:42, Juan A. Suarez Romero > > wrote: > > > Hello list, > > > > > > The candidate for the Mesa 17.3.7 is now available. Currently we

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 12:36 +, Emil Velikov wrote: > On 16 March 2018 at 12:07, Daniel Stone wrote: > > Hi Juan, > > > > On 16 March 2018 at 11:42, Juan A. Suarez Romero > > wrote: > > > Trivial merge conflicts > > > --- > > > > > > [...] > > > > > > commit 5fd11359b66

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 12:17 +, Alex Smith wrote: > Hi Juan, > > On 16 March 2018 at 11:42, Juan A. Suarez Romero wrote: > > Hello list, > > > > The candidate for the Mesa 17.3.7 is now available. Currently we have: > > - 53 queued > > - 9 nominated (outstanding) > > - and 6 rejected patch

Re: [Mesa-dev] [PATCH v0] nir: mako all the intrinsics

2018-03-16 Thread Rob Clark
Ok, I came up with: https://hastebin.com/bulilojupo.cpp if you want to double check my methodology. It spotted a couple small mistakes, which I've fixed up locally. Will resend once I finish autotools build support.. hmm, and I guess there is scons too.. *sigh* Fwiw, one if the bugs was from

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Emil Velikov
On 16 March 2018 at 12:07, Daniel Stone wrote: > Hi Juan, > > On 16 March 2018 at 11:42, Juan A. Suarez Romero wrote: >> Trivial merge conflicts >> --- >> >> [...] >> >> commit 5fd11359b66c8138d2c7ee29bd9740280b02d1e2 >> Author: Daniel Stone >> >> egl/wayland: Fix ARGB/XR

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 12:07 +, Daniel Stone wrote: > Hi Juan, > > On 16 March 2018 at 11:42, Juan A. Suarez Romero wrote: > > Trivial merge conflicts > > --- > > > > [...] > > > > commit 5fd11359b66c8138d2c7ee29bd9740280b02d1e2 > > Author: Daniel Stone > > > > egl/

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Emil Velikov
On 16 March 2018 at 12:17, Alex Smith wrote: > Hi Juan, > > On 16 March 2018 at 11:42, Juan A. Suarez Romero > wrote: >> >> Hello list, >> >> The candidate for the Mesa 17.3.7 is now available. Currently we have: >> - 53 queued >> - 9 nominated (outstanding) >> - and 6 rejected patches >> >> >

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Alex Smith
Hi Juan, On 16 March 2018 at 11:42, Juan A. Suarez Romero wrote: > Hello list, > > The candidate for the Mesa 17.3.7 is now available. Currently we have: > - 53 queued > - 9 nominated (outstanding) > - and 6 rejected patches > > > In the current queue we have lot of fixes, as the latest two r

Re: [Mesa-dev] [PATCH] mesa: RGB9_E5 invalid for CopyTexSubImage* in GLES

2018-03-16 Thread Tapani Pälli
On 03/16/2018 02:10 PM, Juan A. Suarez Romero wrote: On Fri, 2018-03-16 at 13:25 +0200, Tapani Pälli wrote: On 03/16/2018 01:13 PM, Tapani Pälli wrote: On 03/16/2018 01:01 PM, Juan A. Suarez Romero wrote: On Fri, 2018-03-16 at 08:02 +0200, Tapani Pälli wrote: On 03/15/2018 02:04 PM, Juan

[Mesa-dev] [Bug 105551] Unable to compile on FreeBSD 9.x and therefore derivatives... undefined reference to `loader_get_extensions_name'

2018-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105551 Bug ID: 105551 Summary: Unable to compile on FreeBSD 9.x and therefore derivatives... undefined reference to `loader_get_extensions_name' Product: Mesa Version:

Re: [Mesa-dev] [PATCH] mesa: RGB9_E5 invalid for CopyTexSubImage* in GLES

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 13:25 +0200, Tapani Pälli wrote: > > On 03/16/2018 01:13 PM, Tapani Pälli wrote: > > > > > > On 03/16/2018 01:01 PM, Juan A. Suarez Romero wrote: > > > On Fri, 2018-03-16 at 08:02 +0200, Tapani Pälli wrote: > > > > On 03/15/2018 02:04 PM, Juan A. Suarez Romero wrote: > > >

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Daniel Stone
Hi Juan, On 16 March 2018 at 11:42, Juan A. Suarez Romero wrote: > Trivial merge conflicts > --- > > [...] > > commit 5fd11359b66c8138d2c7ee29bd9740280b02d1e2 > Author: Daniel Stone > > egl/wayland: Fix ARGB/XRGB transposition in config map > > (cherry picked from com

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
CCing Bas and Anuj. J.A. On Fri, 2018-03-16 at 12:49 +0100, Juan A. Suarez Romero wrote: > On Fri, 2018-03-16 at 12:42 +0100, Juan A. Suarez Romero wrote: > > Hello list, > > > > The candidate for the Mesa 17.3.7 is now available. Currently we have: > > - 53 queued > > - 9 nominated (

Re: [Mesa-dev] [PATCH 0/6] Shader cache; transform feedback; i965 program-binary

2018-03-16 Thread Tapani Pälli
On 03/15/2018 11:07 AM, Tapani Pälli wrote: Patches 1-3: Reviewed-by: Tapani Pälli Make that the whole series! I tested also briefly with app that uses glProgramBinary to implement cache and verified result of GL_LINK_STATUS query with glGetProgramiv after glProgramBinary. (also verif

Re: [Mesa-dev] [ANNOUNCE] Mesa 17.3.7 release candidate

2018-03-16 Thread Juan A. Suarez Romero
On Fri, 2018-03-16 at 12:42 +0100, Juan A. Suarez Romero wrote: > Hello list, > > The candidate for the Mesa 17.3.7 is now available. Currently we have: > - 53 queued > - 9 nominated (outstanding) > - and 6 rejected patches > > > In the current queue we have lot of fixes, as the latest two re

Re: [Mesa-dev] [PATCH 04/11] nir/spirv: add gl_spirv_validation method

2018-03-16 Thread Alejandro Piñeiro
On 16/03/18 12:39, Rob Clark wrote: > On Fri, Mar 16, 2018 at 7:27 AM, Alejandro Piñeiro > wrote: >> On 16/03/18 12:11, Rob Clark wrote: >>> Just curious, how much different is this from what spirv-val does? >> Well, spirv-val does a full validation of the SPIR-V binary. Here we are >> just doing

  1   2   >