On Fri, Oct 16, 2015 at 08:24:11AM -0700, Jason Ekstrand wrote:
> On Fri, Oct 16, 2015 at 12:35 AM, Pohjolainen, Topi
> wrote:
> > On Fri, Oct 09, 2015 at 05:50:22AM -0700, Jason Ekstrand wrote:
> >> On Fri, Oct 9, 2015 at 12:10 AM, Pohjolainen, Topi
> >> wrote:
> >> > On Thu, Oct 08, 2015 at 05:
On 16.10.2015 18:02, Emil Velikov wrote:
On 16 October 2015 at 15:53, Christian König wrote:
From: Indrajit Das
Reviewed-by: Christian König
Nicely spotted !
For the future use correct prefix for the summary ("st/va:" here, but
git log will show you the way in other places) and add the sta
Hi Samuel,
(some comments further down)
On 11:30 PM - Oct 18 2015, Samuel Pitoiset wrote:
> Like for nvc0, this will allow to split different types of queries and
> to prepare the way for both global performance counters and MP counters.
>
> While we are at it, make use of nv50_query struct inst
On Sun, 2015-10-18 at 21:31 -0700, Kristian Høgsberg Kristensen wrote:
> An immdiate is already uniform so just return it up front. Without this,
> brw_fs_surface_builder ends up passing immediate surface indices through
> SHADER_OPCODE_BROADCAST. This writes to a stride 0 dst, which we can't
> con
On 10/19/2015 10:43 AM, Pierre Moreau wrote:
Hi Samuel,
(some comments further down)
On 11:30 PM - Oct 18 2015, Samuel Pitoiset wrote:
Like for nvc0, this will allow to split different types of queries and
to prepare the way for both global performance counters and MP counters.
While we are
Hi Samuel,
(some comments below)
On 11:36 PM - Oct 18 2015, Samuel Pitoiset wrote:
> While we are at it, store the rotate offset for occlusion queries to
> nv50_hw_query like on nvc0.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/drivers/nouveau/nv50/nv50_query_hw.c | 45
> ++
Fixes a regression added by bb5aeb854915ba67abc56257f830d002c956439e.
Signed-off-by: Samuel Iglesias Gonsalvez
---
Piglit test that checks the regression:
http://patchwork.freedesktop.org/patch/62183/
src/glsl/lower_ubo_reference.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
di
On Mon, 2015-10-19 at 11:00 +0200, Samuel Iglesias Gonsalvez wrote:
> Fixes a regression added by bb5aeb854915ba67abc56257f830d002c956439e.
>
> Signed-off-by: Samuel Iglesias Gonsalvez
Thanks. Reviewed-by: Timothy Arceri
I'm surprised this didn't show up in my piglit regression testing.
> ---
On 10/19/2015 11:01 AM, Pierre Moreau wrote:
Hi Samuel,
(some comments below)
On 11:36 PM - Oct 18 2015, Samuel Pitoiset wrote:
While we are at it, store the rotate offset for occlusion queries to
nv50_hw_query like on nvc0.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/
On 19/10/15 11:10, Timothy Arceri wrote:
> On Mon, 2015-10-19 at 11:00 +0200, Samuel Iglesias Gonsalvez wrote:
>> Fixes a regression added by bb5aeb854915ba67abc56257f830d002c956439e.
>>
>> Signed-off-by: Samuel Iglesias Gonsalvez
>
> Thanks. Reviewed-by: Timothy Arceri
>
> I'm surprised this
Hi Christian,
Am Sonntag, den 18.10.2015, 21:41 +0200 schrieb Christian Gmeiner:
> Hi Thierry,
>
> Thanks for reviewing this patches.
>
> 2015-10-16 15:31 GMT+02:00 Thierry Reding :
> > Hi Christian,
> >
> > First off, thanks for reviving this effort. It's been one of the things
> > that I've ha
On Sun, 2015-10-18 at 21:31 -0700, Kristian Høgsberg Kristensen wrote:
> Instead of looping through single-component reads, read all components
> in one go.
>
> Signed-off-by: Kristian Høgsberg Kristensen
> ---
> src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 25 +++--
> 1 file c
On Sun, 2015-10-18 at 21:31 -0700, Kristian Høgsberg Kristensen wrote:
> We always set the mask to 0x, which is what it defaults to when no
> header is present. Let's drop the header instead.
Right, according to the PRM (3.9.9 Typed/Untyped Surface Read/Write and
Typed/Untyped Atomic Operation
On Sun, 2015-10-18 at 21:31 -0700, Kristian Høgsberg Kristensen wrote:
> I've been giving the SSBO functionality a closer look and I found a
> few problems, each addressed by a patch in this series. The commit
> messages describe the issues and the fixes, but the net effect takes
> my vertex shader
Hi Lucas,
Pardon for jumping it like this.
On 19 October 2015 at 10:25, Lucas Stach wrote:
[snip]
> This series implements a way to fuse multiple DRM devices into a single
> EGL device, solely on the base that current EGL users expect that a
> single device is able to render and scanout at the s
Just a thought, would it be better to move this check into the
eliminate_find_live_channel optimisation? That way it could catch
sources that become immediates through later optimisations. One problem
with this that I've seen before is that eliminating the
FIND_LIVE_CHANNEL doesn't cause the subseq
Hey Kristian,
Kristian Høgsberg Kristensen writes:
> We always set the mask to 0x, which is what it defaults to when no
> header is present. Let's drop the header instead.
>
> Signed-off-by: Kristian Høgsberg Kristensen
> ---
> src/mesa/drivers/dri/i965/brw_eu_emit.c | 3 +--
> src/mesa/dr
On 18 October 2015 at 13:45, Timothy Arceri wrote:
> On Sun, 2015-10-18 at 11:34 +0100, Emil Velikov wrote:
>> Hi Timothy,
>>
>> On 17 October 2015 at 23:19, Timothy Arceri
>> wrote:
>> > Fixes regression cased by bb5aeb854915ba67abc56257f830d002c956439e
>> >
>> > We don't care about the swizzle
On 11:06 AM - Oct 19 2015, Samuel Pitoiset wrote:
>
>
> On 10/19/2015 11:01 AM, Pierre Moreau wrote:
> >Hi Samuel,
> >
> >(some comments below)
> >
> >On 11:36 PM - Oct 18 2015, Samuel Pitoiset wrote:
> >>While we are at it, store the rotate offset for occlusion queries to
> >>nv50_hw_query like
I thought just changing the prim type would cause the loop to not get
closed in the end, albeit I looked at it only briefly (I thought the
DRAW_SPLIT_AFTER/BEFORE flags were supposed to be able to deal with this
but couldn't figure out how). But if this works correctly,
Reviewed-by: Roland Scheide
The amount of memory that the GPU can use in any single batch is the
amount of memory it can map into its page tables, that is the GTT size.
With per-process GTT (i.e. every context has its own set of page tables),
this value may differ from the global GTT size (and is often larger).
When reporting
Reviewed-by: Samuel Iglesias Gonsálvez
On 18/10/15 00:19, Timothy Arceri wrote:
> Fixes regression cased by bb5aeb854915ba67abc56257f830d002c956439e
>
> We don't care about the swizzle when building the name so just skip over it.
> ---
>
> New piglit test: http://patchwork.freedesktop.org/patc
On 17/10/15 03:56, Brian Paul wrote:
If we didn't find a gallium surface format that exactly matched the
glDrawPixels format/type combination, we used some other 32-bit packed
RGBA format and swizzled the whole image in the mesa texstore/format code.
That slow path can be avoided in some common
Neil Roberts writes:
> Just a thought, would it be better to move this check into the
> eliminate_find_live_channel optimisation? That way it could catch
> sources that become immediates through later optimisations. One problem
> with this that I've seen before is that eliminating the
> FIND_LIVE
On 19/10/15 13:18, Samuel Iglesias Gonsálvez wrote:
> Reviewed-by: Samuel Iglesias Gonsálvez
>
Just one thing I forgot to say, Can you be more verbose about what it
fixes in the subject line?
"glsl: fix regression when building interface field name for SSBOs"
or something like that.
Sam
>
On 10/19/2015 12:43 PM, Pierre Moreau wrote:
On 11:06 AM - Oct 19 2015, Samuel Pitoiset wrote:
On 10/19/2015 11:01 AM, Pierre Moreau wrote:
Hi Samuel,
(some comments below)
On 11:36 PM - Oct 18 2015, Samuel Pitoiset wrote:
While we are at it, store the rotate offset for occlusion queries
Hi Abdiel,
Abdiel Janulgue writes:
> Signed-off-by: Abdiel Janulgue
> ---
> src/mesa/drivers/dri/i965/brw_defines.h | 20
> 1 file changed, 20 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
> b/src/mesa/drivers/dri/i965/brw_defines.h
> index a8594a
Assuming no piglit regressions,
Reviewed-by: Samuel Iglesias Gonsálvez
Thanks!
Sam
On 17/10/15 11:51, Timothy Arceri wrote:
> This also removes the validation from the parser as it is not required
> and once arb_enhanced_layouts comes along we wont be able to do validation
> on the stream qual
On 16/10/15 22:25, Brian Paul wrote:
When a long GL_LINE_LOOP prim was split across primitives we drew
stray lines. See previous commit for details.
This patch converts GL_LINE_LOOP prims into GL_LINE_STRIP prims so
that drivers don't have to worry about the _mesa_prim::begin/end flags.
Bugzil
Ping on these two trivial patches ?
-Emil
On 7 October 2015 at 12:38, Emil Velikov wrote:
> There is only one function that can be called, which is well known at
> compilation time.
>
> The abstraction used here seems unnecessary, so let's use a direct call
> to brw_stage_prog_data_free() when a
On 7 October 2015 at 11:11, Emil Velikov wrote:
> Hi all,
>
> A while back we had a case where NIR had C99 initializers in its public
> headers. That's not desired as ISO C++ does not allow them.
>
> So I took a quick look about wiring up a compilation test at 'make
> check' and noticed that we've
Undoes early hacks, and ensures nir/glsl_types.cpp is built once, and
only once.
The root problem is that SCons doesn't know about NIR nor any source
file in the NIR_FILES source list.
Tested with libgl-gdi and libgl-xlib scons targets.
---
src/gallium/targets/libgl-gdi/SConscript | 10
On 17/10/15 17:44, Rob Clark wrote:
On Sat, Oct 17, 2015 at 12:36 PM, Brian Paul wrote:
On 10/17/2015 10:07 AM, Brian Paul wrote:
On 10/17/2015 07:04 AM, Rob Clark wrote:
On Fri, Oct 16, 2015 at 11:11 PM, Brian Paul wrote:
Hi Rob,
Your recent commit "nir: remove dependency on glsl" brok
On 10/19/2015 12:52 AM, Michel Dänzer wrote:
On 17.10.2015 00:10, Brian Paul wrote:
On 10/16/2015 08:17 AM, Brian Paul wrote:
On 10/16/2015 12:36 AM, Michel Dänzer wrote:
Hi Brian,
On 15.10.2015 22:23, Brian Paul wrote:
Module: Mesa
Branch: master
Commit: 0de5e0f3fb0f3671a3ecec6ab4473f9131
On 10/19/2015 08:01 AM, Jose Fonseca wrote:
Undoes early hacks, and ensures nir/glsl_types.cpp is built once, and
only once.
The root problem is that SCons doesn't know about NIR nor any source
file in the NIR_FILES source list.
Tested with libgl-gdi and libgl-xlib scons targets.
---
src/gall
https://bugs.freedesktop.org/show_bug.cgi?id=92536
Bug ID: 92536
Summary: [BSW] [Regrassion] piglit
spec/egl_khr_create_context/valid forward-compatible
flag gl fail
Product: Mesa
Version: 11.0
Hardware:
On 10/18/2015 03:06 PM, Sinclair Yeh wrote:
On Fri, Oct 16, 2015 at 03:25:16PM -0600, Brian Paul wrote:
When long GL_LINE_LOOP primitives don't fit in one vertex buffer they
have to be split across buffers. The code to do this was basically correct
but drivers had to pay special attention to th
On 10/19/2015 04:53 AM, Roland Scheidegger wrote:
I thought just changing the prim type would cause the loop to not get
closed in the end, albeit I looked at it only briefly (I thought the
DRAW_SPLIT_AFTER/BEFORE flags were supposed to be able to deal with this
but couldn't figure out how). But i
Signed-off-by: Emil Velikov
---
docs/relnotes/11.1.0.html| 1 +
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/docs/relnotes/11.1.0.html b/docs/relnotes/11.1.0.html
index dcf425e..3d948eb 100644
--- a/docs/relnotes/11.1.0.htm
Hi all,
This is a resent of the previous RFC series with a few minor changes
- fs_reg:;smear() has been moved out of get_timestamp()
- clock2x32ARB() has proper return value
- the nir intrinsic no longer claims to have a variable
I'm still rocking a SNB hardware so I did not really got the cha
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 9 +
src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 10 ++
2 files changed, 19 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 792663f
Signed-off-by: Emil Velikov
---
src/glsl/glcpp/glcpp-parse.y| 3 +++
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 2 ++
3 files changed, 6 insertions(+)
diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y
index 1d7a3af..4acccf7 100644
--- a/sr
We're about to reuse get_timestamp() for the nir_intrinsic_shader_clock.
The in the latter the generalisation does not apply, so move the smear()
where needed. This also makes the function analogous to the vec4 one.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 30 ++
v2: correctly set the return type
Signed-off-by: Emil Velikov
---
src/glsl/builtin_functions.cpp | 43 ++
1 file changed, 43 insertions(+)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index aae25f8..509a57b 100644
--- a/src
Signed-off-by: Emil Velikov
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/mtypes.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index b2c88c3..64972fa 100644
--- a/src/mesa/main/extensions.c
+++ b/src/mesa/main/extensions.
v2: Add flags and inline comment/description.
v3: None of the input/outputs are variables
Signed-off-by: Emil Velikov
---
src/glsl/nir/glsl_to_nir.cpp | 6 ++
src/glsl/nir/nir_intrinsics.h | 8
2 files changed, 14 insertions(+)
diff --git a/src/glsl/nir/glsl_to_nir.cpp b/src/glsl/
Spotted by Roland. Luckily, this code should never really be hit
since the const buffer size and offset should already be multiples
of 16. I could probably add more assertions to that effect, but
let's just fix the arithmetic for now.
---
src/gallium/drivers/svga/svga_state_constants.c | 2 +-
1
Reviewed-by: Roland Scheidegger
Am 19.10.2015 um 16:42 schrieb Brian Paul:
> Spotted by Roland. Luckily, this code should never really be hit
> since the const buffer size and offset should already be multiples
> of 16. I could probably add more assertions to that effect, but
> let's just fix t
Ping...
>-Original Message-
>From: Liu, Leo
>Sent: Wednesday, October 14, 2015 2:49 PM
>To: mesa-dev@lists.freedesktop.org
>Cc: Liu, Leo; 10.6 11.0
>Subject: [PATCH] st/omx/dec/h264: fix field picture type 0 poc disorder
>
>Signed-off-by: Leo Liu
>Cc: "10.6 11.0"
>---
> src/gallium/state
Reviewed-by: Christian König
For mesa you can push small fixes like this even without a review when
nobody objects after a few days.
Regards,
Christian.
On 19.10.2015 16:55, Liu, Leo wrote:
Ping...
-Original Message-
From: Liu, Leo
Sent: Wednesday, October 14, 2015 2:49 PM
To: mes
On 19 October 2015 at 15:55, Liu, Leo wrote:
> Ping...
>
FYI adding actual commit message does make reviewers happy.
Then again, although it makes sense, I don't feel qualified to review this :(
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedeskto
On 17 October 2015 at 00:14, Julien Isorce wrote:
> Also add RGBA, RGBX and BGRX.
> Also extend ChromaToPipe and implement PipeToYCbCr.
>
> Note that gstreamer-vaapi check all the VAImageFormat fields.
>
> Signed-off-by: Julien Isorce
> ---
> src/gallium/state_trackers/va/image.c | 10 +
On Mon, Oct 19, 2015 at 12:07 AM, Pohjolainen, Topi
wrote:
> On Fri, Oct 16, 2015 at 08:24:11AM -0700, Jason Ekstrand wrote:
>> On Fri, Oct 16, 2015 at 12:35 AM, Pohjolainen, Topi
>> wrote:
>> > On Fri, Oct 09, 2015 at 05:50:22AM -0700, Jason Ekstrand wrote:
>> >> On Fri, Oct 9, 2015 at 12:10 AM,
On 17 October 2015 at 00:14, Julien Isorce wrote:
> Inspired from
> http://cgit.freedesktop.org/vaapi/intel-driver/tree/src/i965_drv_video.c
>
Please mention the actual usecase here and/or how you've tested this.
> Signed-off-by: Julien Isorce
> ---
> src/gallium/state_trackers/va/context.c
On 10/17/2015 04:57 PM, Emil Velikov wrote:
i.e. plug some (hard to hit) memory leaks.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c
On 10/17/2015 04:57 PM, Emil Velikov wrote:
Move the winsys into the pipe-target, bla bla bla
XXX: separate pipe-drivers are likely to be busted
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 94 +-
src/gallium/include/state_tracker/
On Thu, Oct 15, 2015 at 06:03:34PM +0300, Ville Syrjälä wrote:
> On Thu, Oct 15, 2015 at 02:19:09PM +, Predut, Marius wrote:
> > > -Original Message-
> > > From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com]
> > > Sent: Wednesday, October 07, 2015 1:53 PM
> > > To: Predut, Marius
On 17 October 2015 at 00:14, Julien Isorce wrote:
> Signed-off-by: Julien Isorce
> ---
> src/gallium/drivers/nouveau/nvc0/nvc0_resource.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_resource.c
> b/src/gallium/drivers/nouveau/n
On 17 October 2015 at 00:14, Julien Isorce wrote:
> If formats are not the same it seems to re-create the video
> buffer with the right format.
> But if the creation of this new video buffer fails the surface
> loose its video buffer.
> Let's just destroy the previous buffer on success.
>
The form
On 10/17/2015 04:57 PM, Emil Velikov wrote:
Hi all,
Here goes a series (which is slightly overdue), that reworks the
target-helpers into a pipe-loader like interface. As such the
state-tracker do not need to know that there is a difference between the
two and it only uses the pipe-loader interfa
On 17 October 2015 at 00:14, Julien Isorce wrote:
> This patch allows to use gallium vaapi without requiring
> a X server running for your second graphic card.
>
I've sent a lengthy series which should mitigate the need of some hunks.
> Signed-off-by: Julien Isorce
> ---
> src/gallium/state_tra
On 10/18/2015 11:14 AM, Marek Olšák wrote:
From: Marek Olšák
This is more practical and needed by gallium.
---
src/glsl/linker.cpp| 33 +++--
src/glsl/nir/glsl_to_nir.cpp | 3 ++-
src/mesa/drivers/dri/i965/brw_vs.c | 2 +-
src/mesa/main/m
On 17 October 2015 at 00:14, Julien Isorce wrote:
> For now it is limited to RGBA, BGRA, RGBX, BGRX surfaces.
>
> Signed-off-by: Julien Isorce
> ---
> src/gallium/state_trackers/va/surface.c| 90
> +-
> src/gallium/state_trackers/va/va_private.h | 1 +
> 2 files
On Mon, Oct 19, 2015 at 12:08:41PM +0100, Chris Wilson wrote:
> The amount of memory that the GPU can use in any single batch is the
> amount of memory it can map into its page tables, that is the GTT size.
> With per-process GTT (i.e. every context has its own set of page tables),
> this value may
On Mon, Oct 19, 2015 at 10:02 AM, Jose Fonseca wrote:
> On 17/10/15 17:44, Rob Clark wrote:
>>
>> On Sat, Oct 17, 2015 at 12:36 PM, Brian Paul wrote:
>>>
>>> On 10/17/2015 10:07 AM, Brian Paul wrote:
On 10/17/2015 07:04 AM, Rob Clark wrote:
>
>
> On Fri, Oct 16, 2015 at
On 17 October 2015 at 00:14, Julien Isorce wrote:
> Improve following functions to support VA_PROFILE_NONE profile (vpp):
> vlVaQueryConfigProfiles
> vlVaQueryConfigEntrypoints
> vlVaCreateConfig
> vlVaQueryConfigAttributes
>
> Add VADriverVTableVPP and improve following functions to support vpp:
On Mon, Oct 19, 2015 at 1:10 PM, Emil Velikov wrote:
> On 17 October 2015 at 00:14, Julien Isorce wrote:
>> if (!ctx)
>>return VA_STATUS_ERROR_INVALID_CONTEXT;
>>
>> - if (!(picture_width && picture_height))
>> + is_vpp = config_id == PIPE_VIDEO_PROFILE_UNKNOWN &&
>> + pictu
On 19 October 2015 at 18:17, Ilia Mirkin wrote:
> On Mon, Oct 19, 2015 at 1:10 PM, Emil Velikov
> wrote:
>> On 17 October 2015 at 00:14, Julien Isorce wrote:
>>> if (!ctx)
>>>return VA_STATUS_ERROR_INVALID_CONTEXT;
>>>
>>> - if (!(picture_width && picture_height))
>>> + is_vpp =
i.e. plug some (hard to hit) memory leaks.
v2: really teardown the winsys (it rebase fallout), per Brian.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/pipe-loader
On 19 October 2015 at 17:03, Brian Paul wrote:
> On 10/17/2015 04:57 PM, Emil Velikov wrote:
>>
>> Move the winsys into the pipe-target, bla bla bla
>>
>> XXX: separate pipe-drivers are likely to be busted
>>
>> Signed-off-by: Emil Velikov
>> ---
>> src/gallium/auxiliary/pipe-loader/pipe_loader
vec4_live_variables tracks now each flag channel independently, so
vec4_dead_code_eliminate can update the writemask of null registers,
based on which component are alive at the moment. This would allow
vec4_cmod_propagation to optimize out several movs involving null
registers.
v2: added support
On 19 October 2015 at 17:07, Brian Paul wrote:
> On 10/17/2015 04:57 PM, Emil Velikov wrote:
>>
>> Hi all,
>>
>> Here goes a series (which is slightly overdue), that reworks the
>> target-helpers into a pipe-loader like interface. As such the
>> state-tracker do not need to know that there is a di
On Mon, Oct 19, 2015 at 7:45 AM, Emil Velikov wrote:
> v2: Add flags and inline comment/description.
> v3: None of the input/outputs are variables
>
> Signed-off-by: Emil Velikov
> ---
> src/glsl/nir/glsl_to_nir.cpp | 6 ++
> src/glsl/nir/nir_intrinsics.h | 8
> 2 files changed, 14
On Mon, Oct 19, 2015 at 7:45 AM, Emil Velikov wrote:
> We're about to reuse get_timestamp() for the nir_intrinsic_shader_clock.
> The in the latter the generalisation does not apply, so move the smear()
> where needed. This also makes the function analogous to the vec4 one.
>
> Signed-off-by: Emil
This series is Reviewed-by: Ilia Mirkin
provided that you run a couple of piglit tests through valgrind and
get no additional memory leaks.
-ilia
On Sat, Oct 10, 2015 at 9:09 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/mesa/state_tracker/st_cb_program.c | 5 +++--
> src/mesa/s
On Mon, Oct 19, 2015 at 7:46 AM, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
> ---
> src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 9 +
> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 10 ++
> 2 files changed, 19 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_f
On Mon, Oct 19, 2015 at 7:45 AM, Emil Velikov wrote:
> Hi all,
>
> This is a resent of the previous RFC series with a few minor changes
> - fs_reg:;smear() has been moved out of get_timestamp()
> - clock2x32ARB() has proper return value
> - the nir intrinsic no longer claims to have a variable
On Mon, Oct 19, 2015 at 10:46 AM, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
> ---
> src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 9 +
> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 10 ++
> 2 files changed, 19 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_
On Mon, Oct 19, 2015 at 2:29 PM, Matt Turner wrote:
> On Mon, Oct 19, 2015 at 7:45 AM, Emil Velikov
> wrote:
>> We're about to reuse get_timestamp() for the nir_intrinsic_shader_clock.
>> The in the latter the generalisation does not apply, so move the smear()
>> where needed. This also makes th
On Mon, Oct 19, 2015 at 11:55 AM, Connor Abbott wrote:
> On Mon, Oct 19, 2015 at 2:29 PM, Matt Turner wrote:
>> On Mon, Oct 19, 2015 at 7:45 AM, Emil Velikov
>> wrote:
>>> We're about to reuse get_timestamp() for the nir_intrinsic_shader_clock.
>>> The in the latter the generalisation does not
On Mon, Oct 19, 2015 at 3:19 AM, Francisco Jerez wrote:
> Hey Kristian,
>
> Kristian Høgsberg Kristensen writes:
>
>> We always set the mask to 0x, which is what it defaults to when no
>> header is present. Let's drop the header instead.
>>
>> Signed-off-by: Kristian Høgsberg Kristensen
>> -
On Mon, Oct 19, 2015 at 2:31 PM, Ilia Mirkin wrote:
> This series is Reviewed-by: Ilia Mirkin
> provided that you run a couple of piglit tests through valgrind and
> get no additional memory leaks.
>
> -ilia
>
> On Sat, Oct 10, 2015 at 9:09 PM, Marek Olšák wrote:
>> From: Marek Olšák
>>
>> --
---
src/gallium/include/pipe/p_defines.h | 1 +
src/gallium/include/pipe/p_state.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/gallium/include/pipe/p_defines.h
b/src/gallium/include/pipe/p_defines.h
index 29b0bfb..0dbc54c 100644
--- a/src/gallium/include/pipe/p_defines.h
+++ b/src
The goal is to allow the pipe driver to request something other than
TGSI, but detect whether what is getting is TGSI vs what it requested.
The pipe drivers will always have to support TGSI (and convert that into
whatever it is that they prefer), but in some cases we should be able to
skip the TGSI
Complete hack just for debugging. It isn't intended that mixing and
matching glsl->tgsi->nir vs glsl->nir between shader stages should
actually work.
---
src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/gallium/drivers/freedreno/ir
For now under debug flag, since only suitable for debugging/testing.
---
src/gallium/drivers/freedreno/freedreno_screen.c | 5 +++-
src/gallium/drivers/freedreno/freedreno_util.h | 1 +
.../drivers/freedreno/ir3/ir3_compiler_nir.c | 15 ++-
src/gallium/drivers/freedreno/ir3/i
Otherwise, passing -1 gets you:
error: invalid conversion from 'int' to 'nir_variable_mode' [-fpermissive]
Signed-off-by: Rob Clark
---
src/glsl/nir/nir.c | 4
src/glsl/nir/nir.h | 1 +
src/glsl/nir/nir_lower_io.c | 2 +-
src/mesa/drivers/dri/i96
When coming directly from glsl_to_nir (rather than via TGSI where
information about, for example, mat4's is lost), both const_index
fields will be used (vs. tgsi_to_nir where the 2nd is always zero).
For example:
decl_var uniform INTERP_QUALIFIER_NONE mat4 ModelViewProjectionMatrix (0, 0)
dec
We can get the tokens from variant->shader so drop the extra arg. This
will make things easier to support either getting TGSI or NIR as input.
---
src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/src/gallium/dr
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 223 -
src/mesa/state_tracker/st_program.c| 43 +-
2 files changed, 260 insertions(+), 6 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
ind
On Mon, Oct 19, 2015 at 3:51 PM, Ilia Mirkin wrote:
> On Mon, Oct 19, 2015 at 3:47 PM, Rob Clark wrote:
>> Also, there is some trivial shader variant handling in mesa st which
>> would have to be ported to NIR. Or, perhaps, just somehow expose the
>> shader key to the driver. (Currently most dr
From: Rob Clark
(grr, resend to the right list..)
Still quite rough around the edges, but now it's at the point of sorta-
kinda-working-sometimes. So I guess time to get wider feedback.
The initial goal is to figure out things that glsl_to_nir does
differently from tgsi_to_nir, so we can fix t
Am 19.10.2015 um 21:54 schrieb Rob Clark:
> The goal is to allow the pipe driver to request something other than
> TGSI, but detect whether what is getting is TGSI vs what it requested.
> The pipe drivers will always have to support TGSI (and convert that into
> whatever it is that they prefer), bu
Am 19.10.2015 um 21:54 schrieb Rob Clark:
> The goal is to allow the pipe driver to request something other than
> TGSI, but detect whether what is getting is TGSI vs what it requested.
> The pipe drivers will always have to support TGSI (and convert that into
> whatever it is that they prefer), bu
The overall idea looks quite sane to me.
Albeit obviously as you mentioned you'd really have to support all
shader stages and deal with the variants (though there aren't many left).
Roland
Am 19.10.2015 um 21:54 schrieb Rob Clark:
> From: Rob Clark
>
> (grr, resend to the right list..)
>
> Sti
On Mon, Oct 19, 2015 at 4:24 PM, Roland Scheidegger wrote:
> Am 19.10.2015 um 21:54 schrieb Rob Clark:
>> The goal is to allow the pipe driver to request something other than
>> TGSI, but detect whether what is getting is TGSI vs what it requested.
>> The pipe drivers will always have to support T
Acked-by: Sinclair Yeh
On Fri, Oct 16, 2015 at 03:25:17PM -0600, Brian Paul wrote:
> When a long GL_LINE_LOOP prim was split across primitives we drew
> stray lines. See previous commit for details.
>
> This patch converts GL_LINE_LOOP prims into GL_LINE_STRIP prims so
> that drivers don't ha
On Mon, Oct 19, 2015 at 4:19 AM, Francisco Jerez wrote:
> Neil Roberts writes:
>
>> Just a thought, would it be better to move this check into the
>> eliminate_find_live_channel optimisation? That way it could catch
>> sources that become immediates through later optimisations. One problem
>> wit
I don't have anything against new properties but why exactly is it
needed? As far as I can tell, clip/cull distances aren't declared as
arrays but just (up to two each) ordinary outputs, thus using the usage
masks should work just fine. Or is this handled differently by some
shader stages?
Roland
AMD/NVIDIA hardware has a limit on how many cull/clip distances can be
used at a time. On NVIDIA hw, there's a bitmask which you flip to
determine if a distance is meant to clip or cull. It's nice not to
have to analyze the output writes of a shader to tell how many clip
distances are used.
On Mon
1 - 100 of 158 matches
Mail list logo
