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: 0de5e0f3fb0f3671a3ecec6ab4473f9131ecd0ae
UR
Gstreamer-vaapi for one.
-Original Message-
From: Emil Velikov [mailto:emil.l.veli...@gmail.com]
Sent: Friday, October 16, 2015 9:26 PM
To: Christian König
Cc: ML mesa-dev; Das, Indrajit-kumar
Subject: Re: [Mesa-dev] [PATCH 2/2] Added support for NV12 to IYUV conversion
in vlVaGetImage
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/drivers/dri/i965/brw_fs.cpp| 4 ++--
2 files changed, 3 insertion
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 case from this:
add(8) g7<1>D g2<0,1,0>D 16D
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
constant propagate out of, and further, we don't constant propagate into
the typed
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 changed, 7 insertions(+), 18 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_
On Sun, Oct 18, 2015 at 11:14 AM, Eduardo Lima Mitev wrote:
> On 10/16/2015 05:18 PM, Jason Ekstrand wrote:
>> Topi,
>> Seeing as you're on a roll reviewing my move-the-code patches, mind one more?
>> --Jason
>>
>
> I took the liberty to review this one, and looks pretty
> straight-forward. So it
Patches 1-8 look good. I'm spending a bit more time on 10.
Reviewed-by: Sinclair Yeh
From: Brian Paul
Sent: Friday, October 16, 2015 2:25 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; Jose Fonseca; Sinclair Yeh
Subject: [PATCH 01/10] tnl: add
ping?
On 9 October 2015 at 10:38, Dave Airlie wrote:
> From: Dave Airlie
>
> So I've known this was broken before, cogl has a workaround
> for it from what I know, but with the gallium based swrast
> drivers BlitFramebuffer from back to front or vice-versa
> was pretty broken.
>
> The legacy swr
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 +---
src/gallium/drivers/nouveau/nv50/nv50_query_hw.h | 3 +-
2 files changed, 35 insert
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/Makefile.sources | 2 +
src/gallium/drivers/nouveau/nv50/nv50_query.c | 354 ++-
src/gallium/drivers/nouveau/nv50/nv50_query.h | 26 +-
src/gallium/drivers/nouveau/nv50/nv50_query_hw.c | 389 ++
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 instead of pipe_query.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/Makefile.source
Hi there,
As for nvc0, this series moves HW queries to improve readability of this
area of the driver and to prepare the way for both MP counters and global
perf counters.
There are no regressions with piglit.
Feel free to review,
Thanks.
Samuel Pitoiset (4):
nv50: add a header file for nv50_
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nv50/nv50_query.c | 18 --
src/gallium/drivers/nouveau/nv50/nv50_query.h | 3 ---
src/gallium/drivers/nouveau/nv50/nv50_state.c | 18 ++
3 files changed, 18 insertions(+), 21 deletions(-)
diff --git a
On 10/17/2015 07:31 PM, Charmaine Lee wrote:
From: Brian Paul
Sent: Friday, October 16, 2015 2:25 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; Jose Fonseca; Sinclair Yeh
Subject: [PATCH 09/10] vbo: fix GL_LINE_LOOP stray line bug
When long GL_LINE_LOOP primitives don't fit in on
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 the _mesa_prim::begin,end flags
> in order
On 18.10.2015 19:14, 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/mt
On 18.10.2015 19:52, Marek Olšák wrote:
On Sun, Oct 18, 2015 at 7:25 PM, Ilia Mirkin wrote:
Patches 1-3 are Reviewed-by: Ilia Mirkin . Not
familiar with the tgsi scan stuff.
Do you have plans on finishing cull distance support btw? Dave Airlie
and Tobias Klaussman have both made attempts at
Hi Emil,
2015-10-16 1:33 GMT+02:00 Emil Velikov :
> Hi Christian,
>
> Mostly minor suggestions I'm afraid. Things just look too good for
> anything serious.
>
:)
> On 11 October 2015 at 16:09, Christian Gmeiner
> wrote:
>> This commit adds tegra support, which uses the renderonly driver
>> libr
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 had nagging at me for much too long and I think it needs to be
> solved. So I'm hopeful that the more
On Sat, 2015-10-17 at 16:34 -0400, Ilia Mirkin wrote:
> On Sat, Oct 17, 2015 at 4:31 PM, Ilia Mirkin
> wrote:
> > On Sat, Oct 17, 2015 at 4:24 PM, Jan Vesely > > wrote:
> > > On Sat, 2015-10-17 at 15:24 -0400, Ilia Mirkin wrote:
> > > > "compute" in this context is "initialize the compute engine
On 10/16/2015 05:18 PM, Jason Ekstrand wrote:
> Topi,
> Seeing as you're on a roll reviewing my move-the-code patches, mind one more?
> --Jason
>
I took the liberty to review this one, and looks pretty
straight-forward. So it is:
Reviewed-By: Eduardo Lima Mitev
It had a few issues applying you
On Sun, Oct 18, 2015 at 7:25 PM, Ilia Mirkin wrote:
> Patches 1-3 are Reviewed-by: Ilia Mirkin . Not
> familiar with the tgsi scan stuff.
>
> Do you have plans on finishing cull distance support btw? Dave Airlie
> and Tobias Klaussman have both made attempts at it at one point or
> another.
I ha
Patches 1-3 are Reviewed-by: Ilia Mirkin . Not
familiar with the tgsi scan stuff.
Do you have plans on finishing cull distance support btw? Dave Airlie
and Tobias Klaussman have both made attempts at it at one point or
another.
-ilia
On Sun, Oct 18, 2015 at 1:14 PM, Marek Olšák wrote:
> From
From: Marek Olšák
No changes needed for drivers already relying on tgsi_shader_info.
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/
From: Marek Olšák
This reduces the shader key for ES.
Use a fixed attrib location based on (semantic name, index).
The ESGS item size is determined by the physical index of the highest ES
output, so it's almost always larger than before, but I think that
shouldn't matter as long as the ESGS ri
From: Marek Olšák
This solution was recommended by a Catalyst developer.
---
src/gallium/drivers/radeonsi/si_state_shaders.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c
b/src/gallium/drivers/radeonsi/si_state_shaders.c
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.h
b/src/gallium/drivers/radeonsi/si_shader.h
index 82d05e5..b72cb1a 100644
--- a/src/gallium/drivers/radeonsi/si_shader.h
From: Marek Olšák
The TGSI usage mask can't be used, because these are declared as an output
array of 2 elements.
---
src/gallium/auxiliary/tgsi/tgsi_strings.c | 2 ++
src/gallium/docs/source/tgsi.rst | 10 ++
src/gallium/include/pipe/p_shader_tokens.h | 4 +++-
3 files chan
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/mtypes.h | 5 +
src/mesa/main/sha
From: Marek Olšák
---
src/mesa/state_tracker/st_program.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/state_tracker/st_program.c
b/src/mesa/state_tracker/st_program.c
index 87571a8..75ccaf2 100644
--- a/src/mesa/state_tracker/st_program.c
+++ b/src/mesa/state_tracker/st
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c| 43 -
src/gallium/drivers/radeonsi/si_shader.h| 1 -
src/gallium/drivers/radeonsi/si_state_shaders.c | 13
3 files changed, 57 deletions(-)
diff --git a/src/gallium/drivers/radeonsi
Hi Emil,
Thanks for the review!
2015-10-16 1:09 GMT+02:00 Emil Velikov :
> Hi Christian,
>
> I'm glad to see Thierry's work revived. Hopefully this will soon be
> the basis of many more drivers.
I need to look into egl on wayland issue and hope to gain deeper knowledge
about the problem.
>
> On
Ping
On Thu, Oct 15, 2015 at 1:31 PM, Marek Olšák wrote:
> Ping
>
> On Sun, Oct 11, 2015 at 3:09 AM, Marek Olšák wrote:
>> From: Marek Olšák
>>
>> I'll let drivers figure out how to do it.
>> ---
>> src/gallium/docs/source/screen.rst | 2 ++
>> src/gallium/drivers/freedreno/freed
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 when building the name so just skip
> > over it.
> > ---
>
On 10/17/2015 09:11 PM, Jan Vesely wrote:
Does this mean it should be possible to hook up clover with nouveau?
As I said Ilia, this is just the ability to launch compute kernels on Fermi.
Unfortunately, OpenCL is still not supported by Nouveau but I hope this
is going to change.
Jan
On
On 10/17/2015 08:08 PM, Ilia Mirkin wrote:
This seems surprising... could I convince you to trace a test that
executes both a graphics and compute pipeline, which both use
(different) uniforms?
I'm convinced because this seems to be a bit weird for me too.
Anyways, this patch is fine for n
On 18/10/2015 12:25, Emil Velikov wrote:
+/* XXX: Is the owner responsible for FD owndership or are we ?
+ * In the former we're screwing things hard here */
This will be updated to "Although the fd is provided from external
source, mesa/nine takes ownership of it." based on input from
Hi Timothy,
On 17 October 2015 at 23: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/patch/62111/
>
> src/g
On 17 October 2015 at 23:58, Emil Velikov wrote:
> Cc: Axel Davy
> Signed-off-by: Emil Velikov
> ---
> src/gallium/targets/d3dadapter9/Makefile.am | 1 +
> src/gallium/targets/d3dadapter9/drm.c | 2 ++
> 2 files changed, 3 insertions(+)
>
> diff --git a/src/gallium/targets/d3dadapter9/Mak
On 17 October 2015 at 23:57, Emil Velikov wrote:
> Move the winsys into the pipe-target, bla bla bla
>
> XXX: separate pipe-drivers are likely to be busted
>
The XXX: is a left over and the pipe-drivers should be fine.
Upon a second thought this whole "keep the sw winsys+driver into the
pipe-driv
An LLVM IR <-> SPIR-V was in the plans (see [first proposal][0] and [second
proposal][1]). My guess is, it is still being worked on and we should hear some
more around Siggraph Asia (2--5 November), if Khronos is to announce Vulkan and
SPIR-V's final spec at that conference.
Pierre
PS: Just f
42 matches
Mail list logo