Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/main/extensions.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index 553c01e..43f0e8d 100644
--- a/src/mesa/main/extensions.c
+++ b/src/mesa/main/extensions.c
@@ -609,6 +609,15
verify get_xcb_screen() returned pointer before using it.
Signed-off-by: Juha-Pekka Heikkila
---
src/egl/drivers/dri2/platform_x11.c | 30 --
1 file changed, 24 insertions(+), 6 deletions(-)
diff --git a/src/egl/drivers/dri2/platform_x11.c
b/src/egl/drivers/dri2/pla
On default fallback path there was null access on src_rb
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/drivers/common/meta_blit.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/src/mesa/drivers/common/meta_blit.c
b/src/mesa/drivers/common/meta_blit.c
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/program/symbol_table.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/src/mesa/program/symbol_table.c b/src/mesa/program/symbol_table.c
index 9462978..5b22745 100644
--- a/src/mesa/program/symbol_table.c
+++ b/sr
If _mesa_get_tex_image() return NULL there is already error
set in context. Other error pats free allocated texture.
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/drivers/common/meta_copy_image.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/mesa/drivers/common/meta_copy_ima
Signed-off-by: Juha-Pekka Heikkila
---
src/loader/loader.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/loader/loader.c b/src/loader/loader.c
index bdd3906..94c993a 100644
--- a/src/loader/loader.c
+++ b/src/loader/loader.c
@@ -594,6 +594,9 @@ libudev_get_device_name_for_fd(int fd)
Klocwork set again, rebased and added with fixes for meta and egl.
/Juha-Pekka
Juha-Pekka Heikkila (11):
glsl: add missing null check in tfeedback_decl::init()
mesa: add null checks in symbol_table.c
i965: Avoid null access in intelMakeCurrent()
glsl: Check calloc return value in link_int
separate two null checks connected with && to their own if branches.
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/drivers/dri/i965/brw_context.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/mesa/drivers/dri/i965/
Signed-off-by: Juha-Pekka Heikkila
---
src/glsl/link_varyings.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index 54ceae1..a738e2f 100644
--- a/src/glsl/link_varyings.cpp
+++ b/src/glsl/link_varyings.cpp
@@ -318,6 +318,11 @@ t
Signed-off-by: Juha-Pekka Heikkila
---
src/glsl/ir_function.cpp | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/glsl/ir_function.cpp b/src/glsl/ir_function.cpp
index 98bec45..2b2643c 100644
--- a/src/glsl/ir_function.cpp
+++ b/src/glsl/ir_function.cpp
@@ -24,6
Check calloc return value while adding build-in functions.
Signed-off-by: Juha-Pekka Heikkila
---
src/glsl/linker.cpp | 15 +++
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index 57be493..abc1035 100644
--- a/src/glsl/linker
Check calloc returned requested memory.
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/drivers/dri/i965/intel_buffer_objects.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/intel_buffer_objects.c
b/src/mesa/drivers/dri/i965/intel_buffer_objects.c
index 96dacd
https://bugs.freedesktop.org/show_bug.cgi?id=83195
--- Comment #3 from Stanisław Halik ---
Just to be extra-clear:
Talking about GPU reset procedure.
The one where dmesg prints register state.
Is that enough context or is further info needed?
-sh
--
You are receiving this mail because:
You
On 09/09/2014 09:10 AM, Tapani Pälli wrote:
> Hi Frank;
>
> Kalyan sent me a similar fix mentioning that it is related to this test
> case:
> https://www.khronos.org/registry/webgl/sdk/tests/conformance/limits/gl-min-textures.html
>
> If your patch is related, please mention in the commit message.
Hi Frank;
Kalyan sent me a similar fix mentioning that it is related to this test
case:
https://www.khronos.org/registry/webgl/sdk/tests/conformance/limits/gl-min-textures.html
If your patch is related, please mention in the commit message. It helps
to track possible regression in the future.
Wi
From: Michel Dänzer
This lets the kernel know that such BOs can be pinned outside of the CPU
accessible part of VRAM.
Signed-off-by: Michel Dänzer
---
src/gallium/drivers/radeon/r600_buffer_common.c | 1 +
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 12 +---
src/gallium/winsys/r
https://bugs.freedesktop.org/show_bug.cgi?id=83636
Vladimir Ysikov changed:
What|Removed |Added
CC||granti...@gmail.com
--
You are receiv
Am 05.09.2014 00:14, schrieb Glenn Kennard:
Also fixes two sided lighting which was broken at least
on pre-evergreen by commit b1eb00.
Maybe there is a (Mesa-demos) test case around?
Signed-off-by: Glenn Kennard
---
Changes since patch v1:
Factor out and set sample positions also for pre-eve
On Mon, Sep 8, 2014 at 6:21 PM, Matt Turner wrote:
> On Mon, Sep 8, 2014 at 4:56 PM, Ausmus, James wrote:
>> Thanks Emil - works great!
>>
>> Tested-by: James Ausmus
>
> You know, if you care about image size, you shouldn't be building
> Gallium at all. Neither of the two Intel drivers use it.
On Mon, Sep 8, 2014 at 4:56 PM, Ausmus, James wrote:
> Thanks Emil - works great!
>
> Tested-by: James Ausmus
You know, if you care about image size, you shouldn't be building
Gallium at all. Neither of the two Intel drivers use it.
___
mesa-dev mailin
On 09.09.2014 09:47, Michel Dänzer wrote:
> On 09.09.2014 02:36, Alex Deucher wrote:
>>
>> Updated version with comments integrated.
>
> [...]
>
>> @@ -314,10 +314,12 @@ int radeon_bo_pin_restricted(struct radeon_bo
>> *bo, u32 domain, u64 max_offset,
>> unsigned lpfn = 0;
>>
>>
Signed-off-by: Ilia Mirkin
---
Erm... oops, that will obviously modify 10.4.html relnotes, not 10.3. But no
one looks at these nouveau changes anyways...
docs/GL3.txt | 2 +-
docs/relnotes/10.3.html| 1 +
src/gallium/drivers/nouveau/nv50
Signed-off-by: Ilia Mirkin
---
Roland, it was unclear to me whether you were happy with this change or not.
v2 -> v3:
no change
src/mesa/state_tracker/st_atom_texture.c | 30
src/mesa/state_tracker/st_cb_fbo.c | 10 ++
src/mesa/state_tracker/st_cb_texture.c | 62 +
This allows a sampler view to have a different texture target than the
underlying resource. This will be used to implement the type casting
between 2d arrays and cube maps as specified in ARB_texture_view.
Signed-off-by: Ilia Mirkin
Reviewed-by: Roland Scheidegger
---
Roland thought this change
On 09.09.2014 02:36, Alex Deucher wrote:
Updated version with comments integrated.
[...]
@@ -314,10 +314,12 @@ int radeon_bo_pin_restricted(struct radeon_bo *bo, u32
domain, u64 max_offset,
unsigned lpfn = 0;
/* force to pin into visible video ram */
-
Thanks Emil - works great!
Tested-by: James Ausmus
On Mon, Sep 8, 2014 at 3:52 PM, Emil Velikov
wrote:
> With the gallium megadrivers we've converted most ST to optionally
> use either statically linked in or shared pipe-drivers.
>
> The hardcoded switch forgot to conditionally enable the buil
On Sat, Sep 6, 2014 at 4:49 PM, Dave Airlie wrote:
> coverity reported a use before NULL check for src_rb in the below code,
>
> maybe it was there already, but it noticed it now you changed it,
>
> we check src_rb->NumSamples then later src_rb is checked. Could you take a
> look.
>
Yes, the issu
https://bugs.freedesktop.org/show_bug.cgi?id=83636
Priority: medium
Bug ID: 83636
Assignee: mesa-dev@lists.freedesktop.org
Summary: [LLVM 3.5] Mesa32 build failure with LLVM 3.5.
Severity: normal
Classification: Unclassified
Superseded by HAVE_LOADER_GALLIUM. The latter has a *DRM* brethren
making the whose easier on which one to keep.
Signed-off-by: Emil Velikov
---
configure.ac | 1 -
src/gallium/Makefile.am | 2 +-
src/gallium/auxiliary/Makefile.am
With the gallium megadrivers we've converted most ST to optionally
use either statically linked in or shared pipe-drivers.
The hardcoded switch forgot to conditionally enable the build of the
shared pipe-drivers which resulted in them being constantly build.
Cc: "10.3"
Cc: James Ausmus
Reported
https://bugs.freedesktop.org/show_bug.cgi?id=83631
--- Comment #2 from Orion Poplawski ---
/usr/include/GL/glext.h:#define GL_GLEXT_VERSION 20140810
But we're including "GL/glx.h" which includes "GL/glxext.h", not "GL/glext.h".
Too many x's floating around here.
# grep '^typedef .* GLintptr;'
https://bugs.freedesktop.org/show_bug.cgi?id=83631
--- Comment #1 from Brian Paul ---
Which version of glext.h (look for "GL_GLEXT_VERSION")?
In version 20140810 it's defined on line 469:
typedef ptrdiff_t GLintptr;
--
You are receiving this mail because:
You are the assignee for the bug.
___
Le 02/05/2014 00:52, Martin Peres a écrit :
Hello,
I have the pleasure to announce that the X.org Developer Conference 2014
will
be held in Bordeaux, France from October 8th to October 10th. The venue is
located in the campus of the University of Bordeaux 1, in the computer
science
research lab
On Mon, Sep 8, 2014 at 2:28 PM, Kenneth Graunke wrote:
> Excess conversions considered harmful.
>
> Recently Matt reworked the boolean uniform handling to use the value of
> UniformBooleanTrue, rather than integer 1, when uploading uniforms:
>
> mesa: Upload boolean uniforms using UniformBoole
Looks good to me. Matches what ErrorMessageF is doing.
Reviewed-by: Courtney Goeltzenleuchter
On Tue, Aug 26, 2014 at 2:47 PM, Johannes Obermayr
wrote:
> From: Stefan Dirsch
>
> Let's handle LIBGL_DEBUG env. variable in Mesa in a consistent way.
>
> Fixes: https://bugzilla.novell.com/show_bug
Excess conversions considered harmful.
Recently Matt reworked the boolean uniform handling to use the value of
UniformBooleanTrue, rather than integer 1, when uploading uniforms:
mesa: Upload boolean uniforms using UniformBooleanTrue.
glsl: Use UniformBooleanTrue value for uniform initial
https://bugs.freedesktop.org/show_bug.cgi?id=83631
Priority: medium
Bug ID: 83631
Assignee: mesa-dev@lists.freedesktop.org
Summary: /usr/include/GL/glxext.h:480:143: error: 'GLintptr'
has not been declared
Severity: normal
Le 08/09/2014 21:11, Grigori Goronzy a écrit :
On 08.09.2014 21:07, Axel Davy wrote:
On 08/09/2014 20:21, Grigori Goronzy wrote :
On 08.09.2014 14:50, Axel Davy wrote:
Hi,
When reading si_dma.c code, it looks like the requested width of the
copy is ignored except for PIPE_BUFFER.
Perhaps that
Hi,
Thanks to Richard for these patches. Well done! Are these patches in the
latest Mesa version?
Rgds,
Christian
On 07/22/2014 03:51 AM, Richard Sandiford wrote:
>> MESA_FORMAT_LnAn_* puts the luminance in the low part of the integer and
>> the alpha in the high part. The same goes for MES
Given that this was an unintentional relaxation in the 4.2 spec (and
so there is no non-NV way to invoke this behavior), I don't think we
want the line in the 4.2 block of GL3.txt.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.free
https://bugs.freedesktop.org/show_bug.cgi?id=83570
--- Comment #11 from Roland Scheidegger ---
(In reply to comment #10)
> Could be totally borked, but this is what I have at the moment:
>
>if (type.floating) {
> return LLVMBuildFDiv(builder, a, b, "");
>}
>else {
> //The
On Tue, Sep 2, 2014 at 9:34 PM, Matt Turner wrote:
> When the instructions aren't in a flat list, this wouldn't have worked.
> Also, this should be faster.
> ---
> src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 28
> +++
> 1 file changed, 13 insertions(+), 15 deletions(-
When we find a mov.sat, we search backwards. We might as well search
everything else backwards as well and potentially look at fewer
instructions.
This change enables the next patch.
---
src/mesa/drivers/dri/i965/brw_fs_saturate_propagation.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 delet
---
src/mesa/drivers/dri/i965/brw_fs_saturate_propagation.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_saturate_propagation.cpp
b/src/mesa/drivers/dri/i965/brw_fs_saturate_propagation.cpp
index 6f7fb6c..347a78e 100644
--- a/src/mesa/dr
When an instruction's result was consumed by multiple mov.sat
instructions, we would decide that we couldn't move the saturate
modifier because something else was using the result, even though it was
just another mov.sat!
total instructions in shared programs: 4275598 -> 4274842 (-0.02%)
instructi
Helps a handful of programs in Serious Sam 3 that use do-while loops.
instructions in affected programs: 16114 -> 16075 (-0.24%)
---
This patch depends on the save-cfg series.
.../dri/i965/brw_fs_peephole_predicated_break.cpp | 37 ++
1 file changed, 37 insertions(+)
di
pow(x, y) is equivalent to exp(log(x) * y).
instructions in affected programs: 578 -> 458 (-20.76%)
---
src/glsl/opt_algebraic.cpp | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 447618f..0cdb8ec 100644
--- a/sr
Do we have a piglit to confirm unclamped Z values really work with
current hardware drivers?
Marek
On Mon, Sep 8, 2014 at 6:39 PM, Mathias Fröhlich
wrote:
>
> Hi,
>
> On Monday, September 08, 2014 13:16:27 Roland Scheidegger wrote:
>> > Add support for the unclamped versions of glDepthRange
>> >
On 08.09.2014 21:07, Axel Davy wrote:
> On 08/09/2014 20:21, Grigori Goronzy wrote :
>> On 08.09.2014 14:50, Axel Davy wrote:
>>> Hi,
>>>
>>> When reading si_dma.c code, it looks like the requested width of the
>>> copy is ignored except for PIPE_BUFFER.
>>> Perhaps that explains the bugs observed
On 08/09/2014 20:21, Grigori Goronzy wrote :
On 08.09.2014 14:50, Axel Davy wrote:
Hi,
When reading si_dma.c code, it looks like the requested width of the
copy is ignored except for PIPE_BUFFER.
Perhaps that explains the bugs observed ?
It isn't ignored. Partial DMA copies (i.e. operations t
On 08.09.2014 14:50, Axel Davy wrote:
> Hi,
>
> When reading si_dma.c code, it looks like the requested width of the
> copy is ignored except for PIPE_BUFFER.
> Perhaps that explains the bugs observed ?
>
It isn't ignored. Partial DMA copies (i.e. operations that do not copy
whole lines) are simp
Extension enables doing a multisample buffer resolve and buffer
scaling using a single glBlitFrameBuffer() call. Currently, we
have this extension implemented in BLORP which is only used by
SNB and IVB. This patch implements the extension in meta path
which makes it available to Broadwell.
Impleme
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/intel_extensions.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 046d2a1..10fe10e 100644
--- a/src/mesa/drivers/dri/i
Hi Mathias
On 08/09/14 17:39, Mathias Fröhlich wrote:
>
> Hi,
>
> On Monday, September 08, 2014 13:16:27 Roland Scheidegger wrote:
>>> Add support for the unclamped versions of glDepthRange
>>> and relatives. Also starting with OpenGL 4.2 the traditional
>>> functions for this should no longer cl
Series looks good to me too.
Jose
From: srol...@vmware.com
Sent: 08 September 2014 12:56
To: Jose Fonseca; mesa-dev@lists.freedesktop.org
Cc: Roland Scheidegger
Subject: [PATCH 3/3] gallivm: (trivial) don't try to use rcp when the division
1/x is integer
v2: Do not demote items that are already in the pool
Signed-off-by: Niels Ole Salscheider
---
src/gallium/drivers/r600/evergreen_compute.h | 1 +
src/gallium/drivers/r600/r600_blit.c | 59
2 files changed, 43 insertions(+), 17 deletions(-)
diff --git a/src/
On Mon, Sep 8, 2014 at 5:43 AM, Pohjolainen, Topi
wrote:
> On Tue, Sep 02, 2014 at 09:34:17PM -0700, Matt Turner wrote:
>> ---
>> .../drivers/dri/i965/brw_schedule_instructions.cpp | 74
>> --
>> 1 file changed, 42 insertions(+), 32 deletions(-)
>>
>> diff --git a/src/mesa/dr
On Thu, Aug 28, 2014 at 9:46 PM, Michel Dänzer wrote:
> On 29.08.2014 00:01, Alex Deucher wrote:
>> On Thu, Aug 28, 2014 at 4:57 AM, Christian König
>> wrote:
>>> Am 28.08.2014 um 08:56 schrieb Michel Dänzer:
>>>
From: Michel Dänzer
This flag is a hint that userspace expects the B
https://bugs.freedesktop.org/show_bug.cgi?id=83570
--- Comment #10 from rcond...@hotmail.com ---
Could be totally borked, but this is what I have at the moment:
if (type.floating) {
return LLVMBuildFDiv(builder, a, b, "");
}
else {
//The spec says divide by zero is undefined,
Hi,
On Monday, September 08, 2014 13:16:27 Roland Scheidegger wrote:
> > Add support for the unclamped versions of glDepthRange
> > and relatives. Also starting with OpenGL 4.2 the traditional
> > functions for this should no longer clamp the values to [0, 1].
> This looks wrong to me (the NV_dep
On Monday 08 September 2014, 15:19:15, Bruno Jimenez wrote:
> Hi,
>
> I'm not sure if this will work. Imagine this case:
>
> We have an item in the pool, and we want to use
> r600_resource_copy_region with it, for example because we want to demote
> it. This will call r600_copy_global_buffer, an
https://bugs.freedesktop.org/show_bug.cgi?id=83567
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Monday, September 08, 2014 11:10:42 AM Tapani Pälli wrote:
> From: Kalyan Kondapally
>
> According to GLSL-ES Spec(i.e. 1.0, 3.0), gl_Position value is undefined
> after the vertex processing stage if we don't write gl_Position. However,
> GLSL 1.10 Spec mentions that writing to gl_Position is
https://bugs.freedesktop.org/show_bug.cgi?id=83573
Kenneth Graunke changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|mesa-dev@l
https://bugs.freedesktop.org/show_bug.cgi?id=83573
--- Comment #2 from Ilia Mirkin ---
On quick inspection that's happening in
_mesa_propagate_uniforms_to_driver_storage
The format gets set to uniform_bool_float which in turn goes through
((float *) dst)[c] = (float) *isrc;
--
You are receiv
https://bugs.freedesktop.org/show_bug.cgi?id=83573
Kenneth Graunke changed:
What|Removed |Added
CC||bri...@vmware.com,
|
https://bugs.freedesktop.org/show_bug.cgi?id=83570
--- Comment #9 from Roland Scheidegger ---
(In reply to comment #8)
> for the llvm version I was going to return bld->undef. I figure I might as
> well change tgsi_exec.c version to 0x for consistency...but I don't
> have strong feelings
Gents,
Can anyone review this trivial patch ?
Stefan, Johannes
Why do I need to register in order to view the bugreport ? Afaict it is not a
security issue, so I believe the bug should be publicly accessible.
-Emil
On 26/08/14 21:47, Johannes Obermayr wrote:
> From: Stefan Dirsch
>
> Let's ha
On Mon, Sep 8, 2014 at 9:50 AM, Christian König wrote:
> From: Christian König
>
> As long as we don't have a workaround for frame based
> decoding in VDPAU we should not advertise NV_vdpau_interop.
>
> v2: fix commit message, check if get_video_param is present
>
> Signed-off-by: Christian König
On 09/08/2014 05:56 AM, srol...@vmware.com wrote:
From: Roland Scheidegger
This would just crash. Noticed by accident while checking int divisions by zero
with a quickly hacked piglit test.
---
src/gallium/auxiliary/gallivm/lp_bld_arit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
From: Christian König
As long as we don't have a workaround for frame based
decoding in VDPAU we should not advertise NV_vdpau_interop.
v2: fix commit message, check if get_video_param is present
Signed-off-by: Christian König
Cc: mesa-sta...@lists.freedesktop.org
---
src/mesa/state_tracker/s
On 09/06/2014 05:00 PM, Vinson Lee wrote:
This patch fixes a build error on DragonFly.
CC libpipe_loader_la-pipe_loader_drm.lo
pipe_loader_drm.c: In function 'pipe_loader_drm_probe':
pipe_loader_drm.c:207:10: error: implicit declaration of function 'close'
[-Werror=implicit-function-de
On Tue, Sep 02, 2014 at 09:34:22PM -0700, Matt Turner wrote:
> When the instructions aren't in a flat list, this wouldn't have worked.
> Also, this should be faster.
> ---
> src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 28
> +++
> 1 file changed, 13 insertions(+), 15 de
Hi,
I'm not sure if this will work. Imagine this case:
We have an item in the pool, and we want to use
r600_resource_copy_region with it, for example because we want to demote
it. This will call r600_copy_global_buffer, and with your patch it will
call r600_compute_global_demote_or_alloc, which
https://bugs.freedesktop.org/show_bug.cgi?id=83570
--- Comment #8 from rcond...@hotmail.com ---
for the llvm version I was going to return bld->undef. I figure I might as well
change tgsi_exec.c version to 0x for consistency...but I don't have
strong feelings either way.
--
You are recei
On Tue, Sep 02, 2014 at 09:34:21PM -0700, Matt Turner wrote:
> ---
> src/mesa/drivers/dri/i965/brw_shader.cpp| 2 +-
> src/mesa/drivers/dri/i965/brw_vec4.cpp | 14 +++---
> src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp | 3 ++-
> src/mesa/driver
On Tue, Sep 02, 2014 at 09:34:19PM -0700, Matt Turner wrote:
> Now that nothing invalidates the CFG, we can calculate_cfg() immediately
> after emit_fb_writes()/emit_thread_end() and never again.
Reviewed-by: Topi Pohjolainen
___
mesa-dev mailing list
m
On Tue, Sep 02, 2014 at 09:34:20PM -0700, Matt Turner wrote:
> The only trick is changing a break into a return true in register
> coalescing, since the macro is actually a double loop, and break will do
> something different than you expect. (Wish I'd realized that earlier!)
I didn't double check
On Tue, Sep 02, 2014 at 09:34:18PM -0700, Matt Turner wrote:
> Everything has been converted to preserve the CFG.
> ---
Reviewed-by: Topi Pohjolainen
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/
https://bugs.freedesktop.org/show_bug.cgi?id=83463
Abdiel Janulgue changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop. |xynop...@gmail.com
|
Hi,
When reading si_dma.c code, it looks like the requested width of the
copy is ignored except for PIPE_BUFFER.
Perhaps that explains the bugs observed ?
Axel Davy
On 06/09/2014 19:14, Andy Furniss wrote :
Marek Olšák wrote:
From: Marek Olšák
---
Turn this on, run piglit, and pray for m
On Tue, Sep 02, 2014 at 09:34:17PM -0700, Matt Turner wrote:
> ---
> .../drivers/dri/i965/brw_schedule_instructions.cpp | 74
> --
> 1 file changed, 42 insertions(+), 32 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> b/src/mesa/driver
https://bugs.freedesktop.org/show_bug.cgi?id=83570
--- Comment #7 from Roland Scheidegger ---
(In reply to comment #6)
> (In reply to comment #5)
> > Actually, I think it would be a good idea if you could return 0x
> > instead of 0. This is typically what GPUs will do, since the d3d10 doc
https://bugs.freedesktop.org/show_bug.cgi?id=83570
--- Comment #6 from Roland Scheidegger ---
(In reply to comment #5)
> Actually, I think it would be a good idea if you could return 0x
> instead of 0. This is typically what GPUs will do, since the d3d10 docs
> claim this is required, bot
https://bugs.freedesktop.org/show_bug.cgi?id=83570
--- Comment #5 from Roland Scheidegger ---
Actually, I think it would be a good idea if you could return 0x
instead of 0. This is typically what GPUs will do, since the d3d10 docs claim
this is required, both for div and mod -
http://msdn
From: Roland Scheidegger
Forgot to add it when I fixed up the start instance handling in (llvm) draw.
---
docs/GL3.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index f5d5e72..f02ad67 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -138,7 +13
From: Roland Scheidegger
This would just crash. Noticed by accident while checking int divisions by zero
with a quickly hacked piglit test.
---
src/gallium/auxiliary/gallivm/lp_bld_arit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/gallivm/lp_bld_ari
From: Roland Scheidegger
Calling the variable min when it's really max and vice versa seems a bit
confusing.
---
src/gallium/auxiliary/gallivm/lp_bld_arit.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/gallium/auxiliary/gallivm/lp_bld_arit.c
b/src/gal
https://bugs.freedesktop.org/show_bug.cgi?id=83570
--- Comment #4 from Roland Scheidegger ---
Ok, I just was thinking it is probably something along these lines, and could
reproduce this with some hacked up piglit test doing integer division by zero.
Though I got a SIGFPE instead, I guess that's
https://bugs.freedesktop.org/show_bug.cgi?id=81174
--- Comment #15 from Florian Link ---
In addition to the above bug, I found that the stippling pattern is reset when
the intermediate buffer is commited. Thus, GL_LINE_STRIP and GL_LINE_LOOP
stipple patterns jump at the same places where the GL_L
Am 06.09.2014 19:17, schrieb Mathias Froehlich:
>
> Hi,
>
> Please review:
>
> Add support for the unclamped versions of glDepthRange
> and relatives. Also starting with OpenGL 4.2 the traditional
> functions for this should no longer clamp the values to [0, 1].
This looks wrong to me (the NV_de
https://bugs.freedesktop.org/show_bug.cgi?id=83596
--- Comment #1 from Michel Dänzer ---
Please submit the patch directly to the mesa-dev mailing list.
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
From: Kalyan Kondapally
According to GLSL-ES Spec(i.e. 1.0, 3.0), gl_Position value is undefined
after the vertex processing stage if we don't write gl_Position. However,
GLSL 1.10 Spec mentions that writing to gl_Position is mandatory. In case
of GLSL-ES, it's not an error and atleast the linkin
93 matches
Mail list logo
