Re: [Mesa-dev] [PATCH v3 0/8] deal with multiple appearances of the same layout-qualifier-name in a single declaration

2016-11-10 Thread Timothy Arceri
On Wed, 2016-11-09 at 23:09 -0800, Kenneth Graunke wrote: > On Wednesday, October 26, 2016 6:30:43 PM PST Andres Gomez wrote: > [snip] > > > > The main changes in this v3 series are: > >   Patch 2/8 has been modified to use an additional default_layout > > variable instead of a layout_helper a

Re: [Mesa-dev] i965: Use nir_shader_gather_info() over do_set_program_inouts()

2016-11-10 Thread Timothy Arceri
Ping On Thu, 2016-10-27 at 13:00 +1100, Timothy Arceri wrote: > This takes us one step closer to being able to drop the GLSL IR > optimisation passes during linking in favour of the NIR passes. > > Patch 4 has *not* been tested on Vulkan I've asked Mark to setup a > branch > for me to run the Vul

Re: [Mesa-dev] [PATCH 1/3] glsl: don't run loop passes if loop unrolling is disabled

2016-11-10 Thread Kenneth Graunke
On Wednesday, November 9, 2016 1:12:39 PM PST Marek Olšák wrote: > From: Marek Olšák Series is: Reviewed-by: Kenneth Graunke signature.asc Description: This is a digitally signed message part. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH v2 2/2] egl/android: Use gralloc::lock_ycbcr for resolving YUV formats (v2)

2016-11-10 Thread Tapani Pälli
On 11/10/2016 09:55 AM, Tomasz Figa wrote: There is an interface that can be used to query YUV buffers for their internal format. Specifically, if gralloc:lock_ycbcr() is given no SW usage flags, it's supposed to return plane offsets instead of pointers. Let's use this interface to implement su

Re: [Mesa-dev] [PATCH v3 0/8] deal with multiple appearances of the same layout-qualifier-name in a single declaration

2016-11-10 Thread Andres Gomez
On Thu, 2016-11-10 at 19:21 +1100, Timothy Arceri wrote: > On Wed, 2016-11-09 at 23:09 -0800, Kenneth Graunke wrote: > > On Wednesday, October 26, 2016 6:30:43 PM PST Andres Gomez wrote: > > [snip] > > > > > > The main changes in this v3 series are: > > >   Patch 2/8 has been modified to use an ad

[Mesa-dev] [PATCH 2/4] anv: use STATIC_ASSERT instead of static_assert

2016-11-10 Thread Tapani Pälli
fixes following compilation warnings on Android build: "warning: implicit declaration of function 'static_assert' is invalid in C99 [-Wimplicit-function-declaration]" Signed-off-by: Tapani Pälli --- src/intel/vulkan/anv_private.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff -

[Mesa-dev] [PATCH 0/4] android: preparation for vulkan

2016-11-10 Thread Tapani Pälli
Hi; Here are some patches that prepare current build for Vulkan support. These are the things I noticed needing change to be able to build upcoming libmesa_vulkan.intel.so. I'm still wondering a bit what todo with sha1 support, there is sha1 for bionic from netbsd (bionic/libc/upstream-netbsd/

[Mesa-dev] [PATCH 1/4] util: use STATIC_ASSERT instead of static_assert

2016-11-10 Thread Tapani Pälli
fixes following compilation warnings on Android build: "warning: implicit declaration of function 'static_assert' is invalid in C99 [-Wimplicit-function-declaration]" Signed-off-by: Tapani Pälli --- src/util/u_vector.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util

[Mesa-dev] [PATCH 4/4] android/i965: add libmesa_i965_compiler static library

2016-11-10 Thread Tapani Pälli
this will be shared between OpenGL and Vulkan drivers Signed-off-by: Tapani Pälli --- src/mesa/drivers/dri/i965/Android.mk | 32 1 file changed, 28 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/Android.mk b/src/mesa/drivers/dri/i965/Andro

[Mesa-dev] [PATCH 3/4] android: add SPIRV_FILES to libmesa_nir

2016-11-10 Thread Tapani Pälli
Signed-off-by: Tapani Pälli --- src/compiler/Android.nir.mk | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/compiler/Android.nir.mk b/src/compiler/Android.nir.mk index e89a21c..d08f569 100644 --- a/src/compiler/Android.nir.mk +++ b/src/compiler/Android.nir.mk @@ -31,7 +3

Re: [Mesa-dev] [PATCH] Revert "wayland: Block for the frame callback in get_back_bo not dri2_swap_buffers"

2016-11-10 Thread Daniel Stone
Hi, On 10 November 2016 at 06:08, Jonas Ådahl wrote: > On Mon, Oct 24, 2016 at 08:42:59PM +0100, Daniel Stone wrote: >> The intent of moving blocking from SwapBuffers to get_back_bo, was to >> avoid unnecessary triple-buffering by ensuring that the compositor had >> fully processed the previous f

Re: [Mesa-dev] [PATCH 4/6] nir: update nir_gather_info to only mark used array/matrix elements

2016-11-10 Thread Kenneth Graunke
On Thursday, October 27, 2016 1:00:46 PM PST Timothy Arceri wrote: > This is based on the code from the GLSL IR pass however unlike the GLSL IR > pass it also supports arrays of arrays. > > As well as implementing the logic from the GLSL IR pass we add some > additional intrinsic cases to catch mo

Re: [Mesa-dev] [PATCH 4/6] nir: update nir_gather_info to only mark used array/matrix elements

2016-11-10 Thread Timothy Arceri
On Thu, 2016-11-10 at 02:46 -0800, Kenneth Graunke wrote: > On Thursday, October 27, 2016 1:00:46 PM PST Timothy Arceri wrote: > > > > This is based on the code from the GLSL IR pass however unlike the > > GLSL IR > > pass it also supports arrays of arrays. > > > > As well as implementing the log

[Mesa-dev] [PATCH] i965: miptree: prevent potential NULL pointer access

2016-11-10 Thread Lionel Landwerlin
If the mcs buffer allocation fails we might get a NULL pointer. This was reported by Coverity and should only happen if we run out of memory. v2: return failure at the point of allocation (Chris) CID: 1394290 Signed-off-by: Lionel Landwerlin --- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 1

[Mesa-dev] EGL/android: pbuffer implementation.

2016-11-10 Thread Liu Zhiquan
mesa android path didn't support pbuffer, so add pbuffer support to fix most deqp and cts pbuffer test cases fail; add support of front buffer and single buffer config. Test status: android CTS EGL pbuffer test can run without native crash. test:[DEQP,EGL]all deqp pbuffer case passed. Signed-off-

Re: [Mesa-dev] [PATCH 02/59 v2] glsl/standalone: Optimize add-of-neg to subtract

2016-11-10 Thread Nicolai Hähnle
On 27.10.2016 04:17, Ian Romanick wrote: From: Ian Romanick This just makes the output of the standalone compiler a little more compact. v2: Fix indexing typo noticed by Iago. Move the add_neg_to_sub_visitor to it's own header file. Add a unit test that exercises the visitor. Both the neg_a_

Re: [Mesa-dev] [PATCH 5/5] radeonsi: fix r600_texture::tc_compatible_htile

2016-11-10 Thread Nicolai Hähnle
The series is Reviewed-by: Nicolai Hähnle On 09.11.2016 13:09, Marek Olšák wrote: From: Marek Olšák htile_size is now always non-zero if HTILE is allocated. It seems to have caused no issues. --- src/gallium/drivers/radeon/r600_texture.c | 6 +++--- 1 file changed, 3 insertions(+), 3 delet

Re: [Mesa-dev] [PATCH] loader: use drmDeviceUseRevisionFile to prevent waking up the GPUs

2016-11-10 Thread Nicolai Hähnle
On 09.11.2016 19:09, Emil Velikov wrote: From: Emil Velikov Currently the drmGetDevice[s] API parses the config sysfs file to retrieve the revision field. That is required since there's no separate file (nor a libudev call afaict) that can be used. At the same time doing so causes the device t

Re: [Mesa-dev] [PATCH] linker: Remove unnecessary overload of program_resource_visitor::visit_field

2016-11-10 Thread Tapani Pälli
good cleanup; Reviewed-by: Tapani Pälli On 11/09/2016 11:22 PM, Ian Romanick wrote: From: Ian Romanick It looks like I added this version as a short-hand for users that didn't need the fuller version. I don't think there's any real utility in that. I'm not sure what my thinking was there.

Re: [Mesa-dev] [PATCH] loader: use drmDeviceUseRevisionFile to prevent waking up the GPUs

2016-11-10 Thread Emil Velikov
On 10 November 2016 at 12:44, Nicolai Hähnle wrote: > On 09.11.2016 19:09, Emil Velikov wrote: >> >> From: Emil Velikov >> >> Currently the drmGetDevice[s] API parses the config sysfs file to >> retrieve the revision field. >> >> That is required since there's no separate file (nor a libudev call

Re: [Mesa-dev] [PATCH 08/14] swr: [rasterizer core/jitter] fix alpha test bug

2016-11-10 Thread Ilia Mirkin
Right you are - I must have imagined the third place. On Thu, Nov 10, 2016 at 1:06 AM, Rowley, Timothy O wrote: > Ah, yes, this patch missed the 8x2 tile path - I’ve fixed that now. I don’t > see another path to using the blend jit functions. > > Thanks. > > On Nov 9, 2016, at 10:44 PM, Ilia Mir

Re: [Mesa-dev] [PATCH v2 1/8] mesa: removed redundant #else

2016-11-10 Thread Emil Velikov
Reviewed-by: Emil Velikov Thanks Emil ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH v2 2/8] scons: ignore .hpp files in parse_source_list()

2016-11-10 Thread Emil Velikov
Reviewed-by: Emil Velikov Thanks Emil ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH v2 3/8] scons: added llvm 3.9 support.

2016-11-10 Thread Emil Velikov
Reviewed-by: Emil Velikov Thanks Emil P.S. I might be the only one but imperative statements (like you did in 1/8 and 2/8) sound better. https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/SubmittingPatches?id=refs/tags/v4.9-rc4#n155

Re: [Mesa-dev] [PATCH v2 4/8] gallium: Added SWR support for gdi

2016-11-10 Thread Emil Velikov
On 9 November 2016 at 23:18, George Kyriazis wrote: > Added hooks for screen creation and swap. Still keep llvmpipe the default > software renderer. s/gallium/libgl-gdi/ in the summary (do check with git log for the future). Commit message could mention that one cannot (yet) build a SWR based li

Re: [Mesa-dev] [PATCH v2 5/8] gallium scons: Added swr driver to scons

2016-11-10 Thread Emil Velikov
On 9 November 2016 at 23:18, George Kyriazis wrote: > Enable swr builds by "scons swr=1 libgl-gdi" (windows only) Summary is wrong/misleading. Here we add add the driver side plumbing - one cannot build _any_ target yet. Be that libgl-gdi or any other. > +import SCons.Script.SConscript > + > +if

Re: [Mesa-dev] [PATCH] gbm/drm: Pick the oldest available buffer in get_back_bo

2016-11-10 Thread Pekka Paalanen
On Wed, 9 Nov 2016 14:57:22 -0600 Derek Foreman wrote: > We find the oldest backbuffer we can, on the grounds that clients are > only going to keep a fixed history queue, so this gives them the > greatest chance of being able to use that queue via making sure > the age is ~always less than the d

Re: [Mesa-dev] [PATCH v2 6/8] gallium: Enable swr driver

2016-11-10 Thread Emil Velikov
"gallium/targets: scons: wire swr support for libgl-gdi/libgl-xlib/osmesa" ... or alike ? > +import SCons.Script.SConscript > + > +if 'swr' in SCons.Script.ARGUMENTS and SCons.Script.ARGUMENTS['swr']: With the "move this to top level" suggestion this will read roughly as if env['swr']: -Em

Re: [Mesa-dev] [PATCH v2 7/8] swr: Added swr windows support

2016-11-10 Thread Emil Velikov
On 9 November 2016 at 23:18, George Kyriazis wrote: > - moving some header files around for proper inclusion of windows.h > - OS agnostic loading of arch-specific loadable modules > - PUBLIC function declaration > - better handling on NOMINMAX around windows.h inclusion. > --- Hmm having a second

Re: [Mesa-dev] [PATCH v2 8/8] swr: Fix linux build

2016-11-10 Thread Emil Velikov
On 9 November 2016 at 23:18, George Kyriazis wrote: > Added compiler flags for libmesaswr, since it now includes core header > files. > --- > src/gallium/drivers/swr/Makefile.am | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/src/gallium/drivers/swr/Makefile.am > b/src/gallium/dr

Re: [Mesa-dev] [PATCH] egl/android: Remove handling of RGB_888 pixel format

2016-11-10 Thread Rob Clark
On Wed, Nov 9, 2016 at 11:23 PM, Tomasz Figa wrote: > On Thu, Nov 10, 2016 at 5:21 AM, Emil Velikov > wrote: >> On 9 November 2016 at 08:32, Tomasz Figa wrote: >>> It is currently completely broken, as it ends up using RGBX_ on >>> hardware side, due to no way of distinguishing between thes

Re: [Mesa-dev] [PATCH 2/2] egl/android: Use gralloc::lock_ycbcr for resolving YUV formats

2016-11-10 Thread Emil Velikov
On 10 November 2016 at 07:13, Tomasz Figa wrote: > On Thu, Nov 10, 2016 at 1:50 PM, Tomasz Figa wrote: >> On Thu, Nov 10, 2016 at 5:14 AM, Emil Velikov >> wrote: >>> On 9 November 2016 at 08:33, Tomasz Figa wrote: There is an interface that can be used to query YUV buffers for their

Re: [Mesa-dev] Mesa (master): configure.ac: honour LLVM_LIBDIR when linking against LLVM

2016-11-10 Thread Emil Velikov
On 10 November 2016 at 06:46, Michel Dänzer wrote: > On 10/11/16 06:47 AM, Emil Velikov wrote: >> Module: Mesa >> Branch: master >> Commit: a39ad185932eab4f25a0cb2b112c10d8700ef242 >> URL: >> http://cgit.freedesktop.org/mesa/mesa/commit/?id=a39ad185932eab4f25a0cb2b112c10d8700ef242 >> >> Author

Re: [Mesa-dev] [PATCH] egl/android: Remove handling of RGB_888 pixel format

2016-11-10 Thread Tomasz Figa
On Thu, Nov 10, 2016 at 11:42 PM, Rob Clark wrote: > On Wed, Nov 9, 2016 at 11:23 PM, Tomasz Figa wrote: >> On Thu, Nov 10, 2016 at 5:21 AM, Emil Velikov >> wrote: >>> On 9 November 2016 at 08:32, Tomasz Figa wrote: It is currently completely broken, as it ends up using RGBX_ on

Re: [Mesa-dev] [PATCH 4/4] android/i965: add libmesa_i965_compiler static library

2016-11-10 Thread Emil Velikov
On 10 November 2016 at 09:33, Tapani Pälli wrote: > this will be shared between OpenGL and Vulkan drivers > > Signed-off-by: Tapani Pälli > --- > src/mesa/drivers/dri/i965/Android.mk | 32 > 1 file changed, 28 insertions(+), 4 deletions(-) > > diff --git a/src/me

Re: [Mesa-dev] [PATCH 0/4] android: preparation for vulkan

2016-11-10 Thread Jason Ekstrand
On Nov 10, 2016 1:34 AM, "Tapani Pälli" wrote: > > Hi; > > Here are some patches that prepare current build for Vulkan support. Hey, thanks for doing this! Reviewed-by: Jason Ekstrand > These are the things I noticed needing change to be able to build > upcoming libmesa_vulkan.intel.so. I'm st

Re: [Mesa-dev] [PATCH] gbm/drm: Pick the oldest available buffer in get_back_bo

2016-11-10 Thread Emil Velikov
Hi Derek, On 9 November 2016 at 20:57, Derek Foreman wrote: > We find the oldest backbuffer we can, on the grounds that clients are > only going to keep a fixed history queue, so this gives them the > greatest chance of being able to use that queue via making sure > the age is ~always less than t

Re: [Mesa-dev] [PATCH] egl/android: Remove handling of RGB_888 pixel format

2016-11-10 Thread Rob Clark
On Thu, Nov 10, 2016 at 10:29 AM, Tomasz Figa wrote: > On Thu, Nov 10, 2016 at 11:42 PM, Rob Clark wrote: >> On Wed, Nov 9, 2016 at 11:23 PM, Tomasz Figa wrote: >>> On Thu, Nov 10, 2016 at 5:21 AM, Emil Velikov >>> wrote: On 9 November 2016 at 08:32, Tomasz Figa wrote: > It is curren

Re: [Mesa-dev] [PATCH] gbm/drm: Pick the oldest available buffer in get_back_bo

2016-11-10 Thread Daniel Stone
Hi, On Nov 10 2016, at 2:09 pm, Pekka Paalanen wrote: > On Wed, 9 Nov 2016 14:57:22 -0600 Derek Foreman wrote: > > > We find the oldest backbuffer we can, on the grounds that clients are > only going to keep a fixed history queue, so this gives them the > greatest chance of be

Re: [Mesa-dev] EGL/android: pbuffer implementation.

2016-11-10 Thread Jason Ekstrand
+Chad On Nov 10, 2016 3:53 AM, "Liu Zhiquan" wrote: > mesa android path didn't support pbuffer, so add pbuffer support to > fix most deqp and cts pbuffer test cases fail; > add support of front buffer and single buffer config. > > Test status: android CTS EGL pbuffer test can run without native

Re: [Mesa-dev] [PATCH 1/2] gallivm: Fix build after removal of deprecated attribute API v3

2016-11-10 Thread Tom Stellard
On Wed, Nov 09, 2016 at 11:45:38PM +0100, Roland Scheidegger wrote: > Am 09.11.2016 um 16:22 schrieb Tom Stellard: > > v2: > > Fix adding parameter attributes with LLVM < 4.0. > > > > v3: > > Fix typo. > > Fix parameter index. > > Add a gallivm enum for function attributes. > > --- > > sr

Re: [Mesa-dev] [PATCH 1/2] gallivm: Fix build after removal of deprecated attribute API v3

2016-11-10 Thread Roland Scheidegger
Am 10.11.2016 um 17:14 schrieb Tom Stellard: > On Wed, Nov 09, 2016 at 11:45:38PM +0100, Roland Scheidegger wrote: >> Am 09.11.2016 um 16:22 schrieb Tom Stellard: >>> v2: >>> Fix adding parameter attributes with LLVM < 4.0. >>> >>> v3: >>> Fix typo. >>> Fix parameter index. >>> Add a galliv

Re: [Mesa-dev] EGL/android: pbuffer implementation.

2016-11-10 Thread Emil Velikov
On 10 November 2016 at 11:48, Liu Zhiquan wrote: > mesa android path didn't support pbuffer, so add pbuffer support to > fix most deqp and cts pbuffer test cases fail; > add support of front buffer and single buffer config. > > Test status: android CTS EGL pbuffer test can run without native crash

Re: [Mesa-dev] [PATCH v2 6/8] gallium: Enable swr driver

2016-11-10 Thread Kyriazis, George
> -Original Message- > From: Emil Velikov [mailto:emil.l.veli...@gmail.com] > Sent: Thursday, November 10, 2016 8:14 AM > To: Kyriazis, George > Cc: ML mesa-dev > Subject: Re: [Mesa-dev] [PATCH v2 6/8] gallium: Enable swr driver > > "gallium/targets: scons: wire swr support for libgl-g

Re: [Mesa-dev] [PATCH v2 6/8] gallium: Enable swr driver

2016-11-10 Thread Emil Velikov
On 10 November 2016 at 17:27, Kyriazis, George wrote: >> -Original Message- >> From: Emil Velikov [mailto:emil.l.veli...@gmail.com] >> Sent: Thursday, November 10, 2016 8:14 AM >> To: Kyriazis, George >> Cc: ML mesa-dev >> Subject: Re: [Mesa-dev] [PATCH v2 6/8] gallium: Enable swr driver

[Mesa-dev] [PATCH 1/3] i965/glk: Add basic Geminilake support

2016-11-10 Thread Ben Widawsky
From: Ben Widawsky v2: Fix KBL typo while here s/bdw/gen Add the 2x6 config Cc: "13.0" Signed-off-by: Ben Widawsky --- include/pci_ids/i965_pci_ids.h | 2 ++ src/intel/common/gen_device_info.c | 44 ++ 2 files changed, 46 insertions(+) diff --git a/in

Re: [Mesa-dev] [PATCH 3/3] i965: Fix KBL typo in string

2016-11-10 Thread Ilia Mirkin
On Thu, Nov 10, 2016 at 1:20 PM, Ben Widawsky wrote: > From: Ben Widawsky > > Signed-off-by: Ben Widawsky > --- > include/pci_ids/i965_pci_ids.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/pci_ids/i965_pci_ids.h b/include/pci_ids/i965_pci_ids.h > index fbc39

Re: [Mesa-dev] [PATCH v2 1/3] loader: add loader_get_extensions_name(..) helper

2016-11-10 Thread Emil Velikov
On 9 November 2016 at 14:50, Christian Gmeiner wrote: > Changes since v1: > - renamed function to loader_get_extensions_name > - moved function into loader > > Signed-off-by: Christian Gmeiner > --- > src/loader/loader.c | 14 ++ > src/loader/loader.h | 2 ++ > 2 files changed, 16

Re: [Mesa-dev] [PATCH 3/3] i965: Fix KBL typo in string

2016-11-10 Thread Ben Widawsky
On 16-11-10 10:20:13, Ben Widawsky wrote: From: Ben Widawsky Signed-off-by: Ben Widawsky --- include/pci_ids/i965_pci_ids.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/pci_ids/i965_pci_ids.h b/include/pci_ids/i965_pci_ids.h index fbc3999..b938332 100644 --- a/inc

Re: [Mesa-dev] [PATCH v2 2/3] dri: make use of loader_get_extensions_name(..) helper

2016-11-10 Thread Emil Velikov
On 9 November 2016 at 14:50, Christian Gmeiner wrote: > Changes since v1: > - removed not needed includes > They are needed actually, although transitively resolved via the respective headers. If we are to apply some "polish" in the latter this will break. I'd stick with v1 if you don't mind. F

[Mesa-dev] [Bug 98555] NWN won't start on 13.0

2016-11-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98555 Vasyl Demin changed: What|Removed |Added Resolution|NOTOURBUG |--- Status|RESOLVED

[Mesa-dev] [PATCH 2/3] i965: Consolidate GEN9 LP definition

2016-11-10 Thread Ben Widawsky
From: Ben Widawsky Signed-off-by: Ben Widawsky --- src/intel/common/gen_device_info.c | 116 + 1 file changed, 40 insertions(+), 76 deletions(-) diff --git a/src/intel/common/gen_device_info.c b/src/intel/common/gen_device_info.c index 3b09072..055fc24 1006

[Mesa-dev] [PATCH 3/3] i965: Fix KBL typo in string

2016-11-10 Thread Ben Widawsky
From: Ben Widawsky Signed-off-by: Ben Widawsky --- include/pci_ids/i965_pci_ids.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/pci_ids/i965_pci_ids.h b/include/pci_ids/i965_pci_ids.h index fbc3999..b938332 100644 --- a/include/pci_ids/i965_pci_ids.h +++ b/include/

[Mesa-dev] [PATCH] intel: Add Geminilake PCI IDs

2016-11-10 Thread Ben Widawsky
From: Ben Widawsky Signed-off-by: Ben Widawsky --- intel/intel_chipset.h | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/intel/intel_chipset.h b/intel/intel_chipset.h index 514f659..41fc0da 100644 --- a/intel/intel_chipset.h +++ b/intel/intel_chipset.h @@ -218

Re: [Mesa-dev] [PATCH 2/2] egl: Use pkg-config for Android NDK build

2016-11-10 Thread Emil Velikov
On 9 November 2016 at 22:20, Gurchetan Singh wrote: > It's possible to build Mesa for Android using the traditional > autotools workflow. To enable this, let's add the required > pkg-config checks and link against them. > > v2: Bundle pkg-config checks together (Emil) > --- > configure.ac

Re: [Mesa-dev] [PATCH 1/2] configure.ac: Don't set PTHREAD_LIBS when it is not present

2016-11-10 Thread Emil Velikov
On 9 November 2016 at 21:22, Gurchetan Singh wrote: > AX_PTHREADS sets PTHREAD_CFLAGS and PTHREAD_LIBS, and we should > not append to it. > > This breaks building EGL since PTHREAD_CFLAGS isn't specified, so > let's add it here. > > I couldn't find any other places where this breaks the build in >

[Mesa-dev] [Bug 98555] NWN won't start on 13.0

2016-11-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98555 --- Comment #11 from Emil Velikov --- (In reply to Vasyl Demin from comment #9) > Hello Emil, thanks for quick response and sorry for my late reply. > > > You should be able to reproduce with simple glxgears. > glxgears and glxgears32 both work

[Mesa-dev] [PATCH] nvc0: support MP performance counters on Maxwell

2016-11-10 Thread Samuel Pitoiset
Hi, This series adds MP performance counters exposed by SM50 (GM107/GM108) and SM52 (GM200/GM204/GM206). Maxwell introduced some new events like active_ctas, global_store, global_load and inst_issued0, which is (in my opinion) the most important one because it reports the number of cycles that di

[Mesa-dev] [PATCH] nvc0: support MP performance counters on Maxwell

2016-11-10 Thread Samuel Pitoiset
This adds some performance counters/metrics for SM50/SM52. Signed-off-by: Samuel Pitoiset --- .../drivers/nouveau/nvc0/nvc0_query_hw_metric.c| 26 +- .../drivers/nouveau/nvc0/nvc0_query_hw_sm.c| 740 - .../drivers/nouveau/nvc0/nvc0_query_hw_sm.h| 13 +-

Re: [Mesa-dev] [PATCH 00/25] anv: A major rework of color attachment surface states

2016-11-10 Thread Nanley Chery
On Wed, Nov 09, 2016 at 06:43:23PM -0800, Jason Ekstrand wrote: > On Wed, Nov 9, 2016 at 5:09 PM, Nanley Chery wrote: > > > On Sat, Oct 22, 2016 at 10:50:31AM -0700, Jason Ekstrand wrote: > > > This series does some fairly major surgery on color attachment surface > > > state allocation and fill-

[Mesa-dev] [Bug 98555] NWN won't start on 13.0

2016-11-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98555 --- Comment #12 from Alan Swanson --- There's a patch for nwuser to check __nwu_possible in its opendir override; https://github.com/nwnlinux/nwuser/pull/1 Unfortunately with radeonsi I now get a segfault at gelf_getshdr (called via si_init_sha

Re: [Mesa-dev] [PATCH v2] clover: add GetKernelArgInfo (CL 1.2)

2016-11-10 Thread Serge Martin
On Sunday 30 October 2016 16:07:25 Francisco Jerez wrote: > Serge Martin writes: > > --- > > > > src/gallium/state_trackers/clover/api/kernel.cpp | 47 > > -- > > src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++ > > src/gallium/state_trackers/clover/core/kernel.hpp

[Mesa-dev] [Bug 98555] NWN won't start on 13.0

2016-11-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98555 Vasyl Demin changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|---

[Mesa-dev] [PATCH 2/2] egl: Use pkg-config for Android NDK build

2016-11-10 Thread Gurchetan Singh
It's possible to build Mesa for Android using the traditional autotools workflow [1]. ChromiumOS fetches Android prebuilts and puts them in a sysroot. We now want to use pkg-config to specify the location of system headers and libraries [2]. To enable this, let's add the required pkg-config checks

Re: [Mesa-dev] [PATCH 1/2] configure.ac: Don't set PTHREAD_LIBS when it is not present

2016-11-10 Thread Emil Velikov
On 10 November 2016 at 19:04, Emil Velikov wrote: > On 9 November 2016 at 21:22, Gurchetan Singh > wrote: >> AX_PTHREADS sets PTHREAD_CFLAGS and PTHREAD_LIBS, and we should >> not append to it. >> >> This breaks building EGL since PTHREAD_CFLAGS isn't specified, so >> let's add it here. >> >> I c

Re: [Mesa-dev] [PATCH v2 6/8] gallium: Enable swr driver

2016-11-10 Thread Kyriazis, George
Emil, I've followed the approach of trying to localize the swr changes to where they belong in order to avoid "polluting" upper-level files. If you're talking about modifying some of the files in the scons/ directory, I don't see a place where swr fits in, conceptually. I do agree that using

Re: [Mesa-dev] [PATCH 1/2] configure.ac: Don't look for pthreads in Android platform

2016-11-10 Thread Emil Velikov
On 27 October 2016 at 17:06, Gurchetan Singh wrote: > In Android, the pthreads libs are in bionic. When building > Mesa for Android with the autotools workflow, we shouldn't > set -lpthread or -pthread. > --- > configure.ac | 7 +-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff

Re: [Mesa-dev] [PATCH v2 6/8] gallium: Enable swr driver

2016-11-10 Thread Emil Velikov
On 10 November 2016 at 20:18, Kyriazis, George wrote: > Emil, > > I've followed the approach of trying to localize the swr changes to where > they belong in order to avoid "polluting" upper-level files. If you're > talking about modifying some of the files in the scons/ directory, I don't > se

Re: [Mesa-dev] [PATCH 00/25] anv: A major rework of color attachment surface states

2016-11-10 Thread Jason Ekstrand
On Nov 10, 2016 11:44, "Nanley Chery" wrote: > > On Wed, Nov 09, 2016 at 06:43:23PM -0800, Jason Ekstrand wrote: > > On Wed, Nov 9, 2016 at 5:09 PM, Nanley Chery wrote: > > > > > On Sat, Oct 22, 2016 at 10:50:31AM -0700, Jason Ekstrand wrote: > > > > This series does some fairly major surgery on

[Mesa-dev] [Bug 98555] NWN won't start on 13.0

2016-11-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98555 --- Comment #14 from Emil Velikov --- (In reply to Vasyl Demin from comment #13) > @Emil > Sorry, I did not check it before. You are right, NWN run normally without > nwuser.so and nwuser64.so in LD_PRELOAD. > Great, thanks for the confirmation

Re: [Mesa-dev] [PATCH 00/25] anv: A major rework of color attachment surface states

2016-11-10 Thread Nanley Chery
On Thu, Nov 10, 2016 at 12:52:44PM -0800, Jason Ekstrand wrote: > On Nov 10, 2016 11:44, "Nanley Chery" wrote: > > > > On Wed, Nov 09, 2016 at 06:43:23PM -0800, Jason Ekstrand wrote: > > > On Wed, Nov 9, 2016 at 5:09 PM, Nanley Chery > wrote: > > > > > > > On Sat, Oct 22, 2016 at 10:50:31AM -0700

Re: [Mesa-dev] [PATCH 00/25] anv: A major rework of color attachment surface states

2016-11-10 Thread Jason Ekstrand
On Thu, Nov 10, 2016 at 1:56 PM, Nanley Chery wrote: > On Thu, Nov 10, 2016 at 12:52:44PM -0800, Jason Ekstrand wrote: > > On Nov 10, 2016 11:44, "Nanley Chery" wrote: > > > > > > On Wed, Nov 09, 2016 at 06:43:23PM -0800, Jason Ekstrand wrote: > > > > On Wed, Nov 9, 2016 at 5:09 PM, Nanley Chery

[Mesa-dev] [PATCH 2/2] anv/cmd_buffer: Enable a CS stall workaround for Sky Lake gt4

2016-11-10 Thread Jason Ekstrand
This fixes hangs in Dota2 Cc: "12.0 13.0" --- src/intel/vulkan/genX_cmd_buffer.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index 35f5dd6..2bc7e74 100644 --- a/src/intel/vulkan/genX_cmd_buffer.c +++ b/src/int

[Mesa-dev] [PATCH 1/2] anv/cmd_buffer: Take a command buffer instead of a batch in two helpers

2016-11-10 Thread Jason Ekstrand
Cc: "12.0 13.0" --- src/intel/vulkan/genX_cmd_buffer.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index 045cb9d..35f5dd6 100644 --- a/src/intel/vulkan/genX_cmd_buffer.c +++ b/src/int

Re: [Mesa-dev] [PATCH 13/59] glsl/ast/ir: Add 64-bit integer constant support

2016-11-10 Thread Ian Romanick
On 10/27/2016 06:09 AM, Nicolai Hähnle wrote: > On 26.10.2016 02:59, Ian Romanick wrote: >> From: Dave Airlie >> >> This adds support for 64-bit integer constants to the parser, >> ast and ir. >> >> v2: fix a few issues found in testing. >> >> v3: Add missing ir_constant copy contructor support. >

[Mesa-dev] [PATCH 2/2] st/va: fix gop size for rate control

2016-11-10 Thread boyuan.zhang
From: Boyuan Zhang The gop_size in rate control is the budget window for internal rate control calculation, and shouldn't always equal to idr period. Define a coefficient to let budget window contains a number of idr period for proper rate control calculation. Adjust the number of i/p frame rema

Re: [Mesa-dev] [PATCH 12/59] mesa: Add support for 64-bit integer uniforms. (v2)

2016-11-10 Thread Ian Romanick
On 10/27/2016 06:02 AM, Nicolai Hähnle wrote: > On 26.10.2016 02:59, Ian Romanick wrote: >> From: Dave Airlie >> >> This hooks up the API to the internals for 64-bit integer uniforms. >> >> v2: update to use non-strict aliased alternatives >> >> Signed-off-by: Dave Airlie >> Reviewed-by: Ian Roma

Re: [Mesa-dev] [PATCH 31/59] st/glsl_to_tgsi: add support for 64-bit integers. (v2)

2016-11-10 Thread Ian Romanick
On 10/27/2016 06:32 AM, Nicolai Hähnle wrote: > On 26.10.2016 02:59, Ian Romanick wrote: >> From: Dave Airlie >> >> v2: add conversion opcodes. >> >> v3 (idr): Rebase on replacemtn of TGSI_OPCODE_I2U64 with >> TGSI_OPCODE_I2I64. >> >> Signed-off-by: Dave Airlie >> --- >> src/mesa/state_tracker/s

Re: [Mesa-dev] [PATCH 2/2] anv/cmd_buffer: Enable a CS stall workaround for Sky Lake gt4

2016-11-10 Thread Kenneth Graunke
On Thursday, November 10, 2016 2:27:28 PM PST Jason Ekstrand wrote: > This fixes hangs in Dota2 > > Cc: "12.0 13.0" > --- > src/intel/vulkan/genX_cmd_buffer.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/src/intel/vulkan/genX_cmd_buffer.c > b/src/intel/vulkan/genX_cmd_buffer.

Re: [Mesa-dev] [PATCH 12/59] mesa: Add support for 64-bit integer uniforms. (v2)

2016-11-10 Thread Dave Airlie
On 11 November 2016 at 08:57, Ian Romanick wrote: > On 10/27/2016 06:02 AM, Nicolai Hähnle wrote: >> On 26.10.2016 02:59, Ian Romanick wrote: >>> From: Dave Airlie >>> >>> This hooks up the API to the internals for 64-bit integer uniforms. >>> >>> v2: update to use non-strict aliased alternatives

Re: [Mesa-dev] [PATCH 00/25] anv: A major rework of color attachment surface states

2016-11-10 Thread Nanley Chery
On Thu, Nov 10, 2016 at 02:25:37PM -0800, Jason Ekstrand wrote: > On Thu, Nov 10, 2016 at 1:56 PM, Nanley Chery wrote: > > > On Thu, Nov 10, 2016 at 12:52:44PM -0800, Jason Ekstrand wrote: > > > On Nov 10, 2016 11:44, "Nanley Chery" wrote: > > > > > > > > On Wed, Nov 09, 2016 at 06:43:23PM -0800

Re: [Mesa-dev] [PATCH 23/25] anv: Set framebuffer to NULL in secondary command buffers

2016-11-10 Thread Nanley Chery
On Sat, Oct 22, 2016 at 10:50:54AM -0700, Jason Ekstrand wrote: > Nothing that is allowed to be called within a secondary now relies on the > framebuffer. > --- > src/intel/vulkan/genX_cmd_buffer.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) This patch is, Reviewed-by: Nanley Chery

Re: [Mesa-dev] [PATCH 25/25] anv/blorp: Handle VK_ATTACHMENT_UNUSED in CmdClearAttachments

2016-11-10 Thread Nanley Chery
On Sat, Oct 22, 2016 at 10:50:56AM -0700, Jason Ekstrand wrote: > From the Vulkan 1.0.29 spec for vkCmdClearAttachments: > >"If the subpass’s depth/stencil attachment is VK_ATTACHMENT_UNUSED, >then the clear has no effect." > > and > >"If colorAttachment is VK_ATTACHMENT_UNUSED then

[Mesa-dev] [PATCH] glsl: include inttypes.h for PRIx64 macro

2016-11-10 Thread Brian Paul
To fix MinGW build. --- src/compiler/glsl/ir_builder_print_visitor.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/compiler/glsl/ir_builder_print_visitor.cpp b/src/compiler/glsl/ir_builder_print_visitor.cpp index d7bba9b..529466c 100644 --- a/src/compiler/glsl/ir_builder_print_visitor

Re: [Mesa-dev] [PATCH] glsl: include inttypes.h for PRIx64 macro

2016-11-10 Thread Roland Scheidegger
Am 11.11.2016 um 01:30 schrieb Brian Paul: > To fix MinGW build. > --- > src/compiler/glsl/ir_builder_print_visitor.cpp | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/compiler/glsl/ir_builder_print_visitor.cpp > b/src/compiler/glsl/ir_builder_print_visitor.cpp > index d7bba9b..5294

[Mesa-dev] [PATCH 07/70] mesa: remove unused Comment field in prog_instruction

2016-11-10 Thread Timothy Arceri
--- src/mesa/program/prog_instruction.c | 9 - src/mesa/program/prog_instruction.h | 3 --- src/mesa/program/prog_print.c | 38 - 3 files changed, 12 insertions(+), 38 deletions(-) diff --git a/src/mesa/program/prog_instruction.c b/src/mesa/pro

[Mesa-dev] [PATCH 01/70] i965: only try print GLSL IR once when using INTEL_DEBUG to dump ir

2016-11-10 Thread Timothy Arceri
Since we started releasing GLSL IR after linking the only time we can print GLSL IR is during linking. When regenerating variants only NIR will be available. --- src/mesa/drivers/dri/i965/brw_cs.c | 6 -- src/mesa/drivers/dri/i965/brw_gs.c | 3 --- src/mesa/drivers/dri/i965/brw_lin

[Mesa-dev] [PATCH 06/70] i965: get num_abos from shader_info rather than gl_linked_shader

2016-11-10 Thread Timothy Arceri
This is a step towards freeing gl_linked_shader after linking. --- src/mesa/drivers/dri/i965/brw_context.h | 1 + src/mesa/drivers/dri/i965/brw_gs_surface_state.c | 5 +++-- src/mesa/drivers/dri/i965/brw_shader.cpp | 4 ++-- src/mesa/drivers/dri/i965/brw_tcs_surface_state.c

[Mesa-dev] [PATCH 04/70] i965: get num_images from shader_info rather than gl_linked_shader

2016-11-10 Thread Timothy Arceri
This is a step towards freeing gl_linked_shader after linking. --- src/mesa/drivers/dri/i965/brw_context.c | 4 ++-- src/mesa/drivers/dri/i965/brw_context.h | 1 + src/mesa/drivers/dri/i965/brw_cs.c| 5 +++-- src/mesa/drivers/dri/i965/brw_gs.c

[Mesa-dev] [PATCH 05/70] mesa/glsl: copy num_abos to gl_program

2016-11-10 Thread Timothy Arceri
We should be able to free gl_linked_shader after linking in order to do so we need to switch to getting values from gl_program instead. --- src/compiler/glsl/glsl_to_nir.cpp | 1 - src/mesa/main/shaderapi.c | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compiler/gl

[Mesa-dev] [PATCH 09/70] st/mesa/r200/i915/i965: move ARB program fields into a union

2016-11-10 Thread Timothy Arceri
It's common for games to compile 2000 programs or more so at 32bits x 2000 programs x 22 fields x 2 (at least) stages This should give us something like 352 kilobytes in savings. --- src/mesa/drivers/dri/i915/i915_fragprog.c | 12 ++-- src/mesa/drivers/dri/i915/i915_program.c| 16 ++--- sr

[Mesa-dev] [PATCH 02/70] nir: add support for counting AoA uniforms in nir_shader_gather_info()

2016-11-10 Thread Timothy Arceri
--- src/compiler/nir/nir_gather_info.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/compiler/nir/nir_gather_info.c b/src/compiler/nir/nir_gather_info.c index e5cedd9..7d349d5 100644 --- a/src/compiler/nir/nir_gather_info.c +++ b/src/compiler/nir/nir_gather_info.c @@

[Mesa-dev] [PATCH 08/70] mesa: make use of ralloc when creating ARB program instructions

2016-11-10 Thread Timothy Arceri
This will allow us to move the ARB asm fields in gl_program into a union as we will be able call ralloc_free() on the entire struct when destroying the context. --- src/mesa/main/ffvertex_prog.c | 7 +++ src/mesa/program/arbprogparse.c | 9 ++--- src/mesa/program/ir_to_mesa.cpp

[Mesa-dev] Rework use of gl program structures

2016-11-10 Thread Timothy Arceri
I started out with a goal of freeing _LinkedShaders after linking as this would have allowed a bunch of code simplification and also reduced the possibility of subtle bugs in i965 where a program that is currently active is relinked but fails and then we recompile a variant of the still active prog

[Mesa-dev] [PATCH 03/70] mesa/glsl: copy num_images to gl_program

2016-11-10 Thread Timothy Arceri
We should be able to free gl_linked_shader after linking in order to do so we need to switch to getting values from gl_program instead. --- src/compiler/glsl/glsl_to_nir.cpp | 1 - src/mesa/main/shaderapi.c | 2 ++ 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/compiler/

[Mesa-dev] [PATCH 10/70] glsl: stub out _mesa_reference_program() in standalone compiler

2016-11-10 Thread Timothy Arceri
The follow patch will call this directly from the linker, the shader cache will also start calling these from the compiler. --- src/compiler/glsl/standalone_scaffolding.cpp | 8 src/compiler/glsl/standalone_scaffolding.h | 4 2 files changed, 12 insertions(+) diff --git a/src/comp

[Mesa-dev] [PATCH 11/70] glsl: create gl_program at the start of linking rather than the end

2016-11-10 Thread Timothy Arceri
This will allow us to directly store metadata we want to retain in gl_program this metadata is currently stored in gl_linked_shader and will be lost if relinking fails even though the program will remain in use and is still valid according to the spec. "If a program object that is active for any s

[Mesa-dev] [PATCH 16/70] mesa/glsl: add bitmask to track stages a program was linked against

2016-11-10 Thread Timothy Arceri
This will be used to enable us to store the current gl_program rather than gl_shader_program in the gl_pipline_object allowing us to simplify handing of validation. Also we should not be depending on _LinkedShader for this information as it may contain shaders from a failed linking attempt rather

[Mesa-dev] [PATCH 12/70] glsl/i965: move per stage AtomicBuffers list to gl_program

2016-11-10 Thread Timothy Arceri
--- src/compiler/glsl/link_atomics.cpp| 8 src/mesa/drivers/dri/i965/brw_context.h | 1 - src/mesa/drivers/dri/i965/brw_gs_surface_state.c | 8 ++-- src/mesa/drivers/dri/i965/brw_tcs_surface_state.c | 9 +++-- src/mesa/drivers/dri/i965/brw_tes_surfac

[Mesa-dev] [PATCH 18/70] glsl: use linked_shaders bitmask to iterate stages for subroutine fields

2016-11-10 Thread Timothy Arceri
This should be faster than looping over every stage and null checking, but will also make the code a bit cleaner when we switch to getting more fields from gl_program rather than from gl_linked_shader as we can just copy the pointer and not need to worry about null checking then copying. --- src/c

  1   2   >