https://bugs.freedesktop.org/show_bug.cgi?id=94452
Kenneth Graunke changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
On 2016-03-10 18:53:28, Kenneth Graunke wrote:
> Fixes:
> dEQP-GLES3.functional.negative_api.state.get_framebuffer_attachment_parameteriv
>
> Apparently, GL_FRAMEBUFFER_ATTACHMENT_OBJECT_NAME is not allowed when
> GL_FRAMEBUFFER_ATTACHMENT_OBJECT_TYPE is GL_FRAMEBUFFER_DEFAULT, and
> is expected t
Carries across the number of samples and layers state in the
'softpipe_set_framebuffer_state()' callback. This state is
part of 'ARB_framebuffer_no_attachments' support.
Signed-off-by: Edward O'Callaghan
---
src/gallium/drivers/softpipe/sp_state_surface.c | 2 ++
1 file changed, 2 insertions(+)
Signed-off-by: Edward O'Callaghan
---
docs/GL3.txt | 2 +-
docs/relnotes/11.3.0.html | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 3058996..b9fc86b 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -172,7 +172,7 @@ GL 4.3, GLSL 4
Here we store the number of samples and layers directly in the
pipe_framebuffer_state so that in the case of
ARB_framebuffer_no_attachment we may make use of them directly.
Further, we adjust various gallium/auxiliary helper functions
accordingly.
V2:
Convert branches in util_framebuffer_get_nu
Signed-off-by: Edward O'Callaghan
---
src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
index 910143f..4a00d92 100644
--- a/src/gall
From: Bas Nieuwenhuizen
The rasterizer state used Multisample._Enable to enable msaa. However
that gets updated in core mesa before any st validation happens.
Setting it depends on _NumSamples which happens during st validation.
To break this cyclic dependency I replicated the check at the place
From: Ilia Mirkin
Signed-off-by: Ilia Mirkin
Reviewed-by: Edward O'Callaghan
---
src/gallium/drivers/nouveau/nvc0/nvc0_program.c| 7 +++
src/gallium/drivers/nouveau/nvc0/nvc0_state_validate.c | 16 ++--
src/gallium/drivers/nouveau/nvc0/nvc0_surface.c| 4
Change references to gl_framebuffer::Width, Height, MaxNumLayers
and Visual::samples to use the _mesa_geometric_ convenience functions
for those places where the geometry of the gl_framebuffer is needed.
This is in contrast to the geometry of the intersection of the
attachments of the gl_framebuffe
Signed-off-by: Edward O'Callaghan
---
src/gallium/auxiliary/util/u_blitter.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/util/u_blitter.c
b/src/gallium/auxiliary/util/u_blitter.c
index 43fbd8e..c4a32e8 100644
--- a/src/gallium/auxiliary/util/u_bl
Signed-off-by: Edward O'Callaghan
---
src/mesa/main/fbobject.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index d490918..a480a04 100644
--- a/src/mesa/main/fbobject.c
+++ b/src/mesa/main/fbobject.c
@@ -1369,6 +1369,11 @@ _mesa_Bind
Signed-off-by: Edward O'Callaghan
---
src/gallium/drivers/radeonsi/si_pipe.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
b/src/gallium/drivers/radeonsi/si_pipe.c
index 46fa592..1472ccf 100644
--- a/src/gallium/drivers/radeonsi/si_
Handle the case of ARB_framebuffer_no_attachment.
Also, kill off a dead debug printf() call while we are here.
Signed-off-by: Edward O'Callaghan
---
src/mesa/state_tracker/st_atom_framebuffer.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_
Signed-off-by: Edward O'Callaghan
---
src/gallium/drivers/trace/tr_dump_state.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/drivers/trace/tr_dump_state.c
b/src/gallium/drivers/trace/tr_dump_state.c
index 0627e5a..8f827db 100644
--- a/src/gallium/drivers/trace/tr_dump_state.
Due to popular demand this _should_ implement this GL extension
correctly now and include the various community feedback. Sorry
for this taking so very long because of external real-life
pressures.. In any case here it is.
This passes piglit fully on nvc0 (checked awhile ago); on
radeonsi today wi
Signed-off-by: Edward O'Callaghan
---
src/mesa/main/mtypes.h | 8 +++
src/mesa/state_tracker/st_atom_framebuffer.c | 35
src/mesa/state_tracker/st_extensions.c | 4
3 files changed, 47 insertions(+)
diff --git a/src/mesa/main/mt
Upon context creation, internal driver structures are malloc()'ed
and memset() to zero them. This results in a invalid number of
samples 'by default'. Handle this in the simplest way to avoid
elaborate and probably equally sub-optimial solutions.
Signed-off-by: Edward O'Callaghan
---
src/gallium
Signed-off-by: Edward O'Callaghan
---
src/gallium/drivers/r600/r600_pipe.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/r600_pipe.c
b/src/gallium/drivers/r600/r600_pipe.c
index 0d22faa..96c0382 100644
--- a/src/gallium/drivers/r600/r600_pipe.c
+
Set default values for the constants required in
ARB_framebuffer_no_attachments and obtained the number
of layers from ``PIPE_CAP_MAX_TEXTURE_ARRAY_LAYERS``.
Signed-off-by: Edward O'Callaghan
---
src/mesa/state_tracker/st_extensions.c | 19 +++
1 file changed, 19 insertions(+)
d
Add PIPE_CAP to determine the MSAA modes the hardware
supports so that values requested from the application
using GL_ARB_framebuffer_no_attachments may be quantized
to what the hardware expects.
V.2:
Fix doc for a more detailed description of the PIPE_CAP
and the corresponding GL constant.
Sig
---
src/compiler/nir/glsl_to_nir.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/compiler/nir/glsl_to_nir.cpp b/src/compiler/nir/glsl_to_nir.cpp
index 613b138..f592c57 100644
--- a/src/compiler/nir/glsl_to_nir.cpp
+++ b/src/compiler/nir/glsl_to_nir.cpp
@@ -1048,6 +1048,8 @@ nir_visit
Series is:
Reviewed-by: Samuel Pitoiset
On 03/17/2016 10:13 AM, Hans de Goede wrote:
Commit c3083c7082 ("nv50/ir: add support for BUFFER accesses") disabled /
commented out some of the old resource handling code, but not all of it.
Effectively all of it is dead already, if we ever enter the o
This fixes a compile error while building Nouveau with C++11 enabled (and
glibc >= 2.23). This happens if SWR is enabled, as it forces C++11.
Signed-off-by: Pierre Moreau
---
src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/dr
On Thu, 2016-03-17 at 11:43 +0100, Juan A. Suarez Romero wrote:
> ---
> docs/envvars.html | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/docs/envvars.html b/docs/envvars.html
> index 06aa0ac..e21b7c1 100644
> --- a/docs/envvars.html
> +++ b/docs/envvars.html
> @@ -163,6 +163,7 @@ See th
Yuanhan Liu decided these were useful for linear filtering in
commit 76669381 (circa 2011). Prior to that, we never set them;
it seems he tried to preserve that behavior for nearest filtering.
It turns out they're useful for nearest filtering, too: setting
these fixes the following dEQP-GLES3 tes
This is not used outside of prog_parameter.c
---
src/mesa/program/prog_parameter.c | 189 +++---
src/mesa/program/prog_parameter.h | 5 -
2 files changed, 94 insertions(+), 100 deletions(-)
diff --git a/src/mesa/program/prog_parameter.c
b/src/mesa/program/prog_p
---
src/mesa/drivers/dri/i965/brw_vec4.cpp | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp
b/src/mesa/drivers/dri/i965/brw_vec4.cpp
index 155a550..02a00b3 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4.cpp
+++ b/src/mesa/drivers/d
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
On 10/03/16 19:58, Matt Turner wrote:
> On Thu, Mar 10, 2016 at 10:13 AM, Topi Pohjolainen
> wrote:
>> Uniform linking in (see link_assign_uniform_locations()) already
>> stores the index to the storage in ir_variable which is further
>> stored
Mesa version is 11.1.2
LVM - 3.4
Ill see what I can do regarding a gdb backtrace.
Cheers
Jason
On 17/03/2016 11:38 am, "Roland Scheidegger" wrote:
>Am 17.03.2016 um 01:52 schrieb Jason Anderssen:
>> Hi all,
>>
>> I was wondering if anyone knows of or has seen a crash when Mesa is
>> compiled fo
On 2016-03-16 20:32:18, Kenneth Graunke wrote:
> OpenGL ES 3.x contains a table of sized internal formats and their
> required properties. In particular, each format is marked as
> "Color Renderable" or "Texture Filterable".
>
> This patch introduces two functions that can be used to query the
>
Jason Ekstrand writes:
> On Fri, Mar 18, 2016 at 2:32 PM, Francisco Jerez
> wrote:
>
>> Jason Ekstrand writes:
>>
>> > ---
>> > src/compiler/Makefile.sources | 1 +
>> > src/compiler/glsl/glsl_parser_extras.cpp | 1 +
>> > src/compiler/glsl/ir_optimization.h| 1 +
>
On Tue, Mar 1, 2016 at 9:06 AM, Rob Herring wrote:
> On Mon, Feb 29, 2016 at 4:50 AM, Emil Velikov
> wrote:
>> On 25 February 2016 at 01:47, Emil Velikov wrote:
>>> On 24 February 2016 at 18:56, Rob Herring wrote:
>>
Rob Herring (8):
Android: remove dependence on .SECONDEXPANSION
>
On Fri, Mar 18, 2016 at 2:44 PM, Francisco Jerez
wrote:
> Jason Ekstrand writes:
>
> > ---
> > src/compiler/nir/glsl_to_nir.cpp | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/src/compiler/nir/glsl_to_nir.cpp
> b/src/compiler/nir/glsl_to_nir.cpp
> > index 613b138..f592c57 10064
On Wed, Mar 16, 2016 at 04:03:08PM +1000, Peter Hutterer wrote:
> We had a number of last-minute nominations and this did not give all nominees
> the chance to respond to the nominations.
Sheepish Verhaegen.
___
mesa-dev mailing list
mesa-dev@lists.freed
On 03/17/2016 05:07 PM, Hans de Goede wrote:
Hi,
On 14-03-16 21:50, Samuel Pitoiset wrote:
Btw, do you need someone with commit access to push your previous
series (the tgsi thing)? I can do this for you.
Thanks for the offer. IIRC Ilia wanted some minor fixes there, so
I'll do
a v2 tomo
On Tue, 2016-03-15 at 12:38 -0400, Ilia Mirkin wrote:
> Not really an answer to your question, but you may be interested in
> this model of the NVIDIA Tesla FPU that mwk RE'd, including fp64
> (which was available on the G200 only in that series). I have no
> reason to believe that Fermi+ are subst
OpenGL ES 3.x contains a table of sized internal formats and their
required properties. In particular, each format is marked as
"Color Renderable" or "Texture Filterable".
This patch introduces two functions that can be used to query the
information from that table.
Signed-off-by: Kenneth Graunk
Hello,
On 09:55 PM - Mar 15 2016, Samuel Pitoiset wrote:
> Instead make use of constants to improve readability.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/drivers/nouveau/nvc0/nvc0_compute.c| 13 +-
> src/gallium/drivers/nouveau/nvc0/nvc0_context.h| 22 +
The OpenGLES GLSL 3.1 and OpenGL GLSL 4.3 specifications both remove
the requirement for the output and input interpolation qualifiers to
match.
Note: I'm changing desktop OpenGL to allow the interpolation to
differ, starting with 4.3, whereas the code previously used 4.4. (This
was first added in
Having two different variables for the driver constant buffer slot
is confusing and really useless.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir_driver.h | 3 +--
.../drivers/nouveau/codegen/nv50_ir_lowering_nv50.cpp| 4 ++--
.../drivers/nouvea
Because the rules for sRGB are so insane, we change brw_blorp_miptrees
to take decode_srgb and encode_srgb flags, which control linearization
of the source and destination separately.
This should make it easy to implement whatever crazy combination of
rules people throw at us. For now, it should
---
src/gallium/auxiliary/postprocess/pp_colors.h | 3 +++
src/gallium/auxiliary/postprocess/pp_mlaa.h | 6 ++
2 files changed, 9 insertions(+)
diff --git a/src/gallium/auxiliary/postprocess/pp_colors.h
b/src/gallium/auxiliary/postprocess/pp_colors.h
index a79858e..76c4ab4 100644
--- a/src
On Fri, Mar 18, 2016 at 2:32 PM, Francisco Jerez
wrote:
> Jason Ekstrand writes:
>
> > ---
> > src/compiler/Makefile.sources | 1 +
> > src/compiler/glsl/glsl_parser_extras.cpp | 1 +
> > src/compiler/glsl/ir_optimization.h| 1 +
> > src/compiler/glsl/propagate_inva
I have pushed this patch on top of the compute changes for nv50.
On 03/13/2016 10:11 PM, Pierre Moreau wrote:
Signed-off-by: Pierre Moreau
---
src/gallium/drivers/nouveau/nv50/nv50_state_validate.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/nouveau/nv50/nv50_stat
On 2016-03-10 17:24:33, Ilia Mirkin wrote:
> On Thu, Mar 10, 2016 at 7:59 PM, Kenneth Graunke
> wrote:
> > I need to use this in multiple source files.
> >
> > Signed-off-by: Kenneth Graunke
> > ---
> > src/mesa/drivers/dri/i965/brw_state.h | 20
> > src/mesa/drivers/dr
From: Nicolai Hähnle
---
src/mesa/state_tracker/st_cb_texturebarrier.c | 25 -
1 file changed, 24 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_cb_texturebarrier.c
b/src/mesa/state_tracker/st_cb_texturebarrier.c
index 6319b625..fecba65 100644
--- a
From: Nicolai Hähnle
This fixes arb_shader_image_load_store-host-mem-barrier.
---
src/gallium/drivers/radeonsi/si_state.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/radeonsi/si_state.c
index 6dcd532..4
From: Nicolai Hähnle
---
src/gallium/include/pipe/p_defines.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/include/pipe/p_defines.h
b/src/gallium/include/pipe/p_defines.h
index 90af7a7..8257b4a 100644
--- a/src/gallium/include/pipe/p_defines.h
+++ b/src/gallium/include/pipe/p
Hi,
these patches apply on top of my ARB_shader_image_load_store series. Together,
they fix a few remaining fails with piglit's
arb_shader_image_load_store-host-mem-barrier.
You can see them in context at https://cgit.freedesktop.org/~nh/mesa/log/?h=ssbo
The basic assumption for how barrier bi
Fix clang build error.
CXX codegen/nv50_ir_lowering_nvc0.lo
codegen/nv50_ir_lowering_nvc0.cpp:1783:42: error: invalid suffix 'd' on
floating constant
Value *zero = bld.loadImm(NULL, 0.0d);
^
Fixes: c1e4a6bfbf01 ("nv50,nvc0: handle SQRT loweri
On Tue, 2016-03-15 at 12:07 -0700, Kenneth Graunke wrote:
> Now that we implement tessellation shaders, the TES might be the last
> stage enabled. If it's outputting points, then the primitive type
> reaching the SF is points. We need to account for this.
>
> Caught by Ilia Mirkin.
>
> Signed-o
---
src/mesa/state_tracker/st_cb_drawpixels.c| 10 ++
src/mesa/state_tracker/st_cb_drawpixels_shader.c | 10 +-
2 files changed, 19 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_cb_drawpixels.c
b/src/mesa/state_tracker/st_cb_drawpixels.c
index 51d4ae
Am 18.03.2016 um 03:20 schrieb Jason Anderssen:
> Hi,
>
> This is probably a very simple question I hope, using scons, what is the
> easiest way to create a debug build, the Doc on the website I think only
> refers to Make-Config ?
> I am also cross compiling for windows.
>
you control that with
Reviewed-by: Eduardo Lima Mitev
On 03/17/2016 01:01 AM, Kenneth Graunke wrote:
We weren't printing this for some reason.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_state_dump.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/d
This avoids using magic numbers for the driver constbuf slot which
is always 15 except for compute shaders on gk104+ where the slot 0
is used.
For gk104+, some special compute-related values like the thread
index are uploaded to screen->parm which is currently bound on c0.
Signed-off-by: Samuel P
According to the ES 3.0 and GL 4.4 specifications, glBlitFramebuffer
is supposed to perform sRGB decoding and encoding whenever sRGB formats
are in use. The ES 3.0 specification is completely clear, and has
always stated this.
However, the GL specification has changed behavior in 4.1, 4.2, and
4.
Reviewed-by: Ilia Mirkin
On Fri, Mar 18, 2016 at 9:34 PM, Vinson Lee wrote:
> Fix clang build error.
>
> CXX codegen/nv50_ir_lowering_nvc0.lo
> codegen/nv50_ir_lowering_nvc0.cpp:1783:42: error: invalid suffix 'd' on
> floating constant
> Value *zero = bld.loadImm(NULL, 0.0d);
>
---
src/gallium/auxiliary/util/u_simple_shaders.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/util/u_simple_shaders.c
b/src/gallium/auxiliary/util/u_simple_shaders.c
index 7ffb271..76950a1 100644
--- a/src/gallium/auxiliary/util/u_simple_shaders.c
+
---
src/gallium/auxiliary/hud/hud_context.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/auxiliary/hud/hud_context.c
b/src/gallium/auxiliary/hud/hud_context.c
index fb99834..4673458 100644
--- a/src/gallium/auxiliary/hud/hud_context.c
+++ b/src/gallium/auxiliary/hud/hud_context
Previously, we looked at the bound textures (via the pipe_sampler_views)
to determine texture dimensions (1D/2D/3D/etc) and datatype (float vs.
int). But this could fail in out of memory conditions. If we failed to
allocate a texture and didn't create a pipe_sampler_view, we'd default
to using 0
---
src/mesa/state_tracker/st_mesa_to_tgsi.c | 22 ++
1 file changed, 22 insertions(+)
diff --git a/src/mesa/state_tracker/st_mesa_to_tgsi.c
b/src/mesa/state_tracker/st_mesa_to_tgsi.c
index 8a12ce4..7a686b1 100644
--- a/src/mesa/state_tracker/st_mesa_to_tgsi.c
+++ b/src/mesa/
v2: support both TGSI_TEXTURE_2D and _RECT
---
src/mesa/state_tracker/st_cb_drawpixels.c| 10 ++
src/mesa/state_tracker/st_cb_drawpixels_shader.c | 10 +-
2 files changed, 19 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_cb_drawpixels.c
b/src/mesa/st
In June 2015, Rob Clark started updating the tgsi utility code to emit
SVIEW declarations in various shaders (for polygon stipple, blitting,
etc). These patches do the same for the Mesa state tracker.
The VMware driver will use this.
v2: support both TGSI_TEXTURE_2D and _RECT
---
src/mesa/state
---
src/gallium/tests/graw/quad-tex.c| 1 +
src/gallium/tests/graw/tex-srgb.c| 1 +
src/gallium/tests/graw/tex-swizzle.c | 1 +
3 files changed, 3 insertions(+)
diff --git a/src/gallium/tests/graw/quad-tex.c
b/src/gallium/tests/graw/quad-tex.c
index 5f90166..8a9d1b8 100644
--- a/src/gall
Seems reasonable to me... check to make sure that Unreal Engine 4
keeps working, it uses texture views.
FWIW I made a misstep when implementing texture views in that I reused
the ->surface_format thing, which was probably wrong. Perhaps I should
have added a strb->rtt_format or something... dunno.
65 matches
Mail list logo