On Wed, Aug 24, 2016 at 04:41:02PM -0400, Gustavo Padovan wrote:
> 2016-08-24 Daniel Vetter :
>
> > On Tue, Aug 23, 2016 at 01:56:02PM -0400, robert.f...@collabora.com wrote:
> > > From: Robert Foss
> > >
> > > This series implements the sw_sync test and the lib/sw_sync helper
> > > functions
>
https://bugs.freedesktop.org/show_bug.cgi?id=97260
--- Comment #29 from Michel Dänzer ---
(In reply to Kai from comment #28)
> For me it doesn't matter if I set vblank_mode=0 or not. With your patch
> applied I'm getting 30 FPS in XCOM 2 either way [...]
Right, the point being that you were gett
On Thursday, August 25, 2016 1:50:01 AM PDT Connor Abbott wrote:
[snip]
> I think we need to use NIR_VLA here to make MSVC happy. Otherwise,
>
> Reviewed-by: Connor Abbott
Nope - this is in i965, it isn't built with MSVC :)
signature.asc
Description: This is a digitally signed message part.
__
On 2016-08-24 22:54:20, Pohjolainen, Topi wrote:
> On Wed, Aug 24, 2016 at 05:51:22PM -0700, Ian Romanick wrote:
> > On 08/24/2016 04:54 PM, Jordan Justen wrote:
> > > git://people.freedesktop.org/~jljusten/mesa hsw-es31-v1
> > >
> > > Since gen7 doesn't support sampling from a stencil buffer, we
On Thu, Aug 25, 2016 at 12:04 AM, Matt Turner wrote:
> Prior to this commit rename_variables_block() is recursively called,
> performing a depth-first traversal of the control flow graph. The
> function uses a non-trivial amount of stack space for local variables,
> which puts us in danger of smas
On Tue, Aug 23, 2016 at 9:26 PM, Emil Velikov wrote:
> On 22 August 2016 at 07:10, Nicolas Boichat wrote:
>> Hi Emil,
>>
>> On Tue, Aug 16, 2016 at 1:17 AM, Emil Velikov
>> wrote:
>>> Hi Nicolas,
>>>
>>> On 4 August 2016 at 02:51, Nicolas Boichat wrote:
On Thu, Aug 4, 2016 at 9:38 AM, Mic
On Thu, Aug 25, 2016 at 1:49 AM, Jason Ekstrand wrote:
> In some programs, we can have very deep dominance trees and the recursion
> can cause us to risk stack overflows. Instead, we replace the recursion
> with a pair of loops, one at the start and one at the end. This is
> functionally equival
In some programs, we can have very deep dominance trees and the recursion
can cause us to risk stack overflows. Instead, we replace the recursion
with a pair of loops, one at the start and one at the end. This is
functionally equivalent to what we had before and it's actually a bit
easier to read
2016-08-25 7:49 GMT+02:00 Jason Ekstrand :
> In some programs, we can have very deep dominance trees and the recursion
> can cause us to risk stack overflows. Instead, we replace the recursion
> with a pair of loops, one at the start and one at the end. This is
> functionally equivalent to what w
On Wed, Aug 24, 2016 at 05:51:22PM -0700, Ian Romanick wrote:
> On 08/24/2016 04:54 PM, Jordan Justen wrote:
> > git://people.freedesktop.org/~jljusten/mesa hsw-es31-v1
> >
> > Since gen7 doesn't support sampling from a stencil buffer, we copy the
> > stencil texture to an R8_UINT texture, and sam
https://bugs.freedesktop.org/show_bug.cgi?id=97444
Michel Dänzer changed:
What|Removed |Added
Component|Other |Driver/AMDgpu
Assignee|mesa-de
In some programs, we can have very deep dominance trees and the recursion
can cause us to risk stack overflows. Instead, we replace the recursion
with a pair of loops, one at the start and one at the end. This is
functionally equivalent to what we had before and it's actually a bit
easier to read
On Thu, Aug 25, 2016 at 1:25 AM, Kenneth Graunke wrote:
> We want to insert code in each of the predecessors of the end block.
> This code includes a nir_if, which would split the block, altering
> the set. To avoid that, I emitted a dead constant at the end of each
> block before splitting it, s
On Wed, Aug 24, 2016 at 04:55:02PM -0700, Jordan Justen wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 55
> ++-
> src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 4 ++
> 2 files changed, 58 insertions(+), 1 deletion(-)
>
>
On 2016-08-24 17:40:42, Ian Romanick wrote:
> On 08/24/2016 04:55 PM, Jordan Justen wrote:
> > Signed-off-by: Jordan Justen
> > ---
> > src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 13 +++--
> > 1 file changed, 11 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/mesa/drivers/
On 2016-08-24 17:44:44, Matt Turner wrote:
> On Wed, Aug 24, 2016 at 4:55 PM, Jordan Justen
> wrote:
> > Signed-off-by: Jordan Justen
> > ---
> > src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 55
> > ++-
> > src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 4 ++
> > 2
On 2016-08-24 17:54:23, Kenneth Graunke wrote:
> On Wednesday, August 24, 2016 5:49:53 PM PDT Ian Romanick wrote:
> > On 08/24/2016 05:46 PM, Matt Turner wrote:
> > > On Wed, Aug 24, 2016 at 4:54 PM, Jordan Justen
> > > wrote:
> > >> - if (_mesa_format_has_color_component(format, i))
> > >
We want to insert code in each of the predecessors of the end block.
This code includes a nir_if, which would split the block, altering
the set. To avoid that, I emitted a dead constant at the end of each
block before splitting it, so that the set of predecessors remained
unchanged. This was admi
On Wed, Aug 24, 2016 at 05:34:49PM -0700, Ian Romanick wrote:
> On 08/24/2016 04:54 PM, Jordan Justen wrote:
> > Signed-off-by: Jordan Justen
> > ---
> > src/mesa/drivers/dri/i965/brw_sampler_state.c | 8 ++--
> > 1 file changed, 6 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/mesa/
2016-08-24 Daniel Vetter :
> On Tue, Aug 23, 2016 at 01:56:02PM -0400, robert.f...@collabora.com wrote:
> > From: Robert Foss
> >
> > This series implements the sw_sync test and the lib/sw_sync helper functions
> > for said test.
> >
> > Gustavo Padovans sw_sync series was just de-staged in
> >
On Friday, August 12, 2016 3:59:35 PM PDT Kenneth Graunke wrote:
> When generating mipmaps for sRGB textures, force both decode and encode,
> so the filtering is done in linear colorspace, regardless of settings.
>
> Fixes a WebGL conformance test in Chrome:
> https://www.khronos.org/registry/webg
Ilia Mirkin writes:
> On Thu, Aug 25, 2016 at 12:45 AM, Francisco Jerez
> wrote:
>> Ilia Mirkin writes:
>>
>>> On Wed, Aug 24, 2016 at 4:30 PM, Francisco Jerez
>>> wrote:
Ilia Mirkin writes:
> I had trouble getting these to apply, perhaps they were meant to go on
> top of
On Thu, Aug 25, 2016 at 12:45 AM, Francisco Jerez wrote:
> Ilia Mirkin writes:
>
>> On Wed, Aug 24, 2016 at 4:30 PM, Francisco Jerez
>> wrote:
>>> Ilia Mirkin writes:
>>>
I had trouble getting these to apply, perhaps they were meant to go on
top of something else. Anyways, should be
Ilia Mirkin writes:
> On Wed, Aug 24, 2016 at 4:30 PM, Francisco Jerez
> wrote:
>> Ilia Mirkin writes:
>>
>>> I had trouble getting these to apply, perhaps they were meant to go on
>>> top of something else. Anyways, should be fairly easy for you to test
>>> out with llvmpipe.
>>>
>> It should
On Wed, Aug 24, 2016 at 2:42 PM, Marek Olšák wrote:
> On Wed, Aug 24, 2016 at 10:42 PM, Jason Ekstrand
> wrote:
> > On Wed, Aug 24, 2016 at 11:46 AM, Marek Olšák wrote:
> >>
> >> On Fri, Aug 5, 2016 at 1:22 AM, Kenneth Graunke
> >> wrote:
> >> > On Thursday, August 4, 2016 11:28:28 AM PDT Jaso
Jason suggested adding an assert(function->impl) here. All callers
of this function actually want ->impl, so I decided just to change
the API.
We also change the nir_lower_io_to_temporaries API here. All but one
caller passed nir_shader_get_entrypoint(), and with the previous commit,
it now uses
We want to insert the code at the end of the program. Looping over
all the functions (of which there was only one) was the old way of doing
this, but now we have nir_shader_get_entrypoint(), so let's use it.
Suggested by Connor Abbott.
v2: Update for nir_shader_get_entrypoint API change.
Signed
This changes the pass internals to work with a nir_function_impl
directly rather than a nir_function. The next patch will change
the API.
Signed-off-by: Kenneth Graunke
Reviewed-by: Jason Ekstrand
---
src/compiler/nir/nir_lower_io_to_temporaries.c | 8
1 file changed, 4 insertions(+),
On Wed, Aug 24, 2016 at 4:30 PM, Francisco Jerez wrote:
> Ilia Mirkin writes:
>
>> I had trouble getting these to apply, perhaps they were meant to go on
>> top of something else. Anyways, should be fairly easy for you to test
>> out with llvmpipe.
>>
> It should apply cleanly on latest master no
Prior to this commit rename_variables_block() is recursively called,
performing a depth-first traversal of the control flow graph. The
function uses a non-trivial amount of stack space for local variables,
which puts us in danger of smashing the stack, given a sufficiently deep
dominance tree.
XCO
On Wed, Aug 24, 2016 at 6:19 PM, Ian Romanick wrote:
> On 08/24/2016 12:57 PM, Ilia Mirkin wrote:
>> On Wed, Aug 24, 2016 at 3:40 PM, Ian Romanick wrote:
>>> On 08/24/2016 10:40 AM, Ilia Mirkin wrote:
The GL_ARB_compute_shader spec does not make a mention of a #extension
enable, nor a #
Iago Toral writes:
> On Tue, 2016-08-23 at 12:58 -0700, Francisco Jerez wrote:
>> Iago Toral Quiroga writes:
>>
>> >
>> > So we can access it in the vec4 backend to handle byte offsets into
>> > registers.
>> This change has deep implications in the meaning of the vec4 register
>> objects beca
https://bugs.freedesktop.org/show_bug.cgi?id=97444
--- Comment #4 from Fabian Maurer ---
I managed to downgrade mesa to 11.2.2, but the issue persisted. Figured out it
can be fixed by downgrading "xf86-video-amdgpu" from git to stable.
Now I'm not sure wheter this bug is a mesa issue or from that
On 08/24/2016 03:59 PM, Ilia Mirkin wrote:
> Subject should say OES_texture_cube_map_array
Heh... fixed locally. :)
> On Wed, Aug 24, 2016 at 6:12 PM, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> These are the only platforms that current expose OES_geometry_shader.
>> Once OpenGL ES 3.1 and
On Wednesday, August 24, 2016 5:49:53 PM PDT Ian Romanick wrote:
> On 08/24/2016 05:46 PM, Matt Turner wrote:
> > On Wed, Aug 24, 2016 at 4:54 PM, Jordan Justen
> > wrote:
> >> Signed-off-by: Jordan Justen
> >> ---
> >> src/mesa/drivers/dri/i965/brw_sampler_state.c | 8 ++--
> >> 1 file chan
On 08/24/2016 03:59 PM, Ilia Mirkin wrote:
> On Wed, Aug 24, 2016 at 6:12 PM, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> This has a separate enable flag because this extension also requires
>> OES_geometry_shader. It is possible that some drivers may support
>> OpenGL ES 3.1 and ARB_texture
On 08/24/2016 04:54 PM, Jordan Justen wrote:
> git://people.freedesktop.org/~jljusten/mesa hsw-es31-v1
>
> Since gen7 doesn't support sampling from a stencil buffer, we copy the
> stencil texture to an R8_UINT texture, and sample from it instead.
>
> This allows us to enable ARB_stencil_texturing
On 08/24/2016 05:46 PM, Matt Turner wrote:
> On Wed, Aug 24, 2016 at 4:54 PM, Jordan Justen
> wrote:
>> Signed-off-by: Jordan Justen
>> ---
>> src/mesa/drivers/dri/i965/brw_sampler_state.c | 8 ++--
>> 1 file changed, 6 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/mesa/drivers/dri/i9
On Wed, Aug 24, 2016 at 4:54 PM, Jordan Justen
wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/brw_sampler_state.c | 8 ++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_sampler_state.c
> b/src/mesa/drivers/dri/i965
On Wed, Aug 24, 2016 at 4:55 PM, Jordan Justen
wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 55
> ++-
> src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 4 ++
> 2 files changed, 58 insertions(+), 1 deletion(-)
>
> diff --g
On 08/24/2016 04:55 PM, Jordan Justen wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 13 +++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
> b/src/mesa/drivers/dri/
On 08/24/2016 04:55 PM, Jordan Justen wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 55
> ++-
> src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 4 ++
> 2 files changed, 58 insertions(+), 1 deletion(-)
>
> diff --git a/src/
On 08/24/2016 04:54 PM, Jordan Justen wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/brw_sampler_state.c | 8 ++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_sampler_state.c
> b/src/mesa/drivers/dri/i965/brw_samp
On 23 August 2016 at 11:11, Jason Ekstrand wrote:
> This option makes installed Vulkan ICD files contain only a driver library
> name and not a path. This is intended for distros to help them work around
> multi-arch issues.
Can we at least push this for now?
Reviewed-by: Dave Airlie
Dave.
__
On Wed, Aug 24, 2016 at 5:16 PM, Jason Ekstrand wrote:
> The pipeline layout affects shader compilation because it is what
> determines binding table locations as well as whether or not a particular
> buffer has dynamic offsets. Since this affects the generated shader, it
> needs to be in the has
On 24 August 2016 at 20:43, Emil Velikov wrote:
> On 19 July 2016 at 13:41, Eric Engestrom wrote:
>> It seems [0] old versions of Mako are no longer supported. Emil mentioned it
>> might need v0.8.0 [1] for isl_format_layout [2], although I didn't get
>> a confirmation that it's really the minimu
The pipeline layout affects shader compilation because it is what
determines binding table locations as well as whether or not a particular
buffer has dynamic offsets. Since this affects the generated shader, it
needs to be in the hash. This fixes a bunch of CTS tests now that the CTS
is using a
On Thu, 2016-08-25 at 10:00 +1000, Timothy Arceri wrote:
> On Wed, 2016-08-24 at 15:12 -0700, Ian Romanick wrote:
> >
> > From: Ian Romanick
> >
> > Previously we'd munge the interpolation mode so that later checks
> > in
> > the
> > GLSL linker would pass. The caused problems for similar check
On Wednesday, August 24, 2016 3:12:43 PM PDT Ian Romanick wrote:
> From: Ian Romanick
>
> Previously we'd munge the interpolation mode so that later checks in the
> GLSL linker would pass. The caused problems for similar checks in SSO
> IO validation. Instead, make the check smarter, use the sa
On Wed, 2016-08-24 at 15:12 -0700, Ian Romanick wrote:
> From: Ian Romanick
>
> Previously we'd munge the interpolation mode so that later checks in
> the
> GLSL linker would pass. The caused problems for similar checks in
> SSO
> IO validation. Instead, make the check smarter, use the same che
Signed-off-by: Jordan Justen
---
src/mesa/main/stencil.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/main/stencil.c b/src/mesa/main/stencil.c
index 409b2f0..b303bb7 100644
--- a/src/mesa/main/stencil.c
+++ b/src/mesa/main/stencil.c
@@ -109,6 +109,9 @@ _mesa_ClearStencil( GLint
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/intel_extensions.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index aaebd08..756f04d 100644
--- a/src/mesa/drivers/dri/i
Signed-off-by: Jordan Justen
---
src/mesa/main/teximage.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index 8869b3d..efde114 100644
--- a/src/mesa/main/teximage.c
+++ b/src/mesa/main/teximage.c
@@ -5341,6 +5341,11 @@ texture_image_m
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/intel_extensions.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 466c384..aaebd08 100644
--- a/src/mesa/drivers/d
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/intel_screen.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/intel_screen.c
b/src/mesa/drivers/dri/i965/intel_screen.c
index 7876652..3ffbf39 100644
--- a/src/mesa/drivers/dri/i965/intel_screen.c
+++
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/brw_clear.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_clear.c
b/src/mesa/drivers/dri/i965/brw_clear.c
index 1dfff09..18b8fcb 100644
--- a/src/mesa/drivers/dri/i965/brw_clear.c
+++ b/src/mes
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
index 2bafe91..28d0958 100644
-
thread_width_max in the GPGPU walker command limits us to a maximum of
64 threads.
This fixes a crash on Haswell in the OpenGLES 3.1 conformance test
suite which tests the advertised limits of the max invocation counts.
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/brw_context.c |
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/brw_sampler_state.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_sampler_state.c
b/src/mesa/drivers/dri/i965/brw_sampler_state.c
index 9f56c81..8ad34b8 100644
--- a/src/mesa/dr
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/brw_context.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/mesa/drivers/dri/i965/brw_context.c
index d8a3723..d38b483 100644
--- a/src/mesa/drivers/dri/i965/brw_context.c
+++ b/sr
Signed-off-by: Jordan Justen
---
docs/GL3.txt | 6 +++---
docs/relnotes/12.1.0.html | 3 +++
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 084c17e..5dd9e41 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -180,7 +180,7 @@ GL 4.3, GLSL
For gen < 8, we can't sample from the stencil buffer, which is
required for the ARB_stencil_texturing extension. We'll make a copy of
the stencil data into a new texture that we can sample using the
R8_UINT surface type.
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/intel_mipmap_tre
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/intel_tex_image.c| 3 +++
src/mesa/drivers/dri/i965/intel_tex_subimage.c | 7 +--
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_tex_image.c
b/src/mesa/drivers/dri/i965/intel_tex_im
git://people.freedesktop.org/~jljusten/mesa hsw-es31-v1
Since gen7 doesn't support sampling from a stencil buffer, we copy the
stencil texture to an R8_UINT texture, and sample from it instead.
This allows us to enable ARB_stencil_texturing and OpenGLES 3.1 on
Haswell.
Ivy Bridge and Baytrail re
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 55 ++-
src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 4 ++
2 files changed, 58 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
b/src/mesa/drivers/
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/gen7_misc_state.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/gen7_misc_state.c
b/src/mesa/drivers/dri/i965/gen7_misc_state.c
index ffdf6f2..271d962 100644
--- a/src/mesa/drivers/dri/i965/gen7_misc_state
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
index 0836b92..6cae492 100644
--- a/src/mesa/driv
Patches 1-16 are
Reviewed-by: Ilia Mirkin
I left a few comments on a handful, but other than the one you've
already addressed, none are particularly important.
You'll have to sucker someone else into looking at patch 17, I'm afraid.
On Wed, Aug 24, 2016 at 6:12 PM, Ian Romanick wrote:
> This
Subject should say OES_texture_cube_map_array
On Wed, Aug 24, 2016 at 6:12 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> These are the only platforms that current expose OES_geometry_shader.
> Once OpenGL ES 3.1 and OES_geometry_shader are enabled on Gen7, this
> extension can be enabled ther
On Wed, Aug 24, 2016 at 6:12 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> This has a separate enable flag because this extension also requires
> OES_geometry_shader. It is possible that some drivers may support
> OpenGL ES 3.1 and ARB_texture_cube_map but not support
> OES_geometry_shader.
>
On Wed, Aug 24, 2016 at 6:12 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> Signed-off-by: Ian Romanick
> ---
> src/mesa/main/context.h | 5 +
> src/mesa/main/teximage.c | 12 ++--
> src/mesa/main/texobj.c | 4 ++--
> 3 files changed, 13 insertions(+), 8 deletions(-)
>
> diff
On 08/24/2016 03:18 PM, Ilia Mirkin wrote:
> On Wed, Aug 24, 2016 at 6:12 PM, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> Fixes
>>
>> dEQP-GLES31.functional.geometry_shading.layered.fragment_layer_2d_array
>> dEQP-GLES31.functional.geometry_shading.layered.fragment_layer_2d_multisample_array
On 08/24/2016 12:57 PM, Ilia Mirkin wrote:
> On Wed, Aug 24, 2016 at 3:40 PM, Ian Romanick wrote:
>> On 08/24/2016 10:40 AM, Ilia Mirkin wrote:
>>> The GL_ARB_compute_shader spec does not make a mention of a #extension
>>> enable, nor a #define with the ext name. It follows that local_size_*
>>> s
On Wed, Aug 24, 2016 at 6:12 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> Fixes
>
> dEQP-GLES31.functional.geometry_shading.layered.fragment_layer_2d_array
> dEQP-GLES31.functional.geometry_shading.layered.fragment_layer_2d_multisample_array
> dEQP-GLES31.functional.geometry_shading.layered.f
From: Ian Romanick
Previously we'd munge the interpolation mode so that later checks in the
GLSL linker would pass. The caused problems for similar checks in SSO
IO validation. Instead, make the check smarter, use the same check in
both places, and don't modify the interpolation mode.
Fixes pi
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/compiler/glsl/builtin_functions.cpp | 6 ++
src/compiler/glsl/glsl_parser_extras.h | 6 ++
2 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/src/compiler/glsl/builtin_functions.cpp
b/src/compiler/glsl/builtin_function
From: Ian Romanick
glsl/lower_buffer_access.cpp:324:55: warning: unused parameter ‘var’
[-Wunused-parameter]
ir_variable *var,
^
Signed-off-by: Ian Romanick
---
src/compiler/glsl/lower_buffer_acce
From: Ian Romanick
This has a separate enable from ARB_texture_cube_map_array.
Signed-off-by: Ian Romanick
---
src/mesa/main/version.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/version.c b/src/mesa/main/version.c
index a9b4fd1..ba10832 100644
--- a/src/m
This series enables GL_OES_geometry_shader and
GL_OES_texutre_cube_map_array. There are still a number of failures
with this series, but I would like to land it anyway so that I don't
have to keep rebasing it. Here is my analysis of the failures:
ES3-CTS.gtf.GL3Tests.texture_storage.texture_stor
From: Ian Romanick
This has a separate enable flag because this extension also requires
OES_geometry_shader. It is possible that some drivers may support
OpenGL ES 3.1 and ARB_texture_cube_map but not support
OES_geometry_shader.
Signed-off-by: Ian Romanick
---
src/compiler/glsl/builtin_funct
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/main/context.h | 5 +
src/mesa/main/teximage.c | 12 ++--
src/mesa/main/texobj.c | 4 ++--
3 files changed, 13 insertions(+), 8 deletions(-)
diff --git a/src/mesa/main/context.h b/src/mesa/main/context.h
index 9a609dd
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/main/genmipmap.c | 3 +--
src/mesa/main/get.c | 2 +-
src/mesa/main/texobj.c| 2 +-
3 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/genmipmap.c b/src/mesa/main/genmipmap.c
index 97d3c62..5038d5c 1
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/compiler/glsl/builtin_variables.cpp | 3 ++-
src/compiler/glsl/glsl_parser_extras.cpp | 2 ++
src/compiler/glsl/glsl_parser_extras.h | 8 +++-
src/mapi/glapi/gen/es_EXT.xml| 10 ++
src/mesa/main/extensions_tabl
From: Ian Romanick
Fixes
dEQP-GLES31.functional.geometry_shading.layered.fragment_layer_2d_array
dEQP-GLES31.functional.geometry_shading.layered.fragment_layer_2d_multisample_array
dEQP-GLES31.functional.geometry_shading.layered.fragment_layer_3d
dEQP-GLES31.functional.geometry_shading.layered.f
From: Ian Romanick
Gen7 can get this extension (and GL_OES_shader_io_blocks) as soon as the
rest of OpenGL ES 3.1 is enabled.
Signed-off-by: Ian Romanick
---
docs/features.txt| 2 +-
docs/relnotes/12.1.0.html| 1 +
src/mesa/drivers/dri/i965/intel
From: Ian Romanick
Originally this patch added the checks to allow the draw calls with XFB,
but commit 2dabd497 beat me to it.
Signed-off-by: Ian Romanick
---
src/mesa/main/api_validate.c | 33 +++--
1 file changed, 31 insertions(+), 2 deletions(-)
diff --git a/src
From: Ian Romanick
Use bool instead of GLboolean and constify ctx.
Signed-off-by: Ian Romanick
---
src/mesa/main/texstorage.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/src/mesa/main/texstorage.c b/src/mesa/main/texstorage.c
index e319fa5..c4
From: Ian Romanick
All the GLSL 4.x keywords were added to the list of reserved keywords
in GLSL ES 3.10. As far as I can tell, these are the only ones that
were missed.
Signed-off-by: Ian Romanick
Cc: mesa-sta...@lists.freedesktop.org
---
src/compiler/glsl/glsl_lexer.ll | 8
1 file
From: Ian Romanick
Fixes
dEQP-GLES31.functional.geometry_shading.instanced.geometry_1_invocations
dEQP-GLES31.functional.geometry_shading.instanced.invocation_per_layer_2d_array
dEQP-GLES31.functional.geometry_shading.instanced.invocation_per_layer_2d_multisample_array
dEQP-GLES31.functional.geo
From: Ian Romanick
When GL_OES_geometry_shader is enabled, this fixes
dEQP-GLES31.functional.shaders.linkage.geometry.uniform.rules.type_mismatch_1.
Signed-off-by: Ian Romanick
---
src/compiler/glsl/linker.cpp | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/compiler/glsl/linker.
From: Ian Romanick
This makes it a bit easier to add support for more features in different
APIs.
Signed-off-by: Ian Romanick
---
src/mesa/main/texstorage.c | 37 ++---
1 file changed, 26 insertions(+), 11 deletions(-)
diff --git a/src/mesa/main/texstorage.c b/
From: Ian Romanick
These are the only platforms that current expose OES_geometry_shader.
Once OpenGL ES 3.1 and OES_geometry_shader are enabled on Gen7, this
extension can be enabled there as well.
Gen6 will never get OpenGL ES 3.1, so it will never get this
extension... even though it has the d
On Wed, Aug 24, 2016 at 10:42 PM, Jason Ekstrand wrote:
> On Wed, Aug 24, 2016 at 11:46 AM, Marek Olšák wrote:
>>
>> On Fri, Aug 5, 2016 at 1:22 AM, Kenneth Graunke
>> wrote:
>> > On Thursday, August 4, 2016 11:28:28 AM PDT Jason Ekstrand wrote:
>> >> > > - No cut index (GS is supposed to be fin
From: Marek Olšák
Cc: 12.0
---
src/gallium/drivers/radeonsi/cik_sdma.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/cik_sdma.c
b/src/gallium/drivers/radeonsi/cik_sdma.c
index 2eade50..e1fd050 100644
--- a/src/gallium/drivers/radeonsi/cik_sdma.c
+++ b/s
On Wed, Aug 24, 2016 at 11:46 AM, Marek Olšák wrote:
> On Fri, Aug 5, 2016 at 1:22 AM, Kenneth Graunke
> wrote:
> > On Thursday, August 4, 2016 11:28:28 AM PDT Jason Ekstrand wrote:
> >> > > - No cut index (GS is supposed to be fine and Tess looks OK too)
> >> >
> >> > I'd be OK with dropping GS
From: Robert Foss
Add subtest test_sync_merge that tests merging fences and the validity of the
resulting merged fence.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 67 ++-
1 file changed, 67 insertions(+), 0 dele
From: Robert Foss
This subtest verifies merging a fence with itself does not fail.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 27 +++
1 file changed, 27 insertions(+), 0 deletions(-)
diff --git a/tests/sw_sync.c b/tests/sw_sync.c
inde
From: Robert Foss
This subtest verifies that waiting on fences works properly.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 38 ++
1 file changed, 38 insertions(+), 0 deletions(-)
diff --git a/tests/sw_sync.c b/tests/sw_sync
From: Robert Foss
Add subtest alloc_fence that verifies that it's possible to allocate a fence
on a timeline.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 16
1 file changed, 16 insertions(+), 0 deletions(-)
diff --git a/tests/sw_sync.c b/test
From: Robert Foss
This subtest verifies that waiting, timing out on a wait and that counting
fences in various states works.
Signed-off-by: Robert Foss
Reviewed-by: Eric Engestrom
---
tests/sw_sync.c | 66 ++-
1 file changed, 66 insertions(+), 0
1 - 100 of 206 matches
Mail list logo
