Kenneth Graunke kenn...@whitecape.org writes:
The code in glsl_to_nir is entirely dead, as we translate from GLSL to
NIR at link time, when there isn't a _mesa_glsl_parse_state to pass,
so every caller passes NULL.
glsl_to_nir seems like the wrong place to try and create the shader
compiler
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Mesa 10.4.6 has been released. Mesa 10.4.6 is a bug fix release
fixing bugs since the 10.4.5 release, (see below for a list of
changes).
The tag in the git repository for Mesa 10.4.6 is 'mesa-10.4.6'.
Mesa 10.4.6 is available for download at
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Fri, Mar 6, 2015 at 4:53 PM, Tom Stellard thomas.stell...@amd.com wrote:
---
src/gallium/drivers/radeonsi/si_compute.c | 3 ++-
src/gallium/drivers/radeonsi/si_shader.c | 1 +
src/gallium/drivers/radeonsi/si_shader.h | 1 +
3 files
in different fragment shaders. This also applies to a case when gl_FragCoord
is redeclared with no layout qualifiers in one fragment shader and not
declared but used in other fragment shader.
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
Khronos Bug#12957
Cc: 10.5
It looks like this has existed since:
commit f5a477ab76b6e0b268387699cd2253a43db0dfae
Author: Ian Romanick ian.d.roman...@intel.com
Date: Mon Dec 16 11:54:08 2013 -0800
meta: Refactor shader generation code out of mipmap generation path
Valgrind was complaining on the piglit test:
On 03/06/2015 01:58 PM, Chris Wilson wrote:
In order to facilitate the concurrency offered by triple buffering and to
offset the latency induced by swapping via an external process, which
may incur extra rendering itself, only throttle to the previous frame
and not the last. The second issue
On Fri, Feb 27, 2015 at 05:34:49PM +0200, Francisco Jerez wrote:
The generate_untyped_*() methods do nothing useful other than calling
the corresponding function from brw_eu_emit.c. The calls to
brw_mark_surface_used() will go away too in a future commit.
---
On Fri, Feb 27, 2015 at 05:34:46PM +0200, Francisco Jerez wrote:
And calculate the message response size based on the number of
components rather than the other way around. This simplifies their
interface somewhat and allows the caller to request a writeback
message with more than one vector
On Thu, Mar 05, 2015 at 11:49:54AM -0800, Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
Inspired by Damien's recent libdrm changes.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: Damien Lespiau damien.lesp...@intel.com
For the whole series (not that my r-b tag
Piglit's spec/glsl-1.20/compiler/structure-and-array-operations/
array-selection.vert test contains the following code:
gl_Position = (pick_from_a_or_b ? a : b)[i];
where a and b are uniform vec4[2] variables.
ast_to_hir creates a temporary vec4[2] variable, conditional_tmp, and
generates an
Vertex shaders can have shader inputs where location happens to be
VARYING_SLOT_FACE. Without predicating this on the shader stage,
we suddenly end up with load_front_face intrinsics in vertex shaders,
which is nonsensical.
Fixes spec/arb_vertex_buffer_object/pos-array when using NIR for VS.
Nothing actually uses these, and the only caller of glsl_to_nir()
(brw_fs_nir.cpp) always passes NULL for the _mesa_glsl_parse_state
pointer, meaning they'll always be NULL and 0, respectively.
Just delete them.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
glsl_to_nir, tgsi_to_nir, and prog_to_nir all want to know whether the
driver supports native integers. Presumably other passes may as well.
Adding this to nir_shader_compiler_options is an easy way to provide
that information, as it's accessible via nir_shader::options.
Signed-off-by: Kenneth
The code in glsl_to_nir is entirely dead, as we translate from GLSL to
NIR at link time, when there isn't a _mesa_glsl_parse_state to pass,
so every caller passes NULL.
glsl_to_nir seems like the wrong place to try and create the shader
compiler options structure anyway - tgsi_to_nir,
The next commit needs to know the shader stage in glsl_to_nir().
To facilitate that, we pass the gl_shader rather than the raw exec_list
of instructions. This has both the exec_list and the stage.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/nir/glsl_to_nir.cpp
On 03/06/2015 05:34 AM, Emil Velikov wrote:
The C99 header exists on other platforms as well.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/glx/glxclient.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/glx/glxclient.h b/src/glx/glxclient.h
index a140c87..122ae5d
On Thu, Mar 05, 2015 at 02:38:44PM -0800, Ian Romanick wrote:
On 03/04/2015 10:28 AM, Chad Versace wrote:
That text does not appear in the GL spec. When I read the manpage alongside
the GL spec, to get a more complete context, I think the manpage contains
that phrase simply to contrast with
Acked-by: Jason Ekstrand jason.ekstr...@intel.com
On Mar 6, 2015 2:18 AM, Kenneth Graunke kenn...@whitecape.org wrote:
The code in glsl_to_nir is entirely dead, as we translate from GLSL to
NIR at link time, when there isn't a _mesa_glsl_parse_state to pass,
so every caller passes NULL.
On 06/03/15 14:26, Brian Paul wrote:
On 03/06/2015 05:34 AM, Emil Velikov wrote:
Replace the _MSC_VER = 1200 with defined (_MSC_VER) and compact if/else
statements. We require MSVC 2008 or later with commit 46110c5d564.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
When rendering to an fbo, even though it may be acting as a winsys
frontbuffer or just generally, we never throttle. However, when rendering
to an fbo, there is no natural frame boundary. Conventionally we use
SwapBuffers and glFinish, but potential callers avoid often glFinish for
being too heavy
When rendering to an fbo, even though it may be acting as a winsys
frontbuffer or just generally, we never throttle. However, when rendering
to an fbo, there is no natural frame boundary. Conventionally we use
SwapBuffers and glFinish, but potential callers avoid often glFinish for
being too heavy
I gave you an back on 3; I'll let Eric actually review it. The rest are
Reviewed-by: Jason Ekstrand jason.ekstr...@intel.com
On Mar 6, 2015 2:18 AM, Kenneth Graunke kenn...@whitecape.org wrote:
Piglit's spec/glsl-1.20/compiler/structure-and-array-operations/
array-selection.vert test contains
On 03/06/2015 05:34 AM, Emil Velikov wrote:
Replace the _MSC_VER = 1200 with defined (_MSC_VER) and compact if/else
statements. We require MSVC 2008 or later with commit 46110c5d564.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/util/macros.h | 8 +++-
1 file changed, 3
In order to facilitate the concurrency offered by triple buffering and to
offset the latency induced by swapping via an external process, which
may incur extra rendering itself, only throttle to the previous frame
and not the last. This doubles the maximum possible latency at the
benefit of
On 06/03/15 00:11, Mark Janes wrote:
Matt Turner matts...@gmail.com writes:
On Thu, Mar 5, 2015 at 4:54 AM, Jose Fonseca jfons...@vmware.com wrote:
Thanks for doing this. It looks great.
Reviewed-by: Jose Fonseca jfons...@vmware.com
Feel free to push my patch or I'll push it after you
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Mar 06, 2015 at 02:29:15PM +0200, Francisco Jerez wrote:
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Feb 27, 2015 at 05:34:48PM +0200, Francisco Jerez wrote:
Change brw_untyped_atomic() and
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Feb 27, 2015 at 05:34:48PM +0200, Francisco Jerez wrote:
Change brw_untyped_atomic() and brw_untyped_surface_read() to take the
surface index as a register instead of a constant and to use
brw_send_indirect_message() to emit the
The C99 header exists on other platforms as well.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/glx/glxclient.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/glx/glxclient.h b/src/glx/glxclient.h
index a140c87..122ae5d 100644
--- a/src/glx/glxclient.h
+++
On Fri, Mar 06, 2015 at 02:29:15PM +0200, Francisco Jerez wrote:
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Feb 27, 2015 at 05:34:48PM +0200, Francisco Jerez wrote:
Change brw_untyped_atomic() and brw_untyped_surface_read() to take the
surface index as a register instead
On Fri, Mar 06, 2015 at 02:46:51PM +0200, Francisco Jerez wrote:
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Mar 06, 2015 at 02:29:15PM +0200, Francisco Jerez wrote:
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Feb 27, 2015 at 05:34:48PM +0200,
On Fri, Feb 27, 2015 at 05:34:48PM +0200, Francisco Jerez wrote:
Change brw_untyped_atomic() and brw_untyped_surface_read() to take the
surface index as a register instead of a constant and to use
brw_send_indirect_message() to emit the indirect variant of send with
a dynamically calculated
Replace the _MSC_VER = 1200 with defined (_MSC_VER) and compact if/else
statements. We require MSVC 2008 or later with commit 46110c5d564.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/util/macros.h | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Feb 27, 2015 at 05:34:44PM +0200, Francisco Jerez wrote:
---
src/mesa/drivers/dri/i965/brw_eu.h | 19 ++--
src/mesa/drivers/dri/i965/brw_eu_emit.c | 58
++--
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Feb 27, 2015 at 05:34:47PM +0200, Francisco Jerez wrote:
This is currently not a problem because the vec4 visitor happens to
mask out unused components from the destination, but it might become
an issue when we start using atomics
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Feb 27, 2015 at 05:34:51PM +0200, Francisco Jerez wrote:
This doesn't actually enable untyped surface message sends from GRF
yet, the upcoming atomic counter and image intrinsic lowering code
will.
---
On Fri, Feb 27, 2015 at 05:34:44PM +0200, Francisco Jerez wrote:
---
src/mesa/drivers/dri/i965/brw_eu.h | 19 ++--
src/mesa/drivers/dri/i965/brw_eu_emit.c | 58
++--
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 55 +-
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/mesa/drivers/dri/i915/intel_buffer_objects.c | 19 ++-
src/mesa/drivers/dri/i965/intel_mipmap_tree.c| 5 +++--
2 files changed, 13 insertions(+), 11 deletions(-)
diff --git
Convert the code to use the C11 threads implementation, and nuke the
Windows non-pthreads code-path. The c11/threads_win32.h abstraction
should be better than the current code.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/main/eglcurrent.c | 48
Left over from commit 18db13f5865(mapi: THREADS was always defined,
remove it)
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/glx/glxcurrent.c | 4
1 file changed, 4 deletions(-)
diff --git a/src/glx/glxcurrent.c b/src/glx/glxcurrent.c
index dc2acd5..86fb658 100644
---
This has been an implicit rule for building mesa for a long time. Let's
make it official and just bail out at configure time. This way we can
cleaning up some of our glx code.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
configure.ac | 3 +++
1 file changed, 3 insertions(+)
diff
Tom Stellard t...@stellard.net writes:
On Thu, Mar 05, 2015 at 08:42:25PM +0200, Francisco Jerez wrote:
Tom Stellard thomas.stell...@amd.com writes:
This means dropping CL_FP_DENORM from the current return value.
---
src/gallium/state_trackers/clover/api/device.cpp | 4 +++-
1 file
Hi all,
Just accidently pushed the series to master. I'll revert them in a second.
-Emil
On 06/03/15 16:54, Emil Velikov wrote:
Remove the inline wrappers/abstraction layer.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/main/Makefile.sources | 1 -
C11 introduces the aligned_alloc() function, while mesa already has it's
own wrapper. Create a new header, and make use of it.
I was aiming to convert gallium as well, although that code diverges
depending on the debugging state required but wrapping around malloc and
friends.
#This series
On Fri, Mar 6, 2015 at 11:32 AM, Emil Velikov emil.l.veli...@gmail.com wrote:
Upon closer look it seems that TexData is no longer used. Perhaps
we can nuke it ?
http://patchwork.freedesktop.org/patch/43969/
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
Implicitly required for a while, although commit 9385c592c68 (mapi:
remove u_thread.h) was the one that put the final nail on the
coffin.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
docs/dispatch.html| 7 ++-
src/glx/glxclient.h | 18
On 06/03/15 17:05, Emil Velikov wrote:
Hi all,
Just accidently pushed the series to master. I'll revert them in a second.
All done. Apologies for the noise.
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/mesa/main/imports.c | 76 -
src/mesa/main/imports.h | 6
2 files changed, 82 deletions(-)
diff --git a/src/mesa/main/imports.c b/src/mesa/main/imports.c
index 5961587..3937a02
Upon closer look it seems that TexData is no longer used. Perhaps
we can nuke it ?
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/mesa/state_tracker/st_cb_texture.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_texture.c
On Thu, Mar 05, 2015 at 08:42:25PM +0200, Francisco Jerez wrote:
Tom Stellard thomas.stell...@amd.com writes:
This means dropping CL_FP_DENORM from the current return value.
---
src/gallium/state_trackers/clover/api/device.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
---
src/gallium/drivers/radeonsi/si_compute.c | 3 ++-
src/gallium/drivers/radeonsi/si_shader.c | 1 +
src/gallium/drivers/radeonsi/si_shader.h | 1 +
3 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_compute.c
This means dropping CL_FP_DENORM from the current return value.
v2:
- Add comments about minimum values for OpenCL 1.2.
---
src/gallium/state_trackers/clover/api/device.cpp | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/api/device.cpp
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/mesa/main/bufferobj.c | 8 +---
src/mesa/math/m_debug_norm.c | 6 --
src/mesa/math/m_debug_xform.c | 6 --
src/mesa/math/m_matrix.c | 10 ++
src/mesa/math/m_vector.c | 5 +++--
Used for aligned_alloc and other C11 functions missing from the header.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
include/c11_stdlib.h | 118 +++
1 file changed, 118 insertions(+)
create mode 100644 include/c11_stdlib.h
diff --git
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/mesa/main/imports.c | 74 +--
src/mesa/main/imports.h | 7
src/mesa/program/prog_parameter.c | 18 +++---
src/mesa/tnl/t_vertex.c | 5 ++-
4 files changed, 18
Remove the inline wrappers/abstraction layer.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/main/Makefile.sources | 1 -
src/egl/main/eglapi.c | 14 +
src/egl/main/eglcurrent.c | 13 -
src/egl/main/egldisplay.c | 13 +
On 06/03/15 17:05, Emil Velikov wrote:
Hi all,
Just accidently pushed the series to master. I'll revert them in a second.
All done. Apologies for the noise.
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
Series Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
On 2015-03-05 11:49:54, Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
Inspired by Damien's recent libdrm changes.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: Damien Lespiau damien.lesp...@intel.com
On Fri, Feb 27, 2015 at 05:34:52PM +0200, Francisco Jerez wrote:
---
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 5 +++--
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 6 --
On Fri, Mar 06, 2015 at 01:33:05AM -0800, Kenneth Graunke wrote:
On Gen8+, AND/OR/XOR/NOT don't support the abs() source modifier, and
negate changes meaning to bitwise-not (~, not -). This isn't what NIR
expects, so we should resolve the source modifers via a MOV.
+30 Piglits
Pohjolainen, Topi topi.pohjolai...@intel.com writes:
On Fri, Mar 06, 2015 at 10:37:06AM +0200, Pohjolainen, Topi wrote:
On Fri, Feb 27, 2015 at 05:34:44PM +0200, Francisco Jerez wrote:
[..]
+/**
+ * Send message to shared unit \p sfid with a possibly indirect
descriptor \p
+ * desc.
On Fri, Feb 27, 2015 at 05:34:53PM +0200, Francisco Jerez wrote:
This is consistent with the untyped surface read opcode. From now on
all typed and untyped surface access opcodes will follow the same
pattern: src[0] will be the message payload, src[1] will be the
surface index and src[2] will
On Fri, Feb 27, 2015 at 05:34:51PM +0200, Francisco Jerez wrote:
This doesn't actually enable untyped surface message sends from GRF
yet, the upcoming atomic counter and image intrinsic lowering code
will.
---
src/mesa/drivers/dri/i965/brw_vec4.cpp | 7 ---
On Fri, Mar 06, 2015 at 10:37:06AM +0200, Pohjolainen, Topi wrote:
On Fri, Feb 27, 2015 at 05:34:44PM +0200, Francisco Jerez wrote:
---
src/mesa/drivers/dri/i965/brw_eu.h | 19 ++--
src/mesa/drivers/dri/i965/brw_eu_emit.c | 58
++--
On Gen8+, AND/OR/XOR/NOT don't support the abs() source modifier, and
negate changes meaning to bitwise-not (~, not -). This isn't what NIR
expects, so we should resolve the source modifers via a MOV.
+30 Piglits (fs-op-bit{and,or,xor}-not-abs-*).
Signed-off-by: Kenneth Graunke
On Thu, Mar 05, 2015 at 01:48:29PM -0800, Kenneth Graunke wrote:
On Thursday, March 05, 2015 07:41:29 PM Ville Syrjälä wrote:
On Fri, Jan 23, 2015 at 12:12:56PM +0200, ville.syrj...@linux.intel.com
wrote:
From: Ville Syrjälä ville.syrj...@linux.intel.com
Increase the device info
On Fri, Feb 27, 2015 at 05:34:47PM +0200, Francisco Jerez wrote:
This is currently not a problem because the vec4 visitor happens to
mask out unused components from the destination, but it might become
an issue when we start using atomics without writeback message. In
any case it seems
On Fri, Feb 27, 2015 at 05:34:50PM +0200, Francisco Jerez wrote:
---
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
Reviewed-by: Topi Pohjolainen topi.pohjolai...@intel.com
On Thursday, March 05, 2015 09:39:58 PM Jason Ekstrand wrote:
This looks fine to me. I just kicked off a build on our test farm and,
assuming that looks good (I'll send another e-mail in the morning if it
does),
Reviewed-by: Jason Ekstrand jason.ekstr...@intel.com
I ran shader-db on the
Reviewed-by: Timothy Arceri t_arc...@yahoo.com.au
On Tue, 2015-03-03 at 18:57 -0800, Vinson Lee wrote:
i915_debug_fp.c: In function ‘i915_disassemble_program’:
i915_debug_fp.c:302:11: warning: unused variable ‘size’ [-Wunused-variable]
GLuint size = program[0] 0x1ff;
^
On 4 March 2015 at 18:07, Roland Scheidegger srol...@vmware.com wrote:
Am 04.03.2015 um 12:38 schrieb Jose Fonseca:
On 04/03/15 02:00, Emil Velikov wrote:
On 27 February 2015 at 23:28, Sedat Dilek sedat.di...@gmail.com wrote:
On Mon, Feb 9, 2015 at 6:30 PM, Emil Velikov
On Fri, 2015-03-06 at 15:53 +, Tom Stellard wrote:
This means dropping CL_FP_DENORM from the current return value.
v2:
- Add comments about minimum values for OpenCL 1.2.
---
src/gallium/state_trackers/clover/api/device.cpp | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
Jan Vesely jan.ves...@rutgers.edu writes:
On Fri, 2015-03-06 at 15:53 +, Tom Stellard wrote:
This means dropping CL_FP_DENORM from the current return value.
v2:
- Add comments about minimum values for OpenCL 1.2.
---
src/gallium/state_trackers/clover/api/device.cpp | 5 -
1
On Fri, Mar 6, 2015 at 10:14 AM, Emil Velikov emil.l.veli...@gmail.com
wrote:
On 06/03/15 17:05, Emil Velikov wrote:
Hi all,
Just accidently pushed the series to master. I'll revert them in a
second.
All done. Apologies for the noise.
Series looks OK to me. You can re-push with
On Fri, Mar 6, 2015 at 9:32 AM, Emil Velikov emil.l.veli...@gmail.com
wrote:
Used for aligned_alloc and other C11 functions missing from the header.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
include/c11_stdlib.h | 118 ++
I wonder if this should
On Mar 6, 2015, at 8:56 AM, Francisco Jerez curroje...@riseup.net wrote:
Tom Stellard t...@stellard.net mailto:t...@stellard.net writes:
On Thu, Mar 05, 2015 at 08:42:25PM +0200, Francisco Jerez wrote:
Tom Stellard thomas.stell...@amd.com writes:
This means dropping CL_FP_DENORM from
On 03/06/2015 06:56 AM, Chris Wilson wrote:
When rendering to an fbo, even though it may be acting as a winsys
frontbuffer or just generally, we never throttle. However, when rendering
to an fbo, there is no natural frame boundary. Conventionally we use
SwapBuffers and glFinish, but potential
https://bugs.freedesktop.org/show_bug.cgi?id=89477
Bug ID: 89477
Summary: include/no_extern_c.h:47:1: error: template with C
linkage
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
On 03/06/2015 06:56 AM, Chris Wilson wrote:
In order to facilitate the concurrency offered by triple buffering and to
offset the latency induced by swapping via an external process, which
may incur extra rendering itself, only throttle to the previous frame
and not the last. This doubles the
Unfortunately, my build configuration does not build gallium. This
caused me to miss some files which still have extern C includes.
Due to this oversight, master encounters build errors. I'll send a
patch asap.
-Mark
Mark Janes mark.a.ja...@intel.com writes:
Assuming my patch to fix 'extern
In Skylake the order of the arguments for sample messages with the LD
type are u, v, lod, r whereas previously they were u, lod, v, r. This
fixes 82 Piglit tests using texelFetch.
---
I have a feeling this probably isn't the right way to do this patch so
maybe someone who knows the compiler
https://bugs.freedesktop.org/show_bug.cgi?id=89477
--- Comment #2 from Mark Janes mark.a.ja...@intel.com ---
patch sent to list.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
___
mesa-dev
Reviewed-by: Ilia Mirkin imir...@alum.mit.edu
However you should probably split off the r300_public.h change into a
separate commit -- a little awkward to have 'nouveau:' as the subject
of a change to r300.
-ilia
On Fri, Mar 6, 2015 at 4:16 PM, Mark Janes mark.a.ja...@intel.com wrote:
The
The previous patch to fix header inclusion within extern C neglected
to fix the occurences of this pattern in nouveau files.
When the helper to detect this issue was pushed to master, it broke
the build for the nouveau driver. This patch fixes the nouveau build.
Bugzilla:
Series is
Reviewed-by: Ilia Mirkin imir...@alum.mit.edu
Thanks for splitting them up!
On Fri, Mar 6, 2015 at 4:36 PM, Mark Janes mark.a.ja...@intel.com wrote:
A previous patch to fix header inclusion within extern C neglected
to fix the occurences of this pattern in nouveau files.
When the
https://bugs.freedesktop.org/show_bug.cgi?id=89477
--- Comment #1 from Mark Janes mark.a.ja...@intel.com ---
Can you please include your configure line? I'm having trouble reproducing
your failure.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee
A previous patch to fix header inclusion within extern C neglected
to fix the occurences of this pattern in r300 files.
When the helper to detect this issue was pushed to master, it broke
the build for the r300 driver. This patch fixes the r300 build.
Bugzilla:
A previous patch to fix header inclusion within extern C neglected
to fix the occurences of this pattern in nouveau files.
When the helper to detect this issue was pushed to master, it broke
the build for the nouveau driver. This patch fixes the nouveau build.
Bugzilla:
When rendering to an fbo, even though it may be acting as a winsys
frontbuffer or just generally, we never throttle. However, when rendering
to an fbo, there is no natural frame boundary. Conventionally we use
SwapBuffers and glFinish, but potential callers avoid often glFinish for
being too heavy
In order to facilitate the concurrency offered by triple buffering and to
offset the latency induced by swapping via an external process, which
may incur extra rendering itself, only throttle to the previous frame
and not the last. The second issue that mostly affects swap benchmarks,
but also can
brw_shader.cpp: In function ‘bool brw_saturate_immediate(brw_reg_type,
brw_reg*)’:
brw_shader.cpp:618:31: warning: ‘sat_imm.brw_saturate_immediate(brw_reg_type,
brw_reg*)::anonymous union::ud’ may be used uninitialized in this function
[-Wmaybe-uninitialized]
reg-dw1.ud = sat_imm.ud;
egl_dri2.c: In function ‘dri2_bind_tex_image’:
egl_dri2.c:1240:4: warning: ‘format’ may be used uninitialized in this function
[-Wmaybe-uninitialized]
(*dri2_dpy-tex_buffer-setTexBuffer2)(dri2_ctx-dri_context,
^
Signed-off-by: Vinson Lee v...@freedesktop.org
---
On 07/03/15 07:23, Jose Fonseca wrote:
On 06/03/15 18:26, Brian Paul wrote:
On Fri, Mar 6, 2015 at 9:32 AM, Emil Velikov emil.l.veli...@gmail.com
mailto:emil.l.veli...@gmail.com wrote:
Used for aligned_alloc and other C11 functions missing from the
header.
Signed-off-by: Emil Velikov
Silences GCC unused-function warning.
nv30/nvfx_fragprog.c:333:1: warning: ‘nv40_fp_bra’ defined but not used
[-Wunused-function]
nv40_fp_bra(struct nvfx_fpc *fpc, unsigned target)
^
Signed-off-by: Vinson Lee v...@freedesktop.org
---
src/gallium/drivers/nouveau/nv30/nvfx_fragprog.c | 2 +-
1
i915_fragprog.c: In function ‘i915ValidateFragmentProgram’:
i915_fragprog.c:1453:11: warning: variable ‘k’ set but not used
[-Wunused-but-set-variable]
int k;
^
Signed-off-by: Vinson Lee v...@freedesktop.org
---
src/mesa/drivers/dri/i915/i915_fragprog.c | 5 +
1 file
radeon_fbo.c: In function ‘radeon_map_renderbuffer_s8z24’:
radeon_fbo.c:162:9: warning: variable ‘ret’ set but not used
[-Wunused-but-set-variable]
int ret;
^
radeon_fbo.c: In function ‘radeon_map_renderbuffer_z16’:
radeon_fbo.c:200:9: warning: variable ‘ret’ set but not used
egl_dri2.c: In function ‘dri2_bind_tex_image’:
egl_dri2.c:1240:4: warning: ‘format’ may be used uninitialized in this function
[-Wmaybe-uninitialized]
(*dri2_dpy-tex_buffer-setTexBuffer2)(dri2_ctx-dri_context,
^
Suggested-by: Ilia Mirkin imir...@alum.mit.edu
Signed-off-by: Vinson Lee
The pipe's get_vendor method returns something more akin to a driver
vendor string in most cases, instead of the actual device vendor. Use
get_device_vendor instead, which was introduced specifically for this
purpose.
---
src/gallium/state_trackers/clover/core/device.cpp | 2 +-
1 file changed, 1
The only hackish ones are llvmpipe and softpipe, which currently return
the same string as for get_vendor(), while ideally they should return
the CPU vendor.
---
src/gallium/drivers/freedreno/freedreno_screen.c | 8
src/gallium/drivers/galahad/glhd_screen.c| 10 ++
This will be needed by Clover to return the correct information
to CL_DEVICE_VENDOR info queries.
---
src/gallium/include/pipe/p_screen.h | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/gallium/include/pipe/p_screen.h
b/src/gallium/include/pipe/p_screen.h
index 4018f8a..cba4c95
OpenCL (as opposed to OpenGL) has separate vendor strings for the
implementation/driver/platform and the device. CL_PLATFORM_VENDOR
is akin to the GL_VENDOR string, while CL_DEVICE_VENDOR is supposed to
return the actual device vendor.
(For example, the AMD OpenCL platform returns GenuineIntel as
1 - 100 of 118 matches
Mail list logo