On Wed, May 6, 2015 at 4:35 PM, Ian Romanick wrote:
> On 05/06/2015 03:45 PM, Kenneth Graunke wrote:
>> 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.
>>>
>>> Fi
On Wed, May 06, 2015 at 04:12:46PM -0700, Kenneth Graunke wrote:
> On Wednesday, May 06, 2015 10:41:30 PM Topi Pohjolainen wrote:
> > CC: Kenneth Graunke > Signed-off-by: Topi Pohjolainen
> > ---
> > src/mesa/drivers/dri/i965/brw_state.h | 11 +++
> > src/mesa/drivers/dri/i965/gen6_wm_state
Reviewed-by: Ilia Mirkin
On Thu, May 7, 2015 at 1:07 AM, Tapani Pälli wrote:
> This was really the original purpose, for enabling the path for
> ES3.1 tests without the extension being set. Set also fallthrough
> comment for Coverity (caught by Matt).
>
> v2: .. and test the right way, not wrong
This was really the original purpose, for enabling the path for
ES3.1 tests without the extension being set. Set also fallthrough
comment for Coverity (caught by Matt).
v2: .. and test the right way, not wrong one (Ilia Mirkin)
Signed-off-by: Tapani Pälli
---
src/mesa/main/shader_query.cpp | 4
On 05/07/2015 08:02 AM, Ilia Mirkin wrote:
On Thu, May 7, 2015 at 12:55 AM, Tapani Pälli wrote:
This was really the original purpose, for enabling the path for
ES3.1 tests without the extension being set. Set also fallthrough
comment for Coverity (caught by Matt).
Signed-off-by: Tapani Pälli
On Thu, May 7, 2015 at 12:55 AM, Tapani Pälli wrote:
> This was really the original purpose, for enabling the path for
> ES3.1 tests without the extension being set. Set also fallthrough
> comment for Coverity (caught by Matt).
>
> Signed-off-by: Tapani Pälli
> ---
> src/mesa/main/shader_query.c
This was really the original purpose, for enabling the path for
ES3.1 tests without the extension being set. Set also fallthrough
comment for Coverity (caught by Matt).
Signed-off-by: Tapani Pälli
---
src/mesa/main/shader_query.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff -
I know nothing about TGSI but this looks perfectly reasonable to me.
Reviewed-by: Jason Ekstrand
On Wed, May 6, 2015 at 8:33 PM, Ilia Mirkin wrote:
> i2b has to work for all integers, not just 1. INEG would not necessarily
> result with all bits set, which is something that other operations can
i2b has to work for all integers, not just 1. INEG would not necessarily
result with all bits set, which is something that other operations can
rely on by e.g. using AND (or INEG for b2i).
Signed-off-by: Ilia Mirkin
---
Found by observation. Noticed a weird pattern in the generated code,
which l
https://bugs.freedesktop.org/show_bug.cgi?id=90354
Dave Airlie changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Wed, May 6, 2015 at 7:09 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> It's a weird thing that provides some values related to 2**x. It's also
> already handled by a case in the switch.
>
> Signed-off-by: Ian Romanick
The series is
Reviewed-by: Matt Turner
_
From: Ian Romanick
Nothing produces it, and nothing can consume it.
Signed-off-by: Ian Romanick
---
src/glsl/nir/nir_opcodes.py | 1 -
src/glsl/nir/nir_opt_algebraic.py| 7 ---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 1 -
3 files changed, 9 deletions(-)
diff --git
From: Ian Romanick
Nothing produces it, and nothing can consume it.
Signed-off-by: Ian Romanick
---
src/glsl/nir/nir_opcodes.py | 1 -
src/glsl/nir/nir_opt_algebraic.py| 6 --
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 3 ---
3 files changed, 10 deletions(-)
diff --gi
From: Ian Romanick
It's a weird thing that provides some values related to 2**x. It's also
already handled by a case in the switch.
Signed-off-by: Ian Romanick
---
src/mesa/program/prog_to_nir.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/program/prog_to_nir.c
From: Ian Romanick
All paths that produce GLSL IR for NIR lower ir_unop_log. All paths
that consume NIR will explode if they geta nir_op_flog.
Signed-off-by: Ian Romanick
---
src/glsl/nir/glsl_to_nir.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/nir/glsl_to_
From: Ian Romanick
All paths that produce GLSL IR for NIR lower ir_unop_exp. All paths
that consume NIR will explode if they geta nir_op_fexp.
Signed-off-by: Ian Romanick
---
src/glsl/nir/glsl_to_nir.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/nir/glsl_to_
On 05/06/2015 03:45 PM, Kenneth Graunke wrote:
> 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
>>
On 05/06/2015 12:48 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_XML.py", line 120
On 05/06/2015 02:12 PM, 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?
I was recently wondering this as well. The
On Wednesday, May 06, 2015 10:41:30 PM Topi Pohjolainen wrote:
> CC: Kenneth Graunke Signed-off-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_state.h | 11 +++
> src/mesa/drivers/dri/i965/gen6_wm_state.c | 111
> ++
> 2 files changed, 77 insertions(
On Friday, May 01, 2015 01:01:55 PM Chad Versace wrote:
> The first 6 patches are small cleanups. The real implementation is patch
> 7/7. Enabling EGL_KHR_fence_sync also gives us EGL_KHR_wait_sync for
> free.
>
> This series lives on the tag:
> git://github.com/chadversary/mesa
> i965-egl
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
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 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
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
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
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
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 |
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 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.
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
... 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
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
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/
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 +
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
https://bugs.freedesktop.org/show_bug.cgi?id=90346
Vinson Lee changed:
What|Removed |Added
Status|NEEDINFO|NEW
--- Comment #2 from Vinson Lee ---
$ .
https://bugs.freedesktop.org/show_bug.cgi?id=90346
Ian Romanick changed:
What|Removed |Added
Status|NEW |NEEDINFO
Assignee|mesa-dev@list
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
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
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-
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
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 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
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 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
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
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
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
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
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 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
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
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
https://bugs.freedesktop.org/show_bug.cgi?id=90147
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
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
--- 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)
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
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
---
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
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
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
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
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
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
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 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
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
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
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.
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
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
https://bugs.freedesktop.org/show_bug.cgi?id=80183
cgerlac...@gmail.com changed:
What|Removed |Added
Attachment #115590|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 #115588|text/plain |image/png
mime type|
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
--- 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 #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
cgerlac...@gmail.com changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
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
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
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,
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
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
---
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().
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 ++--
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
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 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
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
(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
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-
1 - 100 of 121 matches
Mail list logo
