On 04/11/15 23:49, Matt Turner wrote:
> On Wed, Nov 4, 2015 at 1:01 PM, Alejandro Piñeiro
> wrote:
>> On 04/11/15 20:13, Matt Turner wrote:
>>> On Fri, Oct 23, 2015 at 7:17 AM, Alejandro Piñeiro
>>> wrote:
Equivalent to commit 4eebeb but with sel operations. In this case
>>> That commit
Found by coverity: CID #1337953
Signed-off-by: Julien Isorce
---
src/gallium/state_trackers/va/surface.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/va/surface.c
b/src/gallium/state_trackers/va/surface.c
index 59815aa..3db21c3 1006
Before it was only possible to convert a NV12 surface to
RGBA or BGRA. This patch uses the same post processing
function, "handleVAProcPipelineParameterBufferType", but
add definitions for RGBX and BGRX.
This patch also makes vlVaQuerySurfaceAttributes more generic.
Signed-off-by: Julien Isorce
A few fixes and small improvement from recent changes in st/va:
* Indentation was wrong for some functions.
* Coverity reported a memory leak. (thx to Ilia for pointing this)
* More color formats convertion for Video Post Processing
Julien Isorce (4):
st/va: indent vlVaQuerySurfaceAttributes and
Some lines were using 4 indentation spaces instead of 3.
Signed-off-by: Julien Isorce
---
src/gallium/state_trackers/va/surface.c | 566
1 file changed, 283 insertions(+), 283 deletions(-)
diff --git a/src/gallium/state_trackers/va/surface.c
b/src/gallium/state
Some lines were using 4 indentation spaces instead of 3.
The switch in vlVaAcquireBufferHandle actually had wrong brackets
surrounding case+default.
Signed-off-by: Julien Isorce
---
src/gallium/state_trackers/va/buffer.c | 23 +
src/gallium/state_trackers/va/config.c | 12 ++---
src/g
Reviewed-by: Marek Olšák
Marek
On Thu, Nov 5, 2015 at 6:34 AM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> src/mesa/state_tracker/st_cb_copyimage.c | 8
> 1 file changed, 8 insertions(+)
>
> diff --git a/src/mesa/state_tracker/st_cb_copyimage.c
> b/src/mesa/state_tracker
On Wed, 2015-11-04 at 15:33 -0800, Kristian Høgsberg Kristensen wrote:
> We always pass in shader->ir and we already pass in the shader, so just
> drop the exec_list. Most passes either take just a exec_list or a
> shader, so this seems more consistent.
>
> Signed-off-by: Kristian Høgsberg Kristen
This series adds support for compile time constants and also adds
subroutine index qualifier support which was missing for
ARB_explicit_uniform_location
it doesn't add the missing subroutine location qualifier I'll add that in
a follow-up.
Piglit tests have been reviewed and pushed to master, the
From: Timothy Arceri
---
src/glsl/glsl_parser_extras.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/glsl/glsl_parser_extras.h b/src/glsl/glsl_parser_extras.h
index 684b917..1d8c1b8 100644
--- a/src/glsl/glsl_parser_extras.h
+++ b/src/glsl/glsl_parser_extras.h
@@ -209,6 +209,11 @@
From: Timothy Arceri
Set to dummy_false until the remaining features are added.
---
src/glsl/glcpp/glcpp-parse.y| 1 +
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 2 ++
src/mesa/main/extensions.c | 1 +
4 files changed, 5 insertions(+)
diff --git a/src/glsl
From: Timothy Arceri
---
docs/GL3.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 7abdcd8..416b734 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -179,7 +179,7 @@ GL 4.4, GLSL 4.40:
GL_ARB_buffer_storage
From: Timothy Arceri
In this patch we introduce a new ast type for holding the new
compile-time constant expressions. The main reason for this is that
we can no longer do merging of layout qualifiers before they have been
converted into GLSL IR.
The remainder of the patch replaces all the old in
From: Timothy Arceri
ARB_explicit_uniform_location allows the index for subroutine functions
to be explicitly set in the shader.
This patch reduces the restriction on the index qualifier in
validate_layout_qualifiers() to allow it to be applied to subroutines
and adds the new subroutine qualifie
From: Timothy Arceri
This is in preperation for compile-time constant support.
Also fix up the locations for some of the extension checking
error messages in the parser. We now correctly give the location
of the layout qualifier identifier rather than the integer constant.
The validation is mov
Ben Widawsky writes:
> It seems reasonable to me, but since you touch the same code I touch
> in fast clears, and since I always screw up rebases, any chance I
> could persuade you to wait until I land fast clears?
Sure, I don't mind waiting. I did have a go at rebasing the patch on top
of your
From: Iago Toral Quiroga
Notice that the spec requires that a default precision has been set for every
type used by a shader that can use a precision qualifier and does not have a
predefined precision, however, at the moment, Mesa only checks this for floats
in the fragment shader. This is probab
From: Iago Toral Quiroga
The GLSL ES spec specifies default precision qualifiers for certain types,
so populate the symbol table with these.
Notice that the desktop GLSL spec also indicates defaults for some types
but this is not really useful since precision qualifiers are completely
ignored in
Hi;
I rebased Iago's patches and fixed some issues that I saw. On top of it
is 2 patches that fix precision related ES 3.1 CTS errors.
Any comments appreciated. What should be included in validation seems a
bit vague in the specs and I did not include anything else than precision
since this is wh
From: Iago Toral Quiroga
These have scoping rules that match the ones defined for other things such
as variables, so we want them in the symbol table.
---
src/glsl/glsl_symbol_table.cpp | 24
src/glsl/glsl_symbol_table.h | 2 ++
2 files changed, 26 insertions(+)
diff
From: Iago Toral Quiroga
We will need this later on when we implement proper support for
precision qualifiers in the drivers and also to do link time checks for
uniforms as indicated by the spec.
This patch also adds compile-time checks for variables without precision
information (currently, Mes
Fixes following failing ES3.1 CTS tests:
ES31-CTS.sepshaderobjs.InterfacePrecisionMatchingFloat
ES31-CTS.sepshaderobjs.InterfacePrecisionMatchingInt
ES31-CTS.sepshaderobjs.InterfacePrecisionMatchingUInt
Signed-off-by: Tapani Pälli
---
src/mesa/main/pipelineobj.c| 15
s
This information will be used by cross stage validation of varyings
for pipeline objects.
Signed-off-by: Tapani Pälli
---
src/glsl/lower_packed_varyings.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/glsl/lower_packed_varyings.cpp
b/src/glsl/lower_packed_varyings.cpp
index 5d66ca9.
From: Iago Toral Quiroga
We will need this to build later patches
---
src/glsl/ast_to_hir.cpp | 71 -
1 file changed, 35 insertions(+), 36 deletions(-)
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index d20be0b..b6d662b 100644
--
I believe you meant "do not lose ..." :)
On 05/11/15 13:33, Tapani Pälli wrote:
This information will be used by cross stage validation of varyings
for pipeline objects.
Signed-off-by: Tapani Pälli
---
src/glsl/lower_packed_varyings.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/sr
On 11/05/2015 01:33 PM, Tapani Pälli wrote:
From: Iago Toral Quiroga
The GLSL ES spec specifies default precision qualifiers for certain types,
so populate the symbol table with these.
Notice that the desktop GLSL spec also indicates defaults for some types
but this is not really useful sinc
From: Iago Toral Quiroga
We will need this later on when we implement proper support for
precision qualifiers in the drivers and also to do link time checks for
uniforms as indicated by the spec.
This patch also adds compile-time checks for variables without precision
information (currently, Mes
From: Iago Toral Quiroga
The GLSL ES spec specifies default precision qualifiers for certain types,
so populate the symbol table with these.
Notice that the desktop GLSL spec also indicates defaults for some types
but this is not really useful since precision qualifiers are completely
ignored in
On Tue, 2015-11-03 at 09:19 -0800, Mark Janes wrote:
> Francisco Jerez writes:
>
> > Iago Toral writes:
> >
> >> On Tue, 2015-11-03 at 15:28 +0200, Francisco Jerez wrote:
> >>> Iago Toral writes:
> >>>
> >>> > On Fri, 2015-10-30 at 16:19 +0200, Francisco Jerez wrote:
> >>> >> Iago Toral Quirog
Iago Toral writes:
> On Tue, 2015-11-03 at 09:19 -0800, Mark Janes wrote:
>> Francisco Jerez writes:
>>
>> > Iago Toral writes:
>> >
>> >> On Tue, 2015-11-03 at 15:28 +0200, Francisco Jerez wrote:
>> >>> Iago Toral writes:
>> >>>
>> >>> > On Fri, 2015-10-30 at 16:19 +0200, Francisco Jerez wr
On 17 October 2015 at 10:51, Timothy Arceri wrote:
> This also removes the validation from the parser as it is not required
> and once arb_enhanced_layouts comes along we wont be able to do validation
> on the stream qualifier in the parser anyway as it adds constant expression
> support to the st
Hi All,
This series implements using double immediates in the nouveau codegen code.
This turns the following (nvc0) code:
1: mov u32 $r2 0x (8)
2: mov u32 $r3 0x3fe0 (8)
3: add f64 $r0d $r0d $r2d (8)
Into:
1: add f64 $r0d $r0d 0.50 (8)
This has been t
Add support for encoding double immediates (up to 20 bits of precision)
into the generated gm107 machine-code.
Signed-off-by: Hans de Goede
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/n
Add support for encoding double immediates (up to 20 bits of precision)
into the generated nvc0 machine-code.
Signed-off-by: Hans de Goede
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp | 8
1 file changed, 8 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen
Teach insnCanLoad about double immediates, together with the
"Add support for merge-s to the ConstantFolding pass"
This turns the following (nvc0) code:
1: mov u32 $r2 0x (8)
2: mov u32 $r3 0x3fe0 (8)
3: add f64 $r0d $r0d $r2d (8)
Into:
1: add f64 $r0d $r0d 0.50 (8)
Signe
This allows later passes like LoadPropagation to properly deal with 64
bit immediates.
If the new 64 bit load this introduces does not get optimized away then
split64BitOpPostRA() will split this into 2 instructions again.
Signed-off-by: Hans de Goede
---
src/gallium/drivers/nouveau/codegen/nv5
Now that we support 64 bit immediates in insnCanLoad, we need to swap
64 bit immediate sources too for optimal effect.
Signed-off-by: Hans de Goede
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/gal
Jordan Justen writes:
> For these nir intrinsics, we emit the same code as
> nir_intrinsic_memory_barrier:
>
> * nir_intrinsic_memory_barrier_atomic_counter
> * nir_intrinsic_memory_barrier_buffer
> * nir_intrinsic_memory_barrier_image
>
> We treat these nir intrinsics as no-ops:
>
> * nir_in
Jordan Justen writes:
> When these functions are called in GLSL code, we create an intrinsic
> function call:
>
> * groupMemoryBarrier => __intrinsic_group_memory_barrier
> * memoryBarrierAtomicCounter => __intrinsic_memory_barrier_atomic_counter
> * memoryBarrierBuffer => __intrinsic_memory_b
Matt Turner writes:
> On Tue, Nov 3, 2015 at 5:48 AM, Francisco Jerez wrote:
>> Matt Turner writes:
>>
>>> Generated by
>>>
>>>sed -i -e 's/\.bits\././g' *.c *.h *.cpp
>>>sed -i -e 's/dw1\.//g' *.c *.h *.cpp
>>>
>>> and then reverting changes to comments in gen7_blorp.cpp and
>>> brw_fs
Patches #1-#3 are Reviewed-by: Christian König
Please split patch #4 in two with first adding the new vl_video_buffer
formats and then using them in the state tracker.
Should I commit that for you as well or do you now have an account?
Regards,
Christian.
On 05.11.2015 09:24, Julien Isorce
From: Emil Velikov
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_vec4.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp
b/src/mesa/drivers/dri/i965/brw_vec4.cpp
index e2f8a0a..6975a19 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4.c
Hi all,
This is a series that builds on top of Matts "i965: Refactor register
classes".
As Matt's branch has some additional changes, the first two patches are
fixups that could be merged directly. The follow-up four patches
removes/reworks some of the constructors, in order to avoid reworking
From: Emil Velikov
By using a default value for brw_reg_type the compiler will reuse
the existing constructor.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 9 -
src/mesa/drivers/dri/i965/brw_ir_fs.h | 4 ++--
2 files changed, 2 insertions(+), 11 deletions(-)
From: Emil Velikov
Remove unneeded this->file assignment in dst_reg(const src_reg &f) ctor
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_vec4.cpp | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp
b/src/mesa/drivers/dri/i965/brw_vec4.cpp
From: Emil Velikov
By providing default values for type and writemask, the compiler will
'create' the constructor for us. The values chosen are the exact same
ones, used in the default ctor - dst_reg(), as such the compiler should
be smart enough to optimise the duplicated (identical) assignment.
From: Emil Velikov
As of last commit one can easily the whole brw_reg as two unsigned
values. Lets use that and avoid the nastylooking memcmp + casts.
XXX: reladdr looks a bit strange. we tend to do dst_reg<>src_reg and in
some cases this might cause someheadaches. should we zero it in the
respe
From: Emil Velikov
Rather than doing memset(), implicitly use the backend_reg ctor(s) which
will initialise its variables. For the fs_reg ones, set them manually.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 18 +++---
src/mesa/drivers/dri/i965/brw_ir_fs.
From: Emil Velikov
Just fold the brw_type_for_base_type() and use a delegated
constructor.
Note this is a c++11 feature and the compiler will warn us if it's
set/defaults to an earlier version of the standard.
XXX: Should we just toggle c++11 on ? We seem to be using some of its
functionality a
From: Emil Velikov
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_ir_vec4.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_ir_vec4.h
b/src/mesa/drivers/dri/i965/brw_ir_vec4.h
index b70dca5..2c7293d 100644
--- a/src/mesa/driv
From: Emil Velikov
Just default the size to 1, and let the compiler do its magic.
XXX: Not 100% sure this is correct.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_ir_vec4.h| 3 +--
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 19 +++
2 files chang
From: Emil Velikov
This commits partially reverts "i965: Initialize registers' file to
BAD_FILE."
No longer needed as of commit "i965/fs: properly construct fs_reg" which
removes the memset(...0...), and correctly sets BAD_FILE.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_fs
From: Emil Velikov
Step one towards having a normal constructors for the *_reg primitives,
and removing a handful of somewhat ugly code.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_shader.h | 11 +--
src/mesa/drivers/dri
From: Emil Velikov
Will allow us an easy once-off get/set of the attribs. Thus we can drop
all the memset/memcmp + magic casting.
XXX: alternative name(s) are welcome
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_reg.h | 21 +
1 file changed, 13 insertions(
From: Emil Velikov
Analogous to the previous (fs) commit - drop the explicit memset and
set the non-inherited variables (reladdr) manually.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_ir_vec4.h| 13 +++
src/mesa/drivers/dri/i965/brw_vec4.cpp | 48 -
On 3 November 2015 at 18:02, Matt Turner wrote:
> On Tue, Nov 3, 2015 at 8:05 AM, Emil Velikov wrote:
>> On 3 November 2015 at 00:29, Matt Turner wrote:
>>> The test (file == BAD_FILE) works on registers for which the constructor
>>> has not run because BAD_FILE is zero. The next commit will mo
On 3 November 2015 at 16:04, Emil Velikov wrote:
>> @@ -1596,8 +1586,7 @@ vec4_visitor::lower_attributes_to_hw_regs(const int
>> *attribute_map,
>> reg.type = inst->dst.type;
>> reg.writemask = inst->dst.writemask;
>>
>> -inst->dst.file = HW_REG;
>> -inst->dst.f
Hi Julien,
On 5 November 2015 at 08:24, Julien Isorce wrote:
> Before it was only possible to convert a NV12 surface to
> RGBA or BGRA. This patch uses the same post processing
> function, "handleVAProcPipelineParameterBufferType", but
> add definitions for RGBX and BGRX.
>
> This patch also make
On 5 November 2015 at 08:24, Julien Isorce wrote:
> Some lines were using 4 indentation spaces instead of 3.
>
> The switch in vlVaAcquireBufferHandle actually had wrong brackets
> surrounding case+default.
>
Please don't mix whitespace (trivial) patches and bugfixes.
-Emil
__
On 5 November 2015 at 16:18, Emil Velikov wrote:
> From: Emil Velikov
>
> This commits partially reverts "i965: Initialize registers' file to
> BAD_FILE."
>
> No longer needed as of commit "i965/fs: properly construct fs_reg" which
> removes the memset(...0...), and correctly sets BAD_FILE.
>
> S
This is a very new extension to ES that gives you dual source blending just
like in desktop OpenGL.
This works with the piglits that I've written, which are in the process
of being cleaned up so they can be pushed, and this also works great
with the Dolphin emulator.
---
src/glsl/ast_to_hir.cpp
This can only be used if EXT_blend_func_extended is enabled
---
src/glsl/glsl_parser.yy | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy
index 4636435..40e60e5 100644
--- a/src/glsl/glsl_parser.yy
+++ b/src/glsl/glsl_parser.yy
@@ -1463,6 +1
---
src/mesa/main/tests/dispatch_sanity.cpp | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/main/tests/dispatch_sanity.cpp
b/src/mesa/main/tests/dispatch_sanity.cpp
index abe0f43..97f81f9 100644
--- a/src/mesa/main/tests/dispatch_sanity.cpp
+++ b/src/mesa/main/tests/dispatch_
---
src/mesa/main/extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index bdc6817..f1692ba 100644
--- a/src/mesa/main/extensions.c
+++ b/src/mesa/main/extensions.c
@@ -221,6 +221,7 @@ static const struct extension extension_tab
---
src/mesa/main/get_hash_params.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/get_hash_params.py b/src/mesa/main/get_hash_params.py
index fbc7b8f..9b22b91 100644
--- a/src/mesa/main/get_hash_params.py
+++ b/src/mesa/main/get_hash_params.py
@@ -330,6 +330
gl_MaxDualSourceDrawBuffersEXT - Maximum DS draw buffers supported
Only for ESSL 1.0 it provides two builtins since you can't have user-defined
color output variables
gl_SecondaryFragColorEXT and gl_SecondaryFragDataEXT[MaxDSDrawBuffers]
---
src/glsl/ast_to_hir.cpp| 16 +++
src/gl
This adds a state for the maximum dual source draw variables available
and the variable for determining if the extension has been enabled
in the program shaders.
---
src/glsl/glsl_parser_extras.cpp | 3 +++
src/glsl/glsl_parser_extras.h | 5 +
2 files changed, 8 insertions(+)
diff --git a/s
---
src/mesa/main/blend.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/blend.c b/src/mesa/main/blend.c
index 20aa498..8da81ee 100644
--- a/src/mesa/main/blend.c
+++ b/src/mesa/main/blend.c
@@ -67,7 +67,7 @@ legal_src_factor(const struct gl_context *ctx, GLe
---
src/mapi/glapi/gen/EXT_gpu_shader4.xml | 3 ++-
src/mapi/glapi/gen/es_EXT.xml | 38 ++
2 files changed, 40 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/EXT_gpu_shader4.xml
b/src/mapi/glapi/gen/EXT_gpu_shader4.xml
index b1f7eae..b4120b
This implements support for a very new extension to ES, which allows you
to do dual source blending just like in desktop OpenGL.
This works with the piglits that I am writing that just need to be cleaned
before they are pushed, and it also works fantastically with the
Dolphin emulator (Only known a
subject: glsl: ...
On Thu, Nov 5, 2015 at 12:16 PM, Ryan Houdek wrote:
> This adds a state for the maximum dual source draw variables available
> and the variable for determining if the extension has been enabled
> in the program shaders.
> ---
> src/glsl/glsl_parser_extras.cpp | 3 +++
> src/gl
subject: glapi: add ...
On Thu, Nov 5, 2015 at 12:16 PM, Ryan Houdek wrote:
> ---
> src/mapi/glapi/gen/EXT_gpu_shader4.xml | 3 ++-
> src/mapi/glapi/gen/es_EXT.xml | 38
> ++
> 2 files changed, 40 insertions(+), 1 deletion(-)
>
> diff --git a/src/mapi/g
On Thu, Nov 5, 2015 at 12:16 PM, Ryan Houdek wrote:
subject: glsl: ...
> gl_MaxDualSourceDrawBuffersEXT - Maximum DS draw buffers supported
>
> Only for ESSL 1.0 it provides two builtins since you can't have user-defined
> color output variables
> gl_SecondaryFragColorEXT and gl_SecondaryFragDat
This need to be done along with the patch which adds these to the xml
On Thu, Nov 5, 2015 at 12:16 PM, Ryan Houdek wrote:
> ---
> src/mesa/main/tests/dispatch_sanity.cpp | 8
> 1 file changed, 8 insertions(+)
>
> diff --git a/src/mesa/main/tests/dispatch_sanity.cpp
> b/src/mesa/main/te
You appear to be missing a patch to add this to glcpp:
A new preprocessor #define is added to the OpenGL ES Shading Language:
#define GL_EXT_blend_func_extended 1
On Thu, Nov 5, 2015 at 12:16 PM, Ryan Houdek wrote:
> This implements support for a very new extension to ES, which allows
On Thu, Nov 5, 2015 at 12:16 PM, Ryan Houdek wrote:
> ---
> src/mesa/main/blend.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/main/blend.c b/src/mesa/main/blend.c
> index 20aa498..8da81ee 100644
> --- a/src/mesa/main/blend.c
> +++ b/src/mesa/main/blend.c
On 2015-11-05 06:07:02, Francisco Jerez wrote:
> Jordan Justen writes:
>
> > When these functions are called in GLSL code, we create an intrinsic
> > function call:
> >
> > * groupMemoryBarrier => __intrinsic_group_memory_barrier
> > * memoryBarrierAtomicCounter => __intrinsic_memory_barrier_at
Add a note to release notes as well? Please fix up the subjects
throughout the series to conform to other patches going to those
areas. Generally glsl: or mesa: or glapi: depending on the area (for
the bits you're touching).
On Thu, Nov 5, 2015 at 12:16 PM, Ryan Houdek wrote:
> ---
> src/mesa/ma
Jordan Justen writes:
> On 2015-11-05 06:07:02, Francisco Jerez wrote:
>> Jordan Justen writes:
>>
>> > When these functions are called in GLSL code, we create an intrinsic
>> > function call:
>> >
>> > * groupMemoryBarrier => __intrinsic_group_memory_barrier
>> > * memoryBarrierAtomicCounter
On Sat, Oct 24, 2015 at 10:07 AM, Rob Clark wrote:
> From: Rob Clark
>
> For gallium, at least, we'll need this to manage shader's lifetimes,
> since in some cases both the driver and the state tracker will need
> to hold on to a reference for variant managing.
>
> Use nir_shader_mutable() before
There is no dev in drv, and dev should be from vl_screen here
---
src/gallium/state_trackers/va/context.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_trackers/va/context.c
b/src/gallium/state_trackers/va/context.c
index ec9e048..25fa905 100644
--- a/
On Thu, Nov 5, 2015 at 1:13 PM, Jason Ekstrand wrote:
> On Sat, Oct 24, 2015 at 10:07 AM, Rob Clark wrote:
>> From: Rob Clark
>>
>> For gallium, at least, we'll need this to manage shader's lifetimes,
>> since in some cases both the driver and the state tracker will need
>> to hold on to a refer
On 04.11.2015 00:47, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_blit.c | 55 ++
1 file changed, 55 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_blit.c
b/src/gallium/drivers/radeonsi/si_blit.c
index fce014a..e934146
This will allow the state trackers to use render nodes
with screen creation
Signed-off-by: Leo Liu
---
src/gallium/auxiliary/Makefile.sources | 3 +-
src/gallium/auxiliary/vl/vl_winsys.h | 6 +++
src/gallium/auxiliary/vl/vl_winsys_drm.c | 76
3 files chan
---
src/gallium/state_trackers/va/context.c | 20 +++-
1 file changed, 3 insertions(+), 17 deletions(-)
diff --git a/src/gallium/state_trackers/va/context.c
b/src/gallium/state_trackers/va/context.c
index 25fa905..845b547 100644
--- a/src/gallium/state_trackers/va/context.c
+++ b
On Thu, Nov 5, 2015 at 8:17 AM, Emil Velikov wrote:
> From: Emil Velikov
>
> Just fold the brw_type_for_base_type() and use a delegated
> constructor.
>
> Note this is a c++11 feature and the compiler will warn us if it's
> set/defaults to an earlier version of the standard.
>
> XXX: Should we ju
On Thu, Nov 5, 2015 at 8:18 AM, Emil Velikov wrote:
> From: Emil Velikov
>
> This commits partially reverts "i965: Initialize registers' file to
> BAD_FILE."
>
> No longer needed as of commit "i965/fs: properly construct fs_reg" which
> removes the memset(...0...), and correctly sets BAD_FILE.
I
This will allow dec/enc/transcode without X
Signed-off-by: Leo Liu
---
src/gallium/state_trackers/omx/entrypoint.c | 39 +
1 file changed, 29 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/omx/entrypoint.c
b/src/gallium/state_trackers/omx/ent
+/**
+ *
+ * Copyright 2009 Younes Manton.
+ * All Rights Reserved.
You probably want to change the copyright here.
With that fixed the whole series is Reviewed-by: Christian König
Regards,
Christian.
On 05.11.2015 19:
When these functions are called in glsl-ir, we create a corresponding
nir intrinsic function call.
Signed-off-by: Jordan Justen
Reviewed-by: Francisco Jerez
---
src/glsl/nir/glsl_to_nir.cpp | 15 +++
src/glsl/nir/nir_intrinsics.h | 11 +++
2 files changed, 26 insertions(+)
When these functions are called in GLSL code, we create an intrinsic
function call:
* groupMemoryBarrier => __intrinsic_group_memory_barrier
* memoryBarrierAtomicCounter => __intrinsic_memory_barrier_atomic_counter
* memoryBarrierBuffer => __intrinsic_memory_barrier_buffer
* memoryBarrierImage
For these nir intrinsics, we emit the same code as
nir_intrinsic_memory_barrier:
* nir_intrinsic_memory_barrier_atomic_counter
* nir_intrinsic_memory_barrier_buffer
* nir_intrinsic_memory_barrier_image
We treat these nir intrinsics as no-ops:
* nir_intrinsic_group_memory_barrier
* nir_intrin
On Thu, Nov 5, 2015 at 7:58 PM, Nicolai Hähnle wrote:
> On 04.11.2015 00:47, Marek Olšák wrote:
>>
>> From: Marek Olšák
>>
>> ---
>> src/gallium/drivers/radeonsi/si_blit.c | 55
>> ++
>> 1 file changed, 55 insertions(+)
>>
>> diff --git a/src/gallium/drivers/rad
Hi Boyan,
On 4 November 2015 at 15:25, Boyan Ding wrote:
> In preparation for supporting GL_KHR_debug in OpenGL ES
>
> Signed-off-by: Boyan Ding
> ---
> src/mesa/main/enable.c| 5 +
> src/mesa/main/getstring.c | 5 +
> 2 files changed, 2 insertions(+), 8 deletions(-)
>
> diff --git
As defined in the spec
when implemented in an OpenGL ES context, all entry points defined
by this extension must have a "KHR" suffix.
Signed-off-by: Emil Velikov
---
The final piece afaict for KHR_debug + ES contexts.
Mildly related - the enum tests seems somewhat busted/incomplete. Ext
As defined in the spec
when implemented in an OpenGL ES context, all entry points defined
by this extension must have a "KHR" suffix.
Signed-off-by: Emil Velikov
---
Actually build tested this time.
src/mesa/main/errors.c | 40 ---
src/mesa/mai
On 5 November 2015 at 19:15, Matt Turner wrote:
> On Thu, Nov 5, 2015 at 8:18 AM, Emil Velikov wrote:
>> From: Emil Velikov
>>
>> This commits partially reverts "i965: Initialize registers' file to
>> BAD_FILE."
>>
>> No longer needed as of commit "i965/fs: properly construct fs_reg" which
>> re
On 5 November 2015 at 19:04, Matt Turner wrote:
> On Thu, Nov 5, 2015 at 8:17 AM, Emil Velikov wrote:
>> From: Emil Velikov
>>
>> Just fold the brw_type_for_base_type() and use a delegated
>> constructor.
>>
>> Note this is a c++11 feature and the compiler will warn us if it's
>> set/defaults to
From: Boyuan Zhang
Signed-off-by: Boyuan Zhang
Reviewed-by: Christian König
---
src/gallium/state_trackers/va/buffer.c | 24 +-
src/gallium/state_trackers/va/context.c| 7 ++
src/gallium/state_trackers/va/picture.c| 117 +
src/gallium/state_tracke
On Thu, Nov 5, 2015 at 12:24 PM, Emil Velikov wrote:
> On 5 November 2015 at 19:15, Matt Turner wrote:
>> On Thu, Nov 5, 2015 at 8:18 AM, Emil Velikov
>> wrote:
>>> From: Emil Velikov
>>>
>>> This commits partially reverts "i965: Initialize registers' file to
>>> BAD_FILE."
>>>
>>> No longer n
1 - 100 of 158 matches
Mail list logo
