On Wed, Jul 22, 2015 at 12:41 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez curroje...@riseup.net
wrote:
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 66
+---
1
Use MSVCRT functions instead. Their semantics are slightly
different but they can be made to work as expected.
Also, use the same code paths for both MSVCRT and MinGW.
No testing yet. Just built.
https://bugs.freedesktop.org/show_bug.cgi?id=91418
---
src/gallium/auxiliary/Makefile.sources |
On 22/07/15 16:04, Emil Velikov wrote:
Otherwise the final dri module will have (additional) unresolved
symbols.
Cc: Brian Paul bri...@vmware.com
Cc: Jose Fonseca jfons...@vmware.com
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
We can only fix the remaining unresolved symbols
On Wed, Jul 22, 2015 at 12:31 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
A few comments below. Mostly just asking for explanation.
1-3 are
Reviewed-by: Jason Ekstrand jason.ekstr...@intel.com
Obviously, don't merge 4/4 until it actually
It should be in master now.
-Emil
On 21 July 2015 at 23:34, Dylan Baker baker.dyla...@gmail.com wrote:
Cool, I was under that impression too, but having it confirmed is great.
Since these do matter and I don't have push access, could you make sure this
lands?
On Jul 21, 2015 15:33, Emil
On 21/07/15 15:57, Emil Velikov wrote:
On 18 July 2015 at 08:13, Jose Fonseca jfons...@vmware.com wrote:
On 18/07/15 01:38, Eric Anholt wrote:
Emil Velikov emil.l.veli...@gmail.com writes:
On 14/07/15 19:45, Eric Anholt wrote:
These are really useful hints to the compiler in the absence
On Wed, Jul 22, 2015 at 12:43 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez curroje...@riseup.net
wrote:
So that it's left uninitialized by LOAD_PAYLOAD, we only need to
reserve space for it in
On Wed, Jul 22, 2015 at 12:54 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Thu, Jul 16, 2015 at 8:41 AM, Francisco Jerez curroje...@riseup.net
wrote:
The only non-trivial thing it still has to do is figure out where to
take the src/dst
On Wed, Jul 22, 2015 at 12:55 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez curroje...@riseup.net
wrote:
This should match the set of cases in which we currently call fail()
or no16() from the
https://bugs.freedesktop.org/show_bug.cgi?id=91425
--- Comment #3 from Ilia Mirkin imir...@alum.mit.edu ---
(In reply to Brian Paul from comment #1)
Hmm, I can't reproduce that. The test passes completely for me.
What driver are you using? I'm testing llvmpipe/softpipe.
Odd. I repro with
On Wed, Jul 22, 2015 at 4:26 AM, Eduardo Lima Mitev el...@igalia.com wrote:
On 07/13/2015 01:38 PM, Jason Ekstrand wrote:
On Fri, Jul 10, 2015 at 8:53 AM, Eduardo Lima Mitev el...@igalia.com wrote:
On 06/30/2015 06:58 PM, Jason Ekstrand wrote:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez curroje...@riseup.net wrote:
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 63
+++-
1 file changed, 55 insertions(+), 8 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
On 07/22/2015 11:02 AM, Ilia Mirkin wrote:
This moves the width/height/depth == 0 check to the front and avoids
doing any other checking when that is the case.
Also moves the dimensions check after the format/type checks so that we
don't bail out with success on a width/height/depth == 0
Jason Ekstrand ja...@jlekstrand.net writes:
On Wed, Jul 22, 2015 at 12:31 AM, Francisco Jerez curroje...@riseup.net
wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
A few comments below. Mostly just asking for explanation.
1-3 are
Reviewed-by: Jason Ekstrand jason.ekstr...@intel.com
On Wed, Jul 22, 2015 at 10:02 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez curroje...@riseup.net
wrote:
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 63
+++-
1
Jason Ekstrand ja...@jlekstrand.net writes:
On Wed, Jul 22, 2015 at 12:43 AM, Francisco Jerez curroje...@riseup.net
wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez curroje...@riseup.net
wrote:
So that it's left uninitialized by
On 07/22/2015 10:15 AM, Jose Fonseca wrote:
Use MSVCRT functions instead. Their semantics are slightly
different but they can be made to work as expected.
Also, use the same code paths for both MSVCRT and MinGW.
No testing yet. Just built.
https://bugs.freedesktop.org/show_bug.cgi?id=91418
On 07/22/2015 11:02 AM, Ilia Mirkin wrote:
The current message makes it seem like the zoffset is invalid.
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
src/mesa/main/texgetimage.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/mesa/main/texgetimage.c
https://bugs.freedesktop.org/show_bug.cgi?id=91425
--- Comment #4 from Brian Paul bri...@vmware.com ---
OK, the problem is my local copy of
getteximage-invalid-format-for-packed-type.c has the patch ext_packed_float:
fix getteximage-invalid-format-for-packed-type test which I posted for review
This needs a *lot* more commentary. These calculations are extremely
tricky and there are almost no comments. For instance, you are
turning a 2D offset on a tiled surface into a new 2D address into the
raw view of the surface. Nowhere do you explain what the raw
surface looks like and how its
The current message makes it seem like the zoffset is invalid.
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
---
src/mesa/main/texgetimage.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/mesa/main/texgetimage.c b/src/mesa/main/texgetimage.c
index 59ec091..2f35ac6
This moves the width/height/depth == 0 check to the front and avoids
doing any other checking when that is the case.
Also moves the dimensions check after the format/type checks so that we
don't bail out with success on a width/height/depth == 0 request when
the format/type don't match.
Jason Ekstrand ja...@jlekstrand.net writes:
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez curroje...@riseup.net
wrote:
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 63
+++-
1 file changed, 55 insertions(+), 8 deletions(-)
diff --git
Jason Ekstrand ja...@jlekstrand.net writes:
On Wed, Jul 22, 2015 at 10:02 AM, Francisco Jerez curroje...@riseup.net
wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez curroje...@riseup.net
wrote:
---
src/mesa/drivers/dri/i965/brw_fs.cpp
On Wed, Jul 22, 2015 at 10:16 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Wed, Jul 22, 2015 at 10:02 AM, Francisco Jerez curroje...@riseup.net
wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Tue, Jul 21, 2015 at 9:38 AM, Francisco
On 07/22/2015 11:29 AM, Ilia Mirkin wrote:
On Wed, Jul 22, 2015 at 1:20 PM, Brian Paul bri...@vmware.com wrote:
On 07/22/2015 11:02 AM, Ilia Mirkin wrote:
This moves the width/height/depth == 0 check to the front and avoids
doing any other checking when that is the case.
Also moves the
On Tue, Jul 21, 2015 at 1:18 AM, Iago Toral ito...@igalia.com wrote:
On Tue, 2015-07-21 at 09:24 +0200, Iago Toral wrote:
On Tue, 2015-06-16 at 11:15 -0700, Anuj Phogat wrote:
Without this patch, piglit test arb_color_buffer_float-readpixels
fails, when forced to use the meta pbo path.
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez curroje...@riseup.net wrote:
Define bitfield packing, unpacking and type conversion operations in
terms of which the image format conversion code will be implemented.
These don't directly know about image formats: The packing and
unpacking
On Tue, Jul 21, 2015 at 5:42 PM, Brian Paul bri...@vmware.com wrote:
---
src/mesa/main/fbobject.c | 4 ++--
src/mesa/main/shaderapi.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index f46554b..cc342c2 100644
We can't completely drop this #define because it's used in the xserver
generated code. Until someone gets around to putting xml generation
directly in the server, we're stuck #defining it for them.
---
src/mapi/glapi/gen/gl_XML.py | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
On 01/07/15 01:01, Samuel Pitoiset wrote:
This commit implements the base interface for hardware performance
counters that will be shared between nv50 and nvc0 drivers.
TODO: Bump libdrm version of mesa when nvif will be merged.
Changes since v2:
- remove double-query thing for domains,
On 06/26/2015 07:38 AM, Ilia Mirkin wrote:
On Fri, Jun 26, 2015 at 4:18 AM, Tapani Pälli tapani.pa...@intel.com wrote:
On 06/26/2015 01:06 AM, Ilia Mirkin wrote:
On Thu, Jun 25, 2015 at 4:22 PM, Ilia Mirkin imir...@alum.mit.edu wrote:
On Thu, Jun 25, 2015 at 5:08 AM, Marta Lofstedt
_mesa_meta_pbo_GetTexSubImage() uses _mesa_meta_BlitFrameBuffer(),
which will do fragment clamping if enabled. But fragment clamping
doesn't affect ReadPixels and GetTexImage.
Without this patch, piglit test arb_color_buffer_float-clear fails,
when forced to use the meta pbo path.
v2: Apply this
On Wed, Jul 22, 2015 at 2:55 PM, Ian Romanick i...@freedesktop.org wrote:
On 06/26/2015 07:38 AM, Ilia Mirkin wrote:
On Fri, Jun 26, 2015 at 4:18 AM, Tapani Pälli tapani.pa...@intel.com wrote:
On 06/26/2015 01:06 AM, Ilia Mirkin wrote:
On Thu, Jun 25, 2015 at 4:22 PM, Ilia Mirkin
On 22/07/15 17:13, Jose Fonseca wrote:
On 21/07/15 15:57, Emil Velikov wrote:
On 18 July 2015 at 08:13, Jose Fonseca jfons...@vmware.com wrote:
On 18/07/15 01:38, Eric Anholt wrote:
Emil Velikov emil.l.veli...@gmail.com writes:
On 14/07/15 19:45, Eric Anholt wrote:
These are really
On Wed, Jul 22, 2015 at 1:20 PM, Brian Paul bri...@vmware.com wrote:
On 07/22/2015 11:02 AM, Ilia Mirkin wrote:
This moves the width/height/depth == 0 check to the front and avoids
doing any other checking when that is the case.
Also moves the dimensions check after the format/type checks so
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Wed, Jul 22, 2015 at 7:04 AM, Vinson Lee v...@freedesktop.org wrote:
radeon_fbo.c: In function 'radeon_map_renderbuffer_s8z24':
radeon_fbo.c:162:9: warning: variable 'ret' set but not used
[-Wunused-but-set-variable]
int ret;
On 01/07/15 01:01, Samuel Pitoiset wrote:
This will allow to monitor global performance counters through the
command stream of the GPU instead of using ioctls.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/nv50/nv50_screen.c | 11 +++
On Tue, 2015-07-21 at 11:13 -0700, Anuj Phogat wrote:
On Tue, Jul 21, 2015 at 12:50 AM, Iago Toral ito...@igalia.com wrote:
On Tue, 2015-06-16 at 11:15 -0700, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
src/mesa/drivers/common/meta.c | 13 ++---
1
Jason Ekstrand ja...@jlekstrand.net writes:
A few comments below. Mostly just asking for explanation.
1-3 are
Reviewed-by: Jason Ekstrand jason.ekstr...@intel.com
Obviously, don't merge 4/4 until it actually has users.
--Jason
Thanks.
On Thu, Jul 16, 2015 at 8:35 AM, Francisco Jerez
Jason Ekstrand ja...@jlekstrand.net writes:
On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez curroje...@riseup.net
wrote:
This should match the set of cases in which we currently call fail()
or no16() from the emit_texture_*() methods and the ones in which
emit_texture_gen4() enables the
https://bugs.freedesktop.org/show_bug.cgi?id=90264
--- Comment #38 from Heiko lil_...@web.de ---
(In reply to Antoine Labour from comment #37)
Can you make glXWaitX re-evaluate window sizes? That's the reason we call
it - we just resized the window, we want to make sure we're drawing to the
On Tuesday, July 21, 2015 03:36:32 PM Emil Velikov wrote:
Now that the drm_gralloc module exports the correct includes we can get
rid of this hack.
Cc: Varad Gautam varadgau...@gmail.com
Cc: Chih-Wei Huang cwhu...@android-x86.org
Cc: Eric Anholt e...@anholt.net
Signed-off-by: Emil Velikov
On Tuesday, July 21, 2015 03:27:47 PM Emil Velikov wrote:
See previous two commits for details.
v2: Don't forget git mv, bring back DRM_GRALLOC_TOP. Spotted by Varad.
Cc: Varad Gautam varadgau...@gmail.com
Cc: Chih-Wei Huang cwhu...@android-x86.org
Cc: Eric Anholt e...@anholt.net
On Tue, 2015-07-21 at 17:05 -0700, Anuj Phogat wrote:
On Tue, Jul 21, 2015 at 1:36 AM, Iago Toral ito...@igalia.com wrote:
On Tue, 2015-07-21 at 08:13 +0200, Iago Toral wrote:
On Mon, 2015-07-20 at 10:56 -0700, Anuj Phogat wrote:
On Mon, Jul 20, 2015 at 5:10 AM, Iago Toral
On 07/13/2015 01:38 PM, Jason Ekstrand wrote:
On Fri, Jul 10, 2015 at 8:53 AM, Eduardo Lima Mitev el...@igalia.com wrote:
On 06/30/2015 06:58 PM, Jason Ekstrand wrote:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com
wrote:
This patch makes public the
This seems to have been multiplying by stride twice since
fs_inst::regs_read/regs_written were changed to return the value in
register units rather than in dispatch_width-wide components. The
value returned by fs_inst::regs_read() already takes into account the
stride so it's wrong to do it again
On 17.07.2015 01:54, Marek Olšák wrote:
From: Marek Olšák marek.ol...@amd.com
Broken by f1be3d8cdde17a9b9ae283e1bab2f46b992d3bf3, which returns NULL
if no commands have been submitted.
I've fixed that in radeonsi, so you can drop this patch.
--
Earthling Michel Dänzer |
On 07/13/2015 01:57 PM, Jason Ekstrand wrote:
On Wed, Jul 8, 2015 at 11:54 PM, Eduardo Lima Mitev el...@igalia.com wrote:
On 06/30/2015 06:51 PM, Jason Ekstrand wrote:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev el...@igalia.com
wrote:
The index into the output_reg array where to
Jason Ekstrand ja...@jlekstrand.net writes:
On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez curroje...@riseup.net
wrote:
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 66
+---
1 file changed, 49 insertions(+), 17 deletions(-)
diff --git
Jason Ekstrand ja...@jlekstrand.net writes:
On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez curroje...@riseup.net
wrote:
So that it's left uninitialized by LOAD_PAYLOAD, we only need to
reserve space for it in the message since it will be initialized
implicitly by the generator.
---
Jason Ekstrand ja...@jlekstrand.net writes:
On Thu, Jul 16, 2015 at 8:41 AM, Francisco Jerez curroje...@riseup.net
wrote:
The only non-trivial thing it still has to do is figure out where to
take the src/dst depth values from and predicate the instruction if
discard is in use. The manual
Kenneth Graunke kenn...@whitecape.org writes:
On Thursday, July 16, 2015 06:41:16 PM Francisco Jerez wrote:
The logical variant is largely equivalent to the original opcode but
instead of taking a single payload source it expects the arguments
that make up the payload separately as individual
https://bugs.freedesktop.org/show_bug.cgi?id=91425
Bug ID: 91425
Summary: [regression, bisected] Piglit spec/ext_packed_float/
getteximage-invalid-format-for-packed-type fails
Product: Mesa
Version: git
Hardware: Other
On 01/07/15 01:01, Samuel Pitoiset wrote:
To write data at the right offset, the kernel has to know some
parameters of this ring buffer, like the number of domains and the
maximum number of queries.
Changes since v2:
- only configure the ring buffer if the notifier BO is allocated
- only use
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez curroje...@riseup.net wrote:
v2: Move the image_params array back to brw_stage_prog_data.
---
src/mesa/drivers/dri/i965/brw_fs.h | 1 +
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 50
+++-
2 files changed,
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez curroje...@riseup.net wrote:
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 149
+++
1 file changed, 149 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
https://bugs.freedesktop.org/show_bug.cgi?id=90264
Antoine Labour pi...@chromium.org changed:
What|Removed |Added
CC||pi...@chromium.org
From: Dave Airlie airl...@redhat.com
This fixes make check since the tess changes.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/glsl/tests/varyings_test.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/tests/varyings_test.cpp
From: Dave Airlie airl...@redhat.com
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 6c97933..33469d4 100644
On 23.07.2015 10:14, Dave Airlie wrote:
From: Dave Airlie airl...@redhat.com
This fixes make check since the tess changes.
Signed-off-by: Dave Airlie airl...@redhat.com
Tested-by: Michel Dänzer michel.daen...@amd.com
Please push this to fix make check.
--
Earthling Michel Dänzer
From: Dave Airlie airl...@redhat.com
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 6c97933..33469d4 100644
From: Dave Airlie airl...@redhat.com
This extension is about setting expectation on GL4.1 implementations
rather than actually enforcing things. So once you support GLSL 410
then you support this in theory.
Signed-off-by: Dave Airlie airl...@redhat.com
---
docs/GL3.txt | 2 +-
1 file changed, 1
In actuality it'll only be exposed on nvc0 atm...
On Wed, Jul 22, 2015 at 9:20 PM, Dave Airlie airl...@gmail.com wrote:
From: Dave Airlie airl...@redhat.com
This extension is about setting expectation on GL4.1 implementations
rather than actually enforcing things. So once you support GLSL 410
All vendors I've tested allow token pasting in their ESSL shaders.
At least one application expects this to be supported
even though it is against spec.
This would be the Dolphin GameCube/Wii emulator that requires this.
The vendors I've tested that is known to support this is:
ARM Mali
Qualcomm
Reviewed-by: Iago Toral Quiroga ito...@igalia.com
On Wed, 2015-07-22 at 11:54 -0700, Anuj Phogat wrote:
_mesa_meta_pbo_GetTexSubImage() uses _mesa_meta_BlitFrameBuffer(),
which will do fragment clamping if enabled. But fragment clamping
doesn't affect ReadPixels and GetTexImage.
Without
On Tuesday, July 21, 2015 03:19:25 PM Dave Airlie wrote:
From: Dave Airlie airl...@redhat.com
This fleshes out the APIs, using the program resource
APIs where they should match.
It also sets the default values to valid subroutines.
Signed-off-by: Dave Airlie airl...@redhat.com
---
On Wed, Jul 22, 2015 at 4:37 AM, Eduardo Lima Mitev el...@igalia.com wrote:
On 07/13/2015 01:57 PM, Jason Ekstrand wrote:
On Wed, Jul 8, 2015 at 11:54 PM, Eduardo Lima Mitev el...@igalia.com wrote:
On 06/30/2015 06:51 PM, Jason Ekstrand wrote:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima
https://bugs.freedesktop.org/show_bug.cgi?id=91425
Samuel Iglesias sigles...@igalia.com changed:
What|Removed |Added
Status|NEW |RESOLVED
From: Rhys Kidd rhysk...@gmail.com
Signed-off-by: Rhys Kidd rhysk...@gmail.com
---
src/mesa/math/m_vector.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/math/m_vector.h b/src/mesa/math/m_vector.h
index 3b7f583..5bd76b8 100644
--- a/src/mesa/math/m_vector.h
+++
From: Rhys Kidd rhysk...@gmail.com
Signed-off-by: Rhys Kidd rhysk...@gmail.com
---
src/mesa/math/m_vector.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/math/m_vector.h b/src/mesa/math/m_vector.h
index 8551ee7..3b7f583 100644
--- a/src/mesa/math/m_vector.h
+++
A simple shader such as
vec4 color;
color.xy.x = 1.0;
would cause ir_assignment::set_lhs() to generate bogus IR:
(swiz xy (swiz x (constant float (1.0
We were setting the number of components of each new RHS swizzle based
on the highest channel used in the LHS swizzle. So, .xy.y
On 23 July 2015 at 15:01, Kenneth Graunke kenn...@whitecape.org wrote:
On Tuesday, July 21, 2015 03:19:25 PM Dave Airlie wrote:
From: Dave Airlie airl...@redhat.com
This fleshes out the APIs, using the program resource
APIs where they should match.
It also sets the default values to valid
On 01/07/15 01:01, Samuel Pitoiset wrote:
This commit adds support for both compute and graphics global
performance counters which have been reverse engineered with
CUPTI (Linux) and PerfKit (Windows).
Currently, only one query type can be monitored at the same time because
the Gallium's HUD
On 22/07/15 21:01, Jose Fonseca wrote:
On 22/07/15 17:13, Jose Fonseca wrote:
On 21/07/15 15:57, Emil Velikov wrote:
On 18 July 2015 at 08:13, Jose Fonseca jfons...@vmware.com wrote:
On 18/07/15 01:38, Eric Anholt wrote:
Emil Velikov emil.l.veli...@gmail.com writes:
On 14/07/15 19:45,
On 01/07/15 01:01, Samuel Pitoiset wrote:
This exposes a group of global performance counters that enables
GL_AMD_performance_monitor. All piglit tests are okay.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/nv50/nv50_query.c | 35
On 01/07/15 01:01, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/nv50/nv50_query.c | 41 ++
src/gallium/drivers/nouveau/nv50/nv50_screen.c | 1 +
src/gallium/drivers/nouveau/nv50/nv50_screen.h | 3 ++
---
src/glsl/link_atomics.cpp | 73 +++
1 file changed, 55 insertions(+), 18 deletions(-)
diff --git a/src/glsl/link_atomics.cpp b/src/glsl/link_atomics.cpp
index 100d03c..21f9af7 100644
--- a/src/glsl/link_atomics.cpp
+++ b/src/glsl/link_atomics.cpp
Hi Marek,
This is regarding your commit st/mesa: use pipe_sampler_view_release
for releasing sampler views from last October.
Basically, we have:
void
st_texture_release_all_sampler_views(struct st_context *st,
struct st_texture_object *stObj)
{
GLuint
---
src/glsl/nir/nir_types.cpp | 6 ++
src/glsl/nir/nir_types.h | 2 ++
2 files changed, 8 insertions(+)
diff --git a/src/glsl/nir/nir_types.cpp b/src/glsl/nir/nir_types.cpp
index 62176f5..06f5b0a 100644
--- a/src/glsl/nir/nir_types.cpp
+++ b/src/glsl/nir/nir_types.cpp
@@ -106,6 +106,12 @@
---
src/glsl/nir/nir_lower_atomics.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/src/glsl/nir/nir_lower_atomics.c b/src/glsl/nir/nir_lower_atomics.c
index ce3615a..e5ec008 100644
--- a/src/glsl/nir/nir_lower_atomics.c
+++
https://bugs.freedesktop.org/show_bug.cgi?id=90311
Julien Isorce julien.iso...@gmail.com changed:
What|Removed |Added
CC|
This all looks correct as far as I can tell. However, I'm very
concerned about the number of checks such as
has_matching_typed_format() that are built-in to the compiler (via
surface_builder) where we then go on to do something that is highly
dependant on state setup doing the exact same check
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez curroje...@riseup.net wrote:
Reviewed-by: Paul Berry stereotype...@gmail.com
I'm sure that Paul still thinks this patch does what the commit
message says. However, does the r-b really still apply to the rest of
it?
---
On Tuesday, July 21, 2015 03:19:14 PM Dave Airlie wrote:
From: Dave Airlie airl...@redhat.com
This type will be used to store the name of subroutine types
as in subroutine void myfunc(void);
will store myfunc into a subroutine type.
This is required to the parser can identify a
On Wed, Jul 22, 2015 at 10:05 AM, Francisco Jerez curroje...@riseup.net wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
On Wed, Jul 22, 2015 at 12:31 AM, Francisco Jerez curroje...@riseup.net
wrote:
Jason Ekstrand ja...@jlekstrand.net writes:
A few comments below. Mostly just asking
Hi Guys,
These patches fix up AoA support for atomic counters following
my change in approach to uniforms in V2 of my AoA GLES series.
However I wanted to get some feedback mainly on the first patch
as the end result is the whole AoA is marked as active which
doesnt happen on Nvidia drivers.
I
On 07/22/2015 05:31 PM, Jose Fonseca wrote:
On 22/07/15 23:32, Brian Paul wrote:
Hi Marek,
This is regarding your commit st/mesa: use pipe_sampler_view_release
for releasing sampler views from last October.
Basically, we have:
void
st_texture_release_all_sampler_views(struct st_context *st,
On 22/07/15 23:32, Brian Paul wrote:
Hi Marek,
This is regarding your commit st/mesa: use pipe_sampler_view_release
for releasing sampler views from last October.
Basically, we have:
void
st_texture_release_all_sampler_views(struct st_context *st,
struct
On 23/07/15 01:00, Brian Paul wrote:
On 07/22/2015 05:31 PM, Jose Fonseca wrote:
On 22/07/15 23:32, Brian Paul wrote:
Hi Marek,
This is regarding your commit st/mesa: use pipe_sampler_view_release
for releasing sampler views from last October.
Basically, we have:
void
From: Dave Airlie airl...@redhat.com
These entries were put in the GL4.0 section, so removed the commented
out ones.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/main/tests/dispatch_sanity.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git
Looks good to me, if this did not introduce any regressions:
Reviewed-by: Iago Toral Quiroga ito...@igalia.com
On Tue, 2015-06-16 at 11:15 -0700, Anuj Phogat wrote:
This allows us to handle cases when texImage-_BaseFormat doesn't match
_mesa_format_get_base_format(texImage-Format). _BaseFormat
Reviewed-by: Iago Toral Quiroga ito...@igalia.com
On Tue, 2015-06-16 at 11:15 -0700, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
src/mesa/drivers/common/meta.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git
https://bugs.freedesktop.org/show_bug.cgi?id=91425
--- Comment #2 from Michel Dänzer mic...@daenzer.net ---
FWIW, I got the same regression with radeonsi.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
Otherwise the final dri module will have (additional) unresolved
symbols.
Cc: Brian Paul bri...@vmware.com
Cc: Jose Fonseca jfons...@vmware.com
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
We can only fix the remaining unresolved symbols (_glapi_foo), as we
remove the non-shared
https://bugs.freedesktop.org/show_bug.cgi?id=91425
--- Comment #1 from Brian Paul bri...@vmware.com ---
Hmm, I can't reproduce that. The test passes completely for me.
What driver are you using? I'm testing llvmpipe/softpipe.
--
You are receiving this mail because:
You are the QA Contact for
https://bugs.freedesktop.org/show_bug.cgi?id=91387
--- Comment #3 from Michael Godfrey michaeldgodf...@gmail.com ---
Created attachment 117297
-- https://bugs.freedesktop.org/attachment.cgi?id=117297action=edit
backtrace
--
You are receiving this mail because:
You are the QA Contact for the
The problem here is that the _mesa_meta_BlitFramebuffer is not setting
G/B channels to 0.0 when doing Luminance/Intensity to RGBA conversions,
so why not implement the fix in _mesa_meta_BlitFramebuffer directly? The
GL spec expects frambuffer blits to handle these conversions properly,
so it looks
Patches 10-11 are
Reviewed-by: Iago Toral Quiroga ito...@igalia.com
On Tue, 2015-06-16 at 11:15 -0700, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
src/mesa/drivers/common/meta.c | 12 ++--
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git
https://bugs.freedesktop.org/show_bug.cgi?id=91387
Brian Paul bri...@vmware.com changed:
What|Removed |Added
CC||anuj.pho...@gmail.com
---
1 - 100 of 101 matches
Mail list logo
