Do we really want to wait until a subpass that uses the given view or do we
just want to clear all of the relevant slices in the first subpass that
touches it? I haven't really thought about this much.
On Fri, Jan 5, 2018 at 8:38 AM, Iago Toral Quiroga
wrote:
> When multiview is active a subpas
On Fri, Jan 5, 2018 at 1:28 AM, Iago Toral Quiroga
wrote:
> ---
> src/intel/vulkan/anv_blorp.c | 55 ++
> ++
> 1 file changed, 40 insertions(+), 15 deletions(-)
>
> diff --git a/src/intel/vulkan/anv_blorp.c b/src/intel/vulkan/anv_blorp.c
> index e244468e03
Both patches are:
Reviewed-by: Iago Toral Quiroga
On Tue, 2018-01-23 at 01:33 -0800, Jason Ekstrand wrote:
> Both callers of intel_miptree_prepare/finish_render have to call
> intel_miptree_render_aux_usage anyway for other reasons. They may as
> well pass the result in instead of us calling it
2018-01-21 23:58 GMT+01:00 Eric Anholt :
> Thomas Helland writes:
>
>> Also, allocate worklist_elem in groups of 20, to reduce the burden of
>> allocation. Do not use rzalloc, as there is no need. This lets us drop
>> the number of calls to ralloc from aproximately 10% of all calls to
>> ralloc(13
This is still waiting for review, any takers?
On Wed, 2018-01-17 at 09:08 +0100, Iago Toral Quiroga wrote:
> If a shader only writes to an output via a constant initializer we
> need to lower it before we call nir_remove_dead_variables so that
> this pass sees the stores from the initializer and d
This is still unreviewed, any chances that someone can have a look and
provide feedback? This is required to pass new tests that will be
merged in CTS soon.
Iago
On Fri, 2018-01-05 at 17:38 +0100, Iago Toral Quiroga wrote:
> ---
> src/intel/vulkan/anv_blorp.c | 55 +++
I am not a meson expert but changes look fine. The series is:
Reviewed-by: Samuel Iglesias Gonsálvez
Sam
On 23/01/18 18:44, Dylan Baker wrote:
> This reverts commit 10d1b0be8e9c463dbc35cd66968299f33c76672c.
>
> This is unnecessary, the depend_files argument is for adding
> dependencies on files
Reviewed-by: Tapani Pälli
On 23.01.2018 00:57, Grazvydas Ignotas wrote:
Looks like checking both sources was intended, instead of the first one
twice. Found with Coccinelle, coccinellery/xand/xand.cocci semantic patch.
Signed-off-by: Grazvydas Ignotas
---
src/intel/vulkan/anv_nir_apply_pipe
On Tuesday, January 23, 2018 1:32:53 AM PST Chris Wilson wrote:
> Quoting Kenneth Graunke (2018-01-23 09:28:20)
> > The purpose of the workaround BO is to write to it.
>
> Do you care for the serialisation here? I thought the purpose of the wa
> was just for write-only memory, so lying to the kern
From: Dave Airlie
amdvlk appears to enable these bits always, no explaination of why.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_pipeline.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index 62fa
From: Dave Airlie
This seems to be broken, at least the cts tests fail.
This fixes:
dEQP-VK.renderpass.suballocation.multisample.d32_sfloat_s8_uint.samples_4
dEQP-VK.renderpass.suballocation.multisample.d32_sfloat_s8_uint.samples_8
2 samples seems to pass fine, amdvlk doesn't appear to enable T
For the series:
Tested-by: Dieter Nützel
on RX580
with UH (even wireframe mode works, now - GREAT!), UV, US, glmark2,
Blender 2.79, DiRT Rally, F1 2017 and smoketest --- all I throw at it.
Together with:
[Mesa-dev] [PATCH] ac/nir: call glsl_get_sampler_dim() only once, where
possible (v2)
https://bugs.freedesktop.org/show_bug.cgi?id=101814
--- Comment #4 from Daniel van Vugt ---
Thanks. It looks like mesa-demos hasn't had a release for 2 years...
https://cgit.freedesktop.org/mesa/demos/
So this fix still isn't released either. Any idea of when a release might
happen?
--
You ar
On 01/23/2018 05:59 PM, Clayton Craft wrote:
On Tue, Jan 23, 2018 at 11:44:01AM -0700, Brian Paul wrote:
This fixes another regression from commit 8e4efdc895ea ("vbo: optimize
some display list drawing"). The problem was the min_index, max_index
values passed to the vbo drawing function were no
From: Dave Airlie
This was just found while reading for other stuff,
rc/core/hw/gfxip/gfx6/gfx6DepthStencilView.cpp.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_pipeline.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipelin
Tested-by: Dieter Nützel
on RX580
with UH, UV, US, glmark2, Blender 2.79, DiRT Rally, F1 2017 and
smoketest
Together with:
Mesa-dev] [PATCH 1/3] st/glsl_to_tgsi: move nir detection earlier and
set nir options
Am 23.01.2018 21:14, schrieb Kai Wasserbäch:
Changes since v1:
* Rebased on to
From: Dave Airlie
We need to enable the pos float location 2 mode anytime we have
persample not just when forced by the frag shader.
This fixes:
dEQP-VK.pipeline.multisample.min_sample_shading*
Fixes: 58c97a079 (radv: enable location at sample when persample is forced.)
Signed-off-by: Dave Airl
On 24/01/18 11:44, Dieter Nützel wrote:
Rebase underway? ;-)
Please direct these type of questions to IRC, rather than creating extra
traffic and threads on the mailing list.
Am 15.01.2018 04:46, schrieb Timothy Arceri:
This series is intended to be applied on top of the previous serires
On Tue, Jan 23, 2018 at 11:44:01AM -0700, Brian Paul wrote:
This fixes another regression from commit 8e4efdc895ea ("vbo: optimize
some display list drawing"). The problem was the min_index, max_index
values passed to the vbo drawing function were not computed to compensate
for the biased prim::
Hello Dave,
is this one obsolete or should it go before the 2 committed but without
the first line (duplicate)?
Dieter
Am 23.01.2018 06:29, schrieb Dave Airlie:
From: Dave Airlie
From reading AMDVLK it currently never uses hw resolve paths.
This patch limits the paths we used the hw resol
I should note that compute support is missing (asserts) as I was unable
to test it with radeonsi currently.
On 24/01/18 11:41, Timothy Arceri wrote:
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +-
src/mesa/state_tracker/st_program.c| 6 +-
src/mesa/state_tracker/st_shader
Rebase underway? ;-)
Am 15.01.2018 04:46, schrieb Timothy Arceri:
This series is intended to be applied on top of the previous serires
that ebales glsl 450 starting with:
[PATCH 1/5] ac/radeonsi: add tcs load outputs support
___
mesa-dev mailing list
---
src/gallium/drivers/radeonsi/si_pipe.c | 4
src/mesa/state_tracker/st_program.c| 15 +++
2 files changed, 11 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
b/src/gallium/drivers/radeonsi/si_pipe.c
index 676d199618..c096165b03 100644
--
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +-
src/mesa/state_tracker/st_program.c| 6 +-
src/mesa/state_tracker/st_shader_cache.c | 133 +++--
src/mesa/state_tracker/st_shader_cache.h | 8 +-
4 files changed, 115 insertions(+), 34 deletions(-)
di
We move the nir check before the shader cache call so that we can
call a nir based caching function in a following patch.
---
src/mesa/state_tracker/st_glsl_to_nir.cpp | 10 ++---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 36 +++---
2 files changed, 30 insertions(+)
On Fri, Jan 19, 2018 at 03:47:33PM -0800, Jason Ekstrand wrote:
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 31 ++-
> 1 file changed, 14 insertions(+), 17 deletions(-)
>
This patch is
Reviewed-by: Nanley Chery
> diff --git a/src/intel/vulkan/genX_cmd_buffer.c
> b/
Quoting Kenneth Graunke (2018-01-23 15:04:18)
> On Tuesday, January 23, 2018 10:34:25 AM PST Dylan Baker wrote:
> > There's a bug in our meson build, and I sent a patch. We need to join_dirs
> > with
> > get_option('prefix'). I got a little confused because when you pass a
> > relative
> > direct
Sure enough, it does use ':'
You might want to send the patch to the list and CC Emil, and probably stable.
Although Emil also mentioned that maybe it would just be better to delete this
option since we have LIBGL_DRIVERS_PATH.
Quoting Eric Engestrom (2018-01-23 15:28:50)
> On Tuesday, 2018-01-23
On Wed, Jan 24, 2018 at 12:00 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This is ported from radeonsi and fixes:
> dEQP-VK.pipeline.multisample_shader_builtin.sample_mask.bit_*
>
> v2: don't call this path for radeonsi, it does it in the epilog.
> use the radeonsi code path.
> v3: handle NULL
Quoting Scott D Phillips (2018-01-23 14:42:43)
> TileY's low 6 address bits are: v1 v0 u3 u2 u1 u0
> Thus a cache line in the tiled surface is composed of a 2d area of
> 16x4 bytes of the linear surface.
>
> Add a special case where the area being copied is 4-line aligned
> and a multiple of 4-lin
On 2018-01-23 — 15:16, Francisco Jerez wrote:
> Pierre Moreau writes:
>
> > On 2018-01-23 — 14:04, Francisco Jerez wrote:
> >> Pierre Moreau writes:
> >>
> >> > If creating a library, do not allow non-compiled object in it, as
> >> > executables are not allowed, and libraries would make it real
Pierre Moreau writes:
> On 2018-01-23 — 14:04, Francisco Jerez wrote:
>> Pierre Moreau writes:
>>
>> > If creating a library, do not allow non-compiled object in it, as
>> > executables are not allowed, and libraries would make it really hard to
>> > enforce the "-enable-link-options" flag.
>>
On Tuesday, 2018-01-23 21:53:14 +, Dylan Baker wrote:
> Quoting Eric Engestrom (2018-01-18 15:02:46)
> > On Thursday, 2018-01-18 17:09:33 +, Dylan Baker wrote:
> > > Currently there is not a separate option for setting the search path of
> > > DRI drivers in meson, like there is in scons an
Pierre Moreau writes:
> On 2018-01-23 — 14:03, Francisco Jerez wrote:
>> Pierre Moreau writes:
>>
>> > From the OpenCL 1.2 Specification, Section 5.6.2 (about clBuildProgram:
>> >
>> >> If program is created with clCreateProgramWithBinary, then the
>> >> program binary must be an executable bin
On 2018-01-23 — 14:03, Francisco Jerez wrote:
> Pierre Moreau writes:
>
> > Signed-off-by: Pierre Moreau
> > ---
> > src/gallium/state_trackers/clover/core/device.cpp | 11 +++
> > src/gallium/state_trackers/clover/core/device.hpp | 3 +++
> > 2 files changed, 14 insertions(+)
> >
> >
On 2018-01-23 — 14:03, Francisco Jerez wrote:
> Pierre Moreau writes:
>
> > From the OpenCL 1.2 Specification, Section 5.6.2 (about clBuildProgram:
> >
> >> If program is created with clCreateProgramWithBinary, then the
> >> program binary must be an executable binary (not a compiled binary or
>
On 2018-01-23 — 14:04, Francisco Jerez wrote:
> Pierre Moreau writes:
>
> > If creating a library, do not allow non-compiled object in it, as
> > executables are not allowed, and libraries would make it really hard to
> > enforce the "-enable-link-options" flag.
> >
> > Signed-off-by: Pierre More
Quoting Scott D Phillips (2018-01-10 07:16:59)
> +/* Compute extent parameters for use with tiled_memcpy functions.
> + * xs are in units of bytes and ys are in units of strides. */
> +static inline void
> +tile_extents(struct intel_mipmap_tree *mt, struct intel_miptree_map *map,
> + un
On Tuesday, January 23, 2018 10:34:25 AM PST Dylan Baker wrote:
> There's a bug in our meson build, and I sent a patch. We need to join_dirs
> with
> get_option('prefix'). I got a little confused because when you pass a relative
> directory to meson's `install` option it will autotmatically make i
On Tue, Jan 23, 2018 at 11:46 PM, Francisco Jerez wrote:
> Pierre Moreau writes:
>
>> On 2018-01-23 — 14:02, Francisco Jerez wrote:
>>> Karol Herbst writes:
>>>
>>> > there seem to be some patches missing?
>>> >
>>> > On Tue, Jan 23, 2018 at 1:33 AM, Pierre Moreau
>>> > wrote:
>>> >> Hello,
>>
On Mon, Jan 22, 2018 at 05:25:31PM -0800, Jason Ekstrand wrote:
> On Mon, Jan 22, 2018 at 3:22 PM, Nanley Chery wrote:
>
> > On Fri, Jan 19, 2018 at 03:47:29PM -0800, Jason Ekstrand wrote:
> > > Currently, this helper does nothing but we call it every place where an
> > > image is written through
Pierre Moreau writes:
> On 2018-01-23 — 14:02, Francisco Jerez wrote:
>> Karol Herbst writes:
>>
>> > there seem to be some patches missing?
>> >
>> > On Tue, Jan 23, 2018 at 1:33 AM, Pierre Moreau
>> > wrote:
>> >> Hello,
>> >>
>> >> Here is the second version of my initial series for adding
From: Dave Airlie
This is ported from radeonsi and fixes:
dEQP-VK.pipeline.multisample_shader_builtin.sample_mask.bit_*
v2: don't call this path for radeonsi, it does it in the epilog.
use the radeonsi code path.
v3: handle NULL pCreateInfo->pMultisampleState properly (Samuel)
Signed-off-by: Da
On 2018-01-23 — 14:02, Francisco Jerez wrote:
> Karol Herbst writes:
>
> > there seem to be some patches missing?
> >
> > On Tue, Jan 23, 2018 at 1:33 AM, Pierre Moreau
> > wrote:
> >> Hello,
> >>
> >> Here is the second version of my initial series for adding SPIR-V support
> >> to
> >> clove
TileY's low 6 address bits are: v1 v0 u3 u2 u1 u0
Thus a cache line in the tiled surface is composed of a 2d area of
16x4 bytes of the linear surface.
Add a special case where the area being copied is 4-line aligned
and a multiple of 4-lines so that entire cache lines will be
written at a time.
O
On 2018-01-23 — 14:08, Francisco Jerez wrote:
> Pierre Moreau writes:
>
> > Signed-off-by: Pierre Moreau
> > ---
> > src/gallium/state_trackers/clover/core/program.cpp | 22
> > +-
> > 1 file changed, 17 insertions(+), 5 deletions(-)
> >
> > diff --git a/src/gallium/state_t
Pierre Moreau writes:
> On 2018-01-23 — 14:06, Francisco Jerez wrote:
>> Pierre Moreau writes:
>>
>> > Signed-off-by: Pierre Moreau
>> > ---
>> > .../state_trackers/clover/spirv/spirv.hpp11| 997
>> > +
>> > 1 file changed, 997 insertions(+)
>> > create mode 1006
On 2018-01-23 — 14:08, Francisco Jerez wrote:
> Pierre Moreau writes:
>
> > Signed-off-by: Pierre Moreau
>
> Same comment as Karol, let's update the header instead.
I will do that. Though that won’t impact this patch, as cl_khr_il_program is
not part of the official OpenCL headers. I have sent
Pierre Moreau writes:
> Signed-off-by: Pierre Moreau
> ---
> src/gallium/state_trackers/clover/api/device.cpp | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/src/gallium/state_trackers/clover/api/device.cpp
> b/src/gallium/state_trackers/clover/api/device.cpp
> index 4e274c5005..
Pierre Moreau writes:
> Signed-off-by: Pierre Moreau
> ---
> src/gallium/state_trackers/clover/api/dispatch.hpp | 4 ++
> src/gallium/state_trackers/clover/api/program.cpp | 29 -
> src/gallium/state_trackers/clover/core/program.cpp | 68
> +-
> src/gallium/state_
On 2018-01-23 — 14:06, Francisco Jerez wrote:
> Pierre Moreau writes:
>
> > Signed-off-by: Pierre Moreau
> > ---
> > .../state_trackers/clover/spirv/spirv.hpp11| 997
> > +
> > 1 file changed, 997 insertions(+)
> > create mode 100644 src/gallium/state_trackers/clov
Pierre Moreau writes:
> Signed-off-by: Pierre Moreau
> ---
> src/gallium/state_trackers/clover/core/program.cpp | 22
> +-
> 1 file changed, 17 insertions(+), 5 deletions(-)
>
> diff --git a/src/gallium/state_trackers/clover/core/program.cpp
> b/src/gallium/state_trackers/
Pierre Moreau writes:
> Signed-off-by: Pierre Moreau
Same comment as Karol, let's update the header instead.
> ---
> include/CL/cl_ext.h | 34 ++
> 1 file changed, 34 insertions(+)
>
> diff --git a/include/CL/cl_ext.h b/include/CL/cl_ext.h
> index 710bea8837..2
Pierre Moreau writes:
> OpenCL 2.1, and cl_khr_il_program, gives the ability to query for a
> program’s IL, which is returned as a pointer.
>
> Signed-off-by: Pierre Moreau
I'm not convinced this is correct, but... It seems fully unnecessary, I
think you should be using a property_vector inste
Pierre Moreau writes:
> Creating a program using clCreateProgramWithSource to SPIR-V requires a
> non-upstreamed version of LLVM and clang, therefore it is currently not
> supported.
>
> Signed-off-by: Pierre Moreau
> ---
> src/gallium/state_trackers/clover/core/program.cpp | 4
> 1 file c
Pierre Moreau writes:
> Signed-off-by: Pierre Moreau
> ---
> .../state_trackers/clover/spirv/spirv.hpp11| 997
> +
> 1 file changed, 997 insertions(+)
> create mode 100644 src/gallium/state_trackers/clover/spirv/spirv.hpp11
Can you import this with an hpp extensio
Pierre Moreau writes:
> Signed-off-by: Pierre Moreau
Reviewed-by: Francisco Jerez
> ---
> src/gallium/state_trackers/clover/api/platform.cpp | 4 ++--
> src/gallium/state_trackers/clover/core/platform.cpp | 5 +
> src/gallium/state_trackers/clover/core/platform.hpp | 2 ++
> 3 files cha
Pierre Moreau writes:
> Signed-off-by: Pierre Moreau
Reviewed-by: Francisco Jerez
> ---
> src/gallium/state_trackers/clover/api/device.cpp | 11 +--
> src/gallium/state_trackers/clover/core/device.cpp | 14 ++
> src/gallium/state_trackers/clover/core/device.hpp | 1 +
>
Pierre Moreau writes:
> Signed-off-by: Pierre Moreau
Acked-by: Francisco Jerez
> ---
> src/gallium/state_trackers/clover/api/device.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/state_trackers/clover/api/device.cpp
> b/src/gallium/state_trackers/cl
Pierre Moreau writes:
> If creating a library, do not allow non-compiled object in it, as
> executables are not allowed, and libraries would make it really hard to
> enforce the "-enable-link-options" flag.
>
> Signed-off-by: Pierre Moreau
> ---
> src/gallium/state_trackers/clover/api/program.c
Pierre Moreau writes:
> From the OpenCL 1.2 Specification, Section 5.6.2 (about clBuildProgram:
>
>> If program is created with clCreateProgramWithBinary, then the
>> program binary must be an executable binary (not a compiled binary or
>> library).
>
> Signed-off-by: Pierre Moreau
> ---
> src/
Pierre Moreau writes:
> Signed-off-by: Pierre Moreau
> ---
> src/gallium/state_trackers/clover/core/device.cpp | 11 +++
> src/gallium/state_trackers/clover/core/device.hpp | 3 +++
> 2 files changed, 14 insertions(+)
>
> diff --git a/src/gallium/state_trackers/clover/core/device.cpp
Karol Herbst writes:
> there seem to be some patches missing?
>
> On Tue, Jan 23, 2018 at 1:33 AM, Pierre Moreau wrote:
>> Hello,
>>
>> Here is the second version of my initial series for adding SPIR-V support to
>> clover, after the RFC back in May 2017.
>>
>> For recap, the focus of this serie
On 2018-01-23 — 13:33, Francisco Jerez wrote:
> Pierre Moreau writes:
>
> > On 2018-01-23 — 14:07, Jan Vesely wrote:
> >> On Tue, 2018-01-23 at 01:33 +0100, Pierre Moreau wrote:
> >> > Signed-off-by: Pierre Moreau
> >> > ---
> >> > src/gallium/include/pipe/p_defines.h | 1 +
> >> > 1 file chang
Fixes: d1992255bb29 ("meson: Add build Intel "anv" vulkan driver")
Reported-by: Marc Dietrich
Signed-off-by: Dylan Baker
---
src/util/meson.build | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/util/meson.build b/src/util/meson.build
index fa591c92e56..b23dba3a985
There's a bug in our meson build, and I sent a patch. We need to join_dirs with
get_option('prefix'). I got a little confused because when you pass a relative
directory to meson's `install` option it will autotmatically make it absolute,
but when you pass it to a C define it doesn't.
Quoting Kenne
Quoting Emil Velikov (2018-01-19 10:28:18)
> On 17 January 2018 at 21:34, Dylan Baker wrote:
> > Currently there is not a separate option for setting the search path of
> > DRI drivers in meson, like there is in scons and autotools. This is an
> > oversight and needs to be fixed. This adds an extr
Quoting Eric Engestrom (2018-01-18 15:02:46)
> On Thursday, 2018-01-18 17:09:33 +, Dylan Baker wrote:
> > Currently there is not a separate option for setting the search path of
> > DRI drivers in meson, like there is in scons and autotools. This is an
> > oversight and needs to be fixed. This
Signed-off-by: Dylan Baker
---
src/intel/vulkan/meson.build | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/intel/vulkan/meson.build b/src/intel/vulkan/meson.build
index 1be389b67e3..41dba420cb4 100644
--- a/src/intel/vulkan/meson.build
+++ b/src/intel/vulkan/me
cc: Jason Ekstrand
cc: Samuel Iglesias Gonsálvez
Fixes: f93994080993bda ("anv: Split anv_extensions.py into two files")
Signed-off-by: Dylan Baker
---
src/intel/vulkan/meson.build | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/intel/vulkan/meson.build b/src/intel/vu
cc: Jason Ekstrand
cc: Samuel Iglesias Gonsálvez
Fixes: dd088d4bec74f37ffe4 ("anv/extensions: Generate a header file with
extension tables")
Signed-off-by: Dylan Baker
---
src/intel/vulkan/meson.build | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/intel/vulkan/meso
Hi Marc,
I have two theories
1) Is your locale non-english by chance? generation of drirc support for
translations is still on the list
2) The search path may not be correct, I'll double check that now.
Quoting Marc Dietrich (2018-01-23 03:31:23)
> Hi Dylan,
>
> running "Heaven" fails with "er
This needs to go into the 18.0 release. CC'ing stable.
Quoting Eric Engestrom (2018-01-23 07:46:14)
> On Tuesday, 2018-01-23 15:49:43 +0100, Marc Dietrich wrote:
> > Defines
> > - HAVE_FUNC_ATTRIBUTE_RETURNS_NONNULL
> > - HAVE_FUNC_ATTRIBUTE_VISIBILITY
> > were misspelled.
>
> Wow, good catch!
>
This reverts commit 10d1b0be8e9c463dbc35cd66968299f33c76672c.
This is unnecessary, the depend_files argument is for adding
dependencies on files that are not part of the input, which is already
done.
cc: Jason Ekstrand
cc: Samuel Iglesias Gonsálvez
Fixes: 10d1b0be8e9c463dbc35cd66968299f33c76672
I've added Eric and I's rbs and a fixes tag and pushed to master.
Thanks for fixing this!
Quoting Christoph Haag (2018-01-20 09:53:13)
> ---
> src/gallium/targets/d3dadapter9/meson.build | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/targets/d3dadapter9/meson.build
> b/sr
As far as I can tell this always just reassigns the same value.
Also as we don't curretly store UniformHash in the shader cache
removing this will help with adding a shader cache to gallium
nir drivers.
---
src/mesa/state_tracker/st_glsl_to_nir.cpp | 8
1 file changed, 8 deletions(-)
di
Pierre Moreau writes:
> On 2018-01-23 — 14:07, Jan Vesely wrote:
>> On Tue, 2018-01-23 at 01:33 +0100, Pierre Moreau wrote:
>> > Signed-off-by: Pierre Moreau
>> > ---
>> > src/gallium/include/pipe/p_defines.h | 1 +
>> > 1 file changed, 1 insertion(+)
>> >
>> > diff --git a/src/gallium/include
On 2018-01-23 — 14:20, Jan Vesely wrote:
> On Tue, 2018-01-23 at 01:33 +0100, Pierre Moreau wrote:
> > Signed-off-by: Pierre Moreau
>
> do these tools handle the original SPIR format as well? can this be
> used to support cl_khr_spir?
>
> Jan
They do not I’m afraid. I just saw that SPIR-Tools [
On 2018-01-23 — 14:07, Jan Vesely wrote:
> On Tue, 2018-01-23 at 01:33 +0100, Pierre Moreau wrote:
> > Signed-off-by: Pierre Moreau
> > ---
> > src/gallium/include/pipe/p_defines.h | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/src/gallium/include/pipe/p_defines.h
> > b/src/gall
Also, the associated piglit patches:
https://lists.freedesktop.org/archives/piglit/2017-December/023600.html
On 2018-01-23 01:13 PM, Andres Rodriguez wrote:
On 2018-01-23 01:04 PM, Andres Rodriguez wrote:
Rebased on latest mesa and dropped the gallium fence->semaphore rename.
There is also
On Tue, Jan 23, 2018 at 2:27 PM, Samuel Pitoiset
wrote:
> The Vulkan spec says:
>
>"pipelineBindPoint is a VkPipelineBindPoint indicating whether
> the descriptors will be used by graphics pipelines or compute
> pipelines. There is a separate set of bind points for each of
> graphi
Changes since v1:
* Rebased on top of e68150de263156a3f3d1b609b6506c5649967f61 and
82adf53308c137ce0dc5f2d5da4e7cc40c5b808c.
Cc: Timothy Arceri
Tested-by: Dieter Nützel (v1)
Signed-off-by: Kai Wasserbäch
---
Hey,
if you accept this patch, please commit it for me – I do not have commit
ac
On Jan 22, 2018 09:02, "Kenneth Graunke" wrote:
On Thursday, January 18, 2018 1:44:47 PM PST Chuck Atkins wrote:
> Is there a logging infrastructure currently available to drivers in Mesa?
> I was looking to clean up some of swr's debug / info output and have it
> conditional on the MESA_DEBUG an
On Jan 22, 2018 09:02, "Kenneth Graunke" wrote:
On Thursday, January 18, 2018 1:44:47 PM PST Chuck Atkins wrote:
> Is there a logging infrastructure currently available to drivers in Mesa?
> I was looking to clean up some of swr's debug / info output and have it
> conditional on the MESA_DEBUG an
Reviewed-by: Bas Nieuwenhuizen
On Tue, Jan 23, 2018 at 6:52 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> From reading AMDVLK it currently never uses hw resolve paths.
>
> This patch takes from radeonsi which doesn't use hw resolve
> for integer formats, and does the same for radv.
>
> This fi
On Tue, Jan 23, 2018 at 6:52 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> Some of the hw resolve passes need the SPI color format setup
> correctly.
>
> This fixes lots of 16-bit and 32-bit format tests in
> dEQP-VK.renderpass.suballocation.multisample*
>
> Signed-off-by: Dave Airlie
> ---
>
On Tue, 2018-01-23 at 01:33 +0100, Pierre Moreau wrote:
> Signed-off-by: Pierre Moreau
do these tools handle the original SPIR format as well? can this be
used to support cl_khr_spir?
Jan
> ---
> configure.ac | 5 +
> meson.build | 2 ++
> 2 files changed, 7 insertions(+)
>
> diff --git
Just some nit-picks below.
On 01/23/2018 03:11 AM, Gert Wollny wrote:
Add the equal operator and the "<<" stream write operator for the
st_*_reg classes and the "<<" operator to the instruction class, and
make use of these operators in the debugging output.
---
src/mesa/state_tracker/st_glsl_
On 01/23/2018 03:11 AM, Gert Wollny wrote:
Don't allocate a zero-sized array, when no texture offsets are given.
Reviewed-by: Nicolai Hähnle
Signed-off-by: Gert Wollny
---
.../tests/test_glsl_to_tgsi_lifetime.cpp | 23 +++---
1 file changed, 16 insertions(+), 7 del
On Tue, 2018-01-23 at 01:33 +0100, Pierre Moreau wrote:
> Signed-off-by: Pierre Moreau
> ---
> src/gallium/include/pipe/p_defines.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/include/pipe/p_defines.h
> b/src/gallium/include/pipe/p_defines.h
> index b34e7a8570..082d4c4d
https://bugs.freedesktop.org/show_bug.cgi?id=104752
Brian Paul changed:
What|Removed |Added
Component|Other |New Accounts
Product|Mesa
https://bugs.freedesktop.org/show_bug.cgi?id=104742
--- Comment #1 from Brian Paul ---
Vinson, I posted a patch to mesa-dev to which fixes this and 104746. If you
can test too, that'd be helpful.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee f
On 22 January 2018 at 18:02, Chuck Atkins wrote:
> Hi Emil,
>
>> Fixes: a4be2bcee2f ("swr: allow a single swr architecture to be builtin")
>
>
> It doesn't fix anything that was broken from that commit. The issues with
> error handling were already present before then, it's just that the changes
https://bugs.freedesktop.org/show_bug.cgi?id=104749
--- Comment #1 from Juan A. Suarez ---
Tracked to:
commit 01ab218bbc5c8058a99077a6bc3dc9884e9d218a
Author: George Kyriazis
Date: Fri Jan 19 15:46:59 2018 -0600
swr/rast: Initial work for debugging support.
Adds ability to step into
This fixes another regression from commit 8e4efdc895ea ("vbo: optimize
some display list drawing"). The problem was the min_index, max_index
values passed to the vbo drawing function were not computed to compensate
for the biased prim::start values.
https://bugs.freedesktop.org/show_bug.cgi?id=10
https://bugs.freedesktop.org/show_bug.cgi?id=104752
--- Comment #4 from Emil Velikov ---
Ack from me.
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.___
mesa-dev mailing list
mesa-dev@lists
Memory synchronization is left for a future patch.
v2: flush vertices/bitmaps moved to mesa/main
v3: removed spaces before/after braces
Signed-off-by: Andres Rodriguez
---
src/mesa/main/dd.h | 14 ++
src/mesa/main/externalobjects.c | 38 +
This reverts commit 2ce11ac11fee594ca01608c4006b38c0c8ea37ff.
SteamVR has been updated to correctly use UNDEFINED as the initial
layout transition state. So we should go back to behaving according to
the spec in radv.
Fixes: 44fcf58744 "radv: Disable DCC for GENERAL layout and compute transfer
d
This release corrects the version number, to follow the standard naming scheme.
Emil Velikov (1):
Update version to 18.0.0-rc2
git tag: mesa-18.0.0-rc2
https://mesa.freedesktop.org/archive/mesa-18.0.0-rc2.tar.gz
MD5: 755ced6e9039d2f3394b09a99e3ed2e2 mesa-18.0.0-rc2.tar.gz
SHA1: 0066bb08b
On 01/23/2018 11:05 AM, Andres Rodriguez wrote:
Memory synchronization is left for a future patch.
v2: flush vertices/bitmaps moved to mesa/main
Signed-off-by: Andres Rodriguez
---
src/mesa/main/dd.h | 14 ++
src/mesa/main/externalobjects.c | 38
1 - 100 of 179 matches
Mail list logo
