On 13/07/15 13:06, Samuel Iglesias Gonsálvez wrote:
> On 11/07/15 19:38, Renaud Gaubert wrote:
>> This is done by returning an rvalue of type void in the
>> ast_function_expression::hir function instead of a void expression.
>>
>> This produces (in the case of the ternary) an hir with a call
>> t
On Wed, 2015-07-15 at 11:02 -0700, Connor Abbott wrote:
> On Wed, Jul 15, 2015 at 7:49 AM, Iago Toral wrote:
> > Hi,
> >
> > when we sent the patches for the new nir->vec4 backend we mentioned that
> > we had a few dEQP tests that would fail to link because of register
> > spilling. Now that we ha
---
This will go in before my double promotion series which uses exp2f.
include/c99_math.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/include/c99_math.h b/include/c99_math.h
index 7ed7cc2..0ca5a73 100644
--- a/include/c99_math.h
+++ b/include/c99_math.h
@@ -140,6 +140,12 @@ llrintf
Non-Gallium parts of Mesa require MSVC 2013 which provides these.
---
src/glsl/ir_constant_expression.cpp | 14 +-
src/glsl/nir/nir_constant_expressions.py | 14 +-
2 files changed, 2 insertions(+), 26 deletions(-)
diff --git a/src/glsl/ir_constant_expression.cpp
b/s
Am 16.07.2015 um 02:18 schrieb Ilia Mirkin:
> On Wed, Jul 15, 2015 at 7:40 PM, Brian Paul wrote:
>> This adds the new glGetTextureSubImage() and
>> glGetCompressedTextureSubImage() functions. Also update the
>> dispatch sanity test program.
>>
>> v2: remove stray brace, move xi:include line in gl
Am 13.07.2015 um 17:58 schrieb Emil Velikov:
> On 11 July 2015 at 19:39, wrote:
>
>> @@ -186,17 +172,24 @@ static inline void emit_cb_setup(struct r100_context
>> *r100,
>>
>> /* XXX others? BE/LE? */
> Drop the BE/LE part of the comment ?
>
>> switch (mesa_format) {
>> +/* le *
The reason why you chose to call select_tex_image() several times
through a get_tex_image flow eludes me. Why not just get it once at
the beginning and pass it around? The old APIs also used a
gl_texture_image and not a gl_texture_object. (Obviously the object is
implicitly gettable from the image
On Wed, Jul 15, 2015 at 7:40 PM, Brian Paul wrote:
> This adds the new glGetTextureSubImage() and
> glGetCompressedTextureSubImage() functions. Also update the
> dispatch sanity test program.
>
> v2: remove stray brace, move xi:include line in gl_API.xml, fix extension
> number typo, s/program/te
Needed for GL_ARB_get_texture_sub_image. But at this point, the
offsets are always zero and the sizes match the whole texture image.
v2: Fixes, suggestions from Laura Ekstrand:
* Fix calls to ctx->Driver.UnmapTextureImage() to pass the correct
slice value.
* Added comments and assertions to che
In preparation for decompressing texture sub images.
Reviewed-by: Ilia Mirkin
---
src/mesa/drivers/common/meta.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
index 54c3d5a..34a8e4b 100644
--- a/src/
Same idea as the previous patch.
---
src/mesa/main/texgetimage.c | 345 ++--
1 file changed, 203 insertions(+), 142 deletions(-)
diff --git a/src/mesa/main/texgetimage.c b/src/mesa/main/texgetimage.c
index 49239e7..6ac1779 100644
--- a/src/mesa/main/texgeti
Reviewed-by: Ilia Mirkin
---
src/mesa/main/texgetimage.c | 13 -
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/src/mesa/main/texgetimage.c b/src/mesa/main/texgetimage.c
index fb3c2c8..e180a4c 100644
--- a/src/mesa/main/texgetimage.c
+++ b/src/mesa/main/texgetimage.c
@@
---
docs/GL3.txt | 2 +-
docs/relnotes/10.7.0.html | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 33a282e..2144257 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -202,7 +202,7 @@ GL 4.5, GLSL 4.50:
- Sampler object
---
src/mesa/main/extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index d753e5f..7deb823 100644
--- a/src/mesa/main/extensions.c
+++ b/src/mesa/main/extensions.c
@@ -121,6 +121,7 @@ static const struct extension extension_tab
v2: fix depth, total_depth mix-up in meta.h, per Laura Ekstrand.
---
src/mesa/drivers/common/meta.c | 88 +-
src/mesa/drivers/common/meta.h | 6 +-
src/mesa/drivers/common/meta_generate_mipmap.c | 4 +-
3 files changed, 65 insertions(+), 33
Simple implementations in terms of get_[compressed_]texture_image().
---
src/mesa/main/texgetimage.c | 62 -
src/mesa/main/texgetimage.h | 15 +++
2 files changed, 76 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/texgetimage.c b/src/me
---
src/mesa/main/dd.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/dd.h b/src/mesa/main/dd.h
index ae01770..87eb63e 100644
--- a/src/mesa/main/dd.h
+++ b/src/mesa/main/dd.h
@@ -328,7 +328,7 @@ struct dd_function_table {
void (*CompressedTexSubImage)(struct
The new driver hook has x/y/zoffset and width/height/depth parameters
for the new glGetTextureSubImage() function.
The meta code and gallium state tracker are updated to handle the
new parameters.
Callers to Driver.GetTexSubImage() pass in offsets=0 and sizes equal
to the whole texture size.
v2:
1. Reorganize the error checking code.
2. Lay groundwork for getting sub images by passing image offset and
dimensions to the error checking code.
3. Implement _mesa_GetnTexImageARB(), _mesa_GetTexImage() and
_mesa_GetTextureImage() all in terms of get_texture_image().
v2: pass offset/width/
These functions are only called from teximage.c
Reviewed-by: Ilia Mirkin
---
src/mesa/main/texgetimage.c | 24
src/mesa/main/texgetimage.h | 7 ---
2 files changed, 12 insertions(+), 19 deletions(-)
diff --git a/src/mesa/main/texgetimage.c b/src/mesa/main/texgetima
This adds the new glGetTextureSubImage() and
glGetCompressedTextureSubImage() functions. Also update the
dispatch sanity test program.
v2: remove stray brace, move xi:include line in gl_API.xml, fix extension
number typo, s/program/texture/ in xml file.
---
src/mapi/glapi/gen/ARB_get_texture_sub
For now, pass offsets of zero and width/height/depth equal to the
whole image.
Reviewed-by: Ilia Mirkin
---
src/mesa/drivers/common/driverfuncs.c | 2 +-
src/mesa/main/dd.h | 9 ++---
src/mesa/main/texgetimage.c| 28
src/mesa/ma
On 07/14/2015 03:54 PM, Ilia Mirkin wrote:
On Mon, Jul 13, 2015 at 9:21 PM, Brian Paul wrote:
1. Reorganize the error checking code.
2. Lay groundwork for getting sub images.
3. Implement _mesa_GetnTexImageARB(), _mesa_GetTexImage() and
_mesa_GetTextureImage() all in terms of get_texture_im
On Tue, Jul 14, 2015 at 9:56 AM, Ben Widawsky
wrote:
> At the crux of this change is moving whether or not we can even use the
> hardware
> blitter into the can_blit_slice check. Fundamentally this makes sense as
> blitting a slice is a subset in functionality of being able to use the blitter
> a
On Tue, Jul 14, 2015 at 9:56 AM, Ben Widawsky
wrote:
> With the last few patches a way was provided to influence lower layer miptree
> layout and allocation decisions via flags (replacing bools). For simplicity, I
> chose not to touch the tiling requests because the change was slightly less
> mech
Am 15.07.2015 um 21:58 schrieb Matt Turner:
> On Wed, Jul 15, 2015 at 12:44 PM, Matt Turner wrote:
>> On Mon, Jul 13, 2015 at 5:13 PM, Roland Scheidegger
>> wrote:
>>> Did you replace 2 of them by exp2f but one by exp2f on purpose?
>>>
>>> I don't think we can use exp2/exp2f in gallium. This req
On Wed, Jul 15, 2015 at 11:02:03AM -0700, Connor Abbott wrote:
> On Wed, Jul 15, 2015 at 7:49 AM, Iago Toral wrote:
> > Hi,
> >
> > when we sent the patches for the new nir->vec4 backend we mentioned that
> > we had a few dEQP tests that would fail to link because of register
> > spilling. Now tha
Am 15.07.2015 um 21:44 schrieb Matt Turner:
> On Mon, Jul 13, 2015 at 5:13 PM, Roland Scheidegger
> wrote:
>> Did you replace 2 of them by exp2f but one by exp2f on purpose?
>>
>> I don't think we can use exp2/exp2f in gallium. This requires msvc 2013
>> (all of exp2, exp2f, powf are c99, powf is
On Wed, Jul 15, 2015 at 12:41 PM, Chris Wilson wrote:
> On Wed, Jul 15, 2015 at 12:20:15PM -0700, Kristian Høgsberg wrote:
>> On Wed, Jul 15, 2015 at 10:22 AM, Kenneth Graunke
>> wrote:
>> > From: Chris Wilson
>> >
>> > The kernel actually waits forever when supplied a timeout value < 0,
>> > r
On Wed, Jul 15, 2015 at 12:44 PM, Matt Turner wrote:
> On Mon, Jul 13, 2015 at 5:13 PM, Roland Scheidegger
> wrote:
>> Did you replace 2 of them by exp2f but one by exp2f on purpose?
>>
>> I don't think we can use exp2/exp2f in gallium. This requires msvc 2013
>> (all of exp2, exp2f, powf are c9
On Tue, Jul 14, 2015 at 4:45 AM, Iago Toral wrote:
> On Mon, 2015-07-13 at 16:22 -0700, Matt Turner wrote:
>> case GL_READ_BUFFER:
>> diff --git a/src/mesa/main/light.c b/src/mesa/main/light.c
>> index 4021dbe..fe2ce8c 100644
>> --- a/src/mesa/main/light.c
>> +++ b/src/mesa/main/light.c
>> @@
On Tue, Jul 14, 2015 at 3:53 AM, Iago Toral wrote:
> On Mon, 2015-07-13 at 16:22 -0700, Matt Turner wrote:
>> There are a couple of unrelated changes in t_vb_lighttmp.h that I hope
>> you'll excuse -- there's a block of code that's duplicated modulo a few
>> trivial differences that I took the lib
On Mon, Jul 13, 2015 at 5:13 PM, Roland Scheidegger wrote:
> Did you replace 2 of them by exp2f but one by exp2f on purpose?
>
> I don't think we can use exp2/exp2f in gallium. This requires msvc 2013
> (all of exp2, exp2f, powf are c99, powf is supported by older msvc but
> the others are not). I
On Wed, Jul 15, 2015 at 12:20:15PM -0700, Kristian Høgsberg wrote:
> On Wed, Jul 15, 2015 at 10:22 AM, Kenneth Graunke
> wrote:
> > From: Chris Wilson
> >
> > The kernel actually waits forever when supplied a timeout value < 0,
> > rather than returning immediately. See i915_gem_wait_ioctl() in
On Tue, Jul 14, 2015 at 4:32 AM, Daniel Stone wrote:
> Hi,
>
> On 14 July 2015 at 00:22, Matt Turner wrote:
> but it's not really
>> useful in general because float arguments are always cast to double
>> when passed as arguments to varargs functions like printf (why?), and
>> it warns about that,
On Wed, Jul 15, 2015 at 12:48 AM, Mathias Fröhlich
wrote:
>
>
> Hi Matt,
>
>
>
> On Monday, July 13, 2015 16:22:03 Matt Turner wrote:
>
>> ARB_viewport_array specifies that DEPTH_RANGE consists of double-
>
>> precision parameters (corresponding commit d4dc35987), and a preparatory
>
>> commit (63
On Wed, Jul 15, 2015 at 10:22 AM, Kenneth Graunke wrote:
> From: Chris Wilson
>
> The kernel actually waits forever when supplied a timeout value < 0,
> rather than returning immediately. See i915_gem_wait_ioctl() in
> i915_gem.c's call to __i915_wait_request().
>
> (split by Ken from a large pa
On Wed, Jul 15, 2015 at 11:53 AM, Eric Anholt wrote:
> Timothy Arceri writes:
>
>> Hi guys,
>>
>> As I've mentioned a couple of times in previous patches some of the cts AoA
>> tests are taking very long time to compile. This is due to excessive
>> optimisation passes mainly in the glsl optimisat
Timothy Arceri writes:
> Hi guys,
>
> As I've mentioned a couple of times in previous patches some of the cts AoA
> tests are taking very long time to compile. This is due to excessive
> optimisation passes mainly in the glsl optimisations (there are some slowdowns
> in the intel backend too but
Hi Matt,
I've commented on some of your feedback down below.
The rest is taken note of and I'll be fixing it up later.
2015-07-15 19:18 GMT+02:00 Matt Turner :
> On Tue, Jul 14, 2015 at 4:29 PM, Thomas Helland
> wrote:
>> Signed-off-by: Thomas Helland
>> ---
>> src/glsl/Makefile.sources
On Wed, Jul 15, 2015 at 7:49 AM, Iago Toral wrote:
> Hi,
>
> when we sent the patches for the new nir->vec4 backend we mentioned that
> we had a few dEQP tests that would fail to link because of register
> spilling. Now that we have added GS support we see a few instances of
> this problem popping
On Wed, Jul 15, 2015 at 4:31 AM, Francisco Jerez wrote:
> Connor Abbott writes:
>
>> On Tue, Jul 14, 2015 at 6:02 AM, Francisco Jerez
>> wrote:
>>> Connor Abbott writes:
>>>
sources with file == HW_REG get all their information from the
fixed_hw_reg field, so we need to get the strid
From: Chris Wilson
The kernel actually waits forever when supplied a timeout value < 0,
rather than returning immediately. See i915_gem_wait_ioctl() in
i915_gem.c's call to __i915_wait_request().
(split by Ken from a large patch authored by Chris Wilson)
Reviewed-by: Kenneth Graunke
---
src/
On Tue, Jul 14, 2015 at 4:29 PM, Thomas Helland
wrote:
> Signed-off-by: Thomas Helland
> ---
> src/glsl/Makefile.sources |1 +
> src/glsl/nir/nir.h |2 +
> src/glsl/nir/nir_opt_value_range.c | 1330
>
> 3 files changed, 1333
Otherwise Coverity thinks we'll divide by zero.
---
src/mesa/drivers/dri/i965/brw_reg.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_reg.h
b/src/mesa/drivers/dri/i965/brw_reg.h
index c8b1341..f96b28d 100644
--- a/src/mesa/drivers/dri/i965/brw_r
More.. like in commit 4d93a07c.
---
src/mesa/drivers/dri/i965/brw_cfg.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_cfg.cpp
b/src/mesa/drivers/dri/i965/brw_cfg.cpp
index f1f230e..91d53ef 100644
--- a/src/mesa/drivers/dri/i965/brw_cfg.cpp
+++ b/src/mesa/d
---
src/egl/drivers/dri2/platform_drm.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/egl/drivers/dri2/platform_drm.c
b/src/egl/drivers/dri2/platform_drm.c
index 0d1f4c6..a8c5401 100644
--- a/src/egl/drivers/dri2/platform_drm.c
+++ b/src/egl/drivers/dri2/platform_dr
On Tue, Jul 14, 2015 at 11:45 AM, Eric Anholt wrote:
> These are really useful hints to the compiler in the absence of link-time
> optimization, and I'm going to use them in VC4.
>
> I've made the const attribute be ATTRIBUTE_CONST unlike other function
> attributes, because we have other things i
Acked-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Include stdarg.h for va_list. Unbreaks the build on OpenBSD:
In file included from mesa/program/dummy_errors.c:24:
../src/mesa/main/errors.h:85: error: expected declaration specifiers or '...' be
fore 'va_list'
Signed-off-by: Jonathan Gray
---
src/mesa/main/errors.h | 1 +
1 file changed, 1 in
Hi,
when we sent the patches for the new nir->vec4 backend we mentioned that
we had a few dEQP tests that would fail to link because of register
spilling. Now that we have added GS support we see a few instances of
this problem popping up in a few GS piglit tests too, for example this
one:
tests/
https://bugs.freedesktop.org/show_bug.cgi?id=91337
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
From: Tom Hughes
https://bugs.freedesktop.org/show_bug.cgi?id=90817
Signed-off-by: Jose Fonseca
---
src/glx/dri_common.c | 59 +++-
1 file changed, 58 insertions(+), 1 deletion(-)
diff --git a/src/glx/dri_common.c b/src/glx/dri_common.c
index 63
On 15 July 2015 at 13:23, Brian Paul wrote:
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91337
> Cc: 10.6
Seems like it was broken since day 1 with commit 01dc182ee86(added
OSMesaGetProcAddress())
Reviewed-by: Emil Velikov
-Emil
___
mesa-
On Wed, 2015-07-15 at 16:42 +1000, Timothy Arceri wrote:
> Hi guys,
>
> As I've mentioned a couple of times in previous patches some of the cts AoA
> tests are taking very long time to compile. This is due to excessive
> optimisation passes mainly in the glsl optimisations (there are some
> slowd
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91337
Cc: 10.6
---
src/gallium/state_trackers/osmesa/osmesa.c | 2 +-
src/mesa/drivers/osmesa/osmesa.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_trackers/osmesa/osmesa.c
b/src/gallium/
https://bugs.freedesktop.org/show_bug.cgi?id=91337
--- Comment #1 from Brian Paul ---
Thanks. Patch posted to mesa-dev for review.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
___
mesa-d
Connor Abbott writes:
> On Tue, Jul 14, 2015 at 6:02 AM, Francisco Jerez
> wrote:
>> Connor Abbott writes:
>>
>>> sources with file == HW_REG get all their information from the
>>> fixed_hw_reg field, so we need to get the stride and type from there
>>> when computing the size.
>>>
>>> Signed-
Acked-by: Marek Olšák
Marek
On Wed, Jul 15, 2015 at 7:15 AM, Mario Kleiner
wrote:
> The amdgpu_device for a device node needs its own dup'ed fd, instead
> of using the original fd passed in for a screen, to make multi-x-screen
> ZaphodHeads configurations work on amdgpu.
>
> The original fd's l
https://bugs.freedesktop.org/show_bug.cgi?id=90264
--- Comment #36 from Michel Dänzer ---
(In reply to Heiko from comment #35)
> Iirc from my last post, radeon_draw_buffer() does re-evaluate window sizes.
> That's probably why the reverted commit makes things work again.
Ken, do you think it's p
Hi Matt,
On Monday, July 13, 2015 16:22:03 Matt Turner wrote:
> ARB_viewport_array specifies that DEPTH_RANGE consists of double-
> precision parameters (corresponding commit d4dc35987), and a preparatory
> commit (6340e609a) added _mesa_get_viewport_xform() which returned
> double-precision scale
On Tue, Jul 14, 2015 at 04:48:19PM -0700, Ben Widawsky wrote:
> On Wed, Jul 01, 2015 at 02:46:32PM +0300, Topi Pohjolainen wrote:
> > Signed-off-by: Topi Pohjolainen
>
> I swear I am not trying to nitpick but I don't actually understand what your
> goal of the patch is. Could you maybe elaborate
On 15.07.2015 07:15, Mario Kleiner wrote:
The amdgpu_device for a device node needs its own dup'ed fd, instead
of using the original fd passed in for a screen, to make multi-x-screen
ZaphodHeads configurations work on amdgpu.
The original fd's lifetime differs from that of the amdgpu_device, and
63 matches
Mail list logo
