Re: [Mesa-dev] [PATCH 4/7] anv/pipeline/gen8: Unconditionally set DXMultisampleRasterizaitonEnable

2016-08-06 Thread Jason Ekstrand
On Sat, Aug 6, 2016 at 4:55 PM, Jason Ekstrand wrote: > On Sat, Aug 6, 2016 at 4:15 PM, Kenneth Graunke > wrote: > >> On Saturday, August 6, 2016 10:31:19 AM PDT Jason Ekstrand wrote: >> > This bit doesn't enable multisampling, it simply sets the

[Mesa-dev] [PATCH 3/8] st/mesa: remove TES/TCS/GS state dirtying optimization

2016-08-06 Thread Marek Olšák
From: Marek Olšák This will be replaced with a better mechanism. --- src/mesa/state_tracker/st_atom.c | 8 1 file changed, 8 deletions(-) diff --git a/src/mesa/state_tracker/st_atom.c b/src/mesa/state_tracker/st_atom.c index 7548a29..9985168 100644 ---

[Mesa-dev] [PATCH 1/8] st/mesa: don't update clip state if it has no effect

2016-08-06 Thread Marek Olšák
From: Marek Olšák --- src/mesa/state_tracker/st_atom.c| 3 --- src/mesa/state_tracker/st_context.c | 3 ++- src/mesa/state_tracker/st_context.h | 7 +++ 3 files changed, 9 insertions(+), 4 deletions(-) diff --git a/src/mesa/state_tracker/st_atom.c

[Mesa-dev] [PATCH 4/8] st/mesa: determine states used or affected by shaders at compile time

2016-08-06 Thread Marek Olšák
From: Marek Olšák At compile time, each shader determines which ST_NEW flags should be set at shader bind time. This just sets the new field for all shaders. The next commit will use it. --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 175 -

[Mesa-dev] [PATCH 7/8] st/mesa: don't update atomic, SSBO, UBO and TBO states that have no effect

2016-08-06 Thread Marek Olšák
From: Marek Olšák --- src/mesa/state_tracker/st_atom.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/mesa/state_tracker/st_atom.c b/src/mesa/state_tracker/st_atom.c index dddc5ff..94e012a 100644 --- a/src/mesa/state_tracker/st_atom.c +++

[Mesa-dev] [PATCH 5/8] st/mesa: when changing shaders, only dirty states that are affected by them

2016-08-06 Thread Marek Olšák
From: Marek Olšák This reduces the amount of state processing that has no effect. --- src/mesa/state_tracker/st_atom.c| 78 ++--- src/mesa/state_tracker/st_atom.h| 72 ++

[Mesa-dev] [PATCH 8/8] st/mesa: BufferData should flag NewDriverState

2016-08-06 Thread Marek Olšák
From: Marek Olšák because NewDriverState is filtered depending on active shader states, while st->dirty isn't. --- src/mesa/state_tracker/st_cb_bufferobjects.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[Mesa-dev] [PATCH 6/8] st/mesa: _NEW_TEXTURE & CONSTANTS shouldn't flag states that aren't used

2016-08-06 Thread Marek Olšák
From: Marek Olšák --- src/mesa/state_tracker/st_context.c | 64 - src/mesa/state_tracker/st_context.h | 6 2 files changed, 62 insertions(+), 8 deletions(-) diff --git a/src/mesa/state_tracker/st_context.c

[Mesa-dev] [PATCH 0/8] More state optimizations for st/mesa

2016-08-06 Thread Marek Olšák
Hi, When binding a new shader, st/mesa marks at least 7 states as dirty per shader stage, but usually we just need shader constants and maybe fragment shader textures. Processing states for UBOs, SSBOs, atomics, and images is usually a waste of time. This series resolved that issue as well as

[Mesa-dev] [PATCH 2/8] st/mesa: don't update clip state on VS changes if it has no effect

2016-08-06 Thread Marek Olšák
From: Marek Olšák --- src/mesa/state_tracker/st_atom.c| 4 ++-- src/mesa/state_tracker/st_atom.h| 23 --- src/mesa/state_tracker/st_cb_feedback.c | 2 +- src/mesa/state_tracker/st_cb_program.c | 2 +- 4 files changed, 16

Re: [Mesa-dev] [PATCH 4/7] anv/pipeline/gen8: Unconditionally set DXMultisampleRasterizaitonEnable

2016-08-06 Thread Jason Ekstrand
On Sat, Aug 6, 2016 at 4:15 PM, Kenneth Graunke wrote: > On Saturday, August 6, 2016 10:31:19 AM PDT Jason Ekstrand wrote: > > This bit doesn't enable multisampling, it simply sets the multisampling > > mode to DX which is what we want. > > This in fact does not set the

Re: [Mesa-dev] [PATCH 1/7] anv/pipeline/gen8: Remove an old comment

2016-08-06 Thread Kenneth Graunke
On Saturday, August 6, 2016 10:31:16 AM PDT Jason Ekstrand wrote: > This is now handled in emit_3dstate_clip Not anymore, it's now handled in emit_3dstate_streamout. Otherwise, patches 1-3 get my R-b. > --- > src/intel/vulkan/gen8_pipeline.c | 2 -- > 1 file changed, 2 deletions(-) > > diff

Re: [Mesa-dev] [PATCH 4/7] anv/pipeline/gen8: Unconditionally set DXMultisampleRasterizaitonEnable

2016-08-06 Thread Kenneth Graunke
On Saturday, August 6, 2016 10:31:19 AM PDT Jason Ekstrand wrote: > This bit doesn't enable multisampling, it simply sets the multisampling > mode to DX which is what we want. This in fact does not set the multisampling API mode. That's the DXMultisampleRasterizationMode field, while you're

Re: [Mesa-dev] [PATCH 1/6] gallium/radeon: change the vendor string to Advanced Micro Devices, Inc.

2016-08-06 Thread Axel Davy
Hi, This looks like it will perturb driver detection of already written games. For example from the dolphin sources, this change would make them detect catalyst as driver. As it is known workarounds are applied depending on the driver detected, or some features are disabled, I believe it

[Mesa-dev] [PATCH] st/mesa: use PIPE_USAGE_STREAM for GL_CLIENT_STORAGE_BIT without READ_BIT (v2)

2016-08-06 Thread Marek Olšák
From: Marek Olšák v2: keep STAGING for GL_MAP_READ_BIT --- src/mesa/state_tracker/st_cb_bufferobjects.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/mesa/state_tracker/st_cb_bufferobjects.c

Re: [Mesa-dev] [PATCH 1/3] gallium: add a pipe_context parameter to fence_finish

2016-08-06 Thread Marek Olšák
On Sat, Aug 6, 2016 at 9:07 PM, Rob Clark wrote: > On Sat, Aug 6, 2016 at 11:32 AM, Marek Olšák wrote: >> From: Marek Olšák >> >> required by glClientWaitSync (GL 4.5 Core spec) that can optionally flush >> the context > > Thanks, I've

[Mesa-dev] [PATCH] mesa: simplify ff fs generator a bit

2016-08-06 Thread Miklós Máté
Literally. Signed-off-by: Miklós Máté --- src/mesa/main/ff_fragment_shader.cpp | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/src/mesa/main/ff_fragment_shader.cpp b/src/mesa/main/ff_fragment_shader.cpp index b0ce8c4..ad3d666 100644 ---

[Mesa-dev] [Bug 97231] GL_DEPTH_CLAMP doesn't clamp to the far plane

2016-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97231 --- Comment #1 from Jules Blok --- You can also take a look at this issue on our automated testing system: https://fifoci.dolphin-emu.org/compare/1931541-1924723/ And here is the relevant PR where we are running into this

[Mesa-dev] [Bug 97231] GL_DEPTH_CLAMP doesn't clamp to the far plane

2016-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97231 Bug ID: 97231 Summary: GL_DEPTH_CLAMP doesn't clamp to the far plane Product: Mesa Version: unspecified Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

Re: [Mesa-dev] [PATCH] st/mesa: use PIPE_USAGE_STREAM for GL_CLIENT_STORAGE_BIT

2016-08-06 Thread Fredrik Höglund
On Saturday 06 August 2016, Marek Olšák wrote: > From: Marek Olšák > > STAGING is mainly for fast CPU reads and random access. I don't think > glBufferStorage(GL_CLIENT_STORAGE_BIT) is used for those two purposes. > I'd say GL_CLIENT_STORAGE_BIT is more of a streaming-upload

Re: [Mesa-dev] [PATCH 1/3] gallium: add a pipe_context parameter to fence_finish

2016-08-06 Thread Rob Clark
On Sat, Aug 6, 2016 at 11:32 AM, Marek Olšák wrote: > From: Marek Olšák > > required by glClientWaitSync (GL 4.5 Core spec) that can optionally flush > the context Thanks, I've been meaning to add the same thing but haven't had time to work on the native

[Mesa-dev] [PATCH 5/6] winsys/radeon: track the amount of mapped memory

2016-08-06 Thread Marek Olšák
From: Marek Olšák --- src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 13 - src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 4 src/gallium/winsys/radeon/drm/radeon_drm_winsys.h | 2 ++ 3 files changed, 18 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 4/6] winsys/amdgpu: track the amount of mapped memory

2016-08-06 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeon/radeon_winsys.h| 2 ++ src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 18 +- src/gallium/winsys/amdgpu/drm/amdgpu_bo.h | 1 + src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 4

[Mesa-dev] [PATCH 2/6] gallium/radeon: increase the size of the renderer string

2016-08-06 Thread Marek Olšák
From: Marek Olšák Mine is longer than 64 bytes. --- src/gallium/drivers/radeon/r600_pipe_common.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/r600_pipe_common.h b/src/gallium/drivers/radeon/r600_pipe_common.h index

[Mesa-dev] [PATCH 1/6] gallium/radeon: change the vendor string to Advanced Micro Devices, Inc.

2016-08-06 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c b/src/gallium/drivers/radeon/r600_pipe_common.c index 1c56e6e..8a9ab94 100644 ---

[Mesa-dev] [PATCH 3/6] winsys/amdgpu: don't try to unmap userptr buffers

2016-08-06 Thread Marek Olšák
From: Marek Olšák no app calls this AFAIK --- src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c b/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c index 1805ce6..db2c77f 100644 ---

[Mesa-dev] [PATCH 6/6] gallium/radeon: add HUD queries for mapped VRAM/GTT

2016-08-06 Thread Marek Olšák
From: Marek Olšák mainly for monitoring visible VRAM congestion --- src/gallium/drivers/radeon/r600_query.c | 10 ++ src/gallium/drivers/radeon/r600_query.h | 2 ++ 2 files changed, 12 insertions(+) diff --git a/src/gallium/drivers/radeon/r600_query.c

[Mesa-dev] [PATCH] st/mesa: use PIPE_USAGE_STREAM for GL_CLIENT_STORAGE_BIT

2016-08-06 Thread Marek Olšák
From: Marek Olšák STAGING is mainly for fast CPU reads and random access. I don't think glBufferStorage(GL_CLIENT_STORAGE_BIT) is used for those two purposes. I'd say GL_CLIENT_STORAGE_BIT is more of a streaming-upload kind of thing. ---

Re: [Mesa-dev] non-shared glapi still needed ?

2016-08-06 Thread Emil Velikov
On 3 August 2016 at 16:37, Chad Versace wrote: > On 08/02/2016 06:40 PM, Enrico Weigelt, metux IT consult wrote: >> >> Hi folks, >> >> >> is there still a real need for having non-shared glapi ? >> Otherwise, should we remove it ? > > > +1 > I have the same question. > Think

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Jan Vesely
On Sat, 2016-08-06 at 18:52 +0200, ⚛ wrote: > On Sat, Aug 6, 2016 at 5:52 PM, Jan Vesely > wrote: > > > > The situation I'm concerned about is as > > follows: > > 1.) mesa builds fine using existing build environment. > > > > 2.) I update LLVM. This update changes

Re: [Mesa-dev] oldest gcc version to support ?

2016-08-06 Thread Emil Velikov
On 4 August 2016 at 22:27, Enrico Weigelt, metux IT consult wrote: > On 04.08.2016 17:10, Jason Ekstrand wrote: > > Hi, > >> The BSDs are still stuck at 4.2+patches for licensing reasons and they >> use Mesa for graphics. I think at some point, they'll get clang in core

Re: [Mesa-dev] cairo as state tracker

2016-08-06 Thread Emil Velikov
On 6 August 2016 at 15:59, Enrico Weigelt, metux IT consult wrote: > On 06.08.2016 13:16, Nicolai Hähnle wrote: > >> Well, in your first mail it sounded like you wanted to stabilize the >> Gallium API itself. > > Not actually stabilizing, but making it semi-public. (at

Re: [Mesa-dev] [PATCH 1/3] freedreno/a2xx: fix comparison out of range warnings

2016-08-06 Thread Rob Clark
Thanks.. my expectation is that ~0 gets cast to the same enum on both sides of the != so I *think* we should be good. BR, -R On Sat, Aug 6, 2016 at 9:05 AM, Francesco Ansanelli wrote: > Hello, > > I did more tests... and I didn't experience any error. > I attach last

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Emil Velikov
On 6 August 2016 at 01:42, Jan Ziak <0xe2.0x9a.0...@gmail.com> wrote: > Mesa source code prior to this patch uses both RTLD_NOW and RTLD_LAZY. > This patch removes all RTLD_NOW in favor of RTLD_LAZY. > > In comparison to early binding, lazy binding reduces CPU instruction count > of small GL apps

[Mesa-dev] [PATCH 5/7] anv/pipeline: Unify 3DSTATE_RASTER and 3DSTATE_SF setup between gen7 and gen8

2016-08-06 Thread Jason Ekstrand
--- src/intel/vulkan/gen7_pipeline.c | 43 + src/intel/vulkan/gen8_pipeline.c | 48 src/intel/vulkan/genX_pipeline_util.h | 60 +++ 3 files changed, 61 insertions(+), 90 deletions(-) diff --git

[Mesa-dev] [PATCH 7/7] anv/pipeline/gen7: Set the depth format in 3DSTATE_SF

2016-08-06 Thread Jason Ekstrand
--- src/intel/vulkan/gen7_pipeline.c | 3 ++- src/intel/vulkan/gen8_pipeline.c | 3 ++- src/intel/vulkan/genX_pipeline_util.h | 21 + 3 files changed, 25 insertions(+), 2 deletions(-) diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c

[Mesa-dev] [PATCH 0/7] anv: clean up and unify emit_rs_state

2016-08-06 Thread Jason Ekstrand
This tiny little series cleans up emit_rs_state and unifies it between gen7 and gen8. At the end, we also fix a bug in gen7 setup where we weren't setting the depth format in 3DSTATE_SF. Jason Ekstrand (7): anv/pipeline/gen8: Remove an old comment genxml: Make 3DSTATE_SF more consistent

[Mesa-dev] [PATCH 3/7] anv/pipeline/gen8: Use fewer designated initializers in emit_rs_state

2016-08-06 Thread Jason Ekstrand
--- src/intel/vulkan/gen8_pipeline.c | 53 1 file changed, 27 insertions(+), 26 deletions(-) diff --git a/src/intel/vulkan/gen8_pipeline.c b/src/intel/vulkan/gen8_pipeline.c index 01bab15..6b7936a 100644 --- a/src/intel/vulkan/gen8_pipeline.c +++

[Mesa-dev] [PATCH 1/7] anv/pipeline/gen8: Remove an old comment

2016-08-06 Thread Jason Ekstrand
This is now handled in emit_3dstate_clip --- src/intel/vulkan/gen8_pipeline.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/intel/vulkan/gen8_pipeline.c b/src/intel/vulkan/gen8_pipeline.c index cc10d3a..01bab15 100644 --- a/src/intel/vulkan/gen8_pipeline.c +++

[Mesa-dev] [PATCH 6/7] isl: Add a helper for getting a depth format from an isl_format

2016-08-06 Thread Jason Ekstrand
--- src/intel/isl/isl.h| 2 ++ src/intel/isl/isl_format.c | 24 2 files changed, 26 insertions(+) diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h index 19673f8..4caf0e8 100644 --- a/src/intel/isl/isl.h +++ b/src/intel/isl/isl.h @@ -1016,6 +1016,8 @@

[Mesa-dev] [PATCH 0/7] anv: clean up and unify emit_rs_state

2016-08-06 Thread Jason Ekstrand
This tiny little series cleans up emit_rs_state and unifies it between gen7 and gen8. At the end, we also fix a bug in gen7 setup where we weren't setting the depth format in 3DSTATE_SF. Jason Ekstrand (7): anv/pipeline/gen8: Remove an old comment genxml: Make 3DSTATE_SF more consistent

[Mesa-dev] [PATCH 4/7] anv/pipeline/gen8: Unconditionally set DXMultisampleRasterizaitonEnable

2016-08-06 Thread Jason Ekstrand
This bit doesn't enable multisampling, it simply sets the multisampling mode to DX which is what we want. --- src/intel/vulkan/gen8_pipeline.c | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/src/intel/vulkan/gen8_pipeline.c b/src/intel/vulkan/gen8_pipeline.c index

[Mesa-dev] [PATCH 2/7] genxml: Make 3DSTATE_SF more consistent between gen7 and gen8+

2016-08-06 Thread Jason Ekstrand
--- src/intel/genxml/gen7.xml| 7 +-- src/intel/genxml/gen75.xml | 7 +-- src/intel/vulkan/gen7_pipeline.c | 4 ++-- 3 files changed, 12 insertions(+), 6 deletions(-) diff --git a/src/intel/genxml/gen7.xml b/src/intel/genxml/gen7.xml index 1084093..5e82fab 100644 ---

Re: [Mesa-dev] cairo as state tracker

2016-08-06 Thread Marek Olšák
On Sat, Aug 6, 2016 at 5:03 PM, Enrico Weigelt, metux IT consult wrote: > On 06.08.2016 15:23, Marek Olšák wrote: > >> I'd recommend moving the cairo rendering code into Mesa and have Mesa >> implement and export some kind of a low-level cairo library. > > That would

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread
On Sat, Aug 6, 2016 at 5:52 PM, Jan Vesely wrote: > The situation I'm concerned about is as > follows: > 1.) mesa builds fine using existing build environment. > > 2.) I update LLVM. This update changes symbols (function parameter > changed type, a function was moved to

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Jan Vesely
On Sat, 2016-08-06 at 17:34 +0200, ⚛ wrote: > On Sat, Aug 6, 2016 at 4:32 PM, Jan Vesely > wrote: > > > > On Sat, 2016-08-06 at 13:00 +0200, ⚛ wrote: > > > > > > We could add a verifier to the build process that tests the > > > foo_dri.so > > > libraries (as well as all

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread
On Sat, Aug 6, 2016 at 4:32 PM, Jan Vesely wrote: > On Sat, 2016-08-06 at 13:00 +0200, ⚛ wrote: >> We could add a verifier to the build process that tests the >> foo_dri.so >> libraries (as well as all other libs subject to dlopen by Mesa) for >> undefined symbols: >> >> $

[Mesa-dev] [PATCH 1/3] gallium: add a pipe_context parameter to fence_finish

2016-08-06 Thread Marek Olšák
From: Marek Olšák required by glClientWaitSync (GL 4.5 Core spec) that can optionally flush the context --- src/gallium/docs/source/context.rst | 10 -- src/gallium/drivers/ddebug/dd_draw.c | 2 +- src/gallium/drivers/ddebug/dd_screen.c

[Mesa-dev] [PATCH 3/3] gallium/radeon: use unflushed fences for deferred flushes (v2)

2016-08-06 Thread Marek Olšák
From: Marek Olšák +23% Bioshock Infinite performance. v2: - use the new fence_finish interface - allow deferred fences with multiple contexts - clear the ctx pointer after a deferred flush --- src/gallium/drivers/radeon/r600_pipe_common.c | 44

[Mesa-dev] [PATCH 2/3] st/mesa: set the ctx parameter of fence_finish

2016-08-06 Thread Marek Olšák
From: Marek Olšák for deferred flushes --- src/mesa/state_tracker/st_cb_syncobj.c | 25 ++--- 1 file changed, 18 insertions(+), 7 deletions(-) diff --git a/src/mesa/state_tracker/st_cb_syncobj.c b/src/mesa/state_tracker/st_cb_syncobj.c index

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Jan Vesely
On Sat, 2016-08-06 at 17:20 +0200, ⚛ wrote: > On Sat, Aug 6, 2016 at 4:32 PM, Jan Vesely > wrote: > > > > sure I can set LD_BIND_NOW > > env var, but there are programs that have much stronger case for > > using > > lazy binding (like LO) that would be negatively

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Jan Vesely
On Sat, 2016-08-06 at 07:24 -0700, Jason Ekstrand wrote: > On Aug 6, 2016 4:00 AM, "⚛" <0xe2.0x9a.0...@gmail.com> wrote: > > > > > > On Sat, Aug 6, 2016 at 4:34 AM, Rob Clark > > wrote: > > > > > > On Fri, Aug 5, 2016 at 8:42 PM, Jan Ziak <0xe2.0x9a.0...@gmail.co > > > m>

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread
On Sat, Aug 6, 2016 at 4:32 PM, Jan Vesely wrote: > sure I can set LD_BIND_NOW > env var, but there are programs that have much stronger case for using > lazy binding (like LO) that would be negatively affected. What is "LO"?

Re: [Mesa-dev] cairo as state tracker

2016-08-06 Thread Enrico Weigelt, metux IT consult
On 06.08.2016 15:23, Marek Olšák wrote: > I'd recommend moving the cairo rendering code into Mesa and have Mesa > implement and export some kind of a low-level cairo library. That would essentially create a circular dependency - and cairo's surface backend API is also private. > Please note

Re: [Mesa-dev] cairo as state tracker

2016-08-06 Thread Enrico Weigelt, metux IT consult
On 06.08.2016 13:16, Nicolai Hähnle wrote: > Well, in your first mail it sounded like you wanted to stabilize the > Gallium API itself. Not actually stabilizing, but making it semi-public. (at dist level in an separate package). Of course, external state trackers need to built to the right

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Jan Vesely
On Sat, 2016-08-06 at 13:00 +0200, ⚛ wrote: > On Sat, Aug 6, 2016 at 4:34 AM, Rob Clark > wrote: > > > > On Fri, Aug 5, 2016 at 8:42 PM, Jan Ziak <0xe2.0x9a.0...@gmail.com> > > wrote: > > > > > > Mesa source code prior to this patch uses both RTLD_NOW and > > > RTLD_LAZY. >

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Jason Ekstrand
On Aug 6, 2016 4:00 AM, "⚛" <0xe2.0x9a.0...@gmail.com> wrote: > > On Sat, Aug 6, 2016 at 4:34 AM, Rob Clark wrote: > > On Fri, Aug 5, 2016 at 8:42 PM, Jan Ziak <0xe2.0x9a.0...@gmail.com> wrote: > >> Mesa source code prior to this patch uses both RTLD_NOW and RTLD_LAZY. > >>

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Rob Clark
On Sat, Aug 6, 2016 at 4:28 AM, Enrico Weigelt, metux IT consult wrote: > On 06.08.2016 04:05, ⚛ wrote: > >> Question 2: Exists there a reason for _not_ linking radeonsi_dri.so, >> swrastg_dri.so, etc, directly to Mesa's libGL.so? The Gallium >> *_dri.so libraries are the

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Rob Clark
On Sat, Aug 6, 2016 at 3:01 AM, Eric Anholt wrote: > Rob Clark writes: > >> On Fri, Aug 5, 2016 at 8:42 PM, Jan Ziak <0xe2.0x9a.0...@gmail.com> wrote: >>> Mesa source code prior to this patch uses both RTLD_NOW and RTLD_LAZY. >>> This patch removes all

Re: [Mesa-dev] [PATCH] gallium/radeon: implement ARB_clear_texture (v2)

2016-08-06 Thread Roland Scheidegger
Yes, these commands are modelled after d3d10 clears (the only d3d10 clear methods are clear_rt/clear_dsv), so gallium at some point only had the "ordinary" clear (useful for d3d9 and gl, but doesn't cover everything neither). It is unfortunate all apis handle clear differently. Making them

Re: [Mesa-dev] cairo as state tracker

2016-08-06 Thread Marek Olšák
On Fri, Aug 5, 2016 at 6:48 AM, Enrico Weigelt, metux IT consult wrote: > Hi folks, > > > I'd like to get the cairo's gallium backend running again (hasn't been > maintained for a long time and doesn't fit at all to recent mesa). > > The main problem I've got here is that

Re: [Mesa-dev] [PATCH 1/3] freedreno/a2xx: fix comparison out of range warnings

2016-08-06 Thread Francesco Ansanelli
Hello, I did more tests... and I didn't experience any error. I attach last version of the script. Cheers, Francesco 2016-08-04 19:50 GMT+02:00 Francesco Ansanelli : > Hi Eric, > > after your question: > > [snip] > I expect, with enums having varying size, that this cast

Re: [Mesa-dev] [PATCH] gallium/radeon: implement ARB_clear_texture (v2)

2016-08-06 Thread Nicolai Hähnle
Ah, somehow I missed the earlier email from Ilia, now I see the mention of the d3d10 state tracker. I still think this is a surprising hack. Maybe render condition can be made optional like for blits? Nicolai On 06.08.2016 13:14, Nicolai Hähnle wrote: On 05.08.2016 14:20, Marek Olšák wrote:

Re: [Mesa-dev] cairo as state tracker

2016-08-06 Thread Nicolai Hähnle
On 06.08.2016 10:49, Enrico Weigelt, metux IT consult wrote: On 05.08.2016 17:36, Ilia Mirkin wrote: The idea is that gallium is an unstable unversioned API, and if you want something else, then you make a state tracker which exposes a stable API. That's exactly what I intend: cario. Well,

Re: [Mesa-dev] [PATCH] gallium/radeon: implement ARB_clear_texture (v2)

2016-08-06 Thread Nicolai Hähnle
On 05.08.2016 14:20, Marek Olšák wrote: On Aug 5, 2016 10:54 AM, "Nicolai Hähnle" > wrote: On 04.08.2016 21:42, Marek Olšák wrote: From: Marek Olšák > Some ideas copied from Jakob Sinclair's

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread
On Sat, Aug 6, 2016 at 4:34 AM, Rob Clark wrote: > On Fri, Aug 5, 2016 at 8:42 PM, Jan Ziak <0xe2.0x9a.0...@gmail.com> wrote: >> Mesa source code prior to this patch uses both RTLD_NOW and RTLD_LAZY. >> This patch removes all RTLD_NOW in favor of RTLD_LAZY. >> >> In

Re: [Mesa-dev] cairo as state tracker

2016-08-06 Thread Enrico Weigelt, metux IT consult
On 05.08.2016 17:36, Ilia Mirkin wrote: > The idea is that gallium is an unstable unversioned API, and if you want > something else, then you make a state tracker which exposes a stable > API. That's exactly what I intend: cario. --mtx ___ mesa-dev

[Mesa-dev] [Bug 93103] llvm symbols leak through, cause trouble with software rendering in llvm-linked software

2016-08-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=93103 Vedran Miletić changed: What|Removed |Added CC||ved...@miletic.net

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Enrico Weigelt, metux IT consult
On 06.08.2016 04:05, ⚛ wrote: > Question 2: Exists there a reason for _not_ linking radeonsi_dri.so, > swrastg_dri.so, etc, directly to Mesa's libGL.so? The Gallium > *_dri.so libraries are the same inode on the filesystem. Sure about that ? nekrad@orion:~/MESA/lib/dri$ ls -lai total 508348

[Mesa-dev] [PATCH 3/3] nir: Define system values for vc4's blending-lowering arguments.

2016-08-06 Thread Eric Anholt
In the GLSL-to-NIR conversion of VC4, I had a bit of trouble with what I was calling the "state uniforms" that I was putting into the NIR fighting with its other lowering passes. Instead of using magic uniform base numbers in the backend, follow the lead of load_user_clip_plane and just define

[Mesa-dev] [PATCH 1/3] nir: Move the undef of nir_intrinsics.h macros to the .h.

2016-08-06 Thread Eric Anholt
I wanted to include this from nir_builder as well, so it also needed the undefs. --- src/compiler/nir/nir.h| 3 --- src/compiler/nir/nir_intrinsics.h | 3 +++ 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h index

[Mesa-dev] [PATCH 2/3] nir: Add nir_builder support for individual system value loads.

2016-08-06 Thread Eric Anholt
The previous nir_load_system_value(b, nir_intrinsic_load_whatever), 0) was rather verbose, when system values should be easy to generate. The index is left out because only one system value had an index included in it. --- src/compiler/nir/nir_builder.h | 15 +++

Re: [Mesa-dev] [PATCH] src: replace RTLD_NOW with RTLD_LAZY

2016-08-06 Thread Eric Anholt
Rob Clark writes: > On Fri, Aug 5, 2016 at 8:42 PM, Jan Ziak <0xe2.0x9a.0...@gmail.com> wrote: >> Mesa source code prior to this patch uses both RTLD_NOW and RTLD_LAZY. >> This patch removes all RTLD_NOW in favor of RTLD_LAZY. >> >> In comparison to early binding, lazy

Re: [Mesa-dev] [PATCH] anv: gen7/75: enable multisampling (v3)

2016-08-06 Thread Jason Ekstrand
On Aug 5, 2016 11:37 PM, "Jason Ekstrand" wrote: > > I am going to try and give this a proper look on Monday. Given how sketchy the multisampling support is on gen7 at the moment, I am a but concerned that it doesn't fix more tests. What's left to fix for MSAA? Also, I

[Mesa-dev] [PATCH 2/3] util: Use win32 intrinsics for util_last_bit if present.

2016-08-06 Thread Mathias . Froehlich
From: Mathias Fröhlich v2: Split into two patches. Signed-off-by: Mathias Fröhlich --- src/util/bitscan.h | 12 1 file changed, 12 insertions(+) diff --git a/src/util/bitscan.h b/src/util/bitscan.h index 0743fe7..a5bb34e 100644

[Mesa-dev] [PATCH 1/3] util: Move _mesa_fsl/util_last_bit into util/bitscan.h v2

2016-08-06 Thread Mathias . Froehlich
From: Mathias Fröhlich Following the split up patch as a series and on top the include fix to u_bitscan.h. Please review! Thanks! Mathias As requested with the initial creation of util/bitscan.h now move other bitscan related functions into util. v2: Split into

[Mesa-dev] [PATCH 3/3] gallium: Add c99_compat.h to u_bitcast.h

2016-08-06 Thread Mathias . Froehlich
From: Mathias Fröhlich We need this for 'inline'. Signed-off-by: Mathias Fröhlich --- src/gallium/auxiliary/util/u_bitcast.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/auxiliary/util/u_bitcast.h

Re: [Mesa-dev] [PATCH] anv: gen7/75: enable multisampling (v3)

2016-08-06 Thread Jason Ekstrand
I am going to try and give this a proper look on Monday. Given how sketchy the multisampling support is on gen7 at the moment, I am a but concerned that it doesn't fix more tests. What's left to fix for MSAA? On Aug 5, 2016 4:55 PM, "Lionel Landwerlin" wrote: Fixes the