On Mon, 2014-02-24 at 10:06 -0800, Tom Stellard wrote:
> On Mon, Feb 24, 2014 at 06:02:23PM +0900, Michel Dänzer wrote:
> >
> > diff --git a/test/CodeGen/R600/llvm.AMDGPU.kill.ll
> > b/test/CodeGen/R600/llvm.AMDGPU.kill.ll
> > new file mode 100644
> > index 000..0abe694
> > --- /dev/null
> >
On Die, 2014-02-25 at 00:48 +0100, Marek Olšák wrote:
>
> diff --git a/src/gallium/drivers/radeon/r600_texture.c
> b/src/gallium/drivers/radeon/r600_texture.c
> index 35ad97b..1966251 100644
> --- a/src/gallium/drivers/radeon/r600_texture.c
> +++ b/src/gallium/drivers/radeon/r600_texture.c
> @@ -
https://bugs.freedesktop.org/show_bug.cgi?id=74010
--- Comment #6 from Tibbo ---
I've encountered this segfault before, and i think i can give some useful
informations.
As stated in the file src/mesa/main/varray.c:
/* Page 407 (page 423 of the PDF) of the OpenGL 3.0 spec says:
*
*
GLSL 1.50 spec says:
"If gl_FragCoord is redeclared in any fragment shader in a program,
it must be redeclared in all the fragment shaders in that
program that have a static use gl_FragCoord. All redeclarations of
gl_FragCoord in all fragment shaders in a single program must
have
GLSL 1.50 spec says:
"If gl_FragCoord is redeclared in any fragment shader in a program,
it must be redeclared in all the fragment shaders in that
program that have a static use gl_FragCoord. All redeclarations of
gl_FragCoord in all fragment shaders in a single program must
have
Section 4.3.8.1, page 39 of GLSL 1.50 spec says:
"Within any shader, the first redeclarations of gl_FragCoord
must appear before any use of gl_FragCoord."
GLSL compiler should generate an error in following case:
vec4 p = gl_FragCoord;
layout(origin_upper_left) in vec4 gl_FragCoord;
void ma
Link error conditions added in previous patch are equally applicable
to GL_ARB_fragment_coord_conventions implementation. Extension's spec
says:
"If gl_FragCoord is redeclared in any fragment shader in a program,
it must be redeclared in all the fragment shaders in that program
that have
From: Marek Olšák
The cs_add_reloc change is commented out not to break compilation.
The highest priority of all cs_add_reloc calls is send to the kernel.
---
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 20 +---
src/gallium/winsys/radeon/drm/radeon_winsys.h | 21 +
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 3 ++-
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 24
src/gallium/winsys/radeon/drm/radeon_winsys.h | 8 +++-
3 files changed, 33 insertions(+), 2 deletions(-)
diff --git a/src/gallium/driv
From: Marek Olšák
---
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 40 +++
1 file changed, 28 insertions(+), 12 deletions(-)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c
b/src/gallium/winsys/radeon/drm/radeon_drm_cs.c
index ccba0c0..f6b9d62 100644
--- a
From: Marek Olšák
This updates all occurences of cs_add_reloc.
---
src/gallium/drivers/r300/r300_emit.c | 27 +++
src/gallium/drivers/radeon/r600_cs.h | 3 ++-
src/gallium/drivers/radeon/radeon_uvd.c | 3 ++-
src/gallium/drivers/radeon/radeon_vce
From: Marek Olšák
This depends on the kernel patch.
---
src/gallium/drivers/radeon/r600_texture.c | 2 +-
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 51 +++
src/gallium/winsys/radeon/drm/radeon_winsys.h | 5 +++
3 files changed, 51 insertions(+), 7 deletions(-)
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_pipe_common.c | 6 -
src/gallium/drivers/radeon/r600_pipe_common.h | 4 +++
src/gallium/drivers/radeon/r600_query.c | 32 +++
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 2 ++
src/gallium/w
On Tue, Feb 25, 2014 at 12:41 AM, Ian Romanick wrote:
> On 02/24/2014 03:36 PM, Roland Scheidegger wrote:
>>
>> Am 25.02.2014 00:00, schrieb Matt Turner:
>>>
>>> ---
>>> src/glsl/opt_algebraic.cpp | 3 +++
>>> 1 file changed, 3 insertions(+)
>>>
>>> diff --git a/src/glsl/opt_algebraic.cpp b/src
On Mon, Feb 24, 2014 at 3:36 PM, Roland Scheidegger wrote:
> Am 25.02.2014 00:00, schrieb Matt Turner:
>> ---
>> src/glsl/opt_algebraic.cpp | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
>> index 778638c..50342ac 100644
>> --
On 02/24/2014 03:36 PM, Roland Scheidegger wrote:
Am 25.02.2014 00:00, schrieb Matt Turner:
---
src/glsl/opt_algebraic.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 778638c..50342ac 100644
--- a/src/glsl/opt_algebraic.
Am 25.02.2014 00:00, schrieb Matt Turner:
> ---
> src/glsl/opt_algebraic.cpp | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
> index 778638c..50342ac 100644
> --- a/src/glsl/opt_algebraic.cpp
> +++ b/src/glsl/opt_algebraic.cpp
>
Series is
Reviewed-by: Ian Romanick
Though we'll eventually want to generalize patch 3.
On 02/24/2014 03:00 PM, Matt Turner wrote:
Helps two programs in shader-db:
instructions in affected programs: 254 -> 234 (-7.87%)
---
src/glsl/opt_algebraic.cpp | 2 ++
1 file changed, 2 insertion
The issue is caused by a thinko that an empty string will be
considered of zero length by 'test'. This is not the case,
thus we were building the 'core' of megadrivers even when no
classic drivers were built.
Signed-off-by: Emil Velikov
---
configure.ac | 11 +++
1 file changed, 7 insert
Building to provide accelration using swrast does not make
sense.
Note: update your build script to explicitly mention svga
in the gallium drivers list, if you are building the vmwgfx
xa library.
Signed-off-by: Emil Velikov
---
configure.ac | 3 +++
1 file changed, 3 insertions(+)
diff --git a
All hardware drivers including the virtual vmwgfx require
the drm pipe-loader in order to be properly loaded by xa,
gbm and opencl.
Note this does _not_ add support for the above three it only
allows the pipe driver to be loaded by the library.
Eg. GBM will now properly open the pipe-i915 driver,
Recent patch converted our logic to use test -n and test -z.
An emptry string variable (empty_str="") return true for both
thus making the check unreliable.
Fix this by correctly setting the variable when applicable.
Signed-off-by: Emil Velikov
---
configure.ac | 6 +++---
1 file changed, 3 ins
---
src/glsl/opt_algebraic.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 778638c..50342ac 100644
--- a/src/glsl/opt_algebraic.cpp
+++ b/src/glsl/opt_algebraic.cpp
@@ -421,6 +421,9 @@ ir_algebraic_visitor::handle_expression(
Helps two programs in shader-db:
instructions in affected programs: 254 -> 234 (-7.87%)
---
src/glsl/opt_algebraic.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 1b4d319..28f95a4 100644
--- a/src/glsl/opt_algebraic.cpp
+
Helps one program in shader-db:
instructions in affected programs: 96 -> 92 (-4.17%)
---
src/glsl/opt_algebraic.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 28f95a4..778638c 100644
--- a/src/glsl/opt_algebraic.cpp
+++
On Sat, Feb 22, 2014 at 1:34 AM, Ian Romanick wrote:
> On 02/21/2014 03:15 PM, Marek Olšák wrote:
>>
>> The main reason _min exists is that it evaluates each parameter once,
>> while MIN2 evaluates each parameter twice, so in this case, MIN2 calls
>> the get_param function twice, which may not be
On Mon, Feb 24, 2014 at 2:04 PM, Jon Ashburn wrote:
> Add Intel driver hook for glGetTexImage to accelerate the case of reading
> texture image into a PBO. This case gets huge performance gains by using
> GPU BLIT directly to PBO rather than GPU BLIT to temporary texture followed
> by memcpy.
>
>
On Mon, Feb 24, 2014 at 4:53 PM, Tom Stellard wrote:
> This prevents clover from using unsupported devices.
Candidate for stable?
Reviewed-by: Alex Deucher
> ---
> src/gallium/drivers/r600/r600_pipe.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/driv
Add Intel driver hook for glGetTexImage to accelerate the case of reading
texture image into a PBO. This case gets huge performance gains by using
GPU BLIT directly to PBO rather than GPU BLIT to temporary texture followed
by memcpy.
No regressions on Piglit tests with Intel driver.
---
src/mes
Reviewed-by: Marek Olšák
Marek
On Mon, Feb 24, 2014 at 10:53 PM, Tom Stellard wrote:
> This prevents clover from using unsupported devices.
> ---
> src/gallium/drivers/r600/r600_pipe.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/r600/r600_pip
This prevents clover from using unsupported devices.
---
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 0f75a53..157d560 100644
--- a/src/gallium/drivers/
Am 24.02.2014 21:54, schrieb Marek Olšák:
> On Mon, Feb 24, 2014 at 9:43 PM, Roland Scheidegger
> wrote:
>> Am 24.02.2014 21:23, schrieb Marek Olšák:
>>> Roland,
>>>
>>> The version CAPs that Emil wants to add have very little to do with
>>> which features a driver supports. The versions cannot b
On 24/02/14 20:43, Roland Scheidegger wrote:
> Am 24.02.2014 21:23, schrieb Marek Olšák:
>> Roland,
>>
>> The version CAPs that Emil wants to add have very little to do with
>> which features a driver supports. The versions cannot be derived
>> from other CAPs, instead, it should match what you ge
On Mon, Feb 24, 2014 at 9:43 PM, Roland Scheidegger wrote:
> Am 24.02.2014 21:23, schrieb Marek Olšák:
>> Roland,
>>
>> The version CAPs that Emil wants to add have very little to do with
>> which features a driver supports. The versions cannot be derived
>> from other CAPs, instead, it should mat
The GTT size shouldn't be reported. I think it's 1 GB on r600 and
later. It's not really important to know that, because the memory is
quite slow.
Marek
On Sat, Feb 22, 2014 at 4:04 AM, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
> ---
> src/gallium/drivers/r300/r300_screen.c | 12 ++
Am 24.02.2014 21:23, schrieb Marek Olšák:
> Roland,
>
> The version CAPs that Emil wants to add have very little to do with
> which features a driver supports. The versions cannot be derived
> from other CAPs, instead, it should match what you get after you
> create an OpenGL context. For example
On Sun, Feb 23, 2014 at 4:26 PM, Emil Velikov wrote:
> Now that I think about it, one could copy the extension list and
> (context) constants to pipe_screen. This way on st_api_create_context,
> we'll just copy the initial values over. FWIW we'll still be able to
> amend the extension list via MES
Roland,
The version CAPs that Emil wants to add have very little to do with
which features a driver supports. The versions cannot be derived from
other CAPs, instead, it should match what you get after you create an
OpenGL context. For example, if your driver can do GL 4.0, the driver
cannot repor
On Sat, Feb 22, 2014 at 4:03 AM, Emil Velikov wrote:
> To be used by the upcomming GLX_MESA_query_renderer implementation.
>
> Signed-off-by: Emil Velikov
> ---
> .../drivers/dri/radeon/radeon_common_context.c | 50
> +-
> .../drivers/dri/radeon/radeon_common_context.h
On my gentoo system, llvm libs are in /usr/lib64/llvm, and llvm-config
--ldflags does not provide the rpath (it does, of course, provide a -L).
This adds the llvm dir to the rpath. It should be harmless if the path
is a system path, and should make things work when it's not.
Signed-off-by: Ilia M
On 24 February 2014 00:36, Ilia Mirkin wrote:
> Before you read any further, this is nowhere close to working. However
> it's in
> a state where I think most of the structure is there, albeit with a lot of
> XXX
> comments. And I haven't actually implemented the new opcodes I've added.
>
> I was
Signed-off-by: Adel Gadllah
Reviewed-by: Robert Bragg
---
src/glx/dri2_glx.c | 1 +
src/glx/dri3_glx.c | 18 ++
src/glx/dri3_priv.h | 2 ++
src/glx/glx_pbuffer.c | 33 ++---
src/glx/glxclient.h | 1 +
src/glx/glxextensions.c | 1
Move the pdraw != NULL check out so that they don't
have to be duplicated.
Signed-off-by: Adel Gadllah
Reviewed-by: Ian Romanick
---
src/glx/glx_pbuffer.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/glx/glx_pbuffer.c b/src/glx/glx_pbuffer.c
index 411d6e5.
Hi,
OK this version of the patches address the review comments from Ian and
Robert.
The first two ones are actually unmodified just signed them off and added Ian's
reviewed-by.
The third one now skips the redundant round trip and reports BadDrawable when
the drawable isn't bound to the current
Signed-off-by: Adel Gadllah
Reviewed-by: Ian Romanick
---
include/GL/glxext.h | 32 +---
1 file changed, 29 insertions(+), 3 deletions(-)
diff --git a/include/GL/glxext.h b/include/GL/glxext.h
index 8c642f3..826fda5 100644
--- a/include/GL/glxext.h
+++ b/include/GL/g
Am 24.02.2014 17:58, schrieb Robert Bragg:
Hi Adel,
Hi,
Thanks for look at this; just a few comments...
diff --git a/src/glx/glx_pbuffer.c b/src/glx/glx_pbuffer.c
index 978730c..afb4206 100644
--- a/src/glx/glx_pbuffer.c
+++ b/src/glx/glx_pbuffer.c
@@ -373,6 +373,14 @@ GetDrawableAttribute(Di
Ken,
Just noticed -- all those fields that we're keeping need to be widened
to uint32_t for 32 samplers. None of our sampler quirks get flagged
correctly for `high` samplers on Haswell at present. I'll send some
patches for that this evening if no one beats me to it.
-- Chris
On Tue, Feb 25, 201
Topi Pohjolainen writes:
> Signed-off-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 28
> ++--
> 1 file changed, 26 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
> b/src/mesa/drivers/dri/i9
On 02/24/2014 10:48 AM, Ian Romanick wrote:
> It seems very weird that these are unused. Are GL_YCBCR_MESA just
> completely broken on our driver? If so, we should probably disable that
> extension...
Yeah. The old fragment shader backend had code for this, which used
these fields, but the new
On 02/22/2014 01:05 AM, Topi Pohjolainen wrote:
Most of the logic refers to the local variable 'mt' directly but
a few cases use 'intelObj->mt' instead. These are the same for
now but will be different once stencil miptree gets used.
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i9
Reviewed-by: Ian Romanick
On 02/22/2014 01:05 AM, Topi Pohjolainen wrote:
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/brw_vec4.h | 6 +++---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 8
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git
Reviewed-by: Ian Romanick
On 02/22/2014 01:05 AM, Topi Pohjolainen wrote:
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 6 +++---
src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 4 ++--
src/mesa/drivers/dri/i965/intel_regions.c | 4 ++--
src/mesa/
On 02/22/2014 01:05 AM, Topi Pohjolainen wrote:
This is not utilised yet but will be needed when W-tiled stencil is
sampled as Y-tiled.
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/intel_fbo.h | 2 +-
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 11 +--
It seems very weird that these are unused. Are GL_YCBCR_MESA just
completely broken on our driver? If so, we should probably disable that
extension...
On 02/22/2014 01:05 AM, Topi Pohjolainen wrote:
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/brw_program.h | 6 --
Adel Gadllah writes:
> ---
> src/glx/dri2_glx.c | 1 +
> src/glx/dri3_glx.c | 18 ++
> src/glx/dri3_priv.h | 2 ++
> src/glx/glx_pbuffer.c | 8
> src/glx/glxclient.h | 1 +
> src/glx/glxextensions.c | 1 +
> src/glx/glxextensions.h | 1 +
> 7 fi
On Mon, Feb 24, 2014 at 10:15 AM, Eric Anholt wrote:
> I think we would do better by emitting
> ADD(y_minus_x, y, negate(x))
> MAC(dst, x, y_minus_x, a)
MAC only takes two arguments, so
- if you meant MAD, there's no MAD on platforms that don't have LRP
- if you meant MAC(dst, ...) I don't see
Kenneth Graunke writes:
> While the GL_ARB_stencil_texturing extension does not allow the creation
> of stencil textures, it does allow shaders to sample stencil values
> stored in packed depth/stencil textures.
>
> Specifically, applications can call glTexParameter* with a pname of
> GL_DEPTH_ST
Kenneth Graunke writes:
> When the vec4 backend encountered an ir_triop_lrp, it always emitted an
> actual LRP instruction, which only exists on Gen6+. Gen4-5 used
> lower_instructions() to decompose ir_triop_lrp at the IR level.
>
> Since commit 8d37e9915a3b21 ("glsl: Optimize open-coded lrp in
On Mon, Feb 24, 2014 at 06:02:23PM +0900, Michel Dänzer wrote:
> From: Michel Dänzer
>
> Signed-off-by: Michel Dänzer
> ---
> lib/Target/R600/SILowerControlFlow.cpp | 50
> ++
> test/CodeGen/R600/llvm.AMDGPU.kill.ll | 18
> 2 files changed, 50 inse
On Mon, Feb 24, 2014 at 9:43 AM, Ian Romanick wrote:
> On 02/22/2014 04:35 PM, Matt Turner wrote:
>>
>> Cc: "10.1"
>> ---
>> src/glsl/ir.h | 10 ++
>> 1 file changed, 10 insertions(+)
>>
>> diff --git a/src/glsl/ir.h b/src/glsl/ir.h
>> index e27e30a..55ec5ff 100644
>> --- a/src/glsl/ir
https://bugs.freedesktop.org/show_bug.cgi?id=75226
--- Comment #13 from Erich Hoover ---
Created attachment 94676
--> https://bugs.freedesktop.org/attachment.cgi?id=94676&action=edit
Other user's logs
Attached are the logs for the other user, it sounds like you guys already have
things figured
On 02/23/2014 07:19 PM, Kenneth Graunke wrote:
Tt's kind of a trap---calling do_common_optimization() after
lower_instructions() may cause opt_algebraic() to reintroduce
ir_triop_lrp expressions that were lowered, effectively defeating the
point. Because of this, nobody uses it.
Cc: "10.1"
Sig
On 02/22/2014 04:35 PM, Matt Turner wrote:
Cc: "10.1"
---
src/glsl/ir.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/glsl/ir.h b/src/glsl/ir.h
index e27e30a..55ec5ff 100644
--- a/src/glsl/ir.h
+++ b/src/glsl/ir.h
@@ -1458,6 +1458,16 @@ public:
}
/**
+* Re
https://bugs.freedesktop.org/show_bug.cgi?id=75445
Ian Romanick changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On 02/21/2014 07:15 PM, Kenneth Graunke wrote:
By adding "#define gvec4 %svec4" to the top of our fragment shader, we
can write generic code without needing to specialize it to vec4, ivec4,
or uvec4 via asprintf.
This also makes the INT and UNSIGNED_INT merge function code identical,
so I combin
On 24/02/14 17:04, Ilia Mirkin wrote:
> On Mon, Feb 24, 2014 at 11:46 AM, Emil Velikov
> wrote:
>> Signed-off-by: Emil Velikov
>
> Nice catch! Does this fix any piglits, or just fixes some
> XY_OUT_OF_BOUNDS errors? ms_x == ms_y for ms = 4, which oddly enough
> is where copyteximage passed (sho
On Mon, Feb 24, 2014 at 11:46 AM, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
Nice catch! Does this fix any piglits, or just fixes some
XY_OUT_OF_BOUNDS errors? ms_x == ms_y for ms = 4, which oddly enough
is where copyteximage passed (should have noticed that earlier, but
only made the con
Hi Adel,
Thanks for look at this; just a few comments...
> diff --git a/src/glx/glx_pbuffer.c b/src/glx/glx_pbuffer.c
> index 978730c..afb4206 100644
> --- a/src/glx/glx_pbuffer.c
> +++ b/src/glx/glx_pbuffer.c
> @@ -373,6 +373,14 @@ GetDrawableAttribute(Display * dpy, GLXDrawable drawable,
>
Signed-off-by: Emil Velikov
---
src/gallium/drivers/nouveau/nv50/nv50_transfer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/nv50/nv50_transfer.c
b/src/gallium/drivers/nouveau/nv50/nv50_transfer.c
index a990682..f716052 100644
--- a/src/gallium
Am 24.02.2014 09:38, schrieb Dave Airlie:
> I've taken a quick look at ARB_texture_query_levels,
>
> and the HW generally seems to implement this by returning it in the W
> channel of RESINFO, which would map to the TXQ instruction,
>
> So my initial implementation added a LEVQ TGSI instruction b
On Mon, Feb 24, 2014 at 2:05 AM, Thomas Hellstrom wrote:
> This avoids the kernel driver spewing out errors about the param not being
> supported.
>
> Also correct the max surface size used when the kernel does not support the
> query.
>
> Reported-by: Brian Paul
> Signed-off-by: Thomas Hellstrom
Am 24.02.2014 09:33, schrieb Dave Airlie:
> On Wed, Feb 12, 2014 at 9:10 AM, Roland Scheidegger
> wrote:
>> Am 11.02.2014 22:58, schrieb Dave Airlie:
>dst.z = texture_depth(unit, lod)
>
> +.. opcode:: TG4 - Texture Gather (as per ARB_texture_gather)
> + Gathers t
Am 23.02.2014 16:26, schrieb Emil Velikov:
> On 22/02/14 21:46, Roland Scheidegger wrote:
>>
>>
>> - Original Message -
>>> From: "Emil Velikov"
>>> To: "Roland Scheidegger" ,
>>> mesa-dev@lists.freedesktop.org
>>> Cc: "emil l velikov"
>>> Sent: Saturday, February 22, 2014 2:02:08 PM
>>>
https://bugs.freedesktop.org/show_bug.cgi?id=75356
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=75335
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On 24/02/14 14:26, Emil Velikov wrote:
> With the introduction of the pipe_loader_sw_probe_dri helper we
> require the sw/dri winsys during linking stage despite it being
> unused by any of the targets. This will cause a minor increase
> in the resulting library which will be cleaned up via linker
These four patches fix the OpenCL build errors.
Tested-by: Tom Stellard
On Sat, Feb 22, 2014 at 04:53:47PM +, Emil Velikov wrote:
> When one builds without gallium_drm_loader, the above function will
> not be available, thus we'll segfault in gallium_screen_create due
> to memory access viol
https://bugs.freedesktop.org/show_bug.cgi?id=75445
Priority: medium
Bug ID: 75445
Assignee: mesa-dev@lists.freedesktop.org
Summary: Mesa 10.2.0-devel implementation error: meta program
compile failed:
Severity: normal
Cla
With the introduction of the pipe_loader_sw_probe_dri helper we
require the sw/dri winsys during linking stage despite it being
unused by any of the targets. This will cause a minor increase
in the resulting library which will be cleaned up via linker
options with upcoming patches.
Cc: Tom Stellar
Using TXQ for this sounds good to me.
Marek
On Mon, Feb 24, 2014 at 9:38 AM, Dave Airlie wrote:
> I've taken a quick look at ARB_texture_query_levels,
>
> and the HW generally seems to implement this by returning it in the W
> channel of RESINFO, which would map to the TXQ instruction,
>
> So my
On Mon, Feb 24, 2014 at 5:03 PM, Ilia Mirkin wrote:
> On Mon, Feb 24, 2014 at 3:38 AM, Dave Airlie wrote:
>> I've taken a quick look at ARB_texture_query_levels,
>>
>> and the HW generally seems to implement this by returning it in the W
>> channel of RESINFO, which would map to the TXQ instructi
This avoids the kernel driver spewing out errors about the param not being
supported.
Also correct the max surface size used when the kernel does not support the
query.
Reported-by: Brian Paul
Signed-off-by: Thomas Hellstrom
Reviewed-by: Charmaine Lee
Cc: "10.1"
---
src/gallium/winsys/svga/d
From: Michel Daenzer
Signed-off-by: Michel Dänzer
---
src/gallium/drivers/radeonsi/si_shader.c | 16
1 file changed, 16 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 54270cd..b22db0b 100644
--- a/src/galli
On Mon, Feb 24, 2014 at 3:38 AM, Dave Airlie wrote:
> I've taken a quick look at ARB_texture_query_levels,
>
> and the HW generally seems to implement this by returning it in the W
> channel of RESINFO, which would map to the TXQ instruction,
>
> So my initial implementation added a LEVQ TGSI inst
From: Michel Dänzer
Signed-off-by: Michel Dänzer
---
lib/Target/R600/SILowerControlFlow.cpp | 50 ++
test/CodeGen/R600/llvm.AMDGPU.kill.ll | 18
2 files changed, 50 insertions(+), 18 deletions(-)
create mode 100644 test/CodeGen/R600/llvm.AMDGPU.kil
From: Michel Dänzer
If the SI_KILL operand is constant, we can either clear the exec mask if
the operand is negative, or do nothing otherwise.
Signed-off-by: Michel Dänzer
---
lib/Target/R600/SIInstructions.td | 4 ++--
lib/Target/R600/SILowerControlFlow.cpp | 25 ++--
I've taken a quick look at ARB_texture_query_levels,
and the HW generally seems to implement this by returning it in the W
channel of RESINFO, which would map to the TXQ instruction,
So my initial implementation added a LEVQ TGSI instruction but I do
think this isn't representative of the hw and
Before you read any further, this is nowhere close to working. However it's in
a state where I think most of the structure is there, albeit with a lot of XXX
comments. And I haven't actually implemented the new opcodes I've added.
I was hoping one or two Intel people could take a look at this and
On Wed, Feb 12, 2014 at 9:10 AM, Roland Scheidegger wrote:
> Am 11.02.2014 22:58, schrieb Dave Airlie:
dst.z = texture_depth(unit, lod)
+.. opcode:: TG4 - Texture Gather (as per ARB_texture_gather)
+ Gathers the four texels to be used in a bi-linear
+
89 matches
Mail list logo
