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
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
---
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
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 -
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
+++
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 ++
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
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
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
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
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
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
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
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
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
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
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
---
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
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
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
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
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
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
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
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
---
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
---
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
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.
---
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
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
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
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
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
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
---
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
---
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
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
---
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
+++
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
+++
---
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 @@
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
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
---
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
---
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
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
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
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:
>>
>> $
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
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
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
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
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>
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"?
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
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
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.
>
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.
> >>
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
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
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
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
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
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:
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,
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
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
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
https://bugs.freedesktop.org/show_bug.cgi?id=93103
Vedran Miletić changed:
What|Removed |Added
CC||ved...@miletic.net
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
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
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
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 +++
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
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
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
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
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
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
78 matches
Mail list logo