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
Even if your next patch needs more work this one alone still makes
sense:
Reviewed-by: Iago Toral Quiroga ito...@igalia.com
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
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
Ben Widawsky b...@bwidawsk.net 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
On Thu, Jul 9, 2015 at 8:36 AM, Emil Velikov emil.l.veli...@gmail.com wrote:
On 9 July 2015 at 01:46, Rob Clark robdcl...@gmail.com wrote:
From: Rob Clark robcl...@freedesktop.org
Some, but not all, state trackers will explicitly unref (and set to
NULL) the previous *fence before calling
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 ch...@chris-wilson.co.uk wrote:
static void upload_viewport_state_pointers(struct brw_context *brw)
{
BEGIN_BATCH(4);
brw-batch.map[0] =
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
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
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 el...@igalia.com
wrote:
From: Iago Toral Quiroga ito...@igalia.com
This is based on similar code existing
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 MOCS
From: Ben Widawsky benjamin.widaw...@intel.com
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
On 07/08/2015 10:07 AM, Emil Velikov wrote:
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
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
I can't really speak to the Android.mk or SConscript changes, but the
rest of this patch is
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
You might also see what Jeremy Huddleston Sequoia jerem...@apple.com
thinks, since most of this exists to support his platform. :)
On 07/08/2015 10:07
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(threads,
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
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
Timothy Arceri t_arc...@yahoo.com.au writes:
---
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 |
Don't assume that $(top_srcdir)/.git is a directory. It may be a
gitlink file [1] if $(top_srcdir) is a submodule checkout or a linked
worktree [2].
[1] A gitlink is a text file that specifies the real location of
the gitdir.
[2] Linked worktrees are a new feature in Git 2.5.
Cc: 10.6, 10.5
https://bugs.freedesktop.org/show_bug.cgi?id=91290
Bug ID: 91290
Summary: SIGSEGV glcpp/glcpp-parse.y:1077
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: All
Status: NEW
Keywords: have-backtrace
7-12 inclusive are
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Thu, Jul 9, 2015 at 7:17 PM, Dave Airlie airl...@gmail.com 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
Ohhh ripping out some more of my _very_ old code, how dare you :-).
Too bad some parts (fast z clear for instance) never made it past UMS
for these chips, quake3 never was as fast again ;-). I wonder how well
things actually work these days with enabled hyperz, it has some, let's
say,
Emil Velikov 於 西元2015年07月09日 01:07 寫道:
Set the macro if we're not building swrast alone.
Cc: Chih-Wei Huang cwhu...@linux.org.tw
Cc: Eric Anholt e...@anholt.net
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/gallium/targets/dri/Android.mk | 8 +++-
1 file changed, 3
On 10.07.2015 05:13, Emil Velikov wrote:
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 emil.l.veli...@gmail.com
---
On Wed, Jul 08, 2015 at 05:08:11PM -0700, Matt Turner wrote:
On Wed, Jul 8, 2015 at 4:53 PM, Chris Wilson ch...@chris-wilson.co.uk wrote:
static void upload_viewport_state_pointers(struct brw_context *brw)
{
BEGIN_BATCH(4);
brw-batch.map[0] = (_3DSTATE_VIEWPORT_STATE_POINTERS 16 |
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
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
EGL_EXT_image_dma_buf_import now supports those formats.
CC: Peter Frühberger peter.fruehber...@gmail.com
Cc: Rainer Hochecker rainer.hochec...@onlinehome.de
Signed-off-by: Chad Versace chad.vers...@intel.com
---
include/GL/internal/dri_interface.h | 9 +++--
https://bugs.freedesktop.org/show_bug.cgi?id=90903
Emil Velikov emil.l.veli...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=90249
Emil Velikov emil.l.veli...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
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
https://bugs.freedesktop.org/show_bug.cgi?id=66346
Emil Velikov emil.l.veli...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
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
This applies on top of the two patches I've sent already,
and enables the geometry streams, which is the final
piece missing for ARB_gpu_shader5 on evergreen and cayman.
(I'll do doc update patches later)
Glenn wrote most of this, I just spent some time making it work
and cleaning up the code.
From: Glenn Kennard glenn.kenn...@gmail.com
This adds a peephole and removes an assert that isn't
actually valid with some of the stream emit instructions.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/gallium/drivers/r600/sb/sb_bc_finalize.cpp | 2 --
From: Dave Airlie airl...@redhat.com
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/gallium/drivers/r600/r600_pipe.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/r600_pipe.c
b/src/gallium/drivers/r600/r600_pipe.c
index 67caa69..0db1c1c
On Thu, Jul 9, 2015 at 8:46 AM, Rob Clark robdcl...@gmail.com wrote:
From: Rob Clark robcl...@freedesktop.org
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
From: Glenn Kennard glenn.kenn...@gmail.com
This just adds support to the assembler dumper and allows
stream instructions to be generated. Also fix up the stream
debugging to add stream info.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/gallium/drivers/r600/eg_asm.c | 1 +
From: Glenn Kennard glenn.kenn...@gmail.com
This adds support for queries against the non-0 vertex streams.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/gallium/drivers/radeon/r600_query.c | 18 --
src/gallium/drivers/radeon/r600d_common.h | 3 +++
2 files changed,
From: Glenn Kennard glenn.kenn...@gmail.com
This adds the main chunk of the geometry shader multiple stream
support to the r600 driver.
Glenn wrote the original pass, and I took his code and hacked
it into a working state.
Signed-off-by: Dave Airlie airl...@redhat.com
---
From: Glenn Kennard glenn.kenn...@gmail.com
This just adds to the common radeon streamout code, support
for multiple streams.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/gallium/drivers/radeon/r600_pipe_common.h | 1 +
src/gallium/drivers/radeon/r600_streamout.c | 23
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 store the destination register
is
fetched from the nir_outputs map built during nir_setup_outputs stage.
Bugzilla:
On Thu, 09 Jul 2015 07:37:59 +0200, Dave Airlie airl...@gmail.com wrote:
From: Dave Airlie airl...@redhat.com
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 airl...@redhat.com
---
From: Dave Airlie airl...@redhat.com
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 airl...@redhat.com
---
From: Dave Airlie airl...@redhat.com
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
From: Dave Airlie airl...@redhat.com
Just add support for the subroutine type to the
glsl-tgsi convertor.
v1.1: add subroutine to int support.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
From: Dave Airlie airl...@redhat.com
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 airl...@redhat.com
---
src/mesa/main/shader_query.cpp | 46
From: Dave Airlie airl...@redhat.com
I'm not sure if we shouldn't enable this everywhere
and rip out the API checks,
discuss,
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/state_tracker/st_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git
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
---
src/mesa/main/shaderapi.c | 457
From: Dave Airlie airl...@redhat.com
This adds the necessary storage for subroutine info to gl_shader.
Signed-off-by: Dave Airlie airl...@redhat.com
---
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
From: Dave Airlie airl...@redhat.com
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
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 subroutine
type in a uniform decleration as a valid type, and also for
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.
___
From: Dave Airlie airl...@redhat.com
This handles converting the shader stages to the internal
prefix along with the program resource interfaces.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/main/shaderobj.h | 84 +++
1 file changed, 84
From: Chris Forbes chr...@ijw.co.nz
Reviewed-by: Tapani Pälli tapani.pa...@intel.com
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/main/config.h | 6 ++
From: Chris Forbes chr...@ijw.co.nz
v2: fix output=true and LENGTH typo
Reviewed-by: Tapani Pälli tapani.pa...@intel.com
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Signed-off-by: Dave Airlie airl...@redhat.com
---
From: Dave Airlie airl...@redhat.com
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
From: Chris Forbes chr...@ijw.co.nz
Reviewed-by: Tapani Pälli tapani.pa...@intel.com
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/glsl/glsl_lexer.ll | 2 +-
1 file changed, 1 insertion(+), 1
From: Chris Forbes chr...@ijw.co.nz
v2: [airlied]: merge version check update.
Reviewed-by: Tapani Pälli tapani.pa...@intel.com
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Signed-off-by: Dave Airlie airl...@redhat.com
---
From: Dave Airlie airl...@redhat.com
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 airl...@redhat.com
---
From: Chris Forbes chr...@ijw.co.nz
Reviewed-by: Tapani Pälli tapani.pa...@intel.com
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/main/shaderapi.c | 63
From: Chris Forbes chr...@ijw.co.nz
Reviewed-by: Tapani Pälli tapani.pa...@intel.com
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/glsl/glcpp/glcpp-parse.y| 3 +++
From: Dave Airlie airl...@redhat.com
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 airl...@redhat.com
---
src/glsl/ir.h | 21 -
1
From: Dave Airlie airl...@redhat.com
Add support for the subroutine uniform type ir-mesa.cpp
v1.1: add subroutine to int to switch
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/program/ir_to_mesa.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git
On 08.07.2015 22:34, Rob Clark wrote:
From: Rob Clark robcl...@freedesktop.org
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).
On Thu, 09 Jul 2015 08:00:48 +0200, Dave Airlie airl...@gmail.com wrote:
From: Dave Airlie airl...@redhat.com
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.
On Thu, Jul 9, 2015 at 4:21 AM, Christian König deathsim...@vodafone.de wrote:
On 08.07.2015 22:34, Rob Clark wrote:
From: Rob Clark robcl...@freedesktop.org
This isn't at all clear for pipe driver writers currently, since it
is not documented anywhere. But radeon/nouveau/llvmpipe seem to
On 8 July 2015 at 18:08, Matt Turner matts...@gmail.com wrote:
On Wed, Jul 8, 2015 at 10:07 AM, Emil Velikov emil.l.veli...@gmail.com
wrote:
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/mesa/drivers/dri/i915/intel_context.c | 1 -
src/mesa/drivers/dri/i915/intel_context.h |
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
On 9 July 2015 at 01:46, Rob Clark robdcl...@gmail.com wrote:
From: Rob Clark robcl...@freedesktop.org
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
On 08/07/15 18:57, Emil Velikov wrote:
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
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
On 9 July 2015 at 04:45, Michel Dänzer mic...@daenzer.net 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
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 emil.l.veli...@gmail.com wrote:
On 9 July 2015 at 04:45, Michel Dänzer mic...@daenzer.net wrote:
On 09.07.2015 02:07, Emil Velikov
On 8 July 2015 at 18:35, Eric Anholt e...@anholt.net wrote:
Emil Velikov emil.l.veli...@gmail.com 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
Should expose that only if hw has glsl 130 support?
Roland
Am 09.07.2015 um 09:17 schrieb Dave Airlie:
From: Dave Airlie airl...@redhat.com
I'm not sure if we shouldn't enable this everywhere
and rip out the API checks,
discuss,
Signed-off-by: Dave Airlie airl...@redhat.com
---
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
On 8 July 2015 at 18:55, Axel Davy axel.d...@ens.fr 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 could
On 9 July 2015 at 14:16, Marek Olšák mar...@gmail.com 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
On 9 July 2015 at 09:39, Chad Versace chad.vers...@intel.com 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
On 30 June 2015 at 16:29, Emil Velikov emil.l.veli...@gmail.com wrote:
On 22 June 2015 at 23:19, Dave Airlie airl...@gmail.com wrote:
On 23 June 2015 at 08:16, Ian Romanick i...@freedesktop.org wrote:
On 06/22/2015 11:54 AM, Dave Airlie wrote:
As kindly hinted by Marek, currently we do have a
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 curroje...@riseup.net wrote:
This gets rid of two no16()
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
Do you really need is_subroutine_def ? It seems redundant with
num_subroutine_types0.
On Thu, Jul 9, 2015 at 7:17 PM, Dave Airlie airl...@gmail.com wrote:
From: Dave Airlie airl...@redhat.com
We need to store two sets of info into the ir_function,
if this is a function definition with a
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 on the bus.
On Thu, Jul 9, 2015 at 5:05 PM, Marek Olšák mar...@gmail.com 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
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 emil.l.veli...@gmail.com
---
configure.ac| 21 -
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
On 9 July 2015 at 21:21, Chad Versace chad.vers...@intel.com wrote:
On Thu 09 Jul 2015, Emil Velikov wrote:
On 9 July 2015 at 09:39, Chad Versace chad.vers...@intel.com 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
Ilia Mirkin imir...@alum.mit.edu 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
On 9 July 2015 at 18:50, Ian Romanick i...@freedesktop.org wrote:
On 07/08/2015 10:07 AM, Emil Velikov wrote:
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
include/GL/internal/dri_interface.h | 11 ---
1 file changed, 11 deletions(-)
diff --git
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
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 to
On Jul 9, 2015 7:57 AM, Francisco Jerez curroje...@riseup.net 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.
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
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 emil.l.veli...@gmail.com
---
src/mesa/drivers/dri/r200/r200_context.c | 10
Dead since at least 2009 with commit ccf7814a315(radeon: major cleanups
removing old dead codepaths.)
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/mesa/drivers/dri/r200/r200_context.c | 3 ---
src/mesa/drivers/dri/r200/r200_context.h | 1 -
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.
On Thu 09 Jul 2015, Emil Velikov wrote:
On 9 July 2015 at 09:39, Chad Versace chad.vers...@intel.com 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
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
https://bugs.freedesktop.org/show_bug.cgi?id=66346
--- Comment #19 from José Fonseca jfons...@vmware.com ---
(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
1 - 100 of 117 matches
Mail list logo
