On Die, 2013-10-22 at 22:07 -0400, Tom Stellard wrote:
>
> diff --git a/lib/Target/R600/AMDGPU.td b/lib/Target/R600/AMDGPU.td
> index a722f55..f63617f 100644
> --- a/lib/Target/R600/AMDGPU.td
> +++ b/lib/Target/R600/AMDGPU.td
[...]
> @@ -262,13 +263,24 @@ void
> AMDGPUAsmPrinter::EmitProgramInfoS
Still needs review.
On Wed, Oct 2, 2013 at 5:06 PM, Jonas Ådahl wrote:
> This patch fixes a race when a client releases a named buffer before the
> server had the time to handle 'wl_drm_create_buffer'. When triggered,
> the server would fail to create the buffer since the client already
> having
---
src/egl/drivers/dri2/egl_dri2.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
index b29eb1c..b143a95 100644
--- a/src/egl/drivers/dri2/egl_dri2.c
+++ b/src/egl/drivers/dri2/egl_dri2.c
@@ -1843,10 +1843,12 @@ dri2_bind_wa
On Tuesday, October 22, 2013 12:25:20 PM Chad Versace wrote:
> I'm unsure if it makes sense to build EGL with Wayland support
> but without GBM support. That is, to configure Mesa with
>
>--with-egl-platforms=wayland # no drm
Yeah, I was surprised that this code was being compiled at all. My
The main purpose of this patch is to increase readability of
the array code by introducing is_unsized_array() to glsl_types.
Some redundent is_array() checks are also removed, and small number
of other related clean ups.
The introduction of is_unsized_array() should also make the
ARB_arrays_of_arr
From: Marek Olšák
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi/radeonsi_shader.c
index 80ee325..42db8ca 100644
--- a/src/gallium/drivers
From: Marek Olšák
This fixes a lockup in piglit/spec/glsl-1.40/execution/tf-no-position.
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi/radeonsi_s
On 2013-10-22 21:05, Tom Stellard wrote:
---
src/gallium/drivers/radeonsi/radeonsi_compute.c | 15 ---
1 file changed, 12 insertions(+), 3 deletions(-)
[..]
+ si_pm4_set_reg(pm4, R_00B82C_COMPUTE_MAX_WAVE_ID,
+ 0x190 /*
On 18 October 2013 13:40, Paul Berry wrote:
> On 11 October 2013 11:18, Ian Romanick wrote:
>
>> From: Ian Romanick
>>
>> The unit tests added in the previous commits prove some things about the
>> state of some internal data structures. The most important of these is
>> that all built-in inpu
When lower_named_interface_blocks lowers a built-in interface block
member to an ir_variable, it needs to set explicit_location in the
ir_variable. Otherwise the linker gets confused and treats the
variable as a generic varying.
Fixes the following piglit tests, which were regressed by commit
639
On Mit, 2013-10-23 at 16:15 +0200, Marek Olšák wrote:
> From: Marek Olšák
Both patches are
Reviewed-by: Michel Dänzer
--
Earthling Michel Dänzer| http://www.amd.com
Libre software enthusiast |Mesa and X developer
From: Roland Scheidegger
d3d10 requires that cube corners are filtered with accurate weights (that
is, the weight of the non-existing corner texel should be evenly distributed
to the other 3 texels). OpenGL does not require this (but recommends it).
This requires us to use different filtering cod
---
src/mesa/drivers/dri/i965/brw_context.h | 6 ++
src/mesa/drivers/dri/i965/brw_shader.cpp | 4
src/mesa/drivers/dri/i965/brw_vec4_gs.c | 34
src/mesa/drivers/dri/i965/brw_vec4_gs.h | 4
4 files changed, 48 insertions(+)
diff --git a/src/mesa
This will allow us to re-use it for precompiling geometry shaders.
---
src/mesa/drivers/dri/i965/brw_vec4.cpp | 22 ++
src/mesa/drivers/dri/i965/brw_vec4.h | 4
src/mesa/drivers/dri/i965/brw_vs.c | 15 +--
3 files changed, 27 insertions(+), 14 deletions(
This series adds precompilation for geometry shaders.
I'm not 100% certain this is worth doing, because:
(a) I'm aware of very few programs out in the wild that actually use
geometry shaders right now.
(b) Since i965 hardware has no ability to remap vertex shader outputs
to geometry shader input
Paul Berry writes:
> When a geometry shader is present, the fragment shader gl_PrimitiveID
> input acts like an ordinary varying, receiving data from the gs
> gl_PrimitiveID output. When there's no geometry shader, we have to
> ask the fixed function SF hardware to provide the primitive ID to th
On 22 October 2013 15:52, Brian Paul wrote:
> ---
> src/glsl/builtin_variables.cpp |4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/glsl/builtin_variables.cpp
> b/src/glsl/builtin_variables.cpp
> index fc1115b..1f62fcf 100644
> --- a/src/glsl/builtin_variables.
On Wed, Oct 23, 2013 at 09:47:30AM -0500, Jay Cornwall wrote:
> On 2013-10-22 21:05, Tom Stellard wrote:
>
> > ---
> > src/gallium/drivers/radeonsi/radeonsi_compute.c | 15 ---
> > 1 file changed, 12 insertions(+), 3 deletions(-)
> >
> [..]
> > + si_pm4_set_reg(pm4, R_00B82
On Wed, Oct 23, 2013 at 09:31:53AM +0200, Michel D??nzer wrote:
> On Die, 2013-10-22 at 22:07 -0400, Tom Stellard wrote:
> >
> > diff --git a/lib/Target/R600/AMDGPU.td b/lib/Target/R600/AMDGPU.td
> > index a722f55..f63617f 100644
> > --- a/lib/Target/R600/AMDGPU.td
> > +++ b/lib/Target/R600/AMDGPU
On Sun, Oct 20, 2013 at 7:25 AM, Paul Berry wrote:
> On 14 October 2013 10:12, Anuj Phogat wrote:
>>
>> - Enable GEN6_WM_MSDISPMODE_PERSAMPLE, GEN6_WM_POSOFFSET_SAMPLE,
>> GEN6_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
>> - Don't enable GEN6_WM_16_DISPATCH_ENABLE when
>> GEN6_
This should never have been in the program key in the first place,
since it's determined by the shader source, not by GL state. Change
the code to just refer to gl_program::UsesClipDistanceOut directly.
---
src/mesa/drivers/dri/i965/brw_vec4.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vec4.
This will make it easier for back-ends to share code between geometry
shader and vertex shader compilation. Also, it is renamed to
"UsesClipDistanceOut" to clarify that (a) in geometry shaders, it
refers to the gl_ClipDistance output rather than the gl_ClipDistance
input, and (b) it is irrelevant
Chad Versace writes:
> On 10/11/2013 06:03 PM, Eric Anholt wrote:
>> Previously, we've split things such that mesa core is in libdricore,
>> exposing the whole Mesa core interface in the global namespace, and the
>> i965_dri.so code all links against that. Along with polluting application
>> nam
Emil Velikov writes:
> On 12/10/13 02:03, Eric Anholt wrote:
>> Previously, we've split things such that mesa core is in libdricore,
>> exposing the whole Mesa core interface in the global namespace, and the
>> i965_dri.so code all links against that. Along with polluting application
>> namespac
Chad Versace writes:
> The bits about globalDriverAPI and megadriver_stub.c feel strongly
> logically disjoint from the rest of the patch. And, they really aren't
> related to i965. It looks like they should get separated out into a
> separate patch the immediately precedes this one.
The problem
On 2013-10-23 13:05, Tom Stellard wrote:
> + si_pm4_set_reg(pm4, R_00B82C_COMPUTE_MAX_WAVE_ID,
> + 0x190 /* Default value */);
Is there any visible effect on performance if this is set very low?
I haven't tested with any other values.
OK, I
Ian Romanick writes:
> On 10/01/2013 07:15 PM, Francisco Jerez wrote:
>>[..]
>> +ir_call *
>> +call(ir_function *f, ir_variable *ret, int num_params, ...)
>
> varargs functions that take an explicit count are very susceptible to
> copy-and-past bugs. If you copy-and-paste an invocation, add or r
If a user set MESA_INFO and the OpenGL application uses a
3.0 or later context then the MESA_INFO debug output will have
an error when it queries for extensions using the deprecated
enum GL_EXTENSIONS. Passing context argument allows code
to return extension list directly regardless of profile.
---
Thank you for your comments,
I've rebased my atomic counters branch on top of master [1] and I think
I've taken into account everything you had pointed out.
The rebase has turned out to be especially painful this time. To avoid
wasting more time rebasing patches that no-one is going to have a lo
On 09/26/2013 02:31 PM, Paul Berry wrote:
[snip]
> However, there are two things which I expected to see in this patch
> series but I didn't:
>
> - Do atomic_uints need similar handling to
> ir_sampler_replacement_visitor in opt_function_inlining?
>
> - Do we need to update schedule_node::set_lat
Kenneth Graunke writes:
> On 09/26/2013 02:31 PM, Paul Berry wrote:
> [snip]
>> However, there are two things which I expected to see in this patch
>> series but I didn't:
>>
>> - Do atomic_uints need similar handling to
>> ir_sampler_replacement_visitor in opt_function_inlining?
>>
>> - Do we
Since gl_ClipDistance is lowered from an array of floats to an array
of vec4's during compilation, transform feedback has special logic to
keep track of the pre-lowered array size so that attempting to perform
transform feedback on gl_ClipDistance produces a result with the
correct size.
Previousl
On Mon, Oct 7, 2013 at 8:23 PM, Brian Paul wrote:
> diff --git a/src/gallium/drivers/freedreno/freedreno_texture.c
> b/src/gallium/drivers/freedreno/freedreno_texture.c
> index fc0c8aa..bcc846f 100644
> --- a/src/gallium/drivers/freedreno/freedreno_texture.c
> +++ b/src/gallium/drivers/freedreno/
From: Tom Stellard
This makes it possible to use clover with statically linked LLVM.
---
src/gallium/targets/pipe-loader/Makefile.am | 4
src/gallium/targets/pipe-loader/pipe.link | 3 +++
2 files changed, 7 insertions(+)
create mode 100644 src/gallium/targets/pipe-loader/pipe.link
diff
From: Tom Stellard
This works now that pipe_*.so is no longer exporting LLVM symbols.
---
configure.ac | 6 --
1 file changed, 6 deletions(-)
diff --git a/configure.ac b/configure.ac
index dfa35b4..81403ee 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1530,12 +1530,6 @@ AC_ARG_WITH([llvm
Hi,
The attached patches introduce linker scripts to the pipe-loader and
egl-static targets. The linker scripts prevents these targets from
exporting LLVM (and other) symbols that they shouldn't be. This fixes
several crashes in the radeon drivers when statically linking LLVM.
With these patche
From: Tom Stellard
This fixes a crash in glamor when mesa links against static LLVM.
---
src/gallium/targets/egl-static/Makefile.am | 4
src/gallium/targets/egl-static/egl.link| 3 +++
2 files changed, 7 insertions(+)
create mode 100644 src/gallium/targets/egl-static/egl.link
diff --g
We've always overriden
ctx->Const.{Vertex,Fragment}Program.MaxTextureImageUnits to reflect
the number of texture image units supported by the hardware (rather
than using the default values assigned by Mesa core) so it seems
sensible to do that for GeometryProgram.MaxTextureImageUnits too. We
set i
On Wed, Oct 23, 2013 at 12:41 PM, Courtney Goeltzenleuchter
wrote:
> If a user set MESA_INFO and the OpenGL application uses a
> 3.0 or later context then the MESA_INFO debug output will have
> an error when it queries for extensions using the deprecated
> enum GL_EXTENSIONS. Passing context argum
Hi Matt,
I think the intended use was to provide glxinfo-like information from the
app's perspective, i.e. if it asked for a compatibility context or
something like that. That was how I discovered it, I wanted to get at the
list of extensions that was being given to the app as it was different tha
On Wed, Oct 23, 2013 at 1:26 PM, Tom Stellard wrote:
> From: Tom Stellard
>
> This makes it possible to use clover with statically linked LLVM.
> ---
> src/gallium/targets/pipe-loader/Makefile.am | 4
> src/gallium/targets/pipe-loader/pipe.link | 3 +++
> 2 files changed, 7 insertions(+)
On Wed, Oct 23, 2013 at 1:26 PM, Tom Stellard wrote:
> From: Tom Stellard
>
> This fixes a crash in glamor when mesa links against static LLVM.
> ---
> src/gallium/targets/egl-static/Makefile.am | 4
> src/gallium/targets/egl-static/egl.link| 3 +++
> 2 files changed, 7 insertions(+)
>
On Sun, Oct 20, 2013 at 7:37 AM, Paul Berry wrote:
> On 16 October 2013 15:20, Ian Romanick wrote:
>>
>> On 10/16/2013 02:57 PM, Anuj Phogat wrote:
>> > On Tue, Oct 15, 2013 at 3:48 PM, Kenneth Graunke
>> > wrote:
>> >> On 10/14/2013 10:12 AM, Anuj Phogat wrote:
>> >>> - Enable GEN7_WM_MSDISPMOD
On Fri, Oct 18, 2013 at 2:57 PM, Paul Berry wrote:
> On 14 October 2013 10:12, Anuj Phogat wrote:
>>
>> New builtins added by GL_ARB_sample_shading:
>> in vec2 gl_SamplePosition
>> in int gl_SampleID
>> in int gl_NumSamples
>> out int gl_SampleMask[]
>>
>> Signed-off-by: Anuj Phogat
>> ---
>> s
From: Tom Stellard
This makes it possible to use clover with statically linked LLVM.
v2:
- Inline LINKER_SCRIPT variable
---
src/gallium/targets/pipe-loader/Makefile.am | 2 ++
src/gallium/targets/pipe-loader/pipe.link | 3 +++
2 files changed, 5 insertions(+)
create mode 100644 src/galliu
From: Tom Stellard
This fixes a crash in glamor when mesa links against static LLVM.
v2:
- Inline LINKER_SCRIPT variable
---
src/gallium/targets/egl-static/Makefile.am | 2 ++
src/gallium/targets/egl-static/egl.link| 3 +++
2 files changed, 5 insertions(+)
create mode 100644 src/gallium/
On 10/23/2013 03:14 PM, Courtney Goeltzenleuchter wrote:
On Wed, Oct 23, 2013 at 3:05 PM, Matt Turner mailto:matts...@gmail.com>> wrote:
On Wed, Oct 23, 2013 at 12:41 PM, Courtney Goeltzenleuchter
mailto:court...@lunarg.com>> wrote:
> If a user set MESA_INFO and the OpenGL applicati
From: Tom Stellard
---
lib/Target/R600/AMDGPUISelLowering.cpp | 1 +
test/CodeGen/R600/llvm.sqrt.ll | 54 ++
2 files changed, 55 insertions(+)
create mode 100644 test/CodeGen/R600/llvm.sqrt.ll
diff --git a/lib/Target/R600/AMDGPUISelLowering.cpp
b/lib/T
On 10/23/2013 02:17 PM, Rob Clark wrote:
On Mon, Oct 7, 2013 at 8:23 PM, Brian Paul wrote:
diff --git a/src/gallium/drivers/freedreno/freedreno_texture.c
b/src/gallium/drivers/freedreno/freedreno_texture.c
index fc0c8aa..bcc846f 100644
--- a/src/gallium/drivers/freedreno/freedreno_texture.c
++
The maximum workgroup size for a given kernel is based on the
capabilities of the device that it's being run on. Previously,
we were just returning the maximum value of a size_t which is
obviously wrong.
This patch uses the device's capabilities, but doesn't take into
account any resource usage wh
A few Serious Sam 3 shaders affected:
instructions in affected programs: 4384 -> 4344 (-0.91%)
---
src/glsl/opt_algebraic.cpp | 12
1 file changed, 12 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 37b2f02..3bf0689 100644
--- a/src/glsl/
No shader-db changes, but seems like a good idea.
---
src/glsl/opt_algebraic.cpp | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 3bf0689..1ce9e2d 100644
--- a/src/glsl/opt_algebraic.cpp
+++ b/src/glsl/opt_algebraic.cpp
@@ -
Matt Turner writes:
> A few Serious Sam 3 shaders affected:
These two are:
Reviewed-by: Eric Anholt
pgpSJbcZJCrms.pgp
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/
From: Tom Stellard
v2:
- Fix LDS size calculation
---
lib/Target/R600/AMDGPU.td| 2 ++
lib/Target/R600/AMDGPUAsmPrinter.cpp | 16 ++--
lib/Target/R600/AMDGPUCallingConv.td | 4 ++--
lib/Target/R600/AMDGPUSubtarget.h| 3 ++-
lib/Target/R600/Processors.td|
On Tue, Oct 22, 2013 at 1:46 PM, Eric Anholt wrote:
> The LIFO plan was simple: Take the most recently made available
> instructions, and pick those first.
>
> But because of the order we were pushing things onto our list of
> available-to-schedule instructions, it meant that when a set of
> instr
https://bugs.freedesktop.org/show_bug.cgi?id=70823
Priority: medium
Bug ID: 70823
Assignee: mesa-dev@lists.freedesktop.org
Summary: while configuring the Mesa -9.2.2 file error will be
there in C++ preprocessor .. please suggest me wha
On Wed, Oct 23, 2013 at 06:41:56PM -0500, Aaron Watry wrote:
> The maximum workgroup size for a given kernel is based on the
> capabilities of the device that it's being run on. Previously,
> we were just returning the maximum value of a size_t which is
> obviously wrong.
>
> This patch uses the d
On Mit, 2013-10-23 at 23:04 -0400, Tom Stellard wrote:
> From: Tom Stellard
>
> v2:
> - Fix LDS size calculation
Reviewed-by: Michel Dänzer
--
Earthling Michel Dänzer| http://www.amd.com
Libre software enthusiast |Mesa and X developer
link to similiar bug: https://bugs.freedesktop.org/show_bug.cgi?id=70804
---
src/gallium/state_trackers/clover/core/kernel.cpp | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/ke
From e004b63b2896ac2c8951cfda49d4a54d72b16449 Mon Sep 17 00:00:00 2001
From: David Heidelberger
Date: Wed, 23 Oct 2013 23:35:42 +0200
Subject: [PATCH] clover: mark gcc 4.7.0 as broken
---
configure.ac | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/configure.ac b/confi
60 matches
Mail list logo
