> Sure. Unfortunately, the series does not apply cleanly on ToT master:
>
> % git am p[12].txt
> Applying: scons: whitespace cleanup
> Using index info to reconstruct a base tree...
> error: patch failed: src/gallium/state_trackers/wgl/SConscript:12
> error:
From: Kristian Høgsberg Kristensen
Signed-off-by: Kristian Høgsberg Kristensen
---
src/compiler/.gitignore | 4
src/compiler/nir/tests/.gitignore| 1 +
src/mesa/drivers/dri/i965/.gitignore | 3 +++
src/util/.gitignore
On Tue, May 24, 2016 at 7:48 PM, Kristian Høgsberg
wrote:
> On Tue, May 24, 2016 at 5:01 PM, Jason Ekstrand
> wrote:
> > The buffer_range_* arrays are indexed by buffer index not element index.
>
> Reviewed-by: Kristian Høgsberg
>
From: Dave Airlie
According to GL4.5 spec:
An INVALID_OPERATION error is generated if any part of the speci-
fied buffer range is mapped with MapBufferRange or MapBuffer (see sec-
tion 6.3), unless it was mapped with MAP_PERSISTENT_BIT set in the Map-
BufferRange access
On Tue, May 24, 2016 at 8:19 PM, Ilia Mirkin wrote:
> On Tue, May 24, 2016 at 10:41 PM, Kristian Høgsberg
> wrote:
>> From: Kristian Høgsberg Kristensen
>>
>> GL_KHR_robustness adds the GL_CONTEXT_LOST error and five new entry
>>
Hi Brian,
On Tuesday, May 24, 2016 11:29:56 Brian Paul wrote:
> On 05/24/2016 12:41 AM, mathias.froehl...@gmx.net wrote:
> > From: Mathias Fröhlich
> >
> > Hi all,
> >
> > following a series with performance improvements
> > for cpu/draw bound applications. This part
On Tuesday, May 24, 2016 17:42:17 Roland Scheidegger wrote:
> Am 24.05.2016 um 08:41 schrieb mathias.froehl...@gmx.net:
> > From: Mathias Fröhlich
> >
> > The aim is to replace the CoordReplace array by
> > a bitfield. Until all drivers are converted,
> > establish the
On 25.05.2016 09:09, Mike Lothian wrote:
> Do you need the DRM version number if you'll be displaying the kernel
> version anyway?
Yes, because the DRM version depends on the kernel driver being used.
The patch is
Reviewed-by: Michel Dänzer
--
Earthling Michel Dänzer
On Tue, May 24, 2016 at 10:41 PM, Kristian Høgsberg wrote:
> From: Kristian Høgsberg Kristensen
>
> GL_KHR_robustness adds the GL_CONTEXT_LOST error and five new entry
> points that we already implement. This patch adds a new dispatch
> table that returns
---
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index feea6ca..5d831e5 100644
--- a/src/mesa/drivers/dri/i965/intel_extensions.c
+++
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 33
1 file changed, 29 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 51da3bd..a19ece7 100644
---
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 16 +---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index a19ece7..16fd7d6 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
+++
This offset is used for packing.
---
src/compiler/nir/nir.h| 6 ++
src/compiler/nir/nir_intrinsics.h | 8
src/compiler/nir/nir_lower_io.c | 8
3 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index
---
docs/GL3.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 2dff3cd..edc04a1 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -193,11 +193,11 @@ GL 4.4, GLSL 4.40:
GL_MAX_VERTEX_ATTRIB_STRIDE DONE (all
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index e0d88c6..51da3bd 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
+++
Here we add a new param to the type_size functions in order to pass
in the size of a varying once packing is taken into account.
---
src/compiler/nir/nir.h | 6 +++--
src/compiler/nir/nir_lower_io.c| 35 +-
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 17 +++--
src/mesa/drivers/dri/i965/brw_fs.h | 5 +++--
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 29 -
3 files changed, 34 insertions(+), 17 deletions(-)
diff --git
This will be used to store the total number of components used at this location
when packing via ARB_enhanced_layouts.
---
src/compiler/glsl/ir.h | 5 +++
src/compiler/glsl/link_varyings.cpp | 74 -
src/compiler/glsl/linker.cpp| 2 +
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 38 +++-
1 file changed, 33 insertions(+), 5 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index ce61898..e0d88c6 100644
---
---
src/mesa/drivers/dri/i965/brw_fs.h | 2 +-
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 21 +
2 files changed, 18 insertions(+), 5 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.h
b/src/mesa/drivers/dri/i965/brw_fs.h
index d5d7a77..b5a4fc8 100644
---
---
src/compiler/nir/nir.h| 4 ++--
src/compiler/nir/nir_lower_io.c | 25 -
src/mesa/drivers/dri/i965/brw_nir.c | 12 +++-
src/mesa/state_tracker/st_glsl_to_nir.cpp | 3 +++
4 files changed, 36 insertions(+), 8 deletions(-)
There are a number of packing tests already in piglit but I have a few
more on the way.
This series does not add support for doubles as there is currently a
doubles bug with explicit locations that I've pointed out to the Igalia
guys. Samuel is working on a fix for this after which I will finish
Series is
Reviewed-by: Ilia Mirkin
On Tue, May 24, 2016 at 8:43 PM, Brian Paul wrote:
> Print "GEOM" instead of "2", for example.
>
> v2: also update the text parsing code, per Ilia.
> ---
> src/gallium/auxiliary/tgsi/tgsi_dump.c | 3 +++
>
On Tue, May 24, 2016 at 7:47 PM, Ilia Mirkin wrote:
>
> On May 24, 2016 10:25 PM, "Kristian Høgsberg" wrote:
>>
>> On Tue, May 24, 2016 at 5:22 PM, Ilia Mirkin wrote:
>> > Sorry to be pedantic, but you're enabling the ext for
On Tue, May 24, 2016 at 5:01 PM, Jason Ekstrand wrote:
> The buffer_range_* arrays are indexed by buffer index not element index.
Reviewed-by: Kristian Høgsberg
> ---
> src/mesa/drivers/dri/i965/brw_draw_upload.c | 6 --
> 1 file changed, 4
On May 24, 2016 10:25 PM, "Kristian Høgsberg" wrote:
>
> On Tue, May 24, 2016 at 5:22 PM, Ilia Mirkin wrote:
> > Sorry to be pedantic, but you're enabling the ext for gles1.1, but the
new
> > entrypoint appear to only apply to gles2. I think you also
From: Kristian Høgsberg Kristensen
GL_KHR_robustness adds the GL_CONTEXT_LOST error and five new entry
points that we already implement. This patch adds a new dispatch
table that returns GL_CONTEXT_LOST from all entry points and
implements the GL_LOSE_CONTEXT_ON_RESET by
On Tue, May 24, 2016 at 5:22 PM, Ilia Mirkin wrote:
> Sorry to be pedantic, but you're enabling the ext for gles1.1, but the new
> entrypoint appear to only apply to gles2. I think you also want a es1="1.1"
> or something along those lines.
The extension requires 2.0, so
On Tue, May 24, 2016 at 6:04 PM, Matt Turner wrote:
> On Tue, May 24, 2016 at 4:58 PM, Kristian Høgsberg wrote:
>> diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
>> b/src/mesa/drivers/dri/i965/intel_extensions.c
>> index feea6ca..b8d7517 100644
On Tue, May 17, 2016 at 06:39:02PM +0100, Plamena Manolova wrote:
> eglCreatePbufferSurface should generate an EGL_BAD_MATCH error if:
> 1: The EGL_TEXTURE_FORMAT attribute is EGL_NO_TEXTURE and EGL_TEXTURE_TARGET
> is something other than EGL_NO_TEXTURE
> 2: EGL_TEXTURE_FORMAT is something other
On Tuesday, May 24, 2016 1:37:46 AM PDT Jordan Justen wrote:
> In d8347f12ead89c5a58f69ce9283a54ac8487159c, we added support for
> skipping SIMD8 generation when the program local size is too large for
> SIMD8 to be usable. This change was missed in that commit.
>
> This bug would impact gen7
On Tue, May 24, 2016 at 4:58 PM, Kristian Høgsberg wrote:
> diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
> b/src/mesa/drivers/dri/i965/intel_extensions.c
> index feea6ca..b8d7517 100644
> --- a/src/mesa/drivers/dri/i965/intel_extensions.c
> +++
Jason Ekstrand writes:
> Does this mean we can delete the field from brw_fs_generator?
>
Almost, there is still one use left in fire_fb_write() (for the dual
source blend last-rt hack) which we will be able to get rid of
eventually, but it cannot simply be replaced with
Kenneth Graunke writes:
> On Tuesday, May 24, 2016 5:27:59 PM PDT Francisco Jerez wrote:
>> Jason Ekstrand writes:
>>
>> > On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
>> > wrote:
>> >
>> >> Due to a Gen7-specific
On Tue, May 17, 2016 at 11:50:28AM -0700, Matt Turner wrote:
> On Mon, May 16, 2016 at 4:27 PM, Ardinartsev Nikita
> wrote:
> > Fixes regression introduced by af5ca43f2676bff7499f93277f908b681cb821d0
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=95419
>
> Thank
---
src/gallium/auxiliary/tgsi/tgsi_strings.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_strings.h
b/src/gallium/auxiliary/tgsi/tgsi_strings.h
index 031d322..9a9362e 100644
--- a/src/gallium/auxiliary/tgsi/tgsi_strings.h
+++
Print "GEOM" instead of "2", for example.
v2: also update the text parsing code, per Ilia.
---
src/gallium/auxiliary/tgsi/tgsi_dump.c | 3 +++
src/gallium/auxiliary/tgsi/tgsi_text.c | 22 ++
2 files changed, 25 insertions(+)
diff --git
On Tuesday, May 24, 2016 5:27:59 PM PDT Francisco Jerez wrote:
> Jason Ekstrand writes:
>
> > On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
> > wrote:
> >
> >> Due to a Gen7-specific hardware bug native 32-wide instructions get
> >> the lower 16
Jason Ekstrand writes:
> On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
> wrote:
>
>> Due to a Gen7-specific hardware bug native 32-wide instructions get
>> the lower 16 bits of the execution mask applied incorrectly to both
>> halves of the
On 05/24/2016 04:58 PM, Kristian Høgsberg wrote:
> From: Kristian Høgsberg Kristensen
>
> As per GL_KHR_robustness, we have to return GL_CONTEXT_LOST from all
> entry points when we lose a context. We do this by creating a new
> dispatch table and setting that when we learn
Sorry to be pedantic, but you're enabling the ext for gles1.1, but the new
entrypoint appear to only apply to gles2. I think you also want a es1="1.1"
or something along those lines.
On May 24, 2016 8:02 PM, "Kristian Høgsberg" wrote:
> From: Kristian Høgsberg Kristensen
Jason Ekstrand writes:
> On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
> wrote:
>
>> Alternatively we could have extended the current semantics to 32-wide
>> mode by changing brw_broadcast() to emit multiple indexed MOV
>> instructions in the
Jason Ekstrand writes:
> On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
> wrote:
>
>> This generalizes the current fs_inst::force_sechalf flag to allow
>> specifying channel enable groups other than 0 or 8. At some point it
>> will likely make
Do you need the DRM version number if you'll be displaying the kernel
version anyway?
On Wed, 25 May 2016 at 00:09 Marek Olšák wrote:
> From: Marek Olšák
>
> Example:
> Gallium 0.4 on AMD TONGA (DRM 3.2.0 / 4.5.0, LLVM 3.9.0)
>
> My kernel version is
Signed-off-by: Ilia Mirkin
---
docs/relnotes/11.3.0.html | 1 +
1 file changed, 1 insertion(+)
diff --git a/docs/relnotes/11.3.0.html b/docs/relnotes/11.3.0.html
index 8d6caa2..e6ed538 100644
--- a/docs/relnotes/11.3.0.html
+++ b/docs/relnotes/11.3.0.html
@@ -61,6 +61,7 @@
From: Kristian Høgsberg Kristensen
As per GL_KHR_robustness, we have to return GL_CONTEXT_LOST from all
entry points when we lose a context. We do this by creating a new
dispatch table and setting that when we learn that we've lost the
context.
With the GL_CONTEXT_LOST
Signed-off-by: Ilia Mirkin
---
This addresses the feedback I got after pushing the enablement patch.
docs/relnotes/11.3.0.html | 1 +
src/compiler/glsl/builtin_variables.cpp | 10 --
src/compiler/glsl/glcpp/glcpp-parse.y | 2 ++
The buffer_range_* arrays are indexed by buffer index not element index.
---
src/mesa/drivers/dri/i965/brw_draw_upload.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw_upload.c
b/src/mesa/drivers/dri/i965/brw_draw_upload.c
index
Print "GEOM" instead of "2", for example.
---
src/gallium/auxiliary/tgsi/tgsi_dump.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_dump.c
b/src/gallium/auxiliary/tgsi/tgsi_dump.c
index 0d8bd1b..d59b7ff 100644
--- a/src/gallium/auxiliary/tgsi/tgsi_dump.c
---
src/gallium/auxiliary/tgsi/tgsi_strings.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_strings.h
b/src/gallium/auxiliary/tgsi/tgsi_strings.h
index 031d322..9a9362e 100644
--- a/src/gallium/auxiliary/tgsi/tgsi_strings.h
+++
Please adjust the parser to read this in as well.
On Tue, May 24, 2016 at 7:45 PM, Brian Paul wrote:
> Print "GEOM" instead of "2", for example.
> ---
> src/gallium/auxiliary/tgsi/tgsi_dump.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git
From: Rob Clark
Split standalone glsl_compiler into a libstandalone.la and a thin
main.cpp. This way drivers can re-use the glsl standalone frontend in
their own standalone compilers.
Signed-off-by: Rob Clark
---
v2: fix scons build and fix
On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
wrote:
> Alternatively we could have extended the current semantics to 32-wide
> mode by changing brw_broadcast() to emit multiple indexed MOV
> instructions in the generator copying the selected value to all
> destination
Reviewed-by: Jordan Justen
On 2016-05-24 15:12:25, Matt Turner wrote:
> Coverity warns in multiple places about the potential for division by
> zero, caused by this function's default case.
>
> Cc: Francisco Jerez
> ---
>
I sent a few fairly minor comments that I'd like to see addressed. Other
than those,
Reviewed-by: Jason Ekstrand
On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
wrote:
> This should be enough to get the FS generator emitting 32-wide code
> for
On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
wrote:
> This generalizes the current fs_inst::force_sechalf flag to allow
> specifying channel enable groups other than 0 or 8. At some point it
> will likely make sense to fix the vec4 generator to support arbitrary
>
From: Dave Airlie
The spec says gl_NextBuffer and gl_SkipComponents need to be
returned to userspace in the program interface queries.
We currently throw those away, this requires a complete piglit
run to make sure no drivers fallover due to the extra varyings.
This fixes:
From: Marek Olšák
Example:
Gallium 0.4 on AMD TONGA (DRM 3.2.0 / 4.5.0, LLVM 3.9.0)
My kernel version is pretty long already (4.5.0-amd-01025-g32791c1)
and adding "kernel" into the string would make too it long for glxinfo
to display.
---
On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
wrote:
> Due to a Gen7-specific hardware bug native 32-wide instructions get
> the lower 16 bits of the execution mask applied incorrectly to both
> halves of the instruction, so the MOV trick we currently use wouldn't
>
Reviewed-by: Ian Romanick
On 05/24/2016 03:39 PM, Nanley Chery wrote:
> From: Nanley Chery
>
> Commit eda15abd84af575d3bde432e2163e30d743a7c87 , unintentionally
> advertised these extensions in ES1 contexts. Undo this error.
>
>
According to the EGL specifications before binding an API
we must check whether it's supported first. If not eglBindAPI
should return EGL_FALSE and generate a EGL_BAD_PARAMETER error.
Signed-off-by: Plamena Manolova
---
src/egl/main/eglapi.c | 65
Reviewed-by: Ilia Mirkin
On Tue, May 24, 2016 at 6:39 PM, Nanley Chery wrote:
> From: Nanley Chery
>
> Commit eda15abd84af575d3bde432e2163e30d743a7c87 , unintentionally
> advertised these extensions in ES1 contexts. Undo
From: Nanley Chery
Commit eda15abd84af575d3bde432e2163e30d743a7c87 , unintentionally
advertised these extensions in ES1 contexts. Undo this error.
Signed-off-by: Nanley Chery
---
src/mesa/main/extensions_table.h | 4 ++--
1 file changed, 2
Does this mean we can delete the field from brw_fs_generator?
On Tue, May 24, 2016 at 12:18 AM, Francisco Jerez
wrote:
> Most of these are bugs because the intended execution size of an
> instruction and the dispatch width of the shader aren't necessarily
> the same
On Tue, May 24, 2016 at 2:05 AM, Michael Schellenberger <
mschellenbergerco...@googlemail.com> wrote:
> Hi curro,
>
> Am 24.05.2016 um 09:18 schrieb Francisco Jerez:
> > This implements some simple helper functions that can be used to
> > specify the group of channel enable signals and
On Tue, May 24, 2016 at 2:05 AM, Michael Schellenberger <
mschellenbergerco...@googlemail.com> wrote:
> Hi curro,
>
> Am 24.05.2016 um 09:18 schrieb Francisco Jerez:
> > This implements some simple helper functions that can be used to
> > specify the group of channel enable signals and
Coverity warns in multiple places about the potential for division by
zero, caused by this function's default case.
Cc: Francisco Jerez
---
src/mesa/drivers/dri/i965/brw_reg.h | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git
On Tue, May 24, 2016 at 2:30 PM, Kristian Høgsberg wrote:
> diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
> b/src/mesa/drivers/dri/i965/intel_extensions.c
> index feea6ca..b8d7517 100644
> --- a/src/mesa/drivers/dri/i965/intel_extensions.c
> +++
From: Rob Clark
Some optimizations, like converting integer multiply/divide into left/
right shifts, have additional constraints on the search expression.
Like requiring that a variable is a constant power of two. Support
these cases by allowing a fxn name to be
From: Kristian Høgsberg Kristensen
As per GL_KHR_robustness, we have to return GL_CONTEXT_LOST from all
entry points when we lose a context. We do this by creating a new
dispatch table and setting that when we learn that we've lost the
context.
With the GL_CONTEXT_LOST
On Tue, May 24, 2016 at 3:41 PM, Rob Clark wrote:
> On Mon, May 16, 2016 at 3:16 PM, Jason Ekstrand wrote:
>> On Mon, May 16, 2016 at 9:09 AM, Rob Clark wrote:
>>>
>>> On Mon, May 16, 2016 at 10:45 AM, Jason Ekstrand
On Tue, May 24, 2016 at 1:23 PM, Matt Turner wrote:
> ---
> src/intel/isl/isl_surface_state.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/isl/isl_surface_state.c
> b/src/intel/isl/isl_surface_state.c
> index dd88531..cc9c94a 100644
>
On Tue, May 24, 2016 at 10:16 PM, Emil Velikov wrote:
> On 24 May 2016 at 17:38, Marek Olšák wrote:
>> On Tue, May 24, 2016 at 4:32 PM, Emil Velikov
>> wrote:
>>> From: Emil Velikov
>>>
>>> Using
On 05/24/2016 02:05 PM, Giuseppe Bilotta wrote:
On Tue, May 24, 2016 at 6:13 PM, Brian Paul wrote:
For both,
Reviewed-by: Brian Paul
Do you need me to push these for you?
Well, AFAIK I don't have write access to the mesa git tree so yes
please ;-)
---
src/mesa/drivers/dri/i965/brw_eu_emit.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c
b/src/mesa/drivers/dri/i965/brw_eu_emit.c
index d31943d..20fdfc1 100644
--- a/src/mesa/drivers/dri/i965/brw_eu_emit.c
+++
---
src/intel/isl/isl_surface_state.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/intel/isl/isl_surface_state.c
b/src/intel/isl/isl_surface_state.c
index cc9c94a..e96d3b0 100644
--- a/src/intel/isl/isl_surface_state.c
+++ b/src/intel/isl/isl_surface_state.c
@@
---
src/mesa/drivers/dri/i965/brw_misc_state.c | 1 +
src/mesa/drivers/dri/i965/gen6_depth_state.c | 1 +
src/mesa/drivers/dri/i965/gen7_misc_state.c | 1 +
src/mesa/drivers/dri/i965/gen8_depth_state.c | 1 +
4 files changed, 4 insertions(+)
diff --git
---
src/compiler/nir/nir_lower_clamp_color_outputs.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/compiler/nir/nir_lower_clamp_color_outputs.c
b/src/compiler/nir/nir_lower_clamp_color_outputs.c
index 68bfbed..ab211a4 100644
---
---
src/intel/isl/isl_surface_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/isl/isl_surface_state.c
b/src/intel/isl/isl_surface_state.c
index dd88531..cc9c94a 100644
--- a/src/intel/isl/isl_surface_state.c
+++ b/src/intel/isl/isl_surface_state.c
@@ -90,7
---
src/compiler/spirv/vtn_alu.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/compiler/spirv/vtn_alu.c b/src/compiler/spirv/vtn_alu.c
index 8b9a63c..5730ca4 100644
--- a/src/compiler/spirv/vtn_alu.c
+++ b/src/compiler/spirv/vtn_alu.c
@@ -356,6 +356,7 @@ vtn_handle_alu(struct
On 24 May 2016 at 17:38, Marek Olšák wrote:
> On Tue, May 24, 2016 at 4:32 PM, Emil Velikov
> wrote:
>> From: Emil Velikov
>>
>> Using the macro to set the version is wrong and ill-advised. Please don't
>> do it.
>>
>> Cc:
On Tue, May 24, 2016 at 6:13 PM, Brian Paul wrote:
> For both,
> Reviewed-by: Brian Paul
>
> Do you need me to push these for you?
Well, AFAIK I don't have write access to the mesa git tree so yes
please ;-) (Possibly with the extra commit lines with the
Reviewed-by: Ilia Mirkin
On Tue, May 24, 2016 at 4:03 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> When lowering, we always want to use the clip dist varying.
>
> Signed-off-by: Dave Airlie
> ---
>
From: Dave Airlie
When lowering, we always want to use the clip dist varying.
Signed-off-by: Dave Airlie
---
src/compiler/glsl/lower_distance.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/compiler/glsl/lower_distance.cpp
Indexed primitives were always using cut-aware primitive assembly,
whether primitive_restart was enabled or not. Correctly pass down
primitive_restart and select optimized PA when possible.
---
src/gallium/drivers/swr/rasterizer/core/api.cpp|2 ++
Coverity complains that the computed sizes can lead to negative lengths
passed to memcpy. If that happens we've been handed invalid arguments
anyway, so just bomb out.
The funky "0%s" is because the size string for the variable-length part
of the request is of the form "+ safe_pad() ...", and a
On Mon, May 16, 2016 at 3:16 PM, Jason Ekstrand wrote:
> On Mon, May 16, 2016 at 9:09 AM, Rob Clark wrote:
>>
>> On Mon, May 16, 2016 at 10:45 AM, Jason Ekstrand
>> wrote:
>> >
>> > On May 16, 2016 7:29 AM, "Rob Clark"
On Fri, May 20, 2016 at 6:35 PM, Kenneth Graunke wrote:
> For now, only enable it on platforms that actually support ETC2.
>
> At this point, Broadwell is only failing 5 (out of 8358) dEQP tests:
> dEQP-GLES31.functional.copy_image.non_compressed.viewclass_32_bits.
>
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 05/20/2016 06:03 PM, Timothy Arceri wrote:
> On Fri, 2016-05-20 at 00:26 -0700, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> Fixes the following dEQP tests on SKL:
>>
>> dEQP-
>> GLES31.functional.separate_shader.validation.varying.mismatch_qualifi
>>
On 05/23/2016 07:45 AM, Marek Olšák wrote:
> I think it would be better to put stuff like this only into the
> renderer string of drivers which use LLVM. The majority of drivers
> don't care about the LLVM version.
Yes. In fact, I *really* don't want the LLVM version to show up in the
glxinfo
On 05/20/2016 05:46 PM, Timothy Arceri wrote:
> On Fri, 2016-05-20 at 00:25 -0700, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> There's going to be a second validate_io function, and checking the
>> same
>> thing twice is silly.
>
> I think we should just do this
On Tue, May 24, 2016 at 1:47 PM, Nanley Chery wrote:
> On Mon, May 23, 2016 at 02:07:41PM -0700, Jason Ekstrand wrote:
>> On Mon, May 23, 2016 at 10:41 AM, Nanley Chery
>> wrote:
>>
>> > On Fri, May 20, 2016 at 06:12:34PM -0700, Jason Ekstrand wrote:
On Mon, May 23, 2016 at 02:07:41PM -0700, Jason Ekstrand wrote:
> On Mon, May 23, 2016 at 10:41 AM, Nanley Chery
> wrote:
>
> > On Fri, May 20, 2016 at 06:12:34PM -0700, Jason Ekstrand wrote:
> > > This little series effectively moves the surface format table from
> > >
On 05/24/2016 12:41 AM, mathias.froehl...@gmx.net wrote:
From: Mathias Fröhlich
Hi all,
following a series with performance improvements
for cpu/draw bound applications. This part makes
more use of the bitmask/ffs technique for iterating
a set of enabled items. The
Reviewed-by: Bruce Cherniak minus the knob.h change
to disable cut-aware primitive assembler. New change for that coming today.
> On May 20, 2016, at 2:08 PM, Rowley, Timothy O
> wrote:
>
> Bruce, is cut-aware needed for primitive
On Tue, May 24, 2016 at 03:32:44PM +0100, Emil Velikov wrote:
> From: Emil Velikov
>
> This reverts commit 4e2c9a04354b6b133845b8b93c0c5d34261a91d0.
>
> The solution was incomplete and fragile. An alternative one is coming
> shortly.
Tested-by: Tom Stellard
The series looks good overall.
Patches 2-8, 10-14:
Reviewed-by: Marek Olšák
Marek
On Tue, May 24, 2016 at 4:32 PM, Emil Velikov wrote:
> From: Emil Velikov
>
> This allows clear and easy communication between the
On Tue, May 24, 2016 at 4:32 PM, Emil Velikov wrote:
> From: Emil Velikov
>
> Using the macro to set the version is wrong and ill-advised. Please don't
> do it.
>
> Cc: Marek Olšák
> Signed-off-by: Emil Velikov
On 05/24/2016 02:43 AM, Giuseppe Bilotta wrote:
The main patch is the second, the first is just a cleanup of EOL whitespace.
Giuseppe Bilotta (2):
scons: whitespace cleanup
scons: support 2.5.0
SConstruct| 4 ++--
scons/custom.py
1 - 100 of 219 matches
Mail list logo