Ian Romanick writes:
> From: Kristian Høgsberg
>
> The driverPrivate pointer is opaque to the driver and we can't assume
> it's a struct gl_context in dri_util.c. Instead provide a helper function
> to set the struct gl_context flags from the incoming DRI context flags.
>
> v2 (idr): Modify the
Disabled by default, but it's very useful when needed.
Signed-off-by: Zack Rusin
---
src/gallium/drivers/llvmpipe/lp_setup_point.c | 20
1 file changed, 20 insertions(+)
diff --git a/src/gallium/drivers/llvmpipe/lp_setup_point.c
b/src/gallium/drivers/llvmpipe/lp_setup_poin
Caching in the vbuf module meant that once a vertex has been
emitted it was cached, but it's possible for a vertex at the
same location to be emitted again, but this time with a different
front-face semantic. Caching was causing the first version of the
vertex to be emitted, which resulted in the r
On Tue, Dec 10, 2013 at 02:54:46AM +0400, Vadim Girlin wrote:
> On Mon, 2013-12-09 at 10:56 -0500, Tom Stellard wrote:
> > On Sat, Dec 07, 2013 at 07:06:36PM +0400, Vadim Girlin wrote:
> > > On evergreen we have to reserve 1 stack element in some additional cases
> > > besides the ones mentioned in
The fact that we flush denorms to zero breaks our half-float
conversion and blending. This patches enables denorms for
blending. It's a little tricky due to the llvm bug that makes
it incorrectly reorder the mxcsr intrinsics:
http://llvm.org/bugs/show_bug.cgi?id=6393
Signed-off-by: Zack Rusin
---
OpenGL with ARB_ES2_compatibility extension allows shaders that specify
#version 100 and the shader is treated as targeting version 100 of the
GLSL ES.
state->es_shader flag might change during parsing based on the #version
directive mentioned in the shader program. This patch adds another check
t
https://bugs.freedesktop.org/show_bug.cgi?id=70250
--- Comment #5 from Ian Romanick ---
Since you marked the bug as VERIFIED, I'll call that a Tested-by. Sorry for
the noise.
--
You are receiving this mail because:
You are the QA Contact for the bug.
___
https://bugs.freedesktop.org/show_bug.cgi?id=70250
--- Comment #4 from Ian Romanick ---
Artie,
Did this fix the problem for you? There's no Reviewed-by on the patch, so I
want *something* before I cherry pick it over to stable.
Thanks.
--
You are receiving this mail because:
You are the QA C
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 12/09/2013 04:58 PM, Eric Anholt wrote:
> Ian Romanick writes:
>
>> -BEGIN PGP SIGNED MESSAGE- Hash: SHA1
>>
>> On 12/03/2013 10:44 AM, Eric Anholt wrote:
>>> Siavash Eliasi writes:
>>>
Hello, this is V2 series of patches to accomp
Ian Romanick writes:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA1
>
> On 12/03/2013 10:44 AM, Eric Anholt wrote:
>> Siavash Eliasi writes:
>>
>>> Hello, this is V2 series of patches to accomplish *Enable
>>> ARB_map_buffer_alignment in all drivers* newbie project suggested
>>> by Ian Roma
From: Kristian Høgsberg
The driverPrivate pointer is opaque to the driver and we can't assume
it's a struct gl_context in dri_util.c. Instead provide a helper function
to set the struct gl_context flags from the incoming DRI context flags.
v2 (idr): Modify the other classic drivers to also use
Jordan Justen writes:
> To help the transition period when DRI loaders are being updated
> to support the newer __driDriverExtensions_foo mechanism,
> we populate __driDriverExtensions with the extensions returned
> by __driDriverExtensions_foo during a library contructor
> function.
>
> We find
calculate_attr_overrides is responsible for computing the point sprite
and flat-shading enable bitfields. It does so by OR'ing in a bunch of
bits. However, it relied on the caller to set the initial value to
zero. This is pretty fragile - if the caller neglects to zero out those
variables, then
Calling the local variables flat_enable and point_sprite_enable is
clearer than dw16 and such. It also matches the names used in
calculate_attr_overrides, which computes them.
Signed-off-by: Kenneth Graunke
Cc: Paul Berry
---
src/mesa/drivers/dri/i965/gen6_sf_state.c | 12 +++-
src/mes
On 7 December 2013 17:17, Francisco Jerez wrote:
> And to check if it can have layers at all. This will be used by the
> implementation of ARB_shader_image_load_store.
> ---
> src/mesa/main/teximage.c | 85
>
> src/mesa/main/teximage.h | 6
On 7 December 2013 17:12, Francisco Jerez wrote:
> Paul Berry writes:
> >
> > With those changes, this patch is:
> >
> > Reviewed-by: Paul Berry
>
> I think I've taken into account all your comments here [1], and I've
> changed the value of MAX_IMAGE_UNITS. Let me know if your Reviewed-by
> st
On 7 December 2013 17:11, Francisco Jerez wrote:
> Paul Berry writes:
>
> > On 24 November 2013 21:00, Francisco Jerez
> wrote:
> >[...]
> >> +
> >> + case GL_RGBA16UI:
> >> + return MESA_FORMAT_RGBA_UINT16;
> >> +
> >> + case GL_RGB10_A2UI:
> >> + return MESA_FORMAT_ABGR2101010_U
On 7 December 2013 07:42, Francisco Jerez wrote:
> Paul Berry writes:
>
> > On 24 November 2013 21:00, Francisco Jerez
> wrote:
> >
> >> Including pack/unpack and texstore code. This texture format is a
> >> requirement for ARB_shader_image_load_store.
> >> ---
> >> src/mesa/main/format_pack.
On Mon, 2013-12-09 at 10:56 -0500, Tom Stellard wrote:
> On Sat, Dec 07, 2013 at 07:06:36PM +0400, Vadim Girlin wrote:
> > On evergreen we have to reserve 1 stack element in some additional cases
> > besides the ones mentioned in the docs, but stack size computation was
> > recently reimplemented e
On Mon, Dec 9, 2013 at 12:41 PM, Ian Romanick wrote:
> On 12/09/2013 10:53 AM, Ian Romanick wrote:
>> On 12/07/2013 10:02 PM, Kristian Høgsberg wrote:
>>> The driverPrivate pointer is opaque to the driver and we can't assume
>>> it's a struct gl_context in dri_util.c. Instead provide a helper fun
To help the transition period when DRI loaders are being updated
to support the newer __driDriverExtensions_foo mechanism,
we populate __driDriverExtensions with the extensions returned
by __driDriverExtensions_foo during a library contructor
function.
We find the driver foo's name by using the dl
On 5 December 2013 11:33, Carl Worth wrote:
> Carl Worth writes:
> > So, you should all expect to see these hook messages now:
> >
> >> remote: I: patch # updated using rev .
> > ...
> >> remote: E: failed to find patch for rev .
>
> For anyone who has pushed patches for the past coupl
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 12/03/2013 10:44 AM, Eric Anholt wrote:
> Siavash Eliasi writes:
>
>> Hello, this is V2 series of patches to accomplish *Enable
>> ARB_map_buffer_alignment in all drivers* newbie project suggested
>> by Ian Romanick.
>
> I think there's a piece
On Sat, Dec 07, 2013 at 07:06:36PM +0400, Vadim Girlin wrote:
> On evergreen we have to reserve 1 stack element in some additional cases
> besides the ones mentioned in the docs, but stack size computation was
> recently reimplemented exactly as described in the docs by the patch that
> added worka
On 12/09/2013 10:53 AM, Ian Romanick wrote:
> On 12/07/2013 10:02 PM, Kristian Høgsberg wrote:
>> The driverPrivate pointer is opaque to the driver and we can't assume
>> it's a struct gl_context in dri_util.c. Instead provide a helper function
>> to set the struct gl_context flags from the incomi
Thanks Paul, that makes a lot of sense.
-- Chris
On Tue, Dec 10, 2013 at 8:34 AM, Paul Berry wrote:
> On 9 December 2013 11:06, Chris Forbes wrote:
>>
>> Jordan, Abdiel:
>>
>> Could you provide some context -- the impression I get is that using
>> the tile offsets is undesirable for reasons oth
On 9 December 2013 11:06, Chris Forbes wrote:
> Jordan, Abdiel:
>
> Could you provide some context -- the impression I get is that using
> the tile offsets is undesirable for reasons other than sharing code
> with Gen8, but not familiar enough with the details to know why.
>
> Just curious :)
>
>
Jordan, Abdiel:
Could you provide some context -- the impression I get is that using
the tile offsets is undesirable for reasons other than sharing code
with Gen8, but not familiar enough with the details to know why.
Just curious :)
-- Chris
On Tue, Dec 10, 2013 at 7:21 AM, Jordan Justen wrot
On 12/07/2013 10:02 PM, Kristian Høgsberg wrote:
> The driverPrivate pointer is opaque to the driver and we can't assume
> it's a struct gl_context in dri_util.c. Instead provide a helper function
> to set the struct gl_context flags from the incoming DRI context flags.
>
> Signed-off-by: Kristia
On Mon, Dec 9, 2013 at 8:18 AM, Abdiel Janulgue
wrote:
> (cover letter seems to be lost in the way for the patch series but here it is)
>
> Gen 6/7 hardware support for tile_x and tile_y offset fields in SURFACE_STATE
> has been removed in Gen8. All code dependencies must be removed. This patch
>
https://bugs.freedesktop.org/show_bug.cgi?id=69285
--- Comment #3 from Charles Huber ---
Still broken in 10.0.
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://li
On 12/09/2013 08:19 AM, Pohjolainen, Topi wrote:
On Mon, Dec 09, 2013 at 02:29:21PM +0200, Juha-Pekka Heikkila wrote:
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/main/attrib.c | 320 +
1 file changed, 272 insertions(+), 48 deletions(-)
dif
On 12/09/2013 06:15 AM, Juha-Pekka Heikkila wrote:
Signed-off-by: Juha-Pekka Heikkila
---
src/glx/dri2.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/glx/dri2.c b/src/glx/dri2.c
index bcd1f9c..deb50ab 100644
--- a/src/glx/dri2.c
+++ b/src/glx/dri2.c
@@ -102,6 +102,8 @@ DRI2WireTo
On 12/07/2013 06:17 PM, Francisco Jerez wrote:
And to check if it can have layers at all. This will be used by the
implementation of ARB_shader_image_load_store.
---
src/mesa/main/teximage.c | 85
src/mesa/main/teximage.h | 6
2 files ch
(cover letter seems to be lost in the way for the patch series but here it is)
Gen 6/7 hardware support for tile_x and tile_y offset fields in SURFACE_STATE
has been removed in Gen8. All code dependencies must be removed. This patch
series completely eliminates the tile_x and y fields in the no
On 12/07/2013 11:17 PM, Chris Forbes wrote:
The target parameter to _mesa_get_tex_image() is a target enum, not an index.
When we're setting up faces for a cubemap, it should be
CUBE_MAP_POSITIVE_X .. CUBE_MAP_NEGATIVE_Z; for all other targets it
should be the same as the texobj's target.
Fixes
GEN7+-specific hw-support. This can be copied over to GEN8/Broadwell with few
modifications.
Signed-off-by: Abdiel Janulgue
---
src/mesa/drivers/dri/i965/gen7_blorp.cpp | 24
1 file changed, 8 insertions(+), 16 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen7_
Hi,
The first two patches register the R600EmitClauseMarkers and
AMDGPUCFGStructurizer passes, so that -print-*-all will now work with
them. The third patch is a cleanup of Processors.td and the fourth
patch adds the wavefront size property to the subtargets.
Please Review.
-Tom
>From 28e45d441
We don't need this and the hardware allows us to sample directly from
a mipmap's specific level and layer. We can simplify
the operation by adding a level field in DRIImage instead.
Signed-off-by: Abdiel Janulgue
---
src/mesa/drivers/dri/i965/intel_regions.h |3 +--
src/mesa/drivers/dri/i965
Signed-off-by: Abdiel Janulgue
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c |3 ++-
src/mesa/drivers/dri/i965/gen7_wm_surface_state.c |3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
b/src/mesa/drivers/dri/i9
GEN6-specific. Using the level and lod fields allows us to resolve directly
to mipmap's level. Remove legacy code to specify the level using the
tile_x and y offsets.
Signed-off-by: Abdiel Janulgue
---
src/mesa/drivers/dri/i965/gen6_blorp.cpp | 25 +
1 file changed, 9 i
Remove legacy code when recreating a single-layer miptree from a level
of the source texture using tile_x and tile_y offsets. Save the level field
in intel_texture_object so we can resolve this direcly in the hardware.
Signed-off-by: Abdiel Janulgue
---
src/mesa/drivers/dri/i965/intel_tex_image.
On Sat, Dec 07, 2013 at 07:06:36PM +0400, Vadim Girlin wrote:
> On evergreen we have to reserve 1 stack element in some additional cases
> besides the ones mentioned in the docs, but stack size computation was
> recently reimplemented exactly as described in the docs by the patch that
> added worka
On Mon, Dec 09, 2013 at 02:29:21PM +0200, Juha-Pekka Heikkila wrote:
> Signed-off-by: Juha-Pekka Heikkila
> ---
> src/mesa/main/attrib.c | 320
> +
> 1 file changed, 272 insertions(+), 48 deletions(-)
>
> diff --git a/src/mesa/main/attrib.c b/src/
Hi,
I made this patch a while ago while adding support for eglSwapInterval
to the Wayland platform. The swap_interval member confused me for a
little while before I realised that it's not used anywhere. I think it
would be good to get rid of it.
Regards,
- Neil
--- >8 --- (use gi
Signed-off-by: Juha-Pekka Heikkila
---
src/glx/dri2.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/glx/dri2.c b/src/glx/dri2.c
index bcd1f9c..deb50ab 100644
--- a/src/glx/dri2.c
+++ b/src/glx/dri2.c
@@ -102,6 +102,8 @@ DRI2WireToEvent(Display *dpy, XEvent *event, xEvent *wire)
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/main/attrib.c | 70 ++
1 file changed, 59 insertions(+), 11 deletions(-)
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
index 3ecf533..87797d1 100644
--- a/src/mesa/main/attrib.c
+++ b/s
Change save_attrib_data() to return true/false depending on success.
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/main/attrib.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
index 8a2a268..3ecf533 100644
--- a/src/mesa/m
Simple memory allocation checks became long patches.
Juha-Pekka Heikkila (3):
Mesa: Change save_attrib_data() to return boolean
mesa: Verify memory allocations success in _mesa_PushClientAttrib
mesa: Verify memory allocations success in _mesa_PushAttrib
src/mesa/main/attrib.c | 394 +++
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/main/attrib.c | 320 +
1 file changed, 272 insertions(+), 48 deletions(-)
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
index 87797d1..3083e21 100644
--- a/src/mesa/main/attrib.c
+++ b/
Enhancement: if no function signature is found for a function
name, report that the function is not found instead of printing
an empty list of candidates.
---
src/glsl/ast_function.cpp | 27 +--
1 file changed, 17 insertions(+), 10 deletions(-)
diff --git a/src/glsl/ast_f
This patch changes the error reporting behavior for incorrect
function invocation (triggered by match_function_by_name()
unable to find a matching function call) from using the line
number information associated to the function name term
to using the line number information of the entire function
52 matches
Mail list logo
