Re: [Mesa-dev] [PATCH 2/4] radv/winsys: import the amdgpu winsys for the radv vulkan driver. (v1.1)

On 10/05/2016 11:48 AM, Dave Airlie wrote: > From: Dave Airlie > > This just brings these files into the tree, it doesn't integrate > them with the build system. > > The radv winsys is based on the gallium one with some changes, > due to how command buffers are built and lack of flushing behav

Re: [Mesa-dev] [PATCH] winsys/radeon: (trivial) rename variable for consistency

Am 05.10.2016 um 05:51 schrieb Alexandre Demers: Signed-off-by: Alexandre Demers I don't mind getting this straight, but are we really using "offset" instead of "off" all over the place? Anyway this one is Reviewed-by: Christian König . Regards, Christian. --- src/gallium/drivers/radeo

Re: [Mesa-dev] [PATCH 1/4] amd/common: add nir->llvm translation.

On 04.10.2016 12:58, Grazvydas Ignotas wrote: On Tue, Oct 4, 2016 at 1:09 PM, Nicolai Hähnle wrote: On 04.10.2016 03:48, Dave Airlie wrote: [snip] diff --git a/src/amd/common/ac_llvm_helper.cpp b/src/amd/common/ac_llvm_helper.cpp new file mode 100644 index 000..feafdaf --- /dev/null +++

Re: [Mesa-dev] radv submission 2

On 05.10.2016 02:48, Dave Airlie wrote: Again I'm sure this will hit limits, and I've asked Michel to drop the big patch before it gets here. All of these are in a new branch radv-submit2. https://github.com/airlied/mesa/commits/radv-submit2 I've incorporated all the feedback except two things:

Re: [Mesa-dev] [PATCH] radeonsi: fix texture border colors for compute shaders

On 05.10.2016 01:51, Marek Olšák wrote: From: Marek Olšák There are VM faults without this. Cc: 12.0 Reviewed-by: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_compute.c | 12 1 file changed, 12 insertions(+) ___ mesa-dev ma

Re: [Mesa-dev] [PATCH] winsys/radeon: (trivial) rename variable for consistency

On 05.10.2016 05:51, Alexandre Demers wrote: Signed-off-by: Alexandre Demers I don't think this kind of renaming is too important, but it does make the code slightly more readable. Reviewed-by: Nicolai Hähnle --- src/gallium/drivers/radeon/radeon_uvd.c | 8 1 file changed, 4 in

Re: [Mesa-dev] [PATCH 2/2] i965: use L3 data cache for SSBOs

Curro: Ping? :) On 26/09/16 20:02, Jason Ekstrand wrote: Looks good to me. Curro, do you see anything wrong with this? --Jason On Sep 26, 2016 7:31 AM, "Lionel Landwerlin" > wrote: Anv programs the hardware to use L3 data cache if we use either SSBOs o

[Mesa-dev] [Bug 97542] mesa-12.0.1 with llvm-3.9.0_rc3 - src/gallium/state_trackers/clover/llvm/invocation.cpp:212:75: error: no matching function for call to clang::CompilerInvocation::setLangDefault

https://bugs.freedesktop.org/show_bug.cgi?id=97542 Vedran Miletić changed: What|Removed |Added CC||ved...@miletic.net -- You are receivin

Re: [Mesa-dev] [PATCH 2/2] i965: use L3 data cache for SSBOs

Lionel Landwerlin writes: > Curro: Ping? :) Reviewed-by: Francisco Jerez > > On 26/09/16 20:02, Jason Ekstrand wrote: >> >> Looks good to me. Curro, do you see anything wrong with this? >> >> --Jason >> >> >> On Sep 26, 2016 7:31 AM, "Lionel Landwerlin" > > wrote:

Re: [Mesa-dev] [PATCH] i965/l3: Add explicit way size calculation for bxt

Ben Widawsky writes: > There should be no functional change here because Broxton and CHV are > both gt1. Without this code however, it might seem like broxton support > is missing. > > While here, put the gt1 check in front to hopefully short-circuit the > condition for the mobile cases. > > Cc:

Re: [Mesa-dev] [PATCH 7/7] egl: Unify the EGLint/EGLAttrib paths in eglCreateSync*

On 4 October 2016 at 22:31, Chad Versace wrote: > On Thu 29 Sep 2016, Emil Velikov wrote: >> On 28 September 2016 at 07:28, Chad Versace wrote: >> > Pre-patch, there were two code paths for parsing EGLSync attribute >> > lists: one path for old-style EGLint lists, used by eglCreateSyncKHR, >> > a

Re: [Mesa-dev] radv initial submission

On 4 October 2016 at 23:29, Dave Airlie wrote: >> It would have been great if the build/integration bits were properly >> split, but I won't sweat too much over it. > > I thought they were, all the bits outside src/amd are in this patch really. > > I suppose I could move the Makefile.am and Makefi

Re: [Mesa-dev] [PATCH 2/2] radv: Add new flag for LLVM dependecies with vulkan

On 2 October 2016 at 20:45, Tobias Droste wrote: > This reuse the same logic gallium uses to determine if LLVM is needed or > not: > --enable-vulkan-llvm is set to yes if at least one vulkan driver is > active and the host is i3*6 or x86_64. > To build vulkan drivers without LLVM (e.g. intel) one

[Mesa-dev] [Bug 98002] Mud rendering bug in Portal 2

https://bugs.freedesktop.org/show_bug.cgi?id=98002 Nicolai Hähnle changed: What|Removed |Added Assignee|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.

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

On 10/01/2016 03:54 PM, Serge Martin wrote: > --- > 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 | 1 + > src/gallium/state_trackers/clover/core/modul

Re: [Mesa-dev] [PATCH v3 02/14] mesa/main: add support for ARB_compute_variable_groups_size

On 09/27/2016 08:58 PM, Nicolai Hähnle wrote: On 26.09.2016 19:23, Samuel Pitoiset wrote: v2: - update formatting spec quotations (Ian) - move the total_invocations check outside of the loop (Ian) Signed-off-by: Samuel Pitoiset --- src/mesa/main/api_validate.c | 96 +

Re: [Mesa-dev] [PATCH v3 04/14] glsl: process local_size_variable input qualifier

On 09/27/2016 09:12 PM, Nicolai Hähnle wrote: On 26.09.2016 19:23, Samuel Pitoiset wrote: This is the new layout qualifier introduced by ARB_compute_variable_group_size which allows to use a variable work group size. Signed-off-by: Samuel Pitoiset Reviewed-by: Ian Romanick --- src/compiler

Re: [Mesa-dev] [PATCH v3 00/14] add support for ARB_compute_variable_group_size

On 09/27/2016 09:15 PM, Nicolai Hähnle wrote: On 26.09.2016 19:23, Samuel Pitoiset wrote: v3: - use a new case statement in r600_pipe_common.c - fix compilation with softpipe - initialize max_variable_threads_per_block to 0 I have sent some remarks on patches 2 and 4. Patches 1, 3, 5

Re: [Mesa-dev] [PATCH v2 0/2] Add support for some of the missing CL1.2 queries

On 10/01/2016 06:51 PM, Serge Martin wrote: > Updated serie, please review. > > Serge Martin (2): > clover: add CL_PROGRAM_BINARY_TYPE support (CL1.2) > clover: add missing clGetDeviceInfo CL1.2 queries > > src/gallium/state_trackers/clover/api/device.cpp | 23 > ++ >

Re: [Mesa-dev] [PATCH] clover: clGetExtensionFunctionAddressForPlatform

On 10/01/2016 07:03 PM, Serge Martin wrote: > On Sunday 27 September 2015 11:15:14 Serge Martin wrote: >> add clGetExtensionFunctionAddressForPlatform (CL 1.2) > > ping (one year reminder :p ) > >> --- >> src/gallium/state_trackers/clover/api/dispatch.cpp | 2 +- >> src/gallium/state_trackers/c

Re: [Mesa-dev] [PATCH 13/14] egl: Track EGL_KHR_debug state when going through EGL API calls (v3)

On Fri, 2016-09-16 at 18:33 +0100, Emil Velikov wrote: > But regardless, this and v2 14/14 look a lot better and are > > Reviewed-by: Emil Velikov Merged, thanks. - ajax ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedeskto

Re: [Mesa-dev] [PATCH] ddebug: dump most driver information with GALLIUM_DDEBUG=always

Reviewed-by: Nicolai Hähnle On 05.10.2016 00:37, Marek Olšák wrote: From: Marek Olšák --- src/gallium/drivers/ddebug/dd_draw.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/ddebug/dd_draw.c b/src/gallium/drivers/ddebug/dd_draw.c index 511daf4..

Re: [Mesa-dev] [PATCH] gallium/radeon/winsyses: set reasonable max_alloc_size

On 05.10.2016 00:37, Marek Olšák wrote: From: Marek Olšák which is returned for GL_MAX_TEXTURE_BUFFER_SIZE. It doesn't have any other use at the moment. Bigger allocations are not rejected. This fixes GL45-CTS.texture_buffer.texture_buffer_max_size on Bonaire. Why do you use different multip

[Mesa-dev] [PATCH 4/5] intel: aubinator: enable loading xml files from a given directory

This might be useful for people who debug with out of tree descriptions. Signed-off-by: Lionel Landwerlin --- src/intel/tools/aubinator.c | 18 ++--- src/intel/tools/decoder.c | 64 + src/intel/tools/decoder.h | 2 ++ 3 files changed, 81 i

[Mesa-dev] [PATCH v3 5/5] intel: aubinator: enable loading dumps from standard input

In conjuction with an intel_aubdump change, you can now look at your application's output like this : $ intel_aubdump -c '/path/to/aubinator --gen=hsw' my_gl_app v2: Add print_help() comment about standard input handling (Eero) Remove shrinked gtt space debug workaround (Eero) v3: Use reallo

[Mesa-dev] [PATCH v2 0/5] Enable aubinator to decoder a running application

Hi, Here is another iteration on making aubinator decode its standard input. Hopefully this addresses all the previous comments. While at it, I took on to the move aubinator to use getopt. Thanks, Lionel Landwerlin (5): intel: aubinator: add missing return characters intel: aubinator: use ge

[Mesa-dev] [PATCH 1/5] intel: aubinator: add missing return characters

Signed-off-by: Lionel Landwerlin --- src/intel/tools/aubinator.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c index 6fc0208..b74885a 100644 --- a/src/intel/tools/aubinator.c +++ b/src/intel/tools/aubinator

[Mesa-dev] [PATCH v3 3/5] intel: aubinator: generate a standalone binary

Embed the xml files into the binary, so aubinator can be used from any location. v2: Split generation packing into another patch (Jason) Check for xxd (Jason) v3: Fix out of tree builds (Jason) Generate custom variable name rather than names generated by xxd (Lionel) Signed-off-by: L

[Mesa-dev] [PATCH 2/5] intel: aubinator: use getopt to parse arguments

Signed-off-by: Lionel Landwerlin --- src/intel/tools/aubinator.c | 90 + 1 file changed, 33 insertions(+), 57 deletions(-) diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c index b74885a..44a6bb2 100644 --- a/src/intel/tools/aubina

Re: [Mesa-dev] [PATCH 4/4] radeonsi: fix interpolateAt opcodes for .zw components

On 05.10.2016 00:35, Marek Olšák wrote: From: Marek Olšák Not returning garbage in .zw seems pretty important. Those are the best bugs :) For the series: Reviewed-by: Nicolai Hähnle This fixes: GL45-CTS.shader_multisample_interpolation.render.interpolate_at_*_check.* Cc: 11.2 12.0 ---

Re: [Mesa-dev] [PATCH v3 3/5] intel: aubinator: generate a standalone binary

On Wed, Oct 5, 2016 at 8:55 AM, Lionel Landwerlin wrote: > Embed the xml files into the binary, so aubinator can be used from any > location. > > v2: Split generation packing into another patch (Jason) > Check for xxd (Jason) > > v3: Fix out of tree builds (Jason) > Generate custom variab

Re: [Mesa-dev] [PATCH 2/2] i965: use L3 data cache for SSBOs

On Mon, Sep 26, 2016 at 10:31 AM, Lionel Landwerlin wrote: > Anv programs the hardware to use L3 data cache if we use either SSBOs or > images in the shaders, we can program i965 the same way. > > gl_shader_program has a bit of a confusing named field with > 'NumAtomicBuffers'. It doesn't tell how

Re: [Mesa-dev] [PATCH 2/2] i965: use L3 data cache for SSBOs

On Wed, Oct 5, 2016 at 9:15 AM, Ilia Mirkin wrote: > On Mon, Sep 26, 2016 at 10:31 AM, Lionel Landwerlin > wrote: > > Anv programs the hardware to use L3 data cache if we use either SSBOs or > > images in the shaders, we can program i965 the same way. > > > > gl_shader_program has a bit of a con

Re: [Mesa-dev] [PATCH 2/2] i965: use L3 data cache for SSBOs

On 05/10/16 17:15, Ilia Mirkin wrote: On Mon, Sep 26, 2016 at 10:31 AM, Lionel Landwerlin wrote: Anv programs the hardware to use L3 data cache if we use either SSBOs or images in the shaders, we can program i965 the same way. gl_shader_program has a bit of a confusing named field with 'NumAto

Re: [Mesa-dev] [PATCH 1/5] intel: aubinator: add missing return characters

Reviewed-by: Jason Ekstrand On Wed, Oct 5, 2016 at 8:55 AM, Lionel Landwerlin wrote: > Signed-off-by: Lionel Landwerlin > --- > src/intel/tools/aubinator.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinat

Re: [Mesa-dev] [PATCH v3 3/5] intel: aubinator: generate a standalone binary

On 05/10/16 17:07, Jason Ekstrand wrote: +$(builddir)/%_aubinator_xml.h: $(srcdir)/../genxml/%.xml Makefile + $(MKDIR_GEN) + $(AM_V_GEN) echo -n "static const uint8_t " > $@; \ + echo -n `basename $@` | sed -e 's,_aubinator_xml.h,,' >> $@; \ + echo "

Re: [Mesa-dev] [PATCH 05/11] nir: Add a LCSAA-pass

On Tue, Oct 4, 2016 at 6:46 PM, Timothy Arceri wrote: > On Tue, 2016-10-04 at 16:47 -0700, Jason Ekstrand wrote: > > On Fri, Sep 16, 2016 at 6:24 AM, Timothy Arceri > bora.com> wrote: > > > From: Thomas Helland > > > > > > V2: Do a "depth first search" to convert to LCSSA > > > > > > V3: Small

Re: [Mesa-dev] Mesa include guard style. (Was: [PATCH] i965/cfg: Remove redundant #pragma once.)

Dave Airlie writes: > On 13 March 2016 at 11:29, Ian Romanick wrote: >> On 03/11/2016 03:46 PM, Eric Anholt wrote: >>> Ian Romanick writes: >>> On 03/10/2016 05:53 PM, Francisco Jerez wrote: > Iago Toral writes: > >> On Wed, 2016-03-09 at 19:04 -0800, Francisco Jerez wrote: >>

Re: [Mesa-dev] [PATCH v3 3/5] intel: aubinator: generate a standalone binary

From: Landwerlin, Lionel G Sent: Wednesday, October 05, 2016 9:55 AM To: Jason Ekstrand Cc: mesa-dev@lists.freedesktop.org; Gandikota, Sirisha Subject: Re: [Mesa-dev] [PATCH v3 3/5] intel: aubinator: generate a standalone binary On 05/10/16 17:07, Jason Ekstrand wrote: +$(builddir)/%_aubina

[Mesa-dev] [Bug 97260] R9 290 low performance in Linux 4.7

https://bugs.freedesktop.org/show_bug.cgi?id=97260 --- Comment #59 from Alex Deucher --- does reverting either of these patches help? commit 5e031d9fe8b0741f11d49667dfc3ebf5454121fd Author: Alex Deucher Date: Wed Feb 24 17:38:38 2016 -0500 drm/radeon/pm: update current crtc info after se

Re: [Mesa-dev] [PATCH 1/3] i965: solve cubemap negative x/y/z faces buffer offset issue in dEQP.

Randy, I hadn't realized that we could get images in from EGL where we have a non-zero tile_x and tile_y offset for layer 0 mip 0. That explains things. In that case, I believe this is the correct patch. That said, I would like to see an "assert(tile_x == 0 && tile_y == 0)" right before we do t

Re: [Mesa-dev] [PATCH 1/2] intel: aubinator: generate a standalone binary

On Tue, Oct 4, 2016 at 9:26 AM Ben Widawsky wrote: > On 16-10-04 15:38:52, Lionel Landwerlin wrote: > >Embed the xml files into the binary, so aubinator can be used from any > >location. > > > >Signed-off-by: Lionel Landwerlin > >Cc: Sirisha Gandikota > >--- > > src/intel/Makefile.am

Re: [Mesa-dev] [PATCH] gallium/radeon/winsyses: set reasonable max_alloc_size

On Wed, Oct 5, 2016 at 5:53 PM, Nicolai Hähnle wrote: > On 05.10.2016 00:37, Marek Olšák wrote: >> >> From: Marek Olšák >> >> which is returned for GL_MAX_TEXTURE_BUFFER_SIZE. >> It doesn't have any other use at the moment. >> Bigger allocations are not rejected. >> >> This fixes GL45-CTS.texture

[Mesa-dev] [PATCH v4 04/14] glsl: process local_size_variable input qualifier

This is the new layout qualifier introduced by ARB_compute_variable_group_size which allows to use a variable work group size. v4: - add missing '%s' in the monster format string Signed-off-by: Samuel Pitoiset Reviewed-by: Ian Romanick --- src/compiler/glsl/ast.h | 5 + s

[Mesa-dev] [PATCH v4 03/14] glsl: add enable flags for ARB_compute_variable_group_size

This also initializes the default values for the standalone compiler. Signed-off-by: Samuel Pitoiset Reviewed-by: Ian Romanick Reviewed-by: Nicolai Hähnle --- src/compiler/glsl/glsl_parser_extras.cpp | 1 + src/compiler/glsl/glsl_parser_extras.h | 2 ++ src/compiler/glsl/standalone.c

[Mesa-dev] [PATCH v4 02/14] mesa/main: add support for ARB_compute_variable_groups_size

v4: - slightly indent spec quotes (Nicolai) - drop useless _mesa_has_compute_shaders() check (Nicolai) - move the fixed local size outside of the loop (Nicolai) - add missing check for invalid use of work group count v2: - update formatting spec quotations (Ian) - move the total_inv

[Mesa-dev] [PATCH v4 06/14] glsl/linker: handle errors when a variable local size is used

Compute shaders can now include a fixed local size as defined by ARB_compute_shader or a variable size as defined by ARB_compute_variable_group_size. v2: - update formatting spec quotations (Ian) - various cosmetic changes (Ian) Signed-off-by: Samuel Pitoiset Reviewed-by: Ian Romanick Revie

[Mesa-dev] [PATCH v4 05/14] glsl: reject compute shaders with fixed and variable local size

The ARB_compute_variable_group_size specification explains that when a compute shader includes both a fixed and a variable local size, a compile-time error occurs. v2: - update formatting spec quotations (Ian) Signed-off-by: Samuel Pitoiset Reviewed-by: Nicolai Hähnle --- src/compiler/glsl/ast

[Mesa-dev] [PATCH v4 01/14] glapi: add entry points for GL_ARB_compute_variable_group_size

v2: - correctly sort that new extension (Ian) - fix up the comment (Ian) Signed-off-by: Samuel Pitoiset Reviewed-by: Ian Romanick Reviewed-by: Nicolai Hähnle --- .../glapi/gen/ARB_compute_variable_group_size.xml | 25 ++ src/mapi/glapi/gen/Makefile.am

[Mesa-dev] [PATCH v4 00/14] add support for ARB_compute_variable_group_size

v4: - use 512 threads on Fermi, 2014 on Kepler+ - add missing '%s' in the monster format string - slightly indent spec quotes - drop useless _mesa_has_compute_shaders() check - move the fixed local size outside of the loop - add missing check for invalid use of work group c

[Mesa-dev] [PATCH v4 12/14] nv50/ir: set number of threads/block for variable local size

When a variable local size is defined as specified by ARB_compute_variable_group_size, the fixed local size is set to 0 and a SIGFPE occurs when we compute the maximum number of regs. This allows to use 64 GPRs/thread. v4: - use 512 threads on Fermi, 1024 on Kepler+ Signed-off-by: Samuel Pitoise

[Mesa-dev] [PATCH v4 09/14] st/mesa: add mapping for SYSTEM_VALUE_LOCAL_GROUP_SIZE

gl_LocalGroupSizeARB can be translated into TGSI_SEMANTIC_BLOCK_SIZE which represents the block size in threads. Signed-off-by: Samuel Pitoiset Reviewed-by: Marek Olšák Reviewed-by: Nicolai Hähnle --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --gi

[Mesa-dev] [PATCH v4 14/14] docs: mark ARB_compute_variable_group_size as done for nvc0

Signed-off-by: Samuel Pitoiset Reviewed-by: Marek Olšák --- docs/features.txt | 2 +- docs/relnotes/12.1.0.html | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/docs/features.txt b/docs/features.txt index 85ad1a1..12f0f25 100644 --- a/docs/features.txt +++ b/docs/feat

[Mesa-dev] [PATCH v4 13/14] nvc0: expose ARB_compute_variable_group_size

Only expose 512 threads/block on Fermi to not be limited by 32 GPRs/thread. v4: - use 512 threads on Fermi, 2014 on Kepler+ Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/gallium/d

[Mesa-dev] [PATCH v4 10/14] st/mesa: add support for dispatching a variable local size

Signed-off-by: Samuel Pitoiset Reviewed-by: Marek Olšák Reviewed-by: Nicolai Hähnle --- src/mesa/state_tracker/st_cb_compute.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/src/mesa/state_tracker/st_cb_compute.c b/src/mesa/state_tracker/st_cb_compute.c in

[Mesa-dev] [PATCH v4 07/14] glsl: add gl_LocalGroupSizeARB as a system value

v2: - only add it if the ext is enabled (Ilia) Signed-off-by: Samuel Pitoiset Reviewed-by: Ian Romanick Reviewed-by: Nicolai Hähnle --- src/compiler/glsl/builtin_variables.cpp | 6 ++ src/compiler/shader_enums.h | 1 + 2 files changed, 7 insertions(+) diff --git a/src/compiler

[Mesa-dev] [PATCH v4 08/14] gallium: add PIPE_COMPUTE_CAP_MAX_VARIABLE_THREADS_PER_BLOCK

v3: - use a new case statement in r600_pipe_common.c - fix compilation of softpipe... Signed-off-by: Samuel Pitoiset Reviewed-by: Marek Olšák Reviewed-by: Nicolai Hähnle --- src/gallium/docs/source/screen.rst | 4 src/gallium/drivers/ilo/ilo_screen.c | 2 ++ src/

[Mesa-dev] [PATCH v4 11/14] st/mesa: expose ARB_compute_variable_group_size

This extension is only exposed if the underlying driver supports ARB_compute_shader and if PIPE_COMPUTE_MAX_VARIABLE_THREADS_PER_BLOCK is set. v3: - initialize max_variable_threads_per_block to 0 v2: - expose the ext based on that new cap Signed-off-by: Samuel Pitoiset Reviewed-by: Marek Olšák

Re: [Mesa-dev] [PATCH 08/11] nir: add helper for cloning loops

On Fri, Sep 16, 2016 at 6:24 AM, Timothy Arceri < timothy.arc...@collabora.com> wrote: > --- > src/compiler/nir/nir.h | 2 ++ > src/compiler/nir/nir_clone.c | 41 ++ > --- > 2 files changed, 36 insertions(+), 7 deletions(-) > > diff --git a/src/compiler/

Re: [Mesa-dev] [PATCH v4 13/14] nvc0: expose ARB_compute_variable_group_size

On Wed, Oct 5, 2016 at 2:48 PM, Samuel Pitoiset wrote: > Only expose 512 threads/block on Fermi to not be limited by > 32 GPRs/thread. > > v4: - use 512 threads on Fermi, 2014 on Kepler+ Dyslexics... untie! > > Signed-off-by: Samuel Pitoiset > --- > src/gallium/drivers/nouveau/nvc0/nvc0_screen

Re: [Mesa-dev] [PATCH v4 13/14] nvc0: expose ARB_compute_variable_group_size

On 10/05/2016 08:57 PM, Ilia Mirkin wrote: On Wed, Oct 5, 2016 at 2:48 PM, Samuel Pitoiset wrote: Only expose 512 threads/block on Fermi to not be limited by 32 GPRs/thread. v4: - use 512 threads on Fermi, 2014 on Kepler+ Dyslexics... untie! Ahah! :) Typo... Signed-off-by: Samuel P

[Mesa-dev] [PATCH] nir: Delete open coded type printing.

glsl_print_type() prints arrays of arrays incorrectly. For example, a type with name float[3][7] would be printed as float[7][3]. (This is an array of length 3 containing arrays of 7 floats.) cdecl says that the type name is correct. glsl_print_type() doesn't really do anything above and beyond

[Mesa-dev] [PATCH] nir: Use the correct infos structure for copying atomic sources

Signed-off-by: Jason Ekstrand Cc: "12.0" Cc: Jordan Justen --- src/compiler/nir/nir_lower_io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compiler/nir/nir_lower_io.c b/src/compiler/nir/nir_lower_io.c index d469b61..a3b1423 100644 --- a/src/compiler/nir/nir_lower_io.

[Mesa-dev] [PATCH] nir: Use the correct infos structure for copying atomic sources

Signed-off-by: Jason Ekstrand Cc: "12.0" Cc: Jordan Justen --- src/compiler/nir/nir_lower_io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compiler/nir/nir_lower_io.c b/src/compiler/nir/nir_lower_io.c index d469b61..a3b1423 100644 --- a/src/compiler/nir/nir_lower_io.

Re: [Mesa-dev] [PATCH 1/4] amd/common: add nir->llvm translation.

On Wed, Oct 5, 2016 at 1:58 AM, Dave Airlie wrote: > On 4 October 2016 at 20:09, Nicolai Hähnle wrote: >> On 04.10.2016 03:48, Dave Airlie wrote: >>> >>> From: Bas Nieuwenhuizen >>> >>> This adds the basic files for the NIR->LLVM translation layer, >>> along with some hopefully generic code to l

Re: [Mesa-dev] [PATCH] nir: Use the correct infos structure for copying atomic sources

I don't know anything about this code, but the change seems obviously correct so feel free to add my r-b. On Wed, Oct 5, 2016 at 3:13 PM, Jason Ekstrand wrote: > Signed-off-by: Jason Ekstrand > Cc: "12.0" > Cc: Jordan Justen > --- > src/compiler/nir/nir_lower_io.c | 2 +- > 1 file changed, 1

Re: [Mesa-dev] [PATCH] nir: Use the correct infos structure for copying atomic sources

Tested-by: Mark Janes Jason Ekstrand writes: > Signed-off-by: Jason Ekstrand > Cc: "12.0" > Cc: Jordan Justen > --- > src/compiler/nir/nir_lower_io.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/compiler/nir/nir_lower_io.c b/src/compiler/nir/nir_lower_io.c >

Re: [Mesa-dev] [PATCH 4/4] radv: toplevel configure/make changes required to build (v1.1)

2016-10-05 2:48 GMT+02:00 Dave Airlie : > From: Dave Airlie > > This moves some of the llvm checks around to allow them > to be used for non-gallium drivers as well. > > radv requires llvm 3.9.0 as vulkan requires compute shaders. > > v1.1: add all make infrastructure to this patch for easier > re

Re: [Mesa-dev] [PATCH] nir: Use the correct infos structure for copying atomic sources

Thanks all! Pushed. On Wed, Oct 5, 2016 at 12:29 PM, Mark Janes wrote: > Tested-by: Mark Janes > > Jason Ekstrand writes: > > > Signed-off-by: Jason Ekstrand > > Cc: "12.0" > > Cc: Jordan Justen > > --- > > src/compiler/nir/nir_lower_io.c | 2 +- > > 1 file changed, 1 insertion(+), 1 dele

[Mesa-dev] [PATCH 03/75] st/nine: Unify some check flags

The new defines will be reused in a later patch. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_defines.h | 8 src/gallium/state_trackers/nine/swapchain9.c | 8 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/

[Mesa-dev] [PATCH 02/75] gallium/util: Really allow aliasing for u_box_union_*

Fixes several game bugs with nine. Fixes https://github.com/iXit/Mesa-3D/issues/234 Signed-off-by: Axel Davy Signed-off-by: Patrick Rudolph Cc: "12.0" --- src/gallium/auxiliary/util/u_box.h | 31 --- 1 file changed, 20 insertions(+), 11 deletions(-) diff --git a/s

[Mesa-dev] [PATCH 04/75] st/nine: Fix some check flags

Uses the new defines introduced in previous commit. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/adapter9.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/src/gallium/state_trackers/nine/adapter9.c b/src/gallium/state_trackers/nine/

[Mesa-dev] [PATCH 05/75] st/nine: Disable seamless cubemap for d3d

d3d9 doesn't have seamless cubemap. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 2 +- src/gallium/state_trackers/nine/nine_pipe.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_t

[Mesa-dev] [PATCH 18/75] st/nine: Rework vs int and bool constants buffer

This will help to support swvp constants. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 35 +++ src/gallium/state_trackers/nine/nine_state.c | 4 +-- src/gallium/state_trackers/nine/nine_state.h | 4 +-- src/gallium/state_trackers/nine/s

[Mesa-dev] [PATCH 09/75] st/nine: Implement relative addressing for ps inputs

To implement the feature we copy the ps inputs to a temp array. This is not optimal for performance, but it is the simplest solution. This is a feature that is very very rarely used. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_shader.c | 30 --- 1 f

[Mesa-dev] [PATCH 14/75] st/nine: Initial mixed vertex processing support

In mixed vertex processing, the user can enable or disable software vertex processing. It is on hardware by default. This feature is not a state, and thus the setting doesn't need to be recorded by stateblocks. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 18

[Mesa-dev] [PATCH 07/75] st/nine: Use fixed size arrays for swapchain buffers

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/swapchain9.c | 43 +--- src/gallium/state_trackers/nine/swapchain9.h | 8 +++--- 2 files changed, 11 insertions(+), 40 deletions(-) diff --git a/src/gallium/state_trackers/nine/swapchain9.c b/src/gallium/state

[Mesa-dev] [PATCH 00/75] Gallium Nine patches for Mesa 13

Hi there, here is our serie for Mesa 13. Basically there are a lot of fixes, from solving app crashes, hangs and important visual bugs, to lightning issues on old apps. On the feature side, now the few apps that were missing multisampling settings now have some. Old apps needing Software Vertex

[Mesa-dev] [PATCH 08/75] st/nine: Wait for pending tasks to execute in swapchain

Fixes crash after Reset() when using thread_submit=true Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/swapchain9.c | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/state_trackers/nine/swapchain9.c b/src/gallium/state_trackers/nine/swapchain9.c index bc34dd2..cc

[Mesa-dev] [PATCH 23/75] st/nine: Cleaning code for vs temporaries

This has been a real mess up to now: the temporaries were allocated once, and shared after that between the different parts of the code. To help maintaining the code, the temporaries are now allocated and released on need. As surprising as it could be, this patch, which was supposed to introduce

[Mesa-dev] [PATCH 15/75] st/nine: Add support for swvp constants in shaders

swvp has relaxed limits (more nested loops, etc). In particular it enables more constants. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_shader.c | 156 ++-- src/gallium/state_trackers/nine/nine_shader.h | 2 + src/gallium/state_trackers/nine/vertexs

[Mesa-dev] [PATCH 13/75] st/nine: Implement SetNPatchMode

Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index f2317b6..e4ede6e 100644 --- a/src/gallium/state_trackers/nine/dev

[Mesa-dev] [PATCH 17/75] st/nine: Change dirty tracking for vs int and bool constants

This change makes easier to introduce tracking for swvp constants. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 14 +++--- src/gallium/state_trackers/nine/nine_state.c | 18 + src/gallium/state_trackers/nine/nine_state.h | 8 +++--- src/gallium/

[Mesa-dev] [PATCH 24/75] st/nine: Small simplification for position_t and fog

position_t disables fog computation. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index 7b3c67c..40717f1 1

[Mesa-dev] [PATCH 21/75] st/nine: Handle D3DRS_NORMALIZENORMALS

When this state is set, the normals computed in the vs ff shader should be normalized. Signed-off-by: Axel Davy Signed-off-by: Patrick Rudolph s...@das-labor.org --- src/gallium/state_trackers/nine/nine_ff.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/gallium/s

[Mesa-dev] [PATCH 06/75] st/nine: Fix buffer count check for Ex devices

From: Patrick Rudolph Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/swapchain9.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/gallium/state_trackers/nine/swapchain9.c b/src/gallium/state_trackers/nine/swapchain9.c

[Mesa-dev] [PATCH 25/75] st/nine: Reorganize ff vtx processing

The new order simplified the code a bit for next patches. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 124 +- 1 file changed, 72 insertions(+), 52 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_tr

[Mesa-dev] [PATCH 32/75] st/nine: Use materials if source is not given.

Deduced by test on windows. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index 4c92920..86e1420 100644 ---

[Mesa-dev] [PATCH 11/75] st/nine: Allocate more space for ATI1

From: Patrick Rudolph ATIx are "unknown" formats that do not follow block format conventions. Tests showed that pitch*height bytes are allocated. apitrace used to depend on this behaviour. It used to copy more bytes than it has to for the ATI1 block format, but it didn't crash on Windows. Increa

[Mesa-dev] [PATCH 53/75] st/nine: Print interface id in NineSurface9_GetContainer

From: Patrick Rudolph To ease debugging print interface id. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/surface9.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/state_trackers/nine/surface9.c b/src/gallium/state_trackers/ni

[Mesa-dev] [PATCH 44/75] st/nine: Do always accumulate diffuse

According to spec. Signed-off-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/nine_ff.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index 924cb4c.

[Mesa-dev] [PATCH 16/75] st/nine: Drop unused constant upload path

This path has been disabled for some time because of some bugs with it. It hasn't been updated to the new features, and is not faster. Signed-off-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c| 26 src/gallium/state_trackers/nine/device9.h

[Mesa-dev] [PATCH 50/75] st/nine: Properly declare sampler states for ff

Fixes a softpipe assertion failure with wine tests Signed-off-by: Axel Davy Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/nine_ff.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_track

[Mesa-dev] [PATCH 20/75] st/nine: Initial ProcessVertices support

For now only VS 3 support is implemented. This enables The Sims 2 to work. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 143 +--- src/gallium/state_trackers/nine/device9.h | 7 + src/gallium/state_trackers/nine/nine_shader.c | 69 +++- s

[Mesa-dev] [PATCH 33/75] st/nine: Disable ff vertex blending if required inputs are missing

This behaviour has been partially tested on windows. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c i

[Mesa-dev] [PATCH 72/75] st/nine: Set CLAMP_TO_EDGE on cubetextures

From: Patrick Rudolph Wine tests show that cubetextures always use PIPE_TEX_WRAP_CLAMP_TO_EDGE regardless of set sampler states. Fixes failing d3d9 wine test test_cube_wrap. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/nine_pipe.c | 14 ++

[Mesa-dev] [PATCH 38/75] st/nine: Used normed Vtx for reflectionvector

Fix deduced from the spec. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/nine_ff.c b/src/gallium/state_trackers/nine/nine_ff.c index c2ce352..2c237f4 100644 --- a/

[Mesa-dev] [PATCH 61/75] st/nine: Switch {Set/Get/Free}PrivateData functions

From: Patrick Rudolph Switch {Set/Get/Free}PrivateData function to introduced IUnknown functions. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/cubetexture9.c | 6 +- src/gallium/state_trackers/nine/indexbuffer9.c | 6 +- src/gallium/state_tra

[Mesa-dev] [PATCH 54/75] st/nine: Print interface id in NineVolume9_GetContainer

From: Patrick Rudolph To ease debugging print interface id. Signed-off-by: Patrick Rudolph Reviewed-by: Axel Davy --- src/gallium/state_trackers/nine/volume9.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/gallium/state_trackers/nine/volume9.c b/src/gallium/state_trackers/nine

[Mesa-dev] [PATCH 48/75] st/nine: Fix the calculation of the number of vs inputs

Fixes hangs on radeonsi, and assert on llvmpipe. Signed-off-by: Axel Davy Cc: "12.0" --- src/gallium/state_trackers/nine/nine_shader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/nine_shader.c b/src/gallium/state_trackers/nine/nine_shade

  1   2   >