On Fri, Aug 04, 2017 at 06:54:12PM +0200, Philipp Zabel wrote:
> On Fri, 2017-08-04 at 18:15 +0200, Wladimir J. van der Laan wrote:
> > On Fri, Aug 04, 2017 at 05:07:54PM +0200, Philipp Zabel wrote:
> > > Look up BOs from the name table using the name parameter instead of
> > > req.handle (which at
On Fri, Aug 4, 2017 at 2:16 AM, Rainer Hochecker
wrote:
> This seems to breaks exporting 16bit vaapi images via drm buffers
>
Yes, I'm aware of the problem and there are two patches on the list which
should fix it:
https://patchwork.freedesktop.org/patch/170051/
https://patchwork.freedesktop.or
These shaders have been generated by Dolphin 9649494f67 on Mesa
8c26b52349 for an HD4000 GPU.
They include a lot of uniform branches, mostly on integers, as well as
switch statements branching on small and bounded integers.
Signed-off-by: Emmanuel Gil Peyrot
---
The actual patch isn’t included
Hi Yogesh,
On Sat, Aug 5, 2017 at 1:22 AM, Marathe, Yogesh
wrote:
>> -Original Message-
>> From: Tomasz Figa [mailto:tf...@chromium.org]
>> Sent: Friday, August 4, 2017 9:39 PM
>> On Sat, Aug 5, 2017 at 12:53 AM, Marathe, Yogesh
>> wrote:
>> > Tomasz, Emil,
>> >
>> >> -Original Messa
On 4 August 2017 at 18:53, Nicolai Hähnle wrote:
> On 04.08.2017 04:51, Dave Airlie wrote:
>>
>> From: Dave Airlie
>>
>> This is a bug in the app, but I'd rather avoid hanging the GPU,
>> esp if someone is running in validation and it takes out their
>> development environment.
>>
>> v2: get it r
This seems to breaks exporting 16bit vaapi images via drm buffers
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 5 August 2017 at 00:25, Emil Velikov wrote:
> From: Emil Velikov
>
> The function can handle only window surfaces, so let's rename it
> accordingly, killing the wrapper around it.
>
> Suggested-by: Eric Engestrom
> Signed-off-by: Emil Velikov
> ---
> New patch
> ---
> src/egl/drivers/dri2/p
On Friday, August 4, 2017 12:22:19 PM PDT Chris Wilson wrote:
> Quoting Kenneth Graunke (2017-08-04 19:47:14)
> > On Friday, July 21, 2017 8:36:42 AM PDT Chris Wilson wrote:
> > > Patch reordering from last time so that the cosmetic tweaks are done first
> > > and out of the way. Kenneth has review
Oh, I saw it had the old xml and was assuming it didn't cause any errors,
but clearly I was wrong.
Reviewed-by: Rafael Antognolli
On Fri, Aug 04, 2017 at 10:21:43PM +, Scott D Phillips wrote:
> BLEND_STATE packing was modified to be variable-length in:
>
> 9670124e31 genxml: Make BLEND_STA
On 4 August 2017 at 23:08, Dieter Nützel wrote:
> For the series:
>
> Tested-by: Dieter Nützel
>
> on RX580
>
> with Clover, vdpau and Nine.
>
> ./autogen.sh --prefix=/usr/local --with-dri-drivers=""
> --with-gallium-drivers=r600,radeonsi,swrast --with-platforms=drm,x11
> --enable-nine --enable-t
From: Emil Velikov
The basic (null) check is identical across all backends.
Just move it to the top.
v2:
- Split the WINDOW vs PIXMAP into separate patches
- Move check after the dpy and config - dEQP expects so
Cc: Eric Engestrom
Signed-off-by: Emil Velikov
---
src/egl/drivers/dri2/platfo
From: Emil Velikov
Signed-off-by: Emil Velikov
Reviewed-by: Matt Turner
---
src/egl/drivers/dri2/platform_x11.c | 2 +-
src/egl/drivers/dri2/platform_x11_dri3.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/egl/drivers/dri2/platform_x11.c
b/src/egl/drivers/dr
From: Emil Velikov
The function can handle only window surfaces, so let's rename it
accordingly, killing the wrapper around it.
Suggested-by: Eric Engestrom
Signed-off-by: Emil Velikov
---
New patch
---
src/egl/drivers/dri2/platform_drm.c | 17 -
1 file changed, 4 insertions(+
From: Emil Velikov
The code in _eglCreatePixmapSurfaceCommon() already has a NULL check
which handles the condition. There's no point in checkin again further
down the stack.
v2: Split the WINDOW vs PIXMAP into separate patches
Cc: Eric Engestrom
Signed-off-by: Emil Velikov
---
src/egl/drive
From: Matt Turner
The next patch is going to stop passing XCB_WINDOW_NONE (of type
xcb_window_enum_t) as an argument where these functions expect a void *,
which clang does not appreciate.
This patch cleans things up to better convince me and reviewers that
it's safe to do that.
v2: Emil Veliko
From: Emil Velikov
As mentioned in previous commit the negative tests in dEQP expect the
arguments to be evaluated in particular order.
Namely - first the dpy, then the config, followed by the surface/window.
Move the check further down or executing the test below will produce
the following err
From: Emil Velikov
The function can be called only when the type is EGL_WINDOW_BIT.
Remove the unneeded switch statement.
Signed-off-by: Emil Velikov
---
src/egl/drivers/dri2/platform_drm.c | 20 +++-
1 file changed, 7 insertions(+), 13 deletions(-)
diff --git a/src/egl/driver
From: Emil Velikov
The current two implementations - X11 and Wayland were identical,
barrind the upper limit.
Instead of having same code twice - introduce a helper and pass the
limit as an argument.
Thus as Android/DRM/others get support - they only need to call the
function ;-)
v2: Rebase on
BLEND_STATE packing was modified to be variable-length in:
9670124e31 genxml: Make BLEND_STATE command support variable length array.
The initial gen10.xml still had the old, fixed-length style
definition for BLEND_STATE. So gen10_upload_blend_state would
overwrite the packed BLEND_STATE_ENTRYs
For the series:
Tested-by: Dieter Nützel
on RX580
with Clover, vdpau and Nine.
./autogen.sh --prefix=/usr/local --with-dri-drivers=""
--with-gallium-drivers=r600,radeonsi,swrast --with-platforms=drm,x11
--enable-nine --enable-texture-float --enable-opencl
--with-vulkan-drivers=radeon
Die
On 4 August 2017 at 11:03, Eric Engestrom wrote:
> On Thursday, 2017-08-03 19:29:36 +0100, Emil Velikov wrote:
>> From: Emil Velikov
>>
>> The function can be called only when the type is EGL_WINDOW_BIT.
>> Remove the unneeded switch statement.
>
> I take it we plan on never supporting pbuffers o
On 2017-08-02 13:35:33, Jason Ekstrand wrote:
> Originally, I had moved it to the caller to make some things easier when
> adding the CCS modifier. However, this broke DRI2 because
> intel_process_dri2_buffer calls intel_miptree_create_for_bo but never
> calls intel_miptree_alloc_aux. Also, in hi
---
src/mesa/drivers/dri/i965/brw_bufmgr.c | 36 +++---
1 file changed, 25 insertions(+), 11 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.c
b/src/mesa/drivers/dri/i965/brw_bufmgr.c
index e1036f25a4..844ccaf1e5 100644
--- a/src/mesa/drivers/dri/i965/b
Lots of places open-coded the assumed layout of the predicate/results
within the query object, replace those with simple helpers.
v2: Fix function decl style.
Signed-off-by: Chris Wilson
Cc: Kenneth Graunke
Cc: Matt Turner
---
src/mesa/drivers/dri/i965/brw_conditional_render.c | 10 --
Ony non-llc architectures where we are primarily reading back the
results of the GPU queries, then we can improve performance by using a
cacheable mapping of the results. Unfortunately, enabling snooping makes
the writes from the GPU slower, which may adversely affect pipelined
query operations (wh
Reuse the same query object buffer for multiple queries within the same
batch.
A task for the future is propagating the GL_NO_MEMORY errors.
Signed-off-by: Chris Wilson
Cc: Kenneth Graunke
Cc: Matt Turner
---
src/mesa/drivers/dri/i965/brw_context.c | 4 +++
src/mesa/drivers/dri/i965/brw_co
If we map the bo upon creation, we can avoid the latency of mmapping it
when querying, and later use the asynchronous, persistent map of the
predicate to do a quick query.
v2: Inline the wait on results; it disappears shortly in the next few
patches.
Signed-off-by: Chris Wilson
Cc: Kenneth Graun
If we always write the 'available' flag after writing the final result
of the query, we can probe that predicate to quickly query whether the
result is ready from userspace. The primary advantage of checking the
predicate is that it allows for more fine-grained queries, we do not
have to wait for t
Reviewed-by: Jordan Justen
On 2017-08-02 13:35:26, Jason Ekstrand wrote:
> The docs contain a bunch of commentary about the need to pad various
> surfaces out to multiples of something or other. However, all of those
> requirements are about avoiding GTT errors due to missing pages when the
> da
If we know the bo is idle (that is we have no submitted a command buffer
referencing this bo since the last query) we can skip asking the kernel.
Note this may report a false negative if the target is being shared
between processes (exported via dmabuf or flink). To allow the caller
control over us
Avoid having to clflush after blitting the miptree to a linear buffer
for mapping by enabling snooping on !llc and treating the buffer as
coherent. Similarly, it avoids the clflush afterwards if used for
READ | WRITE.
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 17 +++--
1 file
Be consistent in passing along brw_context rather than switching between
that and gl_context.
Signed-off-by: Chris Wilson
---
src/mesa/drivers/dri/i965/gen6_queryobj.c | 32 ++-
1 file changed, 14 insertions(+), 18 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/
To simplify replacement later, replace repeated use of explicit 0/1 with
local variables of the same value.
Signed-off-by: Chris Wilson
Cc: Kenneth Graunke
Cc: Matt Turner
---
src/mesa/drivers/dri/i965/gen6_queryobj.c | 30 --
1 file changed, 16 insertions(+), 14 de
If we need to stall to read the bo, ask the GPU to copy it into the CPU
cache whilst we wait.
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
b/src/mesa/drivers/dri/i
Skip the next check for brw_batch_references() by recording when we
flush the query.
---
src/mesa/drivers/dri/i965/gen6_queryobj.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen6_queryobj.c
b/src/mesa/drivers/dri/i965/gen6_queryobj.c
i
Quoting Kenneth Graunke (2017-08-04 19:47:14)
> On Friday, July 21, 2017 8:36:42 AM PDT Chris Wilson wrote:
> > Patch reordering from last time so that the cosmetic tweaks are done first
> > and out of the way. Kenneth has reviewed the core NO_RELOC patches, so
> > hopefully it doesn't look too bad
Ken and I had a fairly lengthy conversation about this on IRC today:
https://people.freedesktop.org/~jekstrand/isl-padding
The conclusion was that we both hate the patch but it's probably safe and
it does fix bugs. The thing that really wins me over is that we have
historically done none of this
On 4 August 2017 at 14:23, Tomasz Figa wrote:
>
> If this needs so complicated series of checks, maybe it would make
> more sense to just set enable_out_fence based on availability of the
> capability at initialization time?
>
Either way is fine with me.
>> Did you drop it all together or change
Hi,
I went through most of the series. I think the approach is OK. The
biggest issue I had is with the sequence:
1.) add an interface
2.) implement a feature
3.) change the interface
I gave my rb to 1 and 2, but you might want to consider changing them
as well, if returning int from the functions
On Friday, July 21, 2017 8:36:42 AM PDT Chris Wilson wrote:
> Patch reordering from last time so that the cosmetic tweaks are done first
> and out of the way. Kenneth has reviewed the core NO_RELOC patches, so
> hopefully it doesn't look too bad and we can land at least as far as
> there (patch 8/1
On Sun, 2017-07-30 at 20:26 -0500, Aaron Watry wrote:
> Signed-off-by: Aaron Watry
> CC: Jan Vesely
>
> v2: base it on the device version
> ---
> src/gallium/state_trackers/clover/llvm/invocation.cpp | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/state_t
On 04.08.2017 20:18, Emil Velikov wrote:
From: Emil Velikov
Currently xmlconfig is conditionally used, only when --enable-dri is
available.
As the library has moved to src/util and has wider wisebase, this guard
is no longer correct. Strictly speaking - it wasn't since the
introduction of xmlc
On Sun, 2017-07-30 at 20:26 -0500, Aaron Watry wrote:
> Copying the individual fields from the device when compiling/linking
> will lead to an unnecessarily large number of fields getting passed
> around.
>
> Signed-off-by: Aaron Watry
> Cc: Jan Vesey
I think this should be patch 3/8. It looks
On Sun, 2017-07-30 at 20:26 -0500, Aaron Watry wrote:
> According to section 5.8.4.5 of the 2.0 spec, the CL C version is chosen by:
> 1) If you have -cl-std=CL1.1+ use the version specified
> 2) If not, use the highest 1.x version that the device supports
>
> Curiously, there is no valid value
On 4 August 2017 at 10:53, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> This aligns with the fact that we also check for EXPAT_LIBS
> unconditionally in configure.ac now. It should make all the
> various build permutations of Clover work (whether DRI is
> enabled or disabled in the build).
>
From: Emil Velikov
Brings the Android binaries on par with Autoconf, allowing users to
select their GPU via device_id.
Signed-off-by: Emil Velikov
---
Completely untested. Posting if anyone is interested if polishing it up.
---
src/loader/Android.mk | 3 +++
1 file changed, 3 insertions(+)
di
From: Emil Velikov
Brings the SCons binaries on par with Autoconf, allowing users to select
their GPU via device_id.
Signed-off-by: Emil Velikov
---
Completely untested. Posting if anyone is interested if polishing it up.
---
src/loader/SConscript | 4
1 file changed, 4 insertions(+)
dif
From: Emil Velikov
Currently xmlconfig is conditionally used, only when --enable-dri is
available.
As the library has moved to src/util and has wider wisebase, this guard
is no longer correct. Strictly speaking - it wasn't since the
introduction of xmlconfig into st/nine a while ago.
Unconditio
On Sun, 2017-07-30 at 20:26 -0500, Aaron Watry wrote:
> We'll be using it to select the default language version soon.
>
> Signed-off-by: Aaron Watry
> Cc: Pierre Moreau
> Cc: Jan Vesely
>
> v2: (Pierre) Move changes to create_compiler_instance invocation to correct
> patch to prevent temp
On Sun, 2017-07-30 at 20:26 -0500, Aaron Watry wrote:
> device_version and device_clc_version are not necessarily the same for
> devices that support CL 1.0, but have a 1.1 compiler and the necessary
> extensions.
>
> CC: Jan Vesey
I think you might consider squashing 1/8 and 2/8. squashed or no
On Sun, 2017-07-30 at 20:26 -0500, Aaron Watry wrote:
> The device version is the maximum CL version that the device supports.
>
> Eventually, this will be based on the features/extensions of the actual
> device, but for now move it a bit closer to its eventual destination.
>
> Signed-off-by: Aar
textdata bss dec hex filename
7816886 346248 420496 8583630 82f9ce i965_dri.so before
7816214 346248 420496 8582958 82f72e i965_dri.so after
---
src/intel/compiler/brw_reg_type.h | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/intel/compiler/
---
src/intel/compiler/brw_disasm.c | 47 -
src/intel/compiler/brw_eu_validate.c | 196 ---
src/intel/compiler/brw_reg_type.c| 17 +--
src/intel/compiler/brw_reg_type.h| 11 +-
4 files changed, 113 insertions(+), 158 deletions(-)
diff --git
brw_hw_type_to_reg_type() needs to know only whether the file is
BRW_IMMEDIATE_VALUE or not, which is not a valid file for the
destination. gcc and clang will evaluate __builtin_strcmp() at compile
time, so we can use it to pass a constant file for the destination.
textdata bss dec
So we stop mixing them with the logical enum.
---
src/intel/compiler/brw_eu_defines.h | 31 ---
src/intel/compiler/brw_reg_type.c | 31 +++
2 files changed, 31 insertions(+), 31 deletions(-)
diff --git a/src/intel/compiler/brw_eu_defines.h
And add "to_" to the name for consistency with the other functions in
this file.
---
src/intel/compiler/brw_eu.c | 28
src/intel/compiler/brw_fs.cpp | 4 ++--
src/intel/compiler/brw_reg.h | 1 -
src/intel/compiler/brw_reg_type.c | 30 ++
---
src/intel/compiler/brw_disasm.c | 72 ++-
src/intel/compiler/brw_reg_type.c | 8 +
src/intel/compiler/brw_reg_type.h | 4 +++
3 files changed, 45 insertions(+), 39 deletions(-)
diff --git a/src/intel/compiler/brw_disasm.c b/src/intel/compiler/brw_di
---
src/intel/compiler/brw_eu_compact.c | 27 ---
src/intel/compiler/brw_eu_emit.c| 13 +++--
2 files changed, 19 insertions(+), 21 deletions(-)
diff --git a/src/intel/compiler/brw_eu_compact.c
b/src/intel/compiler/brw_eu_compact.c
index 743ee9519c..7674aa8b85
Put hw_ in the name so that it's clear these are the hardware encodings.
---
src/intel/compiler/brw_disasm.c | 22
src/intel/compiler/brw_eu_compact.c | 22
src/intel/compiler/brw_eu_emit.c| 18 +++
src/intel/compiler/brw_eu_validate.c| 28 +-
Previously the brw_inst{,_set}_{dst,src0,src1}_reg_type() functions
provided access to the hardware encodings for the register types. We
often mixed these with the logical BRW_REGISTER_TYPE_* enums (which
themselves used to be the hardware format!) with bad results.
With that functionality now ava
Will be used in later commits.
---
src/intel/compiler/brw_reg_type.c | 25 +
src/intel/compiler/brw_reg_type.h | 4
2 files changed, 29 insertions(+)
diff --git a/src/intel/compiler/brw_reg_type.c
b/src/intel/compiler/brw_reg_type.c
index b0696570e5..8da93ae1cb 1006
I'll be transitioning everything to use the logical types.
---
src/intel/compiler/brw_reg_type.c | 58 +--
1 file changed, 19 insertions(+), 39 deletions(-)
diff --git a/src/intel/compiler/brw_reg_type.c
b/src/intel/compiler/brw_reg_type.c
index 8da93ae1cb..85
I'm going to encapsulate all of the logic dealing with register types in
this file.
Rename the parameters for the hardware encodings from type -> hw_type at
the same time.
---
src/intel/Makefile.sources| 2 +
src/intel/compiler/brw_eu_emit.c | 102 --
src/in
---
src/intel/compiler/brw_reg_type.c | 65 +--
1 file changed, 29 insertions(+), 36 deletions(-)
diff --git a/src/intel/compiler/brw_reg_type.c
b/src/intel/compiler/brw_reg_type.c
index 8aac0ca009..b0696570e5 100644
--- a/src/intel/compiler/brw_reg_type.c
+++
I think of the initial arguments as "state" and the last as the actual
subject.
---
src/intel/compiler/brw_disasm.c | 4 ++--
src/intel/compiler/brw_eu_emit.c | 14 --
src/intel/compiler/brw_eu_validate.c | 2 +-
src/intel/compiler/brw_reg.h | 8
4 files ch
After the last patch converted things into enums, I helpfully got a
compiler warning about these missing from the switch statement.
---
src/intel/compiler/brw_disasm.c | 6 ++
src/intel/compiler/brw_inst.h | 7 +++
2 files changed, 13 insertions(+)
diff --git a/src/intel/compiler/brw_di
UB and B type encodings are the same as UV and VF. Noticed when writing
the following patch.
---
src/intel/compiler/brw_eu_validate.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/src/intel/compiler/brw_eu_validate.c
b/src/intel/compiler/brw_eu_validate.c
inde
The hardware encodings often mean different things depending on whether
the source is an immediate.
---
src/intel/compiler/brw_disasm.c | 46 ---
src/intel/compiler/brw_eu_compact.c | 8 +--
src/intel/compiler/brw_eu_defines.h | 48 +--
src/intel/compiler/brw_eu_
These vaguely corresponded to the hardware encodings, but that is purely
historical at this point. Reorder them so we stop making things "almost
work" when mixing enums.
The ordering has been closen so that no enum value is the same as a
compatible hardware encoding.
---
src/intel/compiler/brw_eu
The destination stride must be equivalent to a dword if VF is used.
Also, since the only compaction table entires with "i:vf" have the
destination as "r:f" specifically check that the destination is of type
float.
---
src/intel/compiler/brw_eu_compact.c | 3 ++-
1 file changed, 2 insertions(+), 1
These cannot be compacted. A similar mistake was fixed in commit
90eaf01616a8
---
src/intel/compiler/test_eu_compact.cpp | 4
1 file changed, 4 deletions(-)
diff --git a/src/intel/compiler/test_eu_compact.cpp
b/src/intel/compiler/test_eu_compact.cpp
index 668a972bfa..1532e3b984 100644
--- a
---
src/intel/compiler/brw_eu_emit.c| 13 +-
src/intel/compiler/brw_eu_validate.c| 61 +
src/intel/compiler/test_eu_validate.cpp | 79 +
3 files changed, 141 insertions(+), 12 deletions(-)
diff --git a/src/intel/compiler/brw_
Note that there's no point in testing on G45, since its compaction is
the same as Gen5. Same logic applies to Gen7 variants and low-power
parts.
---
src/intel/compiler/test_eu_compact.cpp | 50 --
1 file changed, 42 insertions(+), 8 deletions(-)
diff --git a/src/in
---
src/intel/compiler/brw_eu.h | 2 +-
src/intel/compiler/brw_eu_compact.c | 23 ---
2 files changed, 13 insertions(+), 12 deletions(-)
diff --git a/src/intel/compiler/brw_eu.h b/src/intel/compiler/brw_eu.h
index a3a9c63239..8e597b212a 100644
--- a/src/intel/compiler
The mixture of hardware encodings and logical types has caused lots of
confusion. It's time to fix that.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
---
src/intel/compiler/test_eu_compact.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/compiler/test_eu_compact.cpp
b/src/intel/compiler/test_eu_compact.cpp
index 39e7f1a27c..1ef7e5ae7f 100644
--- a/src/intel/compiler/test_eu_compact.cpp
+++ b/src/intel/compiler/
---
src/intel/compiler/brw_eu_compact.c | 82 -
src/intel/compiler/brw_eu_emit.c| 72 +---
2 files changed, 82 insertions(+), 72 deletions(-)
diff --git a/src/intel/compiler/brw_eu_compact.c
b/src/intel/compiler/brw_eu_compact.c
---
src/intel/vulkan/anv_batch_chain.c | 59 +++
src/intel/vulkan/anv_device.c | 1 +
src/intel/vulkan/anv_private.h | 8
src/intel/vulkan/anv_queue.c | 83 +++---
4 files changed, 128 insertions(+), 23 deletions(-)
dif
---
src/intel/vulkan/anv_extensions.py | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/intel/vulkan/anv_extensions.py
b/src/intel/vulkan/anv_extensions.py
index 00186bc..3252e0f 100644
--- a/src/intel/vulkan/anv_extensions.py
+++ b/src/intel/vulkan/anv_extensions.py
---
src/intel/vulkan/anv_gem.c | 52
src/intel/vulkan/anv_gem_stubs.c | 24 +++
src/intel/vulkan/anv_private.h | 4
3 files changed, 80 insertions(+)
diff --git a/src/intel/vulkan/anv_gem.c b/src/intel/vulkan/anv_gem.c
index 5
Reviewed-by: Chad Versace
---
src/intel/vulkan/anv_gem.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/intel/vulkan/anv_gem.c b/src/intel/vulkan/anv_gem.c
index ac47da4..36692f5 100644
--- a/src/intel/vulkan/anv_gem.c
+++ b/src/intel/vulkan/anv_gem.c
@@ -185,7 +185,1
---
include/drm-uapi/i915_drm.h | 30 --
1 file changed, 28 insertions(+), 2 deletions(-)
diff --git a/include/drm-uapi/i915_drm.h b/include/drm-uapi/i915_drm.h
index c26bf7c..338c8c2 100644
--- a/include/drm-uapi/i915_drm.h
+++ b/include/drm-uapi/i915_drm.h
@@ -431,6
---
src/intel/vulkan/anv_batch_chain.c | 57 +--
src/intel/vulkan/anv_device.c | 1 +
src/intel/vulkan/anv_gem.c | 36
src/intel/vulkan/anv_private.h | 23 +
src/intel/vulkan/anv_queue.c | 69 +
Vulkan allows you to do a submit whose only job is to wait on and
trigger semaphores. The easiest way for us to support that right
now is to insert a dummy execbuf.
---
src/intel/vulkan/anv_batch_chain.c | 28 +---
src/intel/vulkan/anv_device.c | 30 ++
This patch adds an implementation based on DRM BOs. We don't actually
advertise the extension yet because we want to add a couple more paths
first.
---
src/intel/vulkan/anv_batch_chain.c | 31 +++-
src/intel/vulkan/anv_extensions.py | 3 +
src/intel/vulkan/anv_private.h | 3 +
src/in
This series is a quick re-spin of the v2 sent yesterday to address review
feedback from Chris. In particular, we now set EXEC_ASYNC on the trivial
batch and I deleted the syncobj cache. Somehow, when I was working on this
yesterday, I got it into my head that the kernel deduplicates syncobj
handl
The configuration option --with-sha1 is no longer required for the
MESA_SHADER_READ_PATH, MESA_SHADER_DUMP_PATH environment variables
to take effect.
1- removed the "--with-sha1" sentence from docs/shading.html
2- added an extra note: that the corresponding dumped and replacement
shaders must have
Thanks. I wrote the same patch some time ago, but never had a chance
to send it out. I'll send you another patch that I wrote to clear up
some of this confusion. I put it in my series immediately before this
patch. Feel free to add it to yours.
Reviewed-by: Matt Turner
___
The next patch is going to stop passing XCB_WINDOW_NONE (of type
xcb_window_enum_t) as an argument where these functions expect a void *,
which clang does not appreciate.
This patch cleans things up to better convince me and reviewers that
it's safe to do that.
---
src/egl/drivers/dri2/platform_x
https://bugs.freedesktop.org/show_bug.cgi?id=102038
--- Comment #2 from Brian Paul ---
Hmm, swr isn't working at all for me. It's hanging in a swr_fence_finish()
call with everything I've tried. Even glxinfo hangs (but elsewhere).
Maybe one of the swr developers can take a look.
--
You are r
https://bugs.freedesktop.org/show_bug.cgi?id=102038
--- Comment #1 from Brian Paul ---
Sorry for the breakage, Brad.
I'll try to investigate ASAP, but I'm about to leave town for a week.
I can't repro with llmvpipe, fwiw. I suspect it may be an issue in the swr
driver. I'm building it now, bu
On Fri, 2017-08-04 at 18:15 +0200, Wladimir J. van der Laan wrote:
> On Fri, Aug 04, 2017 at 05:07:54PM +0200, Philipp Zabel wrote:
> > Look up BOs from the name table using the name parameter instead of
> > req.handle (which at this point is always zero).
>
> Good catch.
>
> Just out of interest
On Fri, Aug 4, 2017 at 2:03 AM, Chris Wilson
wrote:
> Quoting Jason Ekstrand (2017-08-04 02:25:27)
> > @@ -1497,6 +1569,12 @@ anv_cmd_buffer_execbuf(struct anv_device *device,
> >setup_empty_execbuf(&execbuf, device);
> > }
> >
> > + if (execbuf.fence_count > 0) {
>
> For sanity, si
On Fri, Aug 4, 2017 at 1:59 AM, Chris Wilson
wrote:
> Quoting Jason Ekstrand (2017-08-04 02:25:26)
> > This is mostly a copy+paste of the BO cache but it's a bit simpler
> > because syncobjs don't have actual backing storage so we don't need to
> > check sizes or anything like that. Also, we put
On 2017-08-03 22:26, Alex Deucher wrote:
IIRC, user_ptrs require page alignment.
Alex
I didn't follow the whole discussion (sorry if I'm saying something
redundant), but AMD's older OpenCL Optimization Guide [1] has some notes
regarding the implementation of the USE_HOST_PTR flag.
It initi
> -Original Message-
> From: Tomasz Figa [mailto:tf...@chromium.org]
> Sent: Friday, August 4, 2017 9:39 PM
> On Sat, Aug 5, 2017 at 12:53 AM, Marathe, Yogesh
> wrote:
> > Tomasz, Emil,
> >
> >> -Original Message-
> >> From: Tomasz Figa [mailto:tf...@chromium.org]
> >> On Fri, Aug
On Fri, Aug 4, 2017 at 1:43 AM, Chris Wilson
wrote:
> Quoting Jason Ekstrand (2017-08-04 02:25:21)
> > Vulkan allows you to do a submit whose only job is to wait on and
> > trigger semaphores. The easiest way for us to support that right
> > now is to insert a dummy execbuf.
> > ---
> > diff --g
On Fri, Aug 04, 2017 at 05:07:54PM +0200, Philipp Zabel wrote:
> Look up BOs from the name table using the name parameter instead of
> req.handle (which at this point is always zero).
Good catch.
Just out of interest: when is this used, what problems does this cause?
Regards,
Wladimir
__
On Sat, Aug 5, 2017 at 12:53 AM, Marathe, Yogesh
wrote:
> Tomasz, Emil,
>
>> -Original Message-
>> From: Tomasz Figa [mailto:tf...@chromium.org]
>> Sent: Friday, August 4, 2017 6:54 PM
>> To: Emil Velikov
>> Cc: Marathe, Yogesh ; Antognolli, Rafael
>> ; ML mesa-dev > d...@lists.freedeskto
On Fri, 2017-08-04 at 11:53 +0200, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> This aligns with the fact that we also check for EXPAT_LIBS
> unconditionally in configure.ac now. It should make all the
> various build permutations of Clover work (whether DRI is
> enabled or disabled in the bu
1 - 100 of 171 matches
Mail list logo
