From: Dave Airlie
This fixes: GL43-CTS.compute_shader.resource-ubo
Signed-off-by: Dave Airlie
---
src/mesa/main/uniforms.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/main/uniforms.c b/src/mesa/main/uniforms.c
index
On Wed, May 4, 2016 at 7:54 AM, Eduardo Lima Mitev wrote:
> Patch 2 and 3 are:
>
> Reviewed-by: Eduardo Lima Mitev
>
Thanks! I do not have commit access. Can I ask you to push (again)?
>
> I barely know about coccinelle scripts so I will let others look at
On Wed, May 4, 2016 at 7:46 AM, Eduardo Lima Mitev wrote:
> Good catch!
>
> Reviewed-by: Eduardo Lima Mitev
>
Thanks! I do not have commit access. Can I ask you to push?
>
> On 05/04/2016 05:48 AM, Thomas Hindoe Paaboel Andersen wrote:
> > The assert was
From: Dave Airlie
We already are a GLintptr, casting won't help.
Signed-off-by: Dave Airlie
---
src/mesa/main/api_validate.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/api_validate.c
On Wednesday, May 4, 2016 3:54:13 PM PDT Matt Turner wrote:
> ---
> src/mesa/drivers/dri/i965/brw_defines.h | 5 +
> src/mesa/drivers/dri/i965/brw_disasm.c | 3 +++
> src/mesa/drivers/dri/i965/brw_fs.cpp| 3 +++
>
From: Dave Airlie
This fixes GL43-CTS.compute_shader.api-indirect
which tests the length/4 before anything else.
Signed-off-by: Dave Airlie
---
src/mesa/main/api_validate.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
On Wednesday, May 4, 2016 3:54:14 PM PDT Matt Turner wrote:
> Ken suggested instead of a big and complicated optimization pass, to
> just recognize the operations here. It's certainly less code and a lot
> prettier, but it seems to actually perform worse for currently unknown
> reasons.
One
On Wednesday, May 4, 2016 3:54:12 PM PDT Matt Turner wrote:
> Terrible name. Suggest something else, or even a better way to do this.
>
> Basically, the next patch wants to inspect the LOD argument and do
> something different if it's 0.0f. But at that point we've emitted a MOV
> for it and we
On Tuesday, May 3, 2016 3:00:24 PM PDT Jason Ekstrand wrote:
> The fs_visitor::emit_texture helper originated when we still had both NIR
> and IR visitors for the FS backend. Since the old visitor was removed,
> emit_texture serves no real purpose beyond arbitrarily splitting
> heavily-linked
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_pipe_common.c | 14 ++
src/gallium/drivers/radeonsi/cik_sdma.c | 23 ---
src/gallium/drivers/radeonsi/si_dma.c | 10 --
3 files changed, 14 insertions(+), 33
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_pipe_common.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c
b/src/gallium/drivers/radeon/r600_pipe_common.c
index 3eb12f3..4845587 100644
From: Marek Olšák
This prevents IB rejections due to insane memory usage from
many concecutive texture uploads.
---
src/gallium/drivers/r600/evergreen_hw_context.c | 2 +-
src/gallium/drivers/r600/evergreen_state.c | 2 +-
src/gallium/drivers/r600/r600_hw_context.c
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_pipe_common.c | 16
src/gallium/drivers/radeon/radeon_winsys.h| 2 ++
src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 8
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 8
4
From: Marek Olšák
---
src/gallium/drivers/r600/evergreen_hw_context.c | 1 +
src/gallium/drivers/r600/evergreen_state.c | 1 +
src/gallium/drivers/r600/r600_hw_context.c | 1 +
src/gallium/drivers/r600/r600_state.c | 1 +
From: Marek Olšák
because it doesn't decompress
---
src/gallium/drivers/radeon/r600_texture.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index
From: Marek Olšák
This implements:
- Linear-to-linear partial copies. (unaligned)
- Tiled-to-linear and linear-to-tiled partial copies.
(unaligned except 1-2 Bpp)
- Tiled-to-tiled partial copies aligned to 8x8.
---
src/gallium/drivers/radeonsi/cik_sdma.c | 436
From: Marek Olšák
v2: - adjustments for exercising all important SDMA code paths
- decrease the probability of getting huge sizes (faster testing)
- increase the probability of getting power-of-two dimensions
- change the memory cap to 128MB (faster testing)
From: Marek Olšák
this is more robust and probably fixes some bugs already
---
src/gallium/drivers/r600/evergreen_state.c| 10 ++---
src/gallium/drivers/r600/r600_state.c | 5 ++-
src/gallium/drivers/radeon/r600_pipe_common.h | 7
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 18 --
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 6df013a..ac11380 100644
---
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_texture.c | 46 ++-
1 file changed, 39 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 0dba045..d6d95af
From: Marek Olšák
just normalizing the interfaces
---
src/gallium/drivers/radeonsi/cik_sdma.c | 19 ---
src/gallium/drivers/radeonsi/si_dma.c | 19 ---
src/gallium/drivers/radeonsi/si_pipe.c | 5 +
From: Marek Olšák
Most of this has never worked according to the new test.
The new code will be radically different.
---
src/gallium/drivers/radeonsi/cik_sdma.c | 217 +---
1 file changed, 2 insertions(+), 215 deletions(-)
diff --git
From: Marek Olšák
---
src/gallium/drivers/radeonsi/cik_sdma.c | 4 ++--
src/gallium/drivers/radeonsi/sid.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/cik_sdma.c
b/src/gallium/drivers/radeonsi/cik_sdma.c
index
Hi,
This patch series completely rewrites texture copying with SDMA for CIK & VI.
It only uses the "partial" copy packets, which makes it a lot simpler (one
packet per layered/3D copy). Most of the complexity is in handling hw
limitations, but luckily the SDMA path can be used in the majority
Matt Turner writes:
> Beginning with commit 7b208a73, Unigine Valley began hanging the GPU on
> Gen >= 8 platforms.
>
> Evidently that commit allowed the scheduler to make different choices
> that somehow finally ran afoul of a hardware bug in which POW and FDIV
>
Beginning with commit 7b208a73, Unigine Valley began hanging the GPU on
Gen >= 8 platforms.
Evidently that commit allowed the scheduler to make different choices
that somehow finally ran afoul of a hardware bug in which POW and FDIV
instructions may not be followed by an instruction with two
On Curro's suggestion, I tried reproducing the problem that the patch
fixes. Not only could I not reproduce it, but it seems like the
original workaround that Jason added back when he reworked the fs
backend ages ago is unnecessary at least on my ivybridge and broadwell
machines. Since reproducing
---
src/mesa/drivers/common/meta_blit.c | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/common/meta_blit.c
b/src/mesa/drivers/common/meta_blit.c
index 6761238..bb79c46 100644
--- a/src/mesa/drivers/common/meta_blit.c
+++
---
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 30 ++--
1 file changed, 19 insertions(+), 11 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
b/src/mesa/drivers/dri/i965/brw_blorp_blit.cpp
index 882f9ed..dc68eab 100644
---
---
src/mesa/drivers/dri/i965/brw_defines.h | 5 +
src/mesa/drivers/dri/i965/brw_disasm.c | 3 +++
src/mesa/drivers/dri/i965/brw_fs.cpp| 3 +++
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 14 ++
Ken suggested instead of a big and complicated optimization pass, to
just recognize the operations here. It's certainly less code and a lot
prettier, but it seems to actually perform worse for currently unknown
reasons.
total instructions in shared programs: 8514403 -> 8495373 (-0.22%)
Terrible name. Suggest something else, or even a better way to do this.
Basically, the next patch wants to inspect the LOD argument and do
something different if it's 0.0f. But at that point we've emitted a MOV
for it and we just have a register to look at.
---
This is an alternative approach to
From: Marek Olšák
Vulkan always sets this. It only affects in-place Z decompression.
This is recommended for performance, but what app uses MSAA depth
texturing?
---
src/gallium/drivers/radeonsi/si_state.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #16 from Jose Fonseca ---
(In reply to David Lonie from comment #12)
> Just tried that here, no such luck:
>
> $ ln -s libGL.so libGL.so.1
> $ LD_LIBRARY_PATH=$PWD:$LD_LIBRARY_PATH apitrace trace --api gl
>
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #15 from Emil Velikov ---
Just pointing out something that may not be that obvious - David is using the
xlib powered gallium libGL. That one uses the (iirc) unsupported split shared
LLVM libraries.
--
You
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #14 from David Lonie ---
(In reply to Ilia Mirkin from comment #13)
> FWIW this is what it looks like on i965: http://i.imgur.com/LMmcGkM.png
>
> I'm guessing that's correct, but the "correct" screenshot is
Michael Schellenberger Costa
writes:
> Hi Curro,
>
> Am 04.05.2016 um 06:26 schrieb Francisco Jerez:
>> Instead of using the LOAD_PAYLOAD instruction (emitted through the
>> emit_transpose() helper that is no longer useful and this commit
>> removes) which
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #13 from Ilia Mirkin ---
FWIW this is what it looks like on i965: http://i.imgur.com/LMmcGkM.png
I'm guessing that's correct, but the "correct" screenshot is inside an archive
and I'm lazy. It is indeed totally
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #12 from David Lonie ---
(In reply to Jose Fonseca from comment #10)
> The proper way of using a custom libGL.so is to simply set LD_LIBRARY_PATH
> to the directory that has it:
>
> ln -sf libGL.so
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #11 from Jose Fonseca ---
Created attachment 123472
--> https://bugs.freedesktop.org/attachment.cgi?id=123472=edit
TestTranslucentLUTDepthPeeling.trace
Trace.
--
You are receiving this mail because:
You are
Iago Toral writes:
> On Wed, 2016-05-04 at 01:15 -0700, Francisco Jerez wrote:
>> Iago Toral writes:
>>
>> > On Mon, 2016-05-02 at 18:48 -0700, Francisco Jerez wrote:
>> >> Samuel Iglesias Gonsálvez writes:
>> >>
>> >> > From:
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #10 from Jose Fonseca ---
(In reply to Jose Fonseca from comment #8)
> It looks like apitrace is about to enter an infinite loop: the address of
> the supposedly "real" glXCreateContextAttribsARB function is
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #9 from Roland Scheidegger ---
FWIW it was not the samplers returning NaN for the float texture, but rather
the _coords_ being NaNs (thus the results from sampling could be anything,
though it is possible some
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #8 from Jose Fonseca ---
(In reply to David Lonie from comment #7)
> Is this issue familiar to anyone? I'll keep trying to get this working, but
> would appreciate any tips if this is a known issue :)
It looks
Missed that. Will do.
I should probably also cc this for 11.2
Thanks.
-Brian
On 05/04/2016 02:05 PM, Ilia Mirkin wrote:
My bad (iirc)... copy/paste error. Also need to fix the OES variant. With that,
Reviewed-by: Ilia Mirkin
On Wed, May 4, 2016 at 4:03 PM, Brian
Reviewed-by: Charmaine Lee
From: Brian Paul
Sent: Wednesday, May 4, 2016 1:03 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; Sinclair Yeh
Subject: [PATCH] glapi: fix parameter type for
My bad (iirc)... copy/paste error. Also need to fix the OES variant. With that,
Reviewed-by: Ilia Mirkin
On Wed, May 4, 2016 at 4:03 PM, Brian Paul wrote:
> The function returns GLuint, not GLfloat values.
> ---
> src/mapi/glapi/gen/es_EXT.xml | 2 +-
>
Reviewed-by: Charmaine Lee
From: Brian Paul
Sent: Wednesday, May 4, 2016 1:03 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; Sinclair Yeh
Subject: [PATCH] mesa: include texture format in
The function returns GLuint, not GLfloat values.
---
src/mapi/glapi/gen/es_EXT.xml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/es_EXT.xml b/src/mapi/glapi/gen/es_EXT.xml
index bf67eae..e453401 100644
--- a/src/mapi/glapi/gen/es_EXT.xml
+++
---
src/mesa/main/genmipmap.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/genmipmap.c b/src/mesa/main/genmipmap.c
index 1a6ae9a..d917220 100644
--- a/src/mesa/main/genmipmap.c
+++ b/src/mesa/main/genmipmap.c
@@ -139,7 +139,8 @@
Samuel Iglesias Gonsálvez writes:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA256
>
>
>
> On 03/05/16 20:59, Kenneth Graunke wrote:
>> Other than patches 37, 56, and ones you agreed to drop, the series
>> is: Reviewed-by: Kenneth Graunke
>>
>> I
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #7 from David Lonie ---
I'm trying to get a trace, but apitrace is failing an assertion:
$ apitrace trace --api gl bin/vtkRenderingCoreCxxTests
"TestTranslucentLUTDepthPeeling"
v2: Use == not is for equality testing (Dylan Baker)
Signed-off-by: Adam Jackson
---
src/mapi/glapi/gen/glX_XML.py| 2 +-
src/mapi/glapi/gen/glX_proto_recv.py | 2 --
src/mapi/glapi/gen/glX_proto_send.py | 2 --
src/mapi/glapi/gen/glX_proto_size.py | 24
Signed-off-by: Adam Jackson
---
src/mapi/glapi/gen/gl_XML.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_XML.py b/src/mapi/glapi/gen/gl_XML.py
index 4f35343..8036a02 100644
--- a/src/mapi/glapi/gen/gl_XML.py
+++
Squashes the one remaining warning in the xserver build.
v2: Also clean up some non-standard whitespace (Ian Romanick)
Reviewed-by: Ian Romanick
Signed-off-by: Adam Jackson
---
src/mapi/glapi/gen/glX_proto_recv.py | 12 ++--
1 file changed, 6
Another attempt at syncing the GLX generator scripts with xserver.
Jon mentioned a couple of issues in the last series, namely that these two
patches were still necessary:
https://lists.x.org/archives/xorg-devel/2014-April/041597.html
Otherwise the instances in the extension XML override the core
definitions, and we stop knowing their sizes in indirect_size_get.c
Signed-off-by: Adam Jackson
---
src/mapi/glapi/gen/ARB_viewport_array.xml | 16
1 file changed, 12 insertions(+), 4 deletions(-)
We're about to update the generator scripts to use these, easier not to
vary between client and server.
Signed-off-by: Adam Jackson
---
src/mapi/glapi/gen/glX_proto_send.py | 24
1 file changed, 24 insertions(+)
diff --git
Signed-off-by: Adam Jackson
---
src/mapi/glapi/gen/gl_XML.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_XML.py b/src/mapi/glapi/gen/gl_XML.py
index 2e7123e..e11f6fc 100644
--- a/src/mapi/glapi/gen/gl_XML.py
+++
On Wednesday, May 4, 2016 1:01:33 PM PDT Antía Puentes wrote:
> Hi Kenneth,
>
> thanks for reviewing.
>
> On mié, 2016-05-04 at 03:36 -0700, Kenneth Graunke wrote:
> > On Thursday, April 28, 2016 1:40:32 PM PDT Antia Puentes wrote:
> > >
> > > From the Broadwell specification, structure
On Wednesday, May 4, 2016 3:30:13 PM PDT Alejandro Piñeiro wrote:
>
> On 04/05/16 12:31, Kenneth Graunke wrote:
> > On Thursday, April 28, 2016 1:40:31 PM PDT Antia Puentes wrote:
> >> From: Alejandro Piñeiro
> >>
> >> This commit adds support for PASSTHRU format when
On Sun, May 1, 2016 at 6:03 PM, Axel Davy wrote:
> Do we need the #ifndef WINAPI part ?
>
> Axel
>
>
> On 29/04/2016 20:53, Christian Schmidbauer wrote:
>>
>> As Emil pointed out, only gcc, clang and MSVC compatibility is required.
>> Hence the check for GNUC can be skipped, as
On Sun, May 1, 2016 at 12:02 PM, Emil Velikov wrote:
> Hi Christian,
>
> On 29 April 2016 at 19:53, Christian Schmidbauer
> wrote:
>> As Emil pointed out, only gcc, clang and MSVC compatibility is required.
>> Hence the check for GNUC can be
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #6 from Jose Fonseca ---
(In reply to David Lonie from comment #3)
> I included an apitrace on the last bug, and was told that it was not
> suitable for reproducing bugs (something about apitrace forcing a 4.5
>
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #4 from Ilia Mirkin ---
(In reply to David Lonie from comment #3)
> I included an apitrace on the last bug, and was told that it was not
> suitable for reproducing bugs (something about apitrace forcing a 4.5
>
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #5 from Ilia Mirkin ---
FWIW I was able to run your trace from the other bug with
MESA_GL_VERSION_OVERRIDE=4.5 MESA_GLSL_VERSION_OVERRIDE=450
so I suspect that a trace of this issue would be nice to have as
On Wed, 2016-05-04 at 13:48 -0400, Ilia Mirkin wrote:
> On Wed, May 4, 2016 at 1:41 PM, Connor Abbott
> wrote:
> >
> > On Wed, May 4, 2016 at 1:05 PM, Andres Gomez
> > wrote:
> > >
> > > Hi,
> > >
> > > as part of the work done to "Add FP64 support to
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #3 from David Lonie ---
I included an apitrace on the last bug, and was told that it was not suitable
for reproducing bugs (something about apitrace forcing a 4.5 context IIRC). I
can make one for this issue
Andres Gomez writes:
> Hi,
>
> as part of the work done to "Add FP64 support to the i965 shader
> backends" at:
> https://bugs.freedesktop.org/show_bug.cgi?id=92760
>
> I've been working to add piglit tests that would check the new features
> added by this addition.
>
> Due to
https://bugs.freedesktop.org/show_bug.cgi?id=95266
Ilia Mirkin changed:
What|Removed |Added
Attachment #123465|text/plain |application/bzip2
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #2 from Ilia Mirkin ---
You might consider including an apitrace that reproduces the issue. People are
probably going to be less inclined to run some random binary off the interwebs.
Also, is this only a bug
https://bugs.freedesktop.org/show_bug.cgi?id=94955
--- Comment #23 from David Lonie ---
Opened a new bug:
https://bugs.freedesktop.org/show_bug.cgi?id=95266
Feel free to close this one.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You
https://bugs.freedesktop.org/show_bug.cgi?id=95266
--- Comment #1 from David Lonie ---
The "Reproduction of issue" attachment is a tar.bz2 file, btw. When I try to
download it it just dumps the binary as character data to my browser...Let me
know if I should re-upload it
https://bugs.freedesktop.org/show_bug.cgi?id=95266
Bug ID: 95266
Summary: Geometry missing from rendering, only when using Mesa.
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
On Wed, May 4, 2016 at 1:41 PM, Connor Abbott wrote:
> On Wed, May 4, 2016 at 1:05 PM, Andres Gomez wrote:
>> Hi,
>>
>> as part of the work done to "Add FP64 support to the i965 shader
>> backends" at:
>> https://bugs.freedesktop.org/show_bug.cgi?id=92760
On Wed, May 4, 2016 at 1:05 PM, Andres Gomez wrote:
> Hi,
>
> as part of the work done to "Add FP64 support to the i965 shader
> backends" at:
> https://bugs.freedesktop.org/show_bug.cgi?id=92760
>
> I've been working to add piglit tests that would check the new features
>
On Wed, May 4, 2016 at 10:05 AM, Andres Gomez wrote:
> Hi,
>
> as part of the work done to "Add FP64 support to the i965 shader
> backends" at:
> https://bugs.freedesktop.org/show_bug.cgi?id=92760
>
> I've been working to add piglit tests that would check the new features
>
On Wed, May 4, 2016 at 1:04 PM, Eric Engestrom
wrote:
> On Tue, May 03, 2016 at 11:11:56AM -0400, Rob Clark wrote:
>>
>> Ok, I've renamed to REVIEWERS and scripts/get_reviewer.pl and updated
>> the verbage appropriately.
>
> scripts/get_reviewer.pl still obeys
Hi,
as part of the work done to "Add FP64 support to the i965 shader
backends" at:
https://bugs.freedesktop.org/show_bug.cgi?id=92760
I've been working to add piglit tests that would check the new features
added by this addition.
Due to this, I've been checking and making modifications into the
On Tue, May 03, 2016 at 11:11:56AM -0400, Rob Clark wrote:
>
> Ok, I've renamed to REVIEWERS and scripts/get_reviewer.pl and updated
> the verbage appropriately.
scripts/get_reviewer.pl still obeys .get_maintainer.conf and
.get_maintainer.ignore. Is that something we want to make more
Reviewed-by: Sinclair Yeh
On Mon, May 02, 2016 at 07:15:11PM -0600, Brian Paul wrote:
> Silences warnings with 32-bit Linux gcc builds and MinGW which doesn't
> recognize the ‘t’ conversion character.
> ---
> src/mesa/main/bufferobj.c | 10 ++
> 1 file changed, 6
Reviewed-By: George Kyriazis
> -Original Message-
> From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On
> Behalf Of BruceCherniak
> Sent: Thursday, April 28, 2016 12:13 PM
> To: mesa-dev@lists.freedesktop.org
> Subject: [Mesa-dev] [PATCH] swr:
On Mon, 2016-03-28 at 11:10 -0700, Ian Romanick wrote:
> > @@ -428,7 +428,7 @@ class PrintGlxReqSize_h(PrintGlxReqSize_common):
> def printBody(self, api):
> > for func in api.functionIterateGlx():
> > if not func.ignore and func.has_variable_size_request():
> > -
Hi,
some weeks ago Dave Airlie made the review for the
ARB_internalformat_query2 extension support (again, thanks a lot), and
at the end he suggested to create a "mesa-demos glxinfo super query app.
Something that does what vulkaninfo kinda does" [1]. As I mentioned at
the moment, I had a heavily
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
On 03/05/16 20:59, Kenneth Graunke wrote:
> Other than patches 37, 56, and ones you agreed to drop, the series
> is: Reviewed-by: Kenneth Graunke
>
> I think you can go ahead and land all except those, and we can
> land
https://bugs.freedesktop.org/show_bug.cgi?id=95211
Jose Fonseca changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop. |jfons...@vmware.com
Hi Curro,
Am 04.05.2016 um 06:26 schrieb Francisco Jerez:
> Instead of using the LOAD_PAYLOAD instruction (emitted through the
> emit_transpose() helper that is no longer useful and this commit
> removes) which had to be marked force_writemask_all in some cases,
> emit a series of moves to apply
On fredag 22. april 2016 19.55.10 CEST Emil Velikov wrote:
> From: Emil Velikov
>
> It's used only by dev_icd.json so just call it that way. While we're
> here, manually expand $< (as it might cause issue on some systems)
> and drop the unneeded install_libdir
---
src/intel/vulkan/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am
index fd8aae0..662d720 100644
--- a/src/intel/vulkan/Makefile.am
+++ b/src/intel/vulkan/Makefile.am
@@ -156,7 +156,7 @@ noinst_DATA =
On 04/05/16 12:31, Kenneth Graunke wrote:
> On Thursday, April 28, 2016 1:40:31 PM PDT Antia Puentes wrote:
>> From: Alejandro Piñeiro
>>
>> This commit adds support for PASSTHRU format when pushing
>> double-precision attributes.
>>
>> Check glarray->Doubles in order to
On 05/04/2016 08:52 AM, Rowley, Timothy O wrote:
After shader compilation, the gallivm_state is owned by ShaderVariant<> and
destroyed by its destructor (swr_state.h) so that the generated JIT code and
gallivm_state can be freed when no longer needed.
Oh. Splendid.
Thanks for looking into
From: Robert Foss
The return variable was not set for failure paths.
It has now been changed to VK_ERROR_INITIALIZATION_FAILED
for failure paths.
Coverity: 1358944
Reviewed-by: Eric Engestrom
Signed-off-by: Robert Foss
From: Robert Foss
The return variable was not set for failure paths.
It has now been changed to VK_ERROR_INITIALIZATION_FAILED
for failure paths.
Coverity: 1358944
Signed-off-by: Robert Foss
---
Changes since v1:
- Started using
After shader compilation, the gallivm_state is owned by ShaderVariant<> and
destroyed by its destructor (swr_state.h) so that the generated JIT code and
gallivm_state can be freed when no longer needed.
> On May 3, 2016, at 5:45 PM, robert.f...@collabora.com wrote:
>
> From: Robert Foss
On 4 May 2016 at 13:08, Ilia Mirkin wrote:
> Only if building Intel vulkan, no?
I believe the idea is to move from python2 so this will do no harm as is.
> Could that script just be fixed to work
> with python2 BTW?
>
Possibly. The above 'moving away from python2' topic
Only if building Intel vulkan, no? Could that script just be fixed to work
with python2 BTW?
On May 4, 2016 6:44 AM, "Emil Velikov" wrote:
> From: Emil Velikov
>
> Bail early, as opposed to later on during the build.
>
> Signed-off-by: Emil
On Fri, Apr 29, 2016 at 7:29 AM, Samuel Iglesias Gonsálvez
wrote:
> Lower lrp when operating with double operands because float version of
> lrp is also lowered.
>
> Signed-off-by: Samuel Iglesias Gonsálvez
> CC: Rob Clark
> ---
>
On Wed, 2016-05-04 at 03:43 -0700, Kenneth Graunke wrote:
> On Thursday, April 28, 2016 1:40:42 PM PDT Antia Puentes wrote:
> >
> > From: "Juan A. Suarez Romero"
> >
> > In scalar mode, URB read length limit is 15. Abort if we go beyond
> > it.
> > ---
> >
Hi Kenneth,
thanks for reviewing.
On mié, 2016-05-04 at 03:36 -0700, Kenneth Graunke wrote:
> On Thursday, April 28, 2016 1:40:32 PM PDT Antia Puentes wrote:
> >
> > From the Broadwell specification, structure VERTEX_ELEMENT_STATE
> > description:
> >
> > "When SourceElementFormat is set to
On Thursday, April 28, 2016 1:40:32 PM PDT Antia Puentes wrote:
> From the Broadwell specification, structure VERTEX_ELEMENT_STATE
> description:
>
>"When SourceElementFormat is set to one of the *64*_PASSTHRU
> formats, 64-bit components are stored in the URB without any
>
1 - 100 of 125 matches
Mail list logo