On Thu, Jul 9, 2015 at 8:46 AM, Rob Clark wrote:
> From: Rob Clark
>
> Some, but not all, state trackers will explicitly unref (and set to
> NULL) the previous *fence before calling pipe->flush(). So driver
> should use fence_ref() which will unref the old fence if not NULL.
Looks good. Thanks.
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 in a uniform decleration as a valid type, and also for
looking up the type l
I've rebased this series, it's in my arb_shader_subroutine branch.
I've also implemented Ken's idea for a subroutine->int conversion
and put the changes into each patch that it affects.
Otherwise not much different from when I last posted.
Dave.
___
me
From: Dave Airlie
This handles converting the shader stages to the internal
prefix along with the program resource interfaces.
Signed-off-by: Dave Airlie
---
src/mesa/main/shaderobj.h | 84 +++
1 file changed, 84 insertions(+)
diff --git a/src/mesa/
From: Chris Forbes
Reviewed-by: Tapani Pälli
Reviewed-by: Kenneth Graunke
Signed-off-by: Chris Forbes
Signed-off-by: Dave Airlie
---
src/mesa/main/config.h | 6 ++
src/mesa/main/get.c | 1 +
src/mesa/main/get_hash_params.py | 4
src/mesa/main/tests
From: Chris Forbes
v2: fix output="true" and LENGTH typo
Reviewed-by: Tapani Pälli
Reviewed-by: Kenneth Graunke
Signed-off-by: Chris Forbes
Signed-off-by: Dave Airlie
---
src/mapi/glapi/gen/ARB_shader_subroutine.xml | 84
src/mapi/glapi/gen/Makefile.am
From: Dave Airlie
This is the guts of the GLSL parser and AST support for
shader subroutines.
The code creates a subroutine type in the parser, and
uses that there to validate the identifiers. The parser
also distinguishes between subroutine types/function prototypes
/uniforms and subroutine def
From: Chris Forbes
Reviewed-by: Tapani Pälli
Reviewed-by: Kenneth Graunke
Signed-off-by: Chris Forbes
Signed-off-by: Dave Airlie
---
src/glsl/glsl_lexer.ll | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/glsl_lexer.ll b/src/glsl/glsl_lexer.ll
index 10db5b8..5fd22
From: Chris Forbes
v2: [airlied]: merge version check update.
Reviewed-by: Tapani Pälli
Reviewed-by: Kenneth Graunke
Signed-off-by: Chris Forbes
Signed-off-by: Dave Airlie
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/mtypes.h | 1 +
src/mesa/main/version.c| 2 +-
3 files chan
From: Dave Airlie
We need to store two sets of info into the ir_function,
if this is a function definition with a subroutine list
(subroutine_def) or if it a subroutine prototype.
v1.1: add some more documentation.
Signed-off-by: Dave Airlie
---
src/glsl/ir.cpp | 4
src/gl
From: Chris Forbes
Reviewed-by: Tapani Pälli
Reviewed-by: Kenneth Graunke
Signed-off-by: Chris Forbes
Signed-off-by: Dave Airlie
---
src/mesa/main/shaderapi.c | 63 +++
src/mesa/main/shaderapi.h | 35 ++
2 files changed, 98
From: Chris Forbes
Reviewed-by: Tapani Pälli
Reviewed-by: Kenneth Graunke
Signed-off-by: Chris Forbes
Signed-off-by: Dave Airlie
---
src/glsl/glcpp/glcpp-parse.y| 3 +++
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 2 ++
src/glsl/standalone_scaffol
From: Dave Airlie
This adds a ir_variable which contains the subroutine uniform
and an array rvalue for the deref of that uniform, these
are stored in the ir_call and lowered later.
Signed-off-by: Dave Airlie
---
src/glsl/ir.h | 21 -
1 file changed, 20 insertions(+), 1 del
From: Dave Airlie
Add support for the subroutine uniform type ir->mesa.cpp
v1.1: add subroutine to int to switch
Signed-off-by: Dave Airlie
---
src/mesa/program/ir_to_mesa.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/program/ir_to_mesa.
From: Dave Airlie
This lowers the enhanced ir_call using the lookaside table
of subroutines into an if ladder. This initially was done
at the AST level but it caused some ordering issues so a separate
pass was required.
v2: clone return value derefs.
v2.1: update for subroutine->int convert.
Si
From: Dave Airlie
Just add support for the subroutine type to the
glsl->tgsi convertor.
v1.1: add subroutine to int support.
Signed-off-by: Dave Airlie
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracke
From: Dave Airlie
This fleshes out the ARB_program_query support for the
APIs that ARB_shader_subroutine introduces, leaving
some TODOs for later addition.
Signed-off-by: Dave Airlie
---
src/mesa/main/shader_query.cpp | 46 ++
1 file changed, 46 insertio
From: Dave Airlie
I'm not sure if we shouldn't enable this everywhere
and rip out the API checks,
discuss,
Signed-off-by: Dave Airlie
---
src/mesa/state_tracker/st_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker
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 | 457 +-
src/mesa/main/shaderapi.h
From: Dave Airlie
This adds the necessary storage for subroutine info to gl_shader.
Signed-off-by: Dave Airlie
---
src/mesa/main/mtypes.h | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index a93fe94..c53bf2d 100644
--- a/src
From: Dave Airlie
This adds linker support for subroutine uniforms, they
have some subtle differences from real uniforms, we also hide
them and they are given internal uniform names.
This also adds the subroutine locations and subroutine uniforms
to the program resource tracking for later use.
On Wed, Jul 08, 2015 at 05:08:11PM -0700, Matt Turner wrote:
> On Wed, Jul 8, 2015 at 4:53 PM, Chris Wilson wrote:
> > static void upload_viewport_state_pointers(struct brw_context *brw)
> > {
> >BEGIN_BATCH(4);
> >brw->batch.map[0] = (_3DSTATE_VIEWPORT_STATE_POINTERS << 16 | (4 - 2) |
> >
On 08.07.2015 22:34, Rob Clark wrote:
From: Rob Clark
This isn't at all clear for pipe driver writers currently, since it
is not documented anywhere. But radeon/nouveau/llvmpipe seem to drop
the ref on the **fence passed in to pipe->flush() (if *fence!=NULL).
Freedreno/ilo/vc4 where not doing
Teach EGL_EXT_image_dma_buf_import about DRM_FORMAT_R8 and
DRM_FORMAT_GR88 in egl_dri2.c. Then add the plumbing to i965 to import
R8 and GR88 dma_bufs as textures.
This Mesa series shouldn't land until my kernel patch lands:
To: dri-de...@freedesktop.org
Subject: [PATCH] drm/fourcc: Add fo
EGL_EXT_image_dma_buf_import now supports those formats.
CC: Peter Frühberger
Cc: Rainer Hochecker
Signed-off-by: Chad Versace
---
include/GL/internal/dri_interface.h | 9 +++--
src/mesa/drivers/dri/i965/intel_screen.c | 6 ++
2 files changed, 13 insertions(+), 2 deletions(-)
dif
The Kodi/XBMC developers want to transcode NV12 to RGB with OpenGL shaders,
importing the two source planes through EGL_EXT_image_dma_buf_import. That
requires importing the Y plane as an R8 EGLImage and the UV plane as either an
RG88 or GR88 EGLImage.
This patch teaches the driver-independent par
On Thu, 09 Jul 2015 07:37:59 +0200, Dave Airlie wrote:
From: Dave Airlie
Cayman needs a different method to upload the CF IDX0/1
This fixes 31 piglits when ARB_gpu_shader5 is forced on
with cayman.
Signed-off-by: Dave Airlie
---
src/gallium/drivers/r600/eg_asm.c | 17 +++--
sr
On Thu, 09 Jul 2015 08:00:48 +0200, Dave Airlie wrote:
From: Dave Airlie
temp_reg needs to be last, as we increment things
away from it, otherwise on cayman some tests were overwriting
the index regs.
Fixes 2 piglit with ARB_gpu_shader5 forced on cayman.
Signed-off-by: Dave Airlie
---
src
On 08/07/15 18:57, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
> ---
> configure.ac | 2 +-
> src/mesa/drivers/dri/i965/intel_screen.c | 5 -
> 2 files changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index d819bef..eb71
https://bugs.freedesktop.org/show_bug.cgi?id=90903
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=90249
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
URL|
https://bugs.freedesktop.org/show_bug.cgi?id=66346
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Bug 79706 depends on bug 66346, which changed state.
Bug 66346 Summary: shader_query.cpp:49: error: invalid conversion from 'void*'
to 'GLuint'
https://bugs.freedesktop.org/show_bug.cgi?id=66346
What|Removed |A
On 07/07/15 21:47, 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
> to the void returning function and an assignement of a void v
On Thu, Jul 9, 2015 at 4:21 AM, Christian König wrote:
> On 08.07.2015 22:34, Rob Clark wrote:
>>
>> From: Rob Clark
>>
>> This isn't at all clear for pipe driver writers currently, since it
>> is not documented anywhere. But radeon/nouveau/llvmpipe seem to drop
>> the ref on the **fence passed
Hello list,
The candidate for the Mesa 10.6.2 is now available. Currently we have:
- 37 queued
- 26 nominated (outstanding)
- and 2 rejected/obsolete patches
We have a moderate list of fixes this time around, most of which in
the glsl, core mesa, and the i965 & nouveau.
From a users perspecti
On 9 July 2015 at 01:46, Rob Clark wrote:
> From: Rob Clark
>
> Some, but not all, state trackers will explicitly unref (and set to
> NULL) the previous *fence before calling pipe->flush(). So driver
> should use fence_ref() which will unref the old fence if not NULL.
>
> Signed-off-by: Rob Clar
On 8 July 2015 at 18:08, Matt Turner wrote:
> On Wed, Jul 8, 2015 at 10:07 AM, Emil Velikov
> wrote:
>> Signed-off-by: Emil Velikov
>> ---
>> src/mesa/drivers/dri/i915/intel_context.c | 1 -
>> src/mesa/drivers/dri/i915/intel_context.h | 2 --
>> 2 files changed, 3 deletions(-)
>>
>> diff --gi
On 9 July 2015 at 04:45, Michel Dänzer wrote:
> On 09.07.2015 02:07, Emil Velikov wrote:
>> Most of the data stored(duplicated) was unused, and for the one that is
>> follow the approach set by other drivers.
>> This eliminates the use of legacy (dri1) types.
>
> The commentary below should have b
"major != 2" can't occur. You don't have to check the major version at
all and you can just assume it's always 2.
Marek
On Thu, Jul 9, 2015 at 2:55 PM, Emil Velikov wrote:
> On 9 July 2015 at 04:45, Michel Dänzer wrote:
>> On 09.07.2015 02:07, Emil Velikov wrote:
>>> Most of the data stored(dup
On 8 July 2015 at 18:35, Eric Anholt wrote:
> Emil Velikov writes:
>
>> Hello all,
>>
>> A recent patch by Chris, fixing some libudev fun in our loader, made
>> me think if we can clear it up a bit.
>>
>> Having three different ways of retrieving the vendor/device ID does
>> feel a bit excessive.
Should expose that only if hw has glsl 130 support?
Roland
Am 09.07.2015 um 09:17 schrieb Dave Airlie:
> From: Dave Airlie
>
> I'm not sure if we shouldn't enable this everywhere
> and rip out the API checks,
>
> discuss,
>
> Signed-off-by: Dave Airlie
> ---
> src/mesa/state_tracker/st_exte
For centroid interpolation we can just directly use the values set up
in the shader payload instead of querying the pixel interpolator. To
do this we need to modify brw_compute_barycentric_interp_modes to
detect when interpolateAtCentroid is called.
v2: Rebase on top of changes to set the pulls ba
On 8 July 2015 at 18:55, Axel Davy wrote:
> On 08/07/2015 00:15, Emil Velikov wrote :
>>
>>
>>
>> Can anyone shed a light/cast their 2c ?
>>
>> Thanks
>> Emil
>> ___
>>
>
> The DRI3/Wayland/Gallium Nine DRI_PRIME path uses libudev,
> and I'm not sure it
On 9 July 2015 at 14:16, Marek Olšák wrote:
> "major != 2" can't occur. You don't have to check the major version at
> all and you can just assume it's always 2.
>
That's even better than expected. Thanks !
-Emil
___
mesa-dev mailing list
mesa-dev@lists
On 9 July 2015 at 09:39, Chad Versace wrote:
> EGL_EXT_image_dma_buf_import now supports those formats.
>
Do I have an old version of it (v6) or I simply cannot see those listed ?
Thanks
Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http
On 30 June 2015 at 16:29, Emil Velikov wrote:
> On 22 June 2015 at 23:19, Dave Airlie wrote:
>> On 23 June 2015 at 08:16, Ian Romanick wrote:
>>> On 06/22/2015 11:54 AM, Dave Airlie wrote:
>
> As kindly hinted by Marek, currently we do have a wide selection of
> supported dri <> load
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
to the void returning function and an assignement of a void variable
which will be optimized out (the assignemen
On Thu, Jul 9, 2015 at 8:36 AM, Emil Velikov wrote:
> On 9 July 2015 at 01:46, Rob Clark wrote:
>> From: Rob Clark
>>
>> Some, but not all, state trackers will explicitly unref (and set to
>> NULL) the previous *fence before calling pipe->flush(). So driver
>> should use fence_ref() which will
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
to the void returning function and an assignement of a void variable
which will be optimized out (the assignemen
Even if your next patch needs more work this one alone still makes
sense:
Reviewed-by: Iago Toral Quiroga
Iago
On Wed, 2015-07-08 at 14:00 -0700, Matt Turner wrote:
> It's only used inside #ifdef DEBUG. Cuts ~1.7k of .text, and more
> importantly prevents a larger code size regression in the ne
On 09/07/2015 15:33, Emil Velikov wrote :
Yakes forgot about that one. From a quick look we can (I know it feels
durty) use readlink(
/sys/dev/char/$(major):$(minor)).
Everything seems to be there - bus type and exact location of the
device on the bus. It's limited to sysfs users, but they are
Attached script grabs shaders from shadertoy, and dumps them out as
.shader_test files which can be run through shader-db for compiler
testing.
shadertoy only gives you a fragment shader (which works based on
gl_FragCoord), so a generic vertex shader is used. And a blurb is
inserted for the pre-d
Ben Widawsky writes:
> On Tue, Jul 07, 2015 at 10:21:28PM +0300, Francisco Jerez wrote:
>> Instead of relying on hardware defaults the i915 kernel driver is
>> going program custom MOCS tables system-wide on Gen9 hardware. The
>> "WT" entry previously used for renderbuffers had a number of probl
On 09/07/15 11:13, Chris Wilson wrote:
On Wed, Jul 08, 2015 at 05:08:11PM -0700, Matt Turner wrote:
On Wed, Jul 8, 2015 at 4:53 PM, Chris Wilson wrote:
static void upload_viewport_state_pointers(struct brw_context *brw)
{
BEGIN_BATCH(4);
brw->batch.map[0] = (_3DSTATE_VIEWPORT_STATE_POI
We were passing src0 alpha and oMask in reverse order. There seems to
be no good way to pass them in the correct order to the new-style
LOAD_PAYLOAD (how surprising) because src0 alpha is per-channel while
oMask is not. Just split src0 alpha in fixed-width registers and pass
them to LOAD_PAYLOAD
On Tuesday, June 30, 2015 10:04:47 AM Iago Toral wrote:
> Hi Jason,
>
> On Mon, 2015-06-29 at 16:22 -0700, Jason Ekstrand wrote:
> > On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev
> > wrote:
> > > From: Iago Toral Quiroga
> > >
> > > This is based on similar code existing in vec4_visitor.
While implementing the workaround in the previous patch I noticed things were
starting to get a bit messy. Since gen8 works differently enough from gen7, I
thought splitting it out with be good.
While here, get rid of gen8 MOCS which does nothing and was in the wrong place
anyway.
This patch is t
On Thu, Jul 09, 2015 at 09:44:52AM -0700, Ben Widawsky wrote:
> While implementing the workaround in the previous patch I noticed things were
> starting to get a bit messy. Since gen8 works differently enough from gen7, I
> thought splitting it out with be good.
>
> While here, get rid of gen8 MOC
From: Ben Widawsky
This patch can cause an infinite recursion if the previous patch titled, "i965:
Track finished batch state" isn't present (backporters take notice).
v2: Sent out the wrong patch originally. This patches switches the order of
flushes, doing the generic flush before the CC_STATE
On 07/08/2015 10:07 AM, Emil Velikov wrote:
> Most of the data stored(duplicated) was unused, and for the one that is
> follow the approach set by other drivers.
> This eliminates the use of legacy (dri1) types.
>
> XXX: The radeon code is the only user of __DRIscreen::drm_version (the
> only __DR
On 07/08/2015 10:07 AM, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
> ---
> include/GL/internal/dri_interface.h | 11 ---
> 1 file changed, 11 deletions(-)
>
> diff --git a/include/GL/internal/dri_interface.h
> b/include/GL/internal/dri_interface.h
> index c827bb6..c0545b1 100644
I can't really speak to the Android.mk or SConscript changes, but the
rest of this patch is
Reviewed-by: Ian Romanick
You might also see what Jeremy Huddleston Sequoia
thinks, since most of this exists to support his platform. :)
On 07/08/2015 10:07 AM, Emil Velikov wrote:
> Double negatives i
While implementing the workaround in the previous patch I noticed things were
starting to get a bit messy. Since gen8 works differently enough from gen7, I
thought splitting it out with be good.
While here, get rid of gen8 MOCS which does nothing and was in the wrong place
anyway.
This patch is t
On Thu, Jul 02, 2015 at 11:32:03PM -0700, Jordan Justen wrote:
> On 2015-06-25 11:34:59, Ben Widawsky wrote:
> > On Thu, Jun 11, 2015 at 09:04:45PM -0700, Jordan Justen wrote:
> > > + desc[dw++] = 0;
> > > + const uint32_t media_threads =
> > > + brw->gen >= 8 ?
> > > + SET_FIELD(thre
Aside from the trivial GRF underallocation problem in the
"devinfo->gen < 6 && is_rect" if-block, the texrect scale uniform
look-up code was assuming a one-to-one mapping between UNIFORM
register indices and the param array, which only holds during the
SIMD8 run.
It seems dubious that this needs t
On Jul 9, 2015 7:57 AM, "Francisco Jerez" wrote:
>
> We were passing src0 alpha and oMask in reverse order. There seems to
> be no good way to pass them in the correct order to the new-style
> LOAD_PAYLOAD (how surprising) because src0 alpha is per-channel while
> oMask is not. Just split src0 a
This gets rid of two no16() fall-backs and should allow better
scheduling of the generated IR. There are no uses of usubBorrow() or
uaddCarry() in shader-db so no changes are expected. However the
"arb_gpu_shader5/execution/built-in-functions/fs-usubBorrow" and
"arb_gpu_shader5/execution/built-in
FYI there's already a lowering pass that does this in the GLSL IR
(CARRY_TO_ARITH in lower_instructions). Perhaps the right place to do
this is NIR though, just wanted to let you know.
On Thu, Jul 9, 2015 at 3:51 PM, Francisco Jerez wrote:
> This gets rid of two no16() fall-backs and should allow
As of last commit the only user of it (radeon/r200) no longer uses it.
As such let's remove it and cleanup the nasty hacks that we had in place
to support this.
Signed-off-by: Emil Velikov
---
configure.ac| 21 -
src/mesa/drivers/dri/common/Android
Hi all,
This is a follow up to my earlier "Bye bye __NOT_HAVE_DRM_H" series, on
which thise series depends upon.
As mentioned in there my Michel & Marek, radeon DRM module 2 is only
possible with the in-tree drivers, so I've ripped them out and as a side
effect removed a _very_ nasty hack that we
The original code only half considered hyperz as an option. As per
previous commit "major != 2 cannot occur" we can simply things, and
allow users to set the option if they choose to do so.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/r200/r200_context.c | 10 ++
src/mesa/dri
As mentioned by Michel Dänzer
"FWIW though, any code which is specific to radeon DRM major version 1
can be removed, because that's the UMS major version."
and Marek Olšák
"major != 2" can't occur. You don't have to check the major version at
all and you can just assume it's always 2."
Sign
Dead since at least 2009 with commit ccf7814a315(radeon: major cleanups
removing old dead codepaths.)
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/r200/r200_context.c | 3 ---
src/mesa/drivers/dri/r200/r200_context.h | 1 -
src/mesa/drivers/dri/radeon/radeon_context.c | 3 ---
sr
Ilia Mirkin writes:
> FYI there's already a lowering pass that does this in the GLSL IR
> (CARRY_TO_ARITH in lower_instructions). Perhaps the right place to do
> this is NIR though, just wanted to let you know.
>
Ah, I wasn't aware of that flag, that seems even better. I just tried
it and it see
On 9 July 2015 at 18:50, Ian Romanick wrote:
> On 07/08/2015 10:07 AM, Emil Velikov wrote:
>> Signed-off-by: Emil Velikov
>> ---
>> include/GL/internal/dri_interface.h | 11 ---
>> 1 file changed, 11 deletions(-)
>>
>> diff --git a/include/GL/internal/dri_interface.h
>> b/include/GL/int
These conditionals are used to guard both dri modules and loader(s).
Currently if we try to build the gallium swrast dri module (without glx)
on a system that's missing libdrm the build will fail.
v2: Make sure we assign prior to checking the have_libdrm variable.
Cc: 10.6
Signed-off-by: Emil V
The title should obviously say "dri/common:" consider that fixed.
Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Thu 09 Jul 2015, Emil Velikov wrote:
> On 9 July 2015 at 09:39, Chad Versace wrote:
> > EGL_EXT_image_dma_buf_import now supports those formats.
> >
> Do I have an old version of it (v6) or I simply cannot see those listed ?
I should have been more clear when I said "EGL_EXT_image_dma_buf_impo
On 9 July 2015 at 21:21, Chad Versace wrote:
> On Thu 09 Jul 2015, Emil Velikov wrote:
>> On 9 July 2015 at 09:39, Chad Versace wrote:
>> > EGL_EXT_image_dma_buf_import now supports those formats.
>> >
>> Do I have an old version of it (v6) or I simply cannot see those listed ?
>
> I should have
On 09/07/15 15:40, Axel Davy wrote:
> On 09/07/2015 15:33, Emil Velikov wrote :
>>
>> Yakes forgot about that one. From a quick look we can (I know it feels
>> durty) use readlink(
>> /sys/dev/char/$(major):$(minor)).
>>
>> Everything seems to be there - bus type and exact location of the
>> device
I'd like to discuss one more thing that will affect whether image
slots will be global (shared by all shaders) or not.
Which image unit an image uniform uses is not a compile-time thing,
but it's specified later using glUniform1i. That means we need a
per-shader table that maps image uniforms to g
Do you really need is_subroutine_def ? It seems redundant with
num_subroutine_types>0.
On Thu, Jul 9, 2015 at 7:17 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> We need to store two sets of info into the ir_function,
> if this is a function definition with a subroutine list
> (subroutine_def) o
On Thu, Jul 9, 2015 at 5:05 PM, Marek Olšák wrote:
> I'd like to discuss one more thing that will affect whether image
> slots will be global (shared by all shaders) or not.
>
> Which image unit an image uniform uses is not a compile-time thing,
> but it's specified later using glUniform1i. That m
On Thu, Jul 9, 2015 at 5:30 PM, Ilia Mirkin wrote:
> On Thu, Jul 9, 2015 at 5:05 PM, Marek Olšák wrote:
>> I'd like to discuss one more thing that will affect whether image
>> slots will be global (shared by all shaders) or not.
>>
>> Which image unit an image uniform uses is not a compile-time t
There's a few more cleanups you could do if you wanted, but either way:
Reviewed-by: Dylan Baker
On Thu, Jul 09, 2015 at 10:41:05AM -0400, Rob Clark wrote:
> Attached script grabs shaders from shadertoy, and dumps them out as
> .shader_test files which can be run through shader-db for compiler
>
From: Marek Olšák
PIPE_CAPs and TGSI support will be added later. The TGSI support should be
straightforward. We only need to split TGSI_FILE_RESOURCE into TGSI_FILE_IMAGE
and TGSI_FILE_BUFFER, though duplicating all opcodes shouldn't be necessary.
The idea is:
* ARB_shader_image_load_store shou
From: Marek Olšák
PIPE_CAPs will be added some other time.
---
src/gallium/auxiliary/util/u_debug_describe.c | 9 +++
src/gallium/auxiliary/util/u_debug_describe.h | 2 ++
src/gallium/auxiliary/util/u_dump.h | 3 +++
src/gallium/auxiliary/util/u_dump_state.c | 27 +++
From: Marek Olšák
---
src/gallium/drivers/ilo/core/ilo_state_3d.h | 2 +-
src/gallium/include/pipe/p_state.h| 3 ++-
src/gallium/state_trackers/clover/core/device.cpp | 4 ++--
3 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/ilo/core/ilo_s
From: Marek Olšák
---
src/gallium/include/pipe/p_context.h | 18 ++
src/gallium/include/pipe/p_state.h | 10 ++
2 files changed, 28 insertions(+)
diff --git a/src/gallium/include/pipe/p_context.h
b/src/gallium/include/pipe/p_context.h
index 022ace5..b4512e7 100644
---
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Bug 79706 depends on bug 66346, which changed state.
Bug 66346 Summary: shader_query.cpp:49: error: invalid conversion from 'void*'
to 'GLuint'
https://bugs.freedesktop.org/show_bug.cgi?id=66346
What|Removed |A
https://bugs.freedesktop.org/show_bug.cgi?id=66346
Vinson Lee changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
---
Some of the AoA tests currently get stuck in an optimisation
loop for a long time and this was taking a large amount of time.
Its a symptom not a cause but I don't see any issue with using the faster
version.
src/glsl/ir_validate.cpp | 17 -
1 file changed, 8 insertions(+
For the series:
Reviewed-by: Marek Olšák
Marek
On Thu, Jul 9, 2015 at 10:13 PM, Emil Velikov wrote:
> Dead since at least 2009 with commit ccf7814a315(radeon: major cleanups
> removing old dead codepaths.)
>
> Signed-off-by: Emil Velikov
> ---
> src/mesa/drivers/dri/r200/r200_context.c
This all seems fine to me. This series is:
Reviewed-by: Ilia Mirkin
Might want to wait for someone more experienced with all this to also
take a look before pushing.
-ilia
On Thu, Jul 9, 2015 at 5:46 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/include/pipe/p_context.
https://bugs.freedesktop.org/show_bug.cgi?id=66346
--- Comment #19 from José Fonseca ---
(In reply to Vinson Lee from comment #18)
> mesa: 0166b4c165271bd7525a91049e58e390cb596c60 (master 10.7.0-devel)
>
> Still see this build error. BUILDING_MESA is only defined for darwin DRI
> enabled builds.
30-32 Reviewed-by: Jordan Justen
On 2015-06-03 00:01:20, Iago Toral Quiroga wrote:
> This is the same we do for other things like uniforms because it ensures
> optimal performance.
> ---
> src/mesa/drivers/dri/i965/brw_context.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/
https://bugs.freedesktop.org/show_bug.cgi?id=66346
--- Comment #20 from Vinson Lee ---
(In reply to José Fonseca from comment #19)
> (In reply to Vinson Lee from comment #18)
> > mesa: 0166b4c165271bd7525a91049e58e390cb596c60 (master 10.7.0-devel)
> >
> > Still see this build error. BUILDING_MES
On Thu 09 Jul 2015, Emil Velikov wrote:
> On 9 July 2015 at 21:21, Chad Versace wrote:
> > On Thu 09 Jul 2015, Emil Velikov wrote:
> >> On 9 July 2015 at 09:39, Chad Versace wrote:
> >> > EGL_EXT_image_dma_buf_import now supports those formats.
> >> >
> >> Do I have an old version of it (v6) or I
https://bugs.freedesktop.org/show_bug.cgi?id=91288
Bug ID: 91288
Summary: [swrast] piglit depthfunc double free
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Keywords: hav
1 - 100 of 113 matches
Mail list logo
