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.
---
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
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
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
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/
---
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
---
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
+
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);
>
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.
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
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
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
---
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
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
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
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
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/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
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
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
---
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
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
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
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
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 +
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
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
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 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 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 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
>
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 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
>
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
-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
---
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
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
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
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
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
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
---
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
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
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
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
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_
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/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
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.
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
---
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/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_
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(+)
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
https://bugs.freedesktop.org/show_bug.cgi?id=94452
Kenneth Graunke changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
65 matches
Mail list logo
