Looks good.
Reviewed-by: Martin Peres
On 06/05/15 09:36, Tapani Pälli wrote:
Increases pass rate of ES31-CTS.*program_interface_query* tests
when run with MESA_EXTENSION_OVERRIDE='GL_ARB_compute_shader'. Many
of the negative tests that happen to use compute stage in queries
start passing.
Sig
On 04/29/2015 01:56 AM, kevin.rogo...@intel.com wrote:
> From: Kevin Rogovin
>
> Add convenience helper functions for fetching geometry of gl_framebuffer
> that return the geometry of the gl_framebuffer instead of the geometry of
> the buffers of the gl_framebuffer when then the gl_framebuffer ha
HI,
> For both this and get_framebuffer_parameteriv below, I don't see the value in
> splitting the implementations. Also, these functions need to check that the
> extension is enabled and generate GL_INVALID_OPERATION if it is not.
No worries, I can add the GL_INVALID_OPERATION bit in. I ha
> I'm waffling about this a bit. I'm concerned that people will use
> buffer->Width when they should use the accessor. I don't see any
> changes to core Mesa code to use the accessor, so I'm a little concerned that
> some subtle, incorrect behavior is introduced... but there may well not be.
>
> You haven't been running 'make check'. :) You also need to update
> src/mesa/tests/dispatch_sanity.cpp.
There is something wrong with my box or something... I did run make check and
there were no failures. Out of paranoia, I also ran
src/mesa/main/tests/main-test explicitly and there were no
This increases memory pressure during linking but makes it easier
for backend to free IR after it is not needed anymore.
Signed-off-by: Tapani Pälli
---
src/glsl/linker.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index 330f
Hi;
I've tested this + a modified version (where I just free all
ir_instructions) of Kenneth's:
"i965: Delete most of the linked GLSL IR when using NIR"
No Piglit regressions observed. From here on plan would be to change
ir_variable to a structure which will be hopefully more minimal and
easier
On 05/06/2015 02:53 AM, Robert Bragg wrote:
As we've learned more about the observability capabilities of Gen
graphics we've found that it's not enough to only try and configure the
OA unit from userspace without any dedicated support from the kernel.
Hi Robert,
Yeah, this is the same idea f
GL_ARB_texture_gather is needed for gles 3.1 conformance.
Signed-off-by: Marta Lofstedt
---
src/mesa/main/get.c | 6 ++
src/mesa/main/get_hash_params.py | 4
2 files changed, 10 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index dcf4f0a..95868bf 100
GL_ARB_explicit_uniform_location is required for gles 3.1
conformance
Signed-off-by: Marta Lofstedt
---
src/mesa/main/get.c | 6 ++
src/mesa/main/get_hash_params.py | 2 ++
2 files changed, 8 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index 97d3bf0..6f
GL_ARB_shader_atomic_counters is required for gles 3.1 conformance.
Signed-off-by: Marta Lofstedt
---
src/mesa/main/get.c | 6 ++
src/mesa/main/get_hash_params.py | 12
2 files changed, 18 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index
This series of patches continues on the GLES 3.1 enabling
started by Ian. It focusing of exposing definitions from
previously extisting Open GL extensions that are needed
for OpenGL ES 3.1 conformance.
Marta Lofstedt (6):
mesa/es3.1: enable GL_ARB_shader_image_load_store for gles3.1
mesa/es3.
GL_ARB_compute_shader is needed for gles 3.1 conformance.
Signed-off-by: Marta Lofstedt
---
src/mesa/main/get.c | 6 ++
src/mesa/main/get_hash_params.py | 9 +
2 files changed, 15 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index 95868bf..97d3bf
GL_ARB_shader_image_load store support is required for GLES 3.1
conformance.
Signed-off-by: Marta Lofstedt
---
src/mesa/main/get.c | 6 ++
src/mesa/main/get_hash_params.py | 8
2 files changed, 14 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
ind
From: Tapani Pälli
(increases ES31-CTS.draw_indirect.basic.* passing tests)
Signed-off-by: Tapani Pälli
---
src/mesa/main/get.c | 12
src/mesa/main/get_hash_params.py | 2 ++
2 files changed, 14 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
in
GL_ARB_texture_multisample is required for gles 3.1 conformance.
Signed-off-by: Marta Lofstedt
---
src/mesa/main/get.c | 6 ++
src/mesa/main/get_hash_params.py | 8
2 files changed, 14 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index f5318d5.
On Tue, 2015-05-05 at 13:50 +0300, Juha-Pekka Heikkila wrote:
> I didn't find this being used anywhere.
>
It used to be used by KHR_debug but Chia-I Wu replaced the hash table
with a list for a number of reasons this time last year.
Reviewed-by: Timothy Arceri
> Signed-off-by: Juha-Pekka Heik
Build without st/egl and target/egl.
Signed-off-by: Varad Gautam
---
src/egl/main/Android.mk | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/egl/main/Android.mk b/src/egl/main/Android.mk
index 12b66d0..800d20d 100644
--- a/src/egl/main/Android.mk
+++ b/src/egl/main/Android.mk
@@ -130,1
Hi,
On 6 May 2015 at 07:25, Pekka Paalanen wrote:
> On Wed, 6 May 2015 11:00:13 +1000
> Dave Airlie wrote:
>> On 2 May 2015 at 20:15, Axel Davy wrote:
>> > Only EGL_WINDOW_BIT is supported. Remove tests related.
>>
>> Is this there no plans to support pixmap/pbuffer/ or any of the other bits?
>
One more question:
for patch 2 of the series, there was the request to change the type of
_HasAttachments from GLboolean to bool. If the helper functions "survive",
should the helper functions return unsigned int instead of GLuint?
-Kevin
-Original Message-
From: Rogovin, Kevin
Sent:
I've just noticed that this is wrong, sorry :/ We should to make a new
section for some of the enums that are only in GL core and ES31,
DRAW_INDIRECT_BUFFER_BINDING is one of these cases .. so this patch is
broken. Same may apply for some of your patches, there should not be
duplicates but only
Hi,
> I think this check should be put in a utility function up in core Mesa
> somewhere. It's open-coded twice in this patch, and the check will change
> when GL_ARB_image_load_store lands.
Would a check of the form:
inline bool
_mesa_has_atomic_ops(struct gl_context *ctx)
{
return ctx-
(increases ES31-CTS.draw_indirect.basic.* passing tests)
v2: only expose DRAW_INDIRECT_BUFFER_BINDING for GL core + ES3.1
Signed-off-by: Tapani Pälli
---
src/mesa/main/get.c | 12
src/mesa/main/get_hash_params.py | 8 ++--
2 files changed, 18 insertions(+), 2 dele
Instead of adding new one you should move the existing declaration to
section:
"apis": ["GL", "GL_CORE", "GLES3"]
and make it use the new specified extra
"extra_ARB_explicit_uniform_location_es31".
(same is actually true for all other patches, they all add enums that
should be available for
This series moves all the decision making of values into common
hardware independent dispatcher while leaving the hardware specific
logic to deal with formatting only.
Curro needed a similar refactor for gen7 and gen8. However, that
makes it a harder to apply the changes I needed that expand all t
This reverts commit e17dc004fd96e589e92ee64deeb45339af4bf671.
---
src/mesa/drivers/dri/i965/gen8_surface_state.c | 137 +++--
1 file changed, 60 insertions(+), 77 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen8_surface_state.c
b/src/mesa/drivers/dri/i965/gen8_surface
This reverts commit 6f26ffaf66affb6563e548646fb6d6a049fe6bff.
---
src/mesa/drivers/dri/i965/brw_context.h | 11 --
src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 127 +-
2 files changed, 54 insertions(+), 84 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/b
Also changed a couple of direct shifts into SET_FIELD().
Reviewed-by: Matt Turner
Reviewed-by: Kenneth Graunke
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/brw_context.h | 2 +-
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 12 ++--
src/mesa/drivers/dr
All generations do the same exact dispatch and it could be
therefore done in the hardware independent stage.
Reviewed-by: Matt Turner
Reviewed-by: Kenneth Graunke
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/brw_context.h | 3 ---
src/mesa/drivers/dri/i965/brw_wm_su
Reviewed-by: Matt Turner
Reviewed-by: Kenneth Graunke
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/brw_context.h | 4 ++--
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 20 +++-
src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 17 ++--
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
index 6e0166e..250c61a 100644
--- a/src/mesa/drivers/dri/i965
From: Francisco Jerez
---
src/mesa/drivers/dri/i965/brw_context.h | 2 +
src/mesa/drivers/dri/i965/brw_surface_formats.c | 112 +++
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 77
3 files changed, 191 insertions(+)
diff --git a/src/mesa/d
All hardware platforms have this in common, so do it in the
hardware independent dispatcher.
v2 (Matt): Removed extra whitespace.
Reviewed-by: Matt Turner (v1)
Reviewed-by: Kenneth Graunke (v1)
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/brw_context.h | 4 ++-
src
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c
b/src/mesa/drivers/dri/i965/gen7_wm_surface_state.c
index 1527d12..60f9228 100644
--- a/src/
Also changed a couple of direct shifts into SET_FIELD().
Fixes: arb_copy_image-formats -auto -fbo on ILK. In principle,
minimum level settings are only for TextureView to use. We,
however, also take advantage of that internally when blitting.
Before this patch this wasn't taken into account for IL
No functional change but this allows to drop the texture object's
mesa format in later patches.
Stencil buffer has fixed hardware format corresponding to fixed
mesa format. (This is in fact already specfically tested in
intel_fbo.c::intel_validate_framebuffer()).
Reviewed-by: Matt Turner
Reviewe
Reviewed-by: Matt Turner
Reviewed-by: Kenneth Graunke
Signed-off-by: Topi Pohjolainen
---
src/mesa/drivers/dri/i965/brw_context.h | 1 +
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 12 ++--
src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 7 +--
src/mesa/drive
Sorry, one more question, which I should have asked before, but neglected to:
> If we go that route, we should probably just add a gl_framebuffer::Samples
> field for uniformity.
If this is done, then there are two ways to get the value: from the Visual
field and from the new field; Naturally,
That's OK, Tapani, I will fix up "my" patches once I see what reception you
patch gets.
/Marta
> -Original Message-
> From: Palli, Tapani
> Sent: Wednesday, May 6, 2015 12:19 PM
> To: Lofstedt, Marta
> Cc: mesa-dev@lists.freedesktop.org
> Subject: Re: [PATCH 1/7] mesa/es3.1: enable
> DRA
Hi!
Topi Pohjolainen writes:
> This series moves all the decision making of values into common
> hardware independent dispatcher while leaving the hardware specific
> logic to deal with formatting only.
>
> Curro needed a similar refactor for gen7 and gen8. However, that
> makes it a harder to a
https://bugs.freedesktop.org/show_bug.cgi?id=80183
cgerlac...@gmail.com changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
https://bugs.freedesktop.org/show_bug.cgi?id=80183
--- Comment #10 from cgerlac...@gmail.com ---
Created attachment 115588
--> https://bugs.freedesktop.org/attachment.cgi?id=115588&action=edit
all clipped triangles are within the viewport
--
You are receiving this mail because:
You are the ass
https://bugs.freedesktop.org/show_bug.cgi?id=80183
--- Comment #11 from cgerlac...@gmail.com ---
Created attachment 115589
--> https://bugs.freedesktop.org/attachment.cgi?id=115589&action=edit
triangles get completely clipped on viewport - 1
--
You are receiving this mail because:
You are the
https://bugs.freedesktop.org/show_bug.cgi?id=80183
--- Comment #12 from cgerlac...@gmail.com ---
Created attachment 115590
--> https://bugs.freedesktop.org/attachment.cgi?id=115590&action=edit
triangles get completely clipped on viewport - 2
--
You are receiving this mail because:
You are the
https://bugs.freedesktop.org/show_bug.cgi?id=80183
cgerlac...@gmail.com changed:
What|Removed |Added
Attachment #115588|text/plain |image/png
mime type|
https://bugs.freedesktop.org/show_bug.cgi?id=80183
cgerlac...@gmail.com changed:
What|Removed |Added
Attachment #115589|text/plain |image/png
mime type|
https://bugs.freedesktop.org/show_bug.cgi?id=80183
cgerlac...@gmail.com changed:
What|Removed |Added
Attachment #115590|text/plain |image/png
mime type|
rzalloc_size() call ralloc_size() to allocate memory. ralloc_size()
use calloc to get memory thus zeroing in rzalloc_size is not
necessary.
Signed-off-by: Juha-Pekka Heikkila
---
src/util/ralloc.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/util/ralloc.c b/src/util/ralloc.c
index 01
This should make it more obvious in bug reports while also removing
any sort of guesswork for developers.
Signed-off-by: Martin Peres
---
src/glx/dri2_glx.c | 2 ++
src/glx/dri3_glx.c | 2 ++
2 files changed, 4 insertions(+)
diff --git a/src/glx/dri2_glx.c b/src/glx/dri2_glx.c
index 462d560..53
This provides an easy way to test differences in rendering performance
or conformance between DRI2 and DRI3.
Signed-off-by: Martin Peres
---
docs/envvars.html | 1 +
src/glx/dri3_glx.c | 5 +
2 files changed, 6 insertions(+)
diff --git a/docs/envvars.html b/docs/envvars.html
index 31d14a4.
https://bugs.freedesktop.org/show_bug.cgi?id=9951
marius predut changed:
What|Removed |Added
Assignee|haihao.xi...@intel.com |marius.pre...@intel.com
--
You are recei
https://bugs.freedesktop.org/show_bug.cgi?id=9951
--- Comment #6 from marius predut ---
The patch series that fix this bug are upstream
--
You are receiving this mail because:
You are on the CC list for the bug.
___
mesa-dev mailing list
mesa-dev@list
On Wed, May 06, 2015 at 02:56:53PM +0300, Francisco Jerez wrote:
> Hi!
>
> Topi Pohjolainen writes:
>
> > This series moves all the decision making of values into common
> > hardware independent dispatcher while leaving the hardware specific
> > logic to deal with formatting only.
> >
> > Curro
On Fri, Apr 17, 2015 at 04:51:27PM -0700, Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/brw_tex_layout.c| 2 ++
> src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 6 ++
> 2 files changed, 8 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_te
https://bugs.freedesktop.org/show_bug.cgi?id=80183
--- Comment #13 from Roland Scheidegger ---
Does this happen with other gallium based drivers (softpipe for instance,
though it might hit all the same code)?
I'm afraid noone will look into it without an apitrace trace (or some sample
code like a
On Wed, May 06, 2015 at 10:36:15AM +0200, Samuel Pitoiset wrote:
>
>
> On 05/06/2015 02:53 AM, Robert Bragg wrote:
> >As we've learned more about the observability capabilities of Gen
> >graphics we've found that it's not enough to only try and configure the
> >OA unit from userspace without any
Kenneth Graunke writes:
> On Tuesday, May 05, 2015 03:05:02 PM Matt Turner wrote:
>> On Tue, May 5, 2015 at 2:17 PM, Francisco Jerez
>> wrote:
>> > Kenneth Graunke writes:
>> >> That then begs the question - could we do the format conversion and
>> >> address calculations in a i965-specific NI
From: Roland Scheidegger
Was off-by-one. llvm says inserting an element with an index higher than the
number of elements yields undefined results. Previously such inserts were
ignored but as of llvm revision 235854 the vector gets replaced with undef,
causing failures.
This fixes piglit gl-3.2-la
On 05/06/2015 08:05 AM, srol...@vmware.com wrote:
From: Roland Scheidegger
Was off-by-one. llvm says inserting an element with an index higher than the
number of elements yields undefined results. Previously such inserts were
ignored but as of llvm revision 235854 the vector gets replaced with
Hello list,
The candidate for the Mesa 10.5.5 is now available. The current patch queue
is as follows:
- 10 queued
- 5 nominated (outstanding)
- and 0 rejected (obsolete) patches
The queue consists mostly of nouveau and i965 fixes.
Take a look at section "Mesa stable queue" for more informati
---
src/mesa/drivers/dri/i965/brw_reg.h | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_reg.h
b/src/mesa/drivers/dri/i965/brw_reg.h
index c03a8ae..81a9320 100644
--- a/src/mesa/drivers/dri/i965/brw_reg.h
+++ b/src/mesa/drivers/dri/i965/brw_re
On 06/05/15 17:02, Daniel Vetter wrote:
On Wed, May 06, 2015 at 10:36:15AM +0200, Samuel Pitoiset wrote:
On 05/06/2015 02:53 AM, Robert Bragg wrote:
As we've learned more about the observability capabilities of Gen
graphics we've found that it's not enough to only try and configure the
OA unit
https://bugs.freedesktop.org/show_bug.cgi?id=90147
--- Comment #6 from Brian Paul ---
(In reply to Jeremy Huddleston from comment #5)
> Yeah, I think the sysctl() approach is probably more portable if you want to
> go that route.
>
OK, I can commit that patch, but the cherry-pick info:
(cherry
https://bugs.freedesktop.org/show_bug.cgi?id=90147
--- Comment #7 from Jeremy Huddleston ---
Yes, please remove the 8da9f46f18fd4c73f2a9d708c86e7e3d5535b3d8 before commit
to master. I suspect it refers to the commit on my (unmerged) master branch
from which I cherry-picked to an (also unmerged)
On Wed 06 May 2015, Daniel Stone wrote:
> Hi,
>
> On 6 May 2015 at 07:25, Pekka Paalanen wrote:
> > On Wed, 6 May 2015 11:00:13 +1000
> > Dave Airlie wrote:
> >> On 2 May 2015 at 20:15, Axel Davy wrote:
> >> > Only EGL_WINDOW_BIT is supported. Remove tests related.
> >>
> >> Is this there no pl
https://bugs.freedesktop.org/show_bug.cgi?id=90147
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Le 06/05/2015 14:43, Martin Peres a écrit :
diff --git a/src/glx/dri3_glx.c b/src/glx/dri3_glx.c
index ff77a91..5246737 100644
--- a/src/glx/dri3_glx.c
+++ b/src/glx/dri3_glx.c
@@ -2092,6 +2092,11 @@ dri3_create_display(Display * dpy)
xcb_generic_error_t *error;
cons
On Wed, May 6, 2015 at 2:36 AM, Tapani Pälli wrote:
> Increases pass rate of ES31-CTS.*program_interface_query* tests
> when run with MESA_EXTENSION_OVERRIDE='GL_ARB_compute_shader'. Many
> of the negative tests that happen to use compute stage in queries
> start passing.
>
> Signed-off-by: Tapani
On 05/06/2015 07:54 PM, Ilia Mirkin wrote:
On Wed, May 6, 2015 at 2:36 AM, Tapani Pälli wrote:
Increases pass rate of ES31-CTS.*program_interface_query* tests
when run with MESA_EXTENSION_OVERRIDE='GL_ARB_compute_shader'. Many
of the negative tests that happen to use compute stage in queries
st
Hi,
This optimisation doesn't seem to work with textureGather so a bunch of
Piglit tests are failing for me. I'm not sure why it didn't get picked
up by your Jenkins run.
I can't find anything in the bspec nor a known workaround to suggest
that this shouldn't work so I'm not really sure what to d
On Wednesday, May 06, 2015 03:35:27 PM Juha-Pekka Heikkila wrote:
> rzalloc_size() call ralloc_size() to allocate memory. ralloc_size()
> use calloc to get memory thus zeroing in rzalloc_size is not
> necessary.
>
> Signed-off-by: Juha-Pekka Heikkila
> ---
> src/util/ralloc.c | 2 --
> 1 file ch
On Wednesday, May 06, 2015 03:43:42 PM Martin Peres wrote:
> This should make it more obvious in bug reports while also removing
> any sort of guesswork for developers.
>
> Signed-off-by: Martin Peres
> ---
> src/glx/dri2_glx.c | 2 ++
> src/glx/dri3_glx.c | 2 ++
> 2 files changed, 4 insertions
Fix Clang return-type error introduced with commit
96f164f6f047833091eb98a73aa80c31dc94f962 "gallium: make
pipe_context::begin_query return a boolean".
CC r600_query.lo
r600_query.c:443:3: error: non-void function 'r600_begin_query' should return a
value [-Wreturn-type]
re
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Tue, May 5, 2015 at 11:36 PM, Tapani Pälli wrote:
> Increases pass rate of ES31-CTS.*program_interface_query* tests
> when run with MESA_EXTENSION_OVERRIDE='GL_ARB_compute_shader'. Many
> of the negative tests that happen to use compute stage in queries
> start passing.
>
> Signed-off-by: Tapan
v2: Use LIST_ENTRY instead of container_of in iterators
---
src/util/list.h | 35 ++-
1 file changed, 34 insertions(+), 1 deletion(-)
diff --git a/src/util/list.h b/src/util/list.h
index 191030c..795e98d 100644
--- a/src/util/list.h
+++ b/src/util/list.h
@@ -141,6
On Wednesday, May 06, 2015 09:09:06 AM Tapani Pälli wrote:
> Sorry not, it's just a simple hack patch ATM where I try to preserve
> only the ir_variable::data struct but I forgot that we really need a bit
> more (like name, type ..) so things fall apart. My plan next is first to
> introduce own
https://bugs.freedesktop.org/show_bug.cgi?id=90346
Bug ID: 90346
Summary: DispatchSanity_test.GLES2 regression
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Keywords: bise
On Wed, May 06, 2015 at 10:24:18AM -0700, Vinson Lee wrote:
> Fix Clang return-type error introduced with commit
> 96f164f6f047833091eb98a73aa80c31dc94f962 "gallium: make
> pipe_context::begin_query return a boolean".
>
> CC r600_query.lo
> r600_query.c:443:3: error: non-void function 'r60
On Wed, May 6, 2015 at 1:24 PM, Kenneth Graunke wrote:
> On Wednesday, May 06, 2015 03:35:27 PM Juha-Pekka Heikkila wrote:
>> rzalloc_size() call ralloc_size() to allocate memory. ralloc_size()
>> use calloc to get memory thus zeroing in rzalloc_size is not
>> necessary.
>>
>> Signed-off-by: Juha-
CC: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_state.h | 11 +++
src/mesa/drivers/dri/i965/gen6_wm_state.c | 111 ++
2 files changed, 77 insertions(+), 45 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_state.h
b/src/mesa/drivers/dri/i965/brw_stat
Fix build error introduced with commit 1c5a57a "glapi/es3.1: Add support
for GLES versions > 3.0" with Python < 2.7.
File "src/mapi/glapi/gen/gl_genexec.py", line 230, in
printer.Print(api)
File "src/mapi/glapi/gen/gl_XML.py", line 120, in Print
self.printBody(api)
File "src/mapi/gl
https://bugs.freedesktop.org/show_bug.cgi?id=90346
Ian Romanick changed:
What|Removed |Added
Status|NEW |NEEDINFO
Assignee|mesa-dev@list
https://bugs.freedesktop.org/show_bug.cgi?id=90346
Vinson Lee changed:
What|Removed |Added
Status|NEEDINFO|NEW
--- Comment #2 from Vinson Lee ---
$ .
instructions in affected programs: 380 -> 376 (-1.05%)
helped:2
---
Did we just completely forget these in commit 391fb32b, or is there a
reason to not include them?
src/glsl/nir/nir_opt_algebraic.py | 4
1 file changed, 4 insertions(+)
diff --git a/src/g
Four shaders in Unreal 4's Sun Temple are helped, and gain SIMD16
because we avoid an integer multiplication.
instructions in affected programs: 2353 -> 2245 (-4.59%)
helped:4
GAINED:4
---
src/glsl/nir/nir_opt_algebraic.py | 1 +
No changes, but does prevent some regressions in the next commit.
---
src/glsl/nir/nir_opt_algebraic.py | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/glsl/nir/nir_opt_algebraic.py
b/src/glsl/nir/nir_opt_algebraic.py
index fda4bd5..42f4f47 100644
--- a/src/glsl/nir/nir_opt_algebrai
Helps the same set of programs as the previous commit.
instructions in affected programs: 4490 -> 4346 (-3.21%)
helped:8
---
src/glsl/nir/nir_opt_algebraic.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/glsl/nir/nir_opt_algebraic.py
b/src/glsl/nir/
On Wed, Apr 15, 2015 at 11:46 AM, Anuj Phogat wrote:
> Reduces the number of conditions tested in if to one in case of
> non-integer formats. Makes no functional changes.
>
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/brw_surface_formats.c | 5 +++--
> 1 file changed, 3 inserti
... and (a >= c) || (b >= c) as max(a, b) >= c.
Similar to commit 97e6c1b9.
total instructions in shared programs: 6182276 -> 6182180 (-0.00%)
instructions in affected programs: 6400 -> 6304 (-1.50%)
helped:68
HURT: 4
---
src/g
The standard ICD file path is /etc/OpenCL/vendor/.
However it doesn't fit well with custom build.
This option allow ICD vendor file installation path override
---
configure.ac | 46 +++---
src/gallium/targets/opencl/Makefile.am | 2 +-
2 files
On 2015-05-06 14:12:53, Matt Turner wrote:
> ... and (a >= c) || (b >= c) as max(a, b) >= c.
Looks like maybe you could use two commits, since the subject line
overfloweth. Of course, you have shader-db results for them merged. :\
Series Reviewed-by: Jordan Justen
> Similar to commit 97e6c1b9.
On Wed, May 6, 2015 at 2:19 PM, Matt Turner wrote:
> On Wed, Apr 15, 2015 at 11:46 AM, Anuj Phogat wrote:
>> Reduces the number of conditions tested in if to one in case of
>> non-integer formats. Makes no functional changes.
>>
>> Signed-off-by: Anuj Phogat
>> ---
>> src/mesa/drivers/dri/i965/
On Wed, 06 May 2015 23:12:54 +0200, Matt Turner wrote:
instructions in affected programs: 380 -> 376 (-1.05%)
helped:2
---
Did we just completely forget these in commit 391fb32b, or is there a
reason to not include them?
src/glsl/nir/nir_opt_algebraic.py |
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/brw_cs.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_cs.cpp
b/src/mesa/drivers/dri/i965/brw_cs.cpp
index e2f3d63..70731e4 100644
--- a/src/mesa/drivers/dri/i965/brw_cs.cpp
+++ b/src/mesa/drivers/dri
https://bugs.freedesktop.org/show_bug.cgi?id=90354
Bug ID: 90354
Summary: Software rendering cuses X BadMatch error
Product: Mesa
Version: git
Hardware: All
OS: All
Status: NEW
Severity: normal
P
https://bugs.freedesktop.org/show_bug.cgi?id=90354
--- Comment #1 from Igor Gnatenko ---
Looks like a bug in swrast.
The error was 'BadMatch (invalid parameter attributes)'.
(Details: serial 801 error_code 8 request_code 72 (core protocol) minor_code
0)
(Note to programmers: normally, X erro
On Wednesday, May 06, 2015 12:48:30 PM Vinson Lee wrote:
> Fix build error introduced with commit 1c5a57a "glapi/es3.1: Add support
> for GLES versions > 3.0" with Python < 2.7.
>
> File "src/mapi/glapi/gen/gl_genexec.py", line 230, in
> printer.Print(api)
> File "src/mapi/glapi/gen/gl_XM
Whoops, these should have been added in a later (not yet sent to list)
patch. (I need to look into posting those patches! :)
Reviewed-by: Jordan Justen
On 2015-05-06 15:14:42, Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/brw_cs.cpp | 2 --
> 1 file changed,
On Friday, May 01, 2015 01:01:56 PM Chad Versace wrote:
> From: Chad Versace
>
> If it returns NULL, then return early with an error.
>
> Cc: Marek Olšák
> ---
> src/egl/drivers/dri2/egl_dri2.c | 7 +++
> 1 file changed, 7 insertions(+)
>
> diff --git a/src/egl/drivers/dri2/egl_dri2.c b/s
1 - 100 of 121 matches
Mail list logo