On 04/23/2014 11:39 PM, Matt Turner wrote:
> On Wed, Apr 23, 2014 at 11:25 PM, Kenneth Graunke
> wrote:
>> On 04/18/2014 11:56 AM, Matt Turner wrote:
>>> ---
>>> src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 19 +++
>>> 1 file changed, 15 insertions(+), 4 deletions(-)
>>
>> I origin
On 04/18/2014 11:56 AM, Matt Turner wrote:
> Wouldn't it be nice if case labels could be non-constant expressions.
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 77
> +---
> src/mesa/drivers/dri/i965/brw_fs.h | 10 ++---
> 2 files changed, 31 insertions(+), 56 d
On Wed, Apr 23, 2014 at 11:25 PM, Kenneth Graunke wrote:
> On 04/18/2014 11:56 AM, Matt Turner wrote:
>> ---
>> src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 19 +++
>> 1 file changed, 15 insertions(+), 4 deletions(-)
>
> I originally thought this needed to go earlier in the patch se
On 04/18/2014 11:56 AM, Matt Turner wrote:
> ---
> src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 19 +++
> 1 file changed, 15 insertions(+), 4 deletions(-)
I originally thought this needed to go earlier in the patch series,
since by this point you're emitting opcodes with more than 3
On 04/18/2014 11:56 AM, Matt Turner wrote:
> ---
> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 135
> +++
> 1 file changed, 73 insertions(+), 62 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
On 04/18/2014 11:56 AM, Matt Turner wrote:
> Clean up with with register_coalesce()/dead_code_eliminate().
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 37
>
> src/mesa/drivers/dri/i965/brw_fs.h | 1 +
> 2 files changed, 38 insertions(+)
>
> diff --git a
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
index e076e72..ef0
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 1 +
src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 1 -
2 files changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp
b/src/gallium/dr
Signed-off-by: Ilia Mirkin
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index ee8c54a..e87e761 100644
--- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
+++
Signed-off-by: Ilia Mirkin
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 9 +
src/gallium/include/pipe/p_shader_tokens.h | 3 ++-
2 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_ureg.c
b/src/gallium/auxiliary/tgsi/tgsi_ureg.c
index 38cce5
Signed-off-by: Ilia Mirkin
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 0d69c70..ee8c54a 100644
--- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
+++
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/codegen/nv50_ir.h | 1 +
src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 1 +
src/gallium/drivers/nouveau/codegen/nv50_ir_lowering_nvc0.cpp | 4
src/gallium/drivers/nouveau/codegen/nv50_ir_print.cpp
Signed-off-by: Ilia Mirkin
---
src/gallium/auxiliary/tgsi/tgsi_strings.c | 3 ++-
src/gallium/docs/source/tgsi.rst | 6 ++
src/gallium/include/pipe/p_shader_tokens.h | 3 ++-
3 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_strings.c
This adds support for gl_SampleMaskIn and gl_InvocationID system values to
gallium, mesa/st, and nvc0. The relevant piglit tests pass, except 2
gl_InvocationID-related ones that fail due to the linking being delayed by
mesa/st (I think). I wrote a lame one for gl_SampleMaskIn that just makes sure
t
---
src/gallium/auxiliary/tgsi/tgsi_strings.c| 5 -
src/gallium/docs/source/context.rst | 1 +
src/gallium/docs/source/screen.rst | 3 +++
src/gallium/docs/source/tgsi.rst | 20
src/gallium/drivers/freedreno/freedreno_s
---
src/gallium/auxiliary/cso_cache/cso_context.c | 19 +++
src/gallium/auxiliary/cso_cache/cso_context.h | 4
src/gallium/auxiliary/hud/hud_context.c | 3 +++
src/gallium/auxiliary/postprocess/pp_run.c| 3 +++
src/gallium/auxiliary/util/u_blit.c | 3 ++
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/codegen/nv50_ir.h | 7 +
.../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 13 +
.../drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp | 14 +
.../nouveau/codegen/nv50_ir_lowering_nvc0.cpp | 21 +
---
src/gallium/drivers/nouveau/codegen/nv50_ir.h | 1 +
.../drivers/nouveau/codegen/nv50_ir_driver.h | 3 +-
.../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 10 +-
.../nouveau/codegen/nv50_ir_lowering_nv50.cpp | 12
.../drivers/nouveau/codegen/nv50_ir_print.cpp
This is my latest iteration of the ARB_sample_shading implementation. The only
known defect is that gl_SampleMask doesn't appear to work on nv50 nor
nvc0. I'm fairly sure it's due to some bit of setup I'm missing, but it has
thus far eluded me. I believe I've addressed the various earlier review
co
On Apr 24, 2014, at 0:30 , Jan Vesely wrote:
> On Wed, 2014-04-23 at 19:49 +0200, Francisco Jerez wrote:
>> Jan Vesely writes:
>>
>>> On Tue, 2014-04-22 at 17:50 -0700, Matt Arsenault wrote:
>>>
>>>
>> I think this is what v96:128 is for
> according to [0], it specifies only alignment
On Wed, 2014-04-23 at 19:49 +0200, Francisco Jerez wrote:
> Jan Vesely writes:
>
> > On Tue, 2014-04-22 at 17:50 -0700, Matt Arsenault wrote:
> >
> >
> >> >> I think this is what v96:128 is for
> >> > according to [0], it specifies only alignment, not size. I could not
> >> > find an __attribute
https://bugs.freedesktop.org/show_bug.cgi?id=64386
--- Comment #3 from Götz ---
If this is a Mesa bug, what can we do with this hardware to help debug this?
This error message doesn't appear anymore with the latest mesa version:
"Mesa 9.1.1 implementation error: unexpected format GL_DEPTH_COMPON
Am 23.04.2014 23:10, schrieb Zack Rusin:
> Lets make draw_get_option_use_llvm function available unconditionally
> and use it to avoid useless allocations when LLVM paths are active.
> TGSI machine is never used when we're using LLVM.
>
> Signed-off-by: Zack Rusin
> ---
> src/gallium/auxiliary/d
>
> - tgsi_exec_machine_destroy(draw->vs.tgsi.machine);
> + if (draw_get_option_use_llvm())
> + tgsi_exec_machine_destroy(draw->vs.tgsi.machine);
That part should have used !draw_get_option_use_llvm()
___
mesa-dev mailing list
mesa-dev@lists.
Lets make draw_get_option_use_llvm function available unconditionally
and use it to avoid useless allocations when LLVM paths are active.
TGSI machine is never used when we're using LLVM.
Signed-off-by: Zack Rusin
---
src/gallium/auxiliary/draw/draw_context.c | 6 ++
src/gallium/auxiliary/d
https://bugs.freedesktop.org/show_bug.cgi?id=77582
Benjamin Bellec changed:
What|Removed |Added
Resolution|NOTOURBUG |INVALID
Assignee|mesa-dev@li
Jan Vesely writes:
> On Tue, 2014-04-22 at 17:50 -0700, Matt Arsenault wrote:
>
>
>> >> I think this is what v96:128 is for
>> > according to [0], it specifies only alignment, not size. I could not
>> > find an __attribute__ that would change size either.
>> >
>> > It should be possible to have
Carl Worth writes:
> After I picked this to the 10.1 branch and tested I found the following
> piglit regression:
>
> $ ~/src/piglit/bin/nv_conditional_render-blitframebuffer -auto
> Probe color at (0,16)
> Expected: 0.00 1.00 0.00 0.00
> Observed: 1.00
Richard, Michel,
Apologies for the long silence. I flagged this thread as worth following, but
I failed to noticed the RFC to me.
I glanced over it and the series looks good to me AFAICT. I agree that it is a
better to defer the endianess to C-preprocessing time.
Jose
- Original Message
From: Roland Scheidegger
GL (3.0) allows you to clear individual color buffers in a fb. In fact
for fbs containing both int and float/normalized color buffers this is
required (because the clearing values are otherwise undefined if applied
to all buffers). The gallium interface was changed a whil
From: Roland Scheidegger
util_color often merely represents a collection of bytes, however it is
inconvenient if those bytes can only be accessed as floats/doubles for int
formats exceeding 32bits.
(Note that since rgba8 formats use one uint, not 4 bytes, hence the byte and
short member were left
On Fri, Apr 11, 2014 at 10:37 AM, Matt Turner wrote:
> Someone asked about my Mesa build set up. Rather than sending it
> privately I figured I'd post it for posterity on mesa-dev.
>
> I build with
>
> CFLAGS="-O2 -march=native -pipe" CXXFLAGS="$CFLAGS" ./autogen.sh
> --with-dri-drivers=i965 --wit
Am 23.04.2014 17:22, schrieb Brian Paul:
> On 04/23/2014 09:17 AM, Jose Fonseca wrote:
>> Thanks for the review.
>>
>> - Original Message -
>>> On 04/23/2014 07:55 AM, jfons...@vmware.com wrote:
From: José Fonseca
This prevents buffer overflow w/ llvmpipe when running piglit
Fixes a segmentation fault in conform divzero.c test.
This happens when glTexImage(level, width=0, height=0) is called. We
don't allocate texture memory in that case so the ImageSlices array
was never allocated.
Cc: "10.1"
---
src/mesa/swrast/s_texture.c | 10 ++
1 file changed, 10 in
And add a const qualifier.
---
src/mesa/swrast/s_texture.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/swrast/s_texture.c b/src/mesa/swrast/s_texture.c
index 1d449a2..cbfa26b 100644
--- a/src/mesa/swrast/s_texture.c
+++ b/src/mesa/swrast/s_texture.c
@@ -17
There's no reason to compute texel size, stride, etc. if there's no
image data to map.
---
src/mesa/swrast/s_texture.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/src/mesa/swrast/s_texture.c b/src/mesa/swrast/s_texture.c
index cbfa26b..9273e94 100644
--- a
---
src/mesa/swrast/s_texture.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/src/mesa/swrast/s_texture.c b/src/mesa/swrast/s_texture.c
index 9273e94..5fd80ca 100644
--- a/src/mesa/swrast/s_texture.c
+++ b/src/mesa/swrast/s_texture.c
@@ -269,7 +269,7 @@ _swrast
3 element vectors have the size of 4 element ones.
See Ch 6.1.5 of OCL 1.1+ specs
Thx Matt Arsenault for the hint
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/cl
On Tue, 2014-04-22 at 17:50 -0700, Matt Arsenault wrote:
> >> I think this is what v96:128 is for
> > according to [0], it specifies only alignment, not size. I could not
> > find an __attribute__ that would change size either.
> >
> > It should be possible to have ADMGPUDataLayout: public DataLa
On 04/23/2014 09:17 AM, Jose Fonseca wrote:
Thanks for the review.
- Original Message -
On 04/23/2014 07:55 AM, jfons...@vmware.com wrote:
From: José Fonseca
This prevents buffer overflow w/ llvmpipe when running piglit
bin/gl-3.2-layered-rendering-clear-color-all-types 1d_array
Thanks for the review.
- Original Message -
> On 04/23/2014 07:55 AM, jfons...@vmware.com wrote:
> > From: José Fonseca
> >
> > This prevents buffer overflow w/ llvmpipe when running piglit
> >
> >bin/gl-3.2-layered-rendering-clear-color-all-types 1d_array single_level
> >-fbo -au
On Wed, Apr 23, 2014 at 08:40:22AM -0600, Brian Paul wrote:
> On 04/23/2014 08:18 AM, ville.syrj...@linux.intel.com wrote:
> > From: nick
> >
> > Straightforward fix to properly load dest->color with color data, as
> > opposed to position data as previously implemented.
> >
> > [vsyrjala: I notice
https://bugs.freedesktop.org/show_bug.cgi?id=77749
--- Comment #4 from Brian Paul ---
The patch looks OK to me. I'm always happy to get rid of old system-specific
crud.
But I can't apply the patch here to test it:
$ patch -p0 < ~/mesaport.diff
patching file include/GL/gl.h
patch: malform
https://bugs.freedesktop.org/show_bug.cgi?id=77789
Brian Paul changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop. |sitewranglers@lists.freedes
On 04/23/2014 08:18 AM, ville.syrj...@linux.intel.com wrote:
From: nick
Straightforward fix to properly load dest->color with color data, as
opposed to position data as previously implemented.
[vsyrjala: I noticed the patch languishing in bugzilla.
It looks correct to me so I refreshed it fo
> -Original Message-
> From: Tom Stellard>
> On Wed, Apr 23, 2014 at 01:27:11PM +, Dorrington, Albert wrote:
> > When I try to call clEnqueueReadImage(), after a
> clEnqueueNDRangeKernel(); the clover/aop/transfer.cpp again generates a
> Pixel Shader, which gets integrated into the
On 04/23/2014 07:55 AM, jfons...@vmware.com wrote:
From: José Fonseca
This prevents buffer overflow w/ llvmpipe when running piglit
bin/gl-3.2-layered-rendering-clear-color-all-types 1d_array single_level
-fbo -auto
v2: Compute the framebuffer size as the minimum size, as pointed out by
B
On 04/23/2014 07:55 AM, jfons...@vmware.com wrote:
From: José Fonseca
Courtesy of Clang static analyzer.
---
src/gallium/auxiliary/util/u_debug.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_debug.c
b/src/gallium/auxiliary/util/u_d
From: nick
Straightforward fix to properly load dest->color with color data, as
opposed to position data as previously implemented.
[vsyrjala: I noticed the patch languishing in bugzilla.
It looks correct to me so I refreshed it for master.
Looks like it's a regression introduced in:
commit
From: José Fonseca
This prevents buffer overflow w/ llvmpipe when running piglit
bin/gl-3.2-layered-rendering-clear-color-all-types 1d_array single_level -fbo
-auto
v2: Compute the framebuffer size as the minimum size, as pointed out by
Brian; compacted code; ran piglit quick test list (wi
From: José Fonseca
Courtesy of Clang static analyzer.
---
src/gallium/auxiliary/util/u_debug.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_debug.c
b/src/gallium/auxiliary/util/u_debug.c
index fe51717..dc840e8 100644
--- a/src/gallium/
On Wed, Apr 23, 2014 at 01:27:11PM +, Dorrington, Albert wrote:
> In trying to implement Image support in Clover, I have discovered that the
> existing CL image related calls result in the generation of Pixel Shader
> sequences for copies of images to and from the GPU.
>
> I initially though
In trying to implement Image support in Clover, I have discovered that the
existing CL image related calls result in the generation of Pixel Shader
sequences for copies of images to and from the GPU.
I initially thought that this would be fine, and was able to implement image
read tests that us
Signed-off-by: Petri Latvala
---
src/mesa/drivers/dri/i965/intel_extensions.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 892a048..d6e1494 100644
--- a/src/mesa/drivers/dr
Signed-off-by: Petri Latvala
Reviewed-by: Ian Romanick
---
src/mesa/main/tests/enum_strings.cpp | 18 ++
1 file changed, 18 insertions(+)
diff --git a/src/mesa/main/tests/enum_strings.cpp
b/src/mesa/main/tests/enum_strings.cpp
index 3795700..d16eb36 100644
--- a/src/mesa/main/t
Signed-off-by: Petri Latvala
---
docs/relnotes/10.2.html | 1 +
1 file changed, 1 insertion(+)
diff --git a/docs/relnotes/10.2.html b/docs/relnotes/10.2.html
index d7d557b..473739c 100644
--- a/docs/relnotes/10.2.html
+++ b/docs/relnotes/10.2.html
@@ -47,6 +47,7 @@ Note: some of the new features
Third revision of the patch series. Changes:
- Rebased to current master
- Changes based on Ian's review
- Add the extension to 10.2 release notes
I didn't change patch 5/6 "Enable INTEL_performance_query for Gen5+"
along the review comments yet. It's true that currently drivers can
support both
Using the existing driver hooks made for AMD_performance_monitor, implement
INTEL_performance_query functions.
v2: Whitespace changes.
v3: Whitespace changes, add a _mesa_warning()
Signed-off-by: Petri Latvala
Reviewed-by: Ian Romanick
---
src/mesa/main/performance_monitor.c | 487
Like AMD_performance_monitor, this extension provides an interface for
applications (and OpenGL-based tools) to access GPU performance
counters. Since the exact performance counters available vary between
vendors and hardware generations, the extension provides an API the
application can use to get
Ping
Richard Sandiford writes:
> Ping (with fixed subject)
>
> Richard Sandiford writes:
>> This is a refresh of:
>>
>>http://lists.freedesktop.org/archives/mesa-dev/2013-June/040594.html
>>
>> At the moment the python code uses sys.byteorder to decide whether
>> u_format_table.c should be f
Carl Worth writes:
> After I picked this to the 10.1 branch and tested I found the following
> piglit regression:
...
> So I'm moving this patch off of the "applied" queue and onto a new
> "rejected" queue as can be seen here:
Actually, I replied to the wrong email here. The patch I bisected to w
Anuj Phogat writes:
> + /* Page 679 of OpenGL 4.4 spec says:
> +*"Added BlitFramebuffer to commands affected by conditional
> rendering in
> +* section 10.10 (Bug 9562)."
> +*/
> + if (!_mesa_check_conditional_render(ctx))
> + return; /* Do not blit */
> +
> if (!
62 matches
Mail list logo
