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 multisampling >> > mode to DX which is what we wa

[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 --- a/src/mesa/state_tracker/st_atom.

[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 b/src/mesa/state_tracker/st_atom.c inde

[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 - src/mesa/state_tracker/s

[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 +++ b/src/mesa/state_tracke

[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 ++ src/mesa/state_tracker/st_cb_feedback.c | 6 ++- src

[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 a/src/mesa/state_tracker/st_cb_bufferobjects.c b/src/mesa/

[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 b/src/mesa/state_tracker/st_context.c index 1f

[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 insertions(+), 15 deletions(-) dif

[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 oth

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 multisampling API mode. T

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 chang

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 is

[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 b/src/mesa/state_tracker/st_cb_bufferobjects.c index 6d54326..2f56a

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 been meaning to add the same thing but haven't had time > to

[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 --- a/src/mesa/main/ff_frag

[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 issue: https://github.c

[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 kind of thing. I'd s

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 fd fence stuff in a bit.. mind squashing

[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 a/src/gallium/winsys/ra

[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 src/gallium/winsys/amdgpu/drm/amdgpu_

[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 91e3cb2..e4002f9 100644 --- a

[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 --- a/src/gallium/drivers/radeon/r6

[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 --- a/src/gallium/winsys/amdgpu/drm/

[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 b/src/gallium/drivers/radeon/r600

[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. --- src/mesa/state_tracker/st_cb_bufferobjects.c | 2 +- 1 fil

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 static libOSMesa and x

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 symbols (function parameter > >

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 >> and they can start buil

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 dist level in > an separate

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 version of the script. > > C

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 a/src/in

[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 betwe

[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 +++ b/src/int

[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 +++ b/src/intel/vulkan/gen8_pi

[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 @@ isl_fo

[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 betwe

[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 6

[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 --- a/src/i

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 essentially create a circular

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 header, or devirtualization

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 other libs subject to dlo

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: >> >> $ LD_PRELOAD=libGL.so ldd

[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 | 4 +++- src/

[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 ++- 1 file changed

[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 1fa1403..123925a 100644 --- a/sr

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 affected. > > What is "LO"? lib

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> > wrote: > > > > > >

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"? ___ mesa-dev mailing list m

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 that

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 galli

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. > > > This patch remove

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. > >> This patch removes all

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 same inode on the filesys

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 RTLD_NOW in favor of RTLD_LAZY. >>> >>> In c

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 honoring

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 gallium API is mesa-inter

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 will > not always work.

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, i

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" mailto:nhaeh...@gmail.com>> wrote: On 04.08.2016 21:42, Marek Olšák wrote: From: Marek Olšák mailto:marek.ol...@amd.com>> Some ideas copied from Jakob Sinclair's implementation, but the color clearing is comp

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 comparison to early binding, la

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 m

[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 -- You are receivin

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 6750

[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 sys

[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 379a5f8e02

[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 +++ src/c

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 binding reduces CPU instruc