When hardware-generated binding tables are enabled, use the hw-generated
binding table format when uploading binding table state.
Normally, the CS will will just consume the binding table pointer commands
as pipelined state. When the RS is enabled however, the RS flushes whatever
edited surface st
Check first if the hardware and kernel supports resource streamer. If this
is allowed, tell the kernel to enable the resource streamer enable bit on
MI_BATCHBUFFER_START by specifying I915_EXEC_RESOURCE_STREAMER
execbuffer flags.
v2: - Use new I915_PARAM_HAS_RESOURCE_STREAMER ioctl to check if ker
This patch implements the binding table enable command which is also
used to allocate a binding table pool where where hardware-generated
binding table entries are flushed into. Each binding table offset in
the binding table pool is unique per each shader stage that are
enabled within a batch.
Als
Switch off hardware-generated binding tables and gather push
constants in the blorp. Blorp requires only a minimal set of
simple constants. There is no need for the extra complexity
to program a gather table entry into the pipeline.
Cc: kenn...@whitecape.org
Signed-off-by: Abdiel Janulgue
---
sr
On Thu, 2015-07-02 at 10:11 -0700, Jason Ekstrand wrote:
> On Wed, Jul 1, 2015 at 11:44 PM, Iago Toral wrote:
> > On Tue, 2015-06-30 at 09:30 -0700, Jason Ekstrand wrote:
> >> On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev
> >> wrote:
> >> > From: Iago Toral Quiroga
> >> >
> >> > The same
After c61bc6e ("util: port _mesa_strto[df] to C"), "make check"
fails due to a missing _mesa_locale_init. Fixup this oversight,
by moving the stand-alone compiler initializer inside
initialize_context_to_defaults().
Signed-off-by: Erik Faye-Lund
---
Here's what the latter suggestion would look li
On Wed, Jul 01, 2015 at 03:03:31PM -0700, Kenneth Graunke wrote:
> Jason plumbed this through a while back in the FS backend, but
> apparently we were just passing NULL in the vec4 backend.
>
> This patch passes brw in as intended.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri
On Thu, 2015-07-02 at 09:31 +0200, Iago Toral wrote:
> On Tue, 2015-06-30 at 11:32 -0700, Jason Ekstrand wrote:
> > I'm not sure what I think about adding an is_scalar flag vs. having
> > _scalar and _vec4 versions of each function. My feeling is that once
> > we tweak assign_var_locations as I me
On Wed, Jul 01, 2015 at 03:03:32PM -0700, Kenneth Graunke wrote:
> This patch makes us only issue the performance warning about register
> spilling if we actually spilled registers. We also use scratch space
> for indirect addressing and the like.
>
> This is basically commit c51163b0cf7aff0375b1
On Wed, Jul 01, 2015 at 03:03:33PM -0700, Kenneth Graunke wrote:
> This is more consistent with how we do it in the FS backend, and reduces
> a tiny bit of duplication. It'll also allow for a bit more tidying.
And it also makes it clearer that code generation doesn't have anything to do
with the
On Wed, Jul 01, 2015 at 03:03:34PM -0700, Kenneth Graunke wrote:
> Nothing outside of vec4_visitor uses it, so we may as well keep it
> internal.
>
> Commit db9c915abcc5ad78d2d11d0e732f04cc94631350 for the vec4 backend.
>
> (The empty class will be going away soon.)
>
> Signed-off-by: Kenneth Gr
On Wed, Jul 01, 2015 at 03:03:35PM -0700, Kenneth Graunke wrote:
> At this point, the brw_vs_compile structure only contains the key and
> gl_vertex_program pointer. We may as well pass and store them directly;
> it's simpler and more convenient (key-> instead of vs_compile->key...).
>
> Signed-o
On Fri, Jul 03, 2015 at 11:29:33AM +0300, Pohjolainen, Topi wrote:
> On Wed, Jul 01, 2015 at 03:03:35PM -0700, Kenneth Graunke wrote:
> > At this point, the brw_vs_compile structure only contains the key and
> > gl_vertex_program pointer. We may as well pass and store them directly;
> > it's simpl
On Wed, Jul 01, 2015 at 03:03:36PM -0700, Kenneth Graunke wrote:
> After tearing it out another level or two, and just passing the key and
> vp directly, we can finally remove this struct. It also eliminates a
> pointless memcpy() of the key.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/
Same check is made for glBindFragDataLocationIndexed but it was missing
when using layout qualifiers.
Fixes following Piglit test:
arb_blend_func_extended-output-location
Signed-off-by: Tapani Pälli
---
src/glsl/linker.cpp | 19 +++
1 file changed, 19 insertions(+)
diff
Change function to get all gl_constants for inspection, this is used
by follow-up patch.
Signed-off-by: Tapani Pälli
---
src/glsl/linker.cpp | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index 71a45e8..aae0c0d 1006
On Fri, Jul 03, 2015 at 10:00:31AM +0300, Abdiel Janulgue wrote:
> +void
> +gen7_enable_hw_binding_tables(struct brw_context *brw)
> +{
> + if (!brw->use_resource_streamer)
> + return;
> +
> + if (!brw->hw_bt_pool.bo) {
> + /* We use a single re-usable buffer object for the lifetime o
Samuel Iglesias Gonsálvez writes:
> On 29/06/15 09:11, Jordan Justen wrote:
>> On 2015-06-24 07:36:24, Iago Toral wrote:
>>> On Wed, 2015-06-24 at 15:43 +0300, Francisco Jerez wrote:
AFAICT the reason why this (and many of the other changes in GLSL
optimization passes) is needed is beca
On Fri, 2015-07-03 at 13:23 +0300, Francisco Jerez wrote:
> Samuel Iglesias Gonsálvez writes:
>
> > On 29/06/15 09:11, Jordan Justen wrote:
> >> On 2015-06-24 07:36:24, Iago Toral wrote:
> >>> On Wed, 2015-06-24 at 15:43 +0300, Francisco Jerez wrote:
> AFAICT the reason why this (and many of
Neil Roberts writes:
> There was a comment saying that in SIMD16 mode the pixel interpolator
> returns coords interleaved 8 channels at a time and that this requires
> extra work to support. However, this interleaved format is exactly
> what the PLN instruction requires so I don't think anything
Cc: Martin Peres
Cc: Tapani Pälli
---
src/glsl/linker.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index 71a45e8..d8f1689 100644
--- a/src/glsl/linker.cpp
+++ b/src/glsl/linker.cpp
@@ -462,6 +462,10 @@ parse_program_resource_name(const GLc
This change removes multiple functions designed to validate an array
subscript and replaces them with a call to a single function.
The change also means that validation is now only done once and the index
is retrived at the same time, as a result the getUniformLocation code can
be merged saving an
On 03/07/15 13:55, Timothy Arceri wrote:
This change removes multiple functions designed to validate an array
subscript and replaces them with a call to a single function.
The change also means that validation is now only done once and the index
is retrived at the same time, as a result the ge
On Gen9+ there is a new bit in 3DSTATE_PS_EXTRA that must be set if
the shader sends a message to the pixel interpolator. This fixes the
interpolateAt* tests on SKL, apart from interpolateatsample-nonconst
but that is not implemented anywhere so it's not a regression.
---
src/mesa/drivers/dri/i965
On Wed, Jul 1, 2015 at 4:27 PM, Eric Anholt wrote:
> Rob Clark writes:
>
>> From: Ilia Mirkin
>>
>> We need to distinguish a shader that has separate writes to each MRT
>> from one which is supposed to write the data from MRT 0 to all the MRTs.
>> In TGSI this is done with a property. NIR doesn'
On Fri, 2015-07-03 at 15:08 +0300, Martin Peres wrote:
>
> On 03/07/15 13:55, Timothy Arceri wrote:
> > This change removes multiple functions designed to validate an array
> > subscript and replaces them with a call to a single function.
> >
> > The change also means that validation is now only
On 02/07/15 18:25, Dylan Baker wrote:
> One of the plugins I use with vim "helpfully" added an underscore to the
> front of mode for kicks.
>
> Obviously this isn't a feature used very often because it's been broken
> since d986cb7c70db (since May 20th), and no one has noticed.
>
Iirc these have
Hi Boyan,
Thank you for doing this ! A few suggestions which you might be interesting:
Considering that the backend has handled more than dri2 perhaps we can
do a s/dri2/dri/ :-) That obviously is independent of your work.
On 01/07/15 16:31, Boyan Ding wrote:
> Signed-off-by: Boyan Ding
> ---
>
On 02/07/15 16:32, Jon TURNEY wrote:
> On 02/07/2015 13:42, Emil Velikov wrote:
>> On 27 June 2015 at 12:21, Jon TURNEY wrote:
>>> This smells to me like there's something not quite right about the
>>> partitioning of stuff between libmesa and libglapi, but I don't have any
>>> insight into what ex
On Friday, July 03, 2015 10:50:52 AM Pohjolainen, Topi wrote:
> On Wed, Jul 01, 2015 at 03:03:31PM -0700, Kenneth Graunke wrote:
> > Jason plumbed this through a while back in the FS backend, but
> > apparently we were just passing NULL in the vec4 backend.
> >
> > This patch passes brw in as inte
On Friday, July 03, 2015 11:11:45 AM Pohjolainen, Topi wrote:
> On Wed, Jul 01, 2015 at 03:03:32PM -0700, Kenneth Graunke wrote:
> > This patch makes us only issue the performance warning about register
> > spilling if we actually spilled registers. We also use scratch space
> > for indirect addre
Hi Jason,
On mar, 2015-06-30 at 14:18 -0700, Jason Ekstrand wrote:
> On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev wrote:
> > From: Antia Puentes
> >
> > For operations that have a predefined operand size > 0, defined in
> > glsl/nir/nir_opcodes.c, NIR returns a swizzle containing zeros in
On Fri, Jul 3, 2015 at 12:46 AM, Erik Faye-Lund wrote:
> After c61bc6e ("util: port _mesa_strto[df] to C"), "make check"
> fails due to a missing _mesa_locale_init. Fixup this oversight,
> by moving the stand-alone compiler initializer inside
> initialize_context_to_defaults().
>
> Signed-off-by:
On Fri, Jul 03, 2015 at 01:15:21PM +0100, Neil Roberts wrote:
> On Gen9+ there is a new bit in 3DSTATE_PS_EXTRA that must be set if
> the shader sends a message to the pixel interpolator. This fixes the
> interpolateAt* tests on SKL, apart from interpolateatsample-nonconst
> but that is not impleme
On Friday, July 03, 2015 01:15:21 PM Neil Roberts wrote:
> On Gen9+ there is a new bit in 3DSTATE_PS_EXTRA that must be set if
> the shader sends a message to the pixel interpolator. This fixes the
> interpolateAt* tests on SKL, apart from interpolateatsample-nonconst
> but that is not implemented
I asked about dumping them and was told that they would leave mesa only
when x itself died. If be more than happy to see them go if they're not
useful
On Jul 3, 2015 06:28, "Emil Velikov" wrote:
> On 02/07/15 18:25, Dylan Baker wrote:
> > One of the plugins I use with vim "helpfully" added an und
https://bugs.freedesktop.org/show_bug.cgi?id=91222
Bug ID: 91222
Summary: lp_test_format regression on CentOS 7
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Keywords: bis
Cc: Tapani Pälli
---
src/mesa/main/uniform_query.cpp | 75 -
src/mesa/main/uniforms.h| 4 ---
2 files changed, 79 deletions(-)
diff --git a/src/mesa/main/uniform_query.cpp b/src/mesa/main/uniform_query.cpp
index cab5083..680ba16 100644
--- a/src/m
This change removes multiple functions designed to validate an array
subscript and replaces them with a call to a single function.
The change also means that validation is now only done once and the index
is retrived at the same time, as a result the getUniformLocation code can
be simplified savin
Cc: Tapani Pälli
---
src/mesa/main/shader_query.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/mesa/main/shader_query.cpp b/src/mesa/main/shader_query.cpp
index a6246a3..0473c2e 100644
--- a/src/mesa/main/shader_query.cpp
+++ b/src/mesa/main/shader_query.cpp
@@ -787,8 +787,6 @@ prog
Cc: Tapani Pälli
---
src/glsl/linker.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index 71a45e8..d8f1689 100644
--- a/src/glsl/linker.cpp
+++ b/src/glsl/linker.cpp
@@ -462,6 +462,10 @@ parse_program_resource_name(const GLchar *name,
if
From: Roland Scheidegger
The expansion should always be to the same width as the input arguments
no matter what, since these functions should work with any bit width of
the arguments (the sext is a no-op on any sane simd architecture).
Thus, fix the caller expecting differently.
This fixes https
Reviewed-by: Ilia Mirkin
On Sat, Jun 20, 2015 at 8:33 AM, Timothy Arceri wrote:
> ---
> src/glsl/ast_to_hir.cpp | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
> index f1c3e4a..0d3cbac 100644
> --- a/src/glsl/ast_to_h
Hi Emil,
On 07/03/2015 10:36 PM, Emil Velikov wrote:
> Hi Boyan,
>
> Thank you for doing this ! A few suggestions which you might be interesting:
>
> Considering that the backend has handled more than dri2 perhaps we can
> do a s/dri2/dri/ :-) That obviously is independent of your work.
>
> On 01
https://bugs.freedesktop.org/show_bug.cgi?id=90162
Tapani Pälli changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
On Fri, Jul 3, 2015 at 6:05 PM, wrote:
> From: Roland Scheidegger
>
> The expansion should always be to the same width as the input arguments
> no matter what, since these functions should work with any bit width of
> the arguments (the sext is a no-op on any sane simd architecture).
> Thus, fix
This fixes: ES31-CTS.program_interface_query.uniform-types
Cc: Tapani Pälli
---
Note: This only fixes the remaining CTS subtests for uniform-types the
other fixes are part of my clean-up series:
http://lists.freedesktop.org/archives/mesa-dev/2015-July/088122.html
src/glsl/linker.cpp | 16 +++
I didn't notice earlier but it seems this series also fixes some CTS subtests
for example some test are fixed in ES31-CTS.program_interface_query.uniform
-types.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman
48 matches
Mail list logo
