v2:
- Add tessellation shader constants support
Signed-off-by: Samuel Iglesias Gonsalvez
---
src/mesa/main/get.c | 1 +
src/mesa/main/get_hash_params.py | 14 ++
2 files changed, 15 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index 56cc3f2..a75
v2:
- Add tessellation shader constants assignment
Signed-off-by: Samuel Iglesias Gonsalvez
---
src/mesa/drivers/dri/i965/brw_context.c | 12
1 file changed, 12 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/mesa/drivers/dri/i965/brw_context.c
index b08a5
This fixes my HSW getting dropped back to 3.2 most of the time, and
seems like the reasonable thing to do.
Tested-and-acked-by: Chris Forbes
On Tue, Jul 21, 2015 at 11:58 PM, Chris Wilson wrote:
> I was mistaken, I thought we already had fixed this in the kernel a
> couple of years ago. We had
Reviewed-by: Iago Toral Quiroga
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 this patch, pigl
On 23 July 2015 at 15:01, Kenneth Graunke wrote:
> On Tuesday, July 21, 2015 03:19:25 PM Dave Airlie wrote:
>> From: Dave Airlie
>>
>> This fleshes out the APIs, using the program resource
>> APIs where they should match.
>>
>> It also sets the default values to valid subroutines.
>>
>> Signed-of
https://bugs.freedesktop.org/show_bug.cgi?id=91425
Samuel Iglesias changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Tuesday, July 21, 2015 03:19:25 PM Dave Airlie wrote:
> From: Dave Airlie
>
> 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
> ---
> src/mesa/main/shaderapi.c | 4
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 w
From: Rhys Kidd
Signed-off-by: Rhys Kidd
---
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
+++ b/src/mesa/math/m_vector.h
@@ -51,7 +51,
From: Rhys Kidd
Signed-off-by: Rhys Kidd
---
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
+++ b/src/mesa/math/m_vector.h
@@ -61,7 +61,
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 Ad
On Wed, Jul 22, 2015 at 4:37 AM, Eduardo Lima Mitev wrote:
> On 07/13/2015 01:57 PM, Jason Ekstrand wrote:
>> On Wed, Jul 8, 2015 at 11:54 PM, Eduardo Lima Mitev wrote:
>>> On 06/30/2015 06:51 PM, Jason Ekstrand wrote:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev
wrote:
> T
On 23.07.2015 10:14, Dave Airlie wrote:
> From: Dave Airlie
>
> This fixes make check since the tess changes.
>
> Signed-off-by: Dave Airlie
Tested-by: Michel Dänzer
Please push this to fix make check.
--
Earthling Michel Dänzer | http://www.amd.com
Libre soft
In actuality it'll only be exposed on nvc0 atm...
On Wed, Jul 22, 2015 at 9:20 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> 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 theo
From: Dave Airlie
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
---
docs/GL3.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --
From: Dave Airlie
Signed-off-by: Dave Airlie
---
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
--- a/src/mesa/drivers/dri/i965/brw_
From: Dave Airlie
Signed-off-by: Dave Airlie
---
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
--- a/src/mesa/drivers/dri/i965/brw_
From: Dave Airlie
This fixes make check since the tess changes.
Signed-off-by: Dave Airlie
---
src/glsl/tests/varyings_test.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/tests/varyings_test.cpp b/src/glsl/tests/varyings_test.cpp
index 4573529..62f8c6b 100644
From: Dave Airlie
These entries were put in the GL4.0 section, so removed the commented
out ones.
Signed-off-by: Dave Airlie
---
src/mesa/main/tests/dispatch_sanity.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/mesa/main/tests/dispatch_sanity.cpp
b/src/mesa/main/tests/dispatch_
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
st_texture_release_al
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
https://bugs.freedesktop.org/show_bug.cgi?id=90264
Antoine Labour changed:
What|Removed |Added
CC||pi...@chromium.org
--- Comment #39 from
https://bugs.freedesktop.org/show_bug.cgi?id=90311
Julien Isorce changed:
What|Removed |Added
CC||emil.l.veli...@gmail.com
--- Comment #3
---
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
+++ b/src/glsl/nir/nir_lower_atomi
---
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 @@
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)
{
GLuin
---
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 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 g
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
---
src/gallium/drivers/nouveau/nv50/nv50_query.c | 35 ++
src/gallium/dri
On 01/07/15 01:01, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset
---
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 ++
3 files changed, 45 ins
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 wrote:
On 18/07/15 01:38, Eric Anholt wrote:
Emil Velikov writes:
On 14/07/15 19:45, Eric Anholt wrote:
These are really useful hi
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 doe
On Wed, Jul 22, 2015 at 10:05 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> On Wed, Jul 22, 2015 at 12:31 AM, Francisco Jerez
>> wrote:
>>> Jason Ekstrand writes:
>>>
A few comments below. Mostly just asking for explanation.
1-3 are
Reviewed-by: Jason Ekstr
*whew*, I've made it through the entire series...
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez wrote:
> Another resend of the i965 compiler-related changes for
> ARB_shader_image_load_store, reworked to make use of the SIMD lowering
> infrastructure introduced in a previous series [1]. For a
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez wrote:
> Reviewed-by: Paul Berry
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?
> ---
> src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
> 1 file chan
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez 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
> b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
> index 31
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez 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, 43 insertions(+),
On Tuesday, July 21, 2015 03:19:14 PM Dave Airlie wrote:
> From: Dave Airlie
>
> 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 subroutine
> type i
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 (no
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 o
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez 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 functions take a 4-tupl
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
---
src/gallium/drivers/nouveau/nv50/nv50_screen.c | 11 +++
src/gallium/drivers/nouveau/nv50
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, signa
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 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 wrote:
On 18/07/15 01:38, Eric Anholt wrote:
Emil Velikov writes:
On 14/07/15 19:45, Eric Anholt wrote:
These are really useful hints to the compiler in the absence of
li
On Tue, Jul 21, 2015 at 5:42 PM, Brian Paul 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
> --- a/src/me
Reviewed-by: Marek Olšák
Marek
On Wed, Jul 22, 2015 at 7:04 AM, Vinson Lee 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;
> ^
> radeon_fbo.c: In function '
On Wed, Jul 22, 2015 at 2:55 PM, Ian Romanick wrote:
> On 06/26/2015 07:38 AM, Ilia Mirkin wrote:
>> On Fri, Jun 26, 2015 at 4:18 AM, Tapani Pälli wrote:
>>>
>>>
>>> On 06/26/2015 01:06 AM, Ilia Mirkin wrote:
On Thu, Jun 25, 2015 at 4:22 PM, Ilia Mirkin wrote:
>
> On Thu, Jun 2
On 06/26/2015 07:38 AM, Ilia Mirkin wrote:
> On Fri, Jun 26, 2015 at 4:18 AM, Tapani Pälli wrote:
>>
>>
>> On 06/26/2015 01:06 AM, Ilia Mirkin wrote:
>>>
>>> On Thu, Jun 25, 2015 at 4:22 PM, Ilia Mirkin wrote:
On Thu, Jun 25, 2015 at 5:08 AM, Marta Lofstedt
wrote:
>
> From
_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 Tue, Jul 21, 2015 at 1:18 AM, Iago Toral 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.
>> >
>> > Sign
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
On 07/22/2015 11:29 AM, Ilia Mirkin wrote:
On Wed, Jul 22, 2015 at 1:20 PM, Brian Paul 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
On Wed, Jul 22, 2015 at 1:20 PM, Brian Paul 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 that w
Jason Ekstrand writes:
> On Wed, Jul 22, 2015 at 12:43 AM, Francisco Jerez
> wrote:
>> Jason Ekstrand writes:
>>
>>> On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez
>>> wrote:
So that it's left uninitialized by LOAD_PAYLOAD, we only need to
reserve space for it in the message sinc
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 request
On Wed, Jul 22, 2015 at 10:16 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> On Wed, Jul 22, 2015 at 10:02 AM, Francisco Jerez
>> wrote:
>>> Jason Ekstrand writes:
>>>
On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez
wrote:
> ---
> src/mesa/drivers/dri/i965/brw_f
Jason Ekstrand writes:
> On Wed, Jul 22, 2015 at 10:02 AM, Francisco Jerez
> wrote:
>> Jason Ekstrand writes:
>>
>>> On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez
>>> wrote:
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 63
+++-
1 file ch
https://bugs.freedesktop.org/show_bug.cgi?id=91425
--- Comment #4 from Brian Paul ---
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
(and was R-b'd by
Jason Ekstrand writes:
> On Wed, Jul 22, 2015 at 12:31 AM, Francisco Jerez
> wrote:
>> Jason Ekstrand writes:
>>
>>> A few comments below. Mostly just asking for explanation.
>>>
>>> 1-3 are
>>>
>>> Reviewed-by: Jason Ekstrand
>>>
>>> Obviously, don't merge 4/4 until it actually has users.
>
On Wed, Jul 22, 2015 at 10:02 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez
>> wrote:
>>> ---
>>> src/mesa/drivers/dri/i965/brw_fs.cpp | 63
>>> +++-
>>> 1 file changed, 55 insertions(+), 8 deletions(-
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
---
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
Jason Ekstrand writes:
> On Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez
> 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
>> b/src
The current message makes it seem like the zoffset is invalid.
Signed-off-by: Ilia Mirkin
---
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 100644
--- a/src/mesa/m
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.
Bugzilla:
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 Tue, Jul 21, 2015 at 9:38 AM, Francisco Jerez 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
> b/src/mesa/drivers/dri/i965/brw_fs.cpp
> i
On Wed, Jul 22, 2015 at 4:26 AM, Eduardo Lima Mitev wrote:
> On 07/13/2015 01:38 PM, Jason Ekstrand wrote:
>> On Fri, Jul 10, 2015 at 8:53 AM, Eduardo Lima Mitev wrote:
>>> On 06/30/2015 06:58 PM, Jason Ekstrand wrote:
On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev
wrote:
> T
https://bugs.freedesktop.org/show_bug.cgi?id=91425
--- Comment #3 from Ilia Mirkin ---
(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 llvmpipe. I guess t
On 22/07/15 16:04, Emil Velikov wrote:
Otherwise the final dri module will have (additional) unresolved
symbols.
Cc: Brian Paul
Cc: Jose Fonseca
Signed-off-by: Emil Velikov
---
We can only fix the remaining unresolved symbols (_glapi_foo), as we
remove the non-shared glapi when building with
On Wed, Jul 22, 2015 at 12:55 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez
>> 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
>>> em
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 Wed, Jul 22, 2015 at 12:54 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> On Thu, Jul 16, 2015 at 8:41 AM, Francisco Jerez
>> 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
>>>
On 21/07/15 15:57, Emil Velikov wrote:
On 18 July 2015 at 08:13, Jose Fonseca wrote:
On 18/07/15 01:38, Eric Anholt wrote:
Emil Velikov writes:
On 14/07/15 19:45, Eric Anholt wrote:
These are really useful hints to the compiler in the absence of
link-time
optimization, and I'm going to u
On Wed, Jul 22, 2015 at 12:43 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez
>> 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
>>> implici
On Wed, Jul 22, 2015 at 12:41 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez
>> wrote:
>>> ---
>>> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 66
>>> +---
>>> 1 file changed, 49 insertions(+), 17 deletions(
On Wed, Jul 22, 2015 at 12:31 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> A few comments below. Mostly just asking for explanation.
>>
>> 1-3 are
>>
>> Reviewed-by: Jason Ekstrand
>>
>> Obviously, don't merge 4/4 until it actually has users.
>> --Jason
>
> Thanks.
>
>>
>> On Thu, J
It should be in master now.
-Emil
On 21 July 2015 at 23:34, Dylan Baker 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 Velikov" wrote:
>>
Otherwise the final dri module will have (additional) unresolved
symbols.
Cc: Brian Paul
Cc: Jose Fonseca
Signed-off-by: Emil Velikov
---
We can only fix the remaining unresolved symbols (_glapi_foo), as we
remove the non-shared glapi when building with DRI.
With this we at least match the a
Reviewed-by: Iago Toral Quiroga
On Tue, 2015-06-16 at 11:15 -0700, Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/common/meta.c | 7 ++-
> 1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/met
https://bugs.freedesktop.org/show_bug.cgi?id=91425
--- Comment #2 from Michel Dänzer ---
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.
___
m
Looks good to me, if this did not introduce any regressions:
Reviewed-by: Iago Toral Quiroga
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 is what we
>
https://bugs.freedesktop.org/show_bug.cgi?id=91387
--- Comment #5 from Michael Godfrey ---
I should have added that the graphics in my system is
Intel HD 6000 in an Intel NUC5i5RYH.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
___
https://bugs.freedesktop.org/show_bug.cgi?id=91387
Brian Paul changed:
What|Removed |Added
CC||anuj.pho...@gmail.com
--- Comment #4 from B
https://bugs.freedesktop.org/show_bug.cgi?id=91387
--- Comment #3 from Michael Godfrey ---
Created attachment 117297
--> https://bugs.freedesktop.org/attachment.cgi?id=117297&action=edit
backtrace
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee
https://bugs.freedesktop.org/show_bug.cgi?id=91425
--- Comment #1 from Brian Paul ---
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 the bug.
You are
Patches 10-11 are
Reviewed-by: Iago Toral Quiroga
On Tue, 2015-06-16 at 11:15 -0700, Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/common/meta.c | 12 ++--
> 1 file changed, 2 insertions(+), 10 deletions(-)
>
> diff --git a/src/mesa/drivers/common/meta.c b/s
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
On 07/13/2015 01:57 PM, Jason Ekstrand wrote:
> On Wed, Jul 8, 2015 at 11:54 PM, Eduardo Lima Mitev wrote:
>> On 06/30/2015 06:51 PM, Jason Ekstrand wrote:
>>> On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev
>>> wrote:
The index into the output_reg array where to store the destination
On 07/13/2015 01:38 PM, Jason Ekstrand wrote:
> On Fri, Jul 10, 2015 at 8:53 AM, Eduardo Lima Mitev wrote:
>> On 06/30/2015 06:58 PM, Jason Ekstrand wrote:
>>> On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev
>>> wrote:
This patch makes public the is_scalar_shader_stage() method in brw_s
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
>
> 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 | http:/
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
Kenneth Graunke 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 sources, like:
Jason Ekstrand writes:
> On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez
> 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 SIMD16 workaround.
>>
>> Hint for re
Jason Ekstrand writes:
> On Thu, Jul 16, 2015 at 8:41 AM, Francisco Jerez
> 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 SIMD unrolling logic in the dual-so
Jason Ekstrand writes:
> On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez
> 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.
>> ---
>> src/mesa/drivers/dri/i965/brw_fs_
Jason Ekstrand writes:
> On Sat, Jul 18, 2015 at 7:34 AM, Francisco Jerez
> wrote:
>> ---
>> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 66
>> +---
>> 1 file changed, 49 insertions(+), 17 deletions(-)
>>
>> diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
Jason Ekstrand writes:
> A few comments below. Mostly just asking for explanation.
>
> 1-3 are
>
> Reviewed-by: Jason Ekstrand
>
> Obviously, don't merge 4/4 until it actually has users.
> --Jason
Thanks.
>
> On Thu, Jul 16, 2015 at 8:35 AM, Francisco Jerez
> wrote:
>> This lowering pass im
100 matches
Mail list logo
