On Sat, Nov 05, 2016 at 11:48:57PM +0100, Alejandro Piñeiro wrote:
> On 05/11/16 19:57, Chris Wilson wrote:
> > Currently we signal the availabilty of the query result using an
> typo: availability (this also affects commit message)
> > unordered pipe-control write. As it is unordered, it may be ex
https://bugs.freedesktop.org/show_bug.cgi?id=98774
Timothy Arceri changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=98632
Kenneth Graunke changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
Add functions to add and check targets/components.
Not used in this patch.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 36
1 file changed, 36 insertions(+)
diff --git a/configure.ac b/configure.ac
index 5a11798..0c955bf 100644
There's no harm in always searching llvm-config.
This way it's available as soon as possible for all functions.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/configure.ac b/configure.a
LLVM components are only added when LLVM is needed.
This means gallium adds this as soon as "--enable-gallium-llvm"
is "yes" and radv + opencl add it explicitly.
v5:
Removed hunk that disabled LLVM for gallium if it was not found.
Signed-off-by: Tobias Droste
---
configure.ac | 15 -
Use the same order as llvm_check_version_for.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/configure.ac b/configure.ac
index 5b299d0..5042bf9 100644
--- a/configure.ac
+++ b/configure
The LLVM checks can be anywhere below line 1161 now.
Move the openCL LLVM checks to the section with the other openCL checks.
No functional change.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 41 +++--
1 file changed, 19 insertio
This moves the function to get the LLVM environment variables higher
in the file. It still needs to be below the "--enable-opencl" because
it uses $enable_opencl.
It can be called without condition now as it only throws errors if
openCL is enabled.
v5:
HAVE_MESA_LLVM is only used for gallium. Rena
No need for an additional function here.
Use the same style for LLVM checks as the other drivers
(e.g. r300, llvmpipe) that don't need a load of other checks.
Instead of open conding the LLVM version check, use the
function used by other drivers.
"enable_gallium_llvm" is checked by gallium_require
This moves llvm_set_environment_variables to its final destination
and moves all the LLVM AC_SUBST() below the function call.
No functional change.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 29 ++---
1 file changed, 14 insertions(+), 15 de
Patches 1-11 and 13-16 are only rebased, no change in functionality and have a
reviewed by.
Patch 12 is changed based on review of Emil (I used "HAVE_GALLIUM_LLVM" instead
of "USE_MESA_LLVM").
Patch 17 is changed based on review of Emil
Patch 18 is split in 18, 19 and 20 as these were independen
This is handled by "llvm_check_version_for" for openCL.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 4
1 file changed, 4 deletions(-)
diff --git a/configure.ac b/configure.ac
index 6f9ebed..a511c68 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2296,10 +229
This moves the LLVM check for radv to the corresponding driver section.
No functional change.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/configure.ac b/configure.ac
index 00a1791..5b299d0 100644
Major and minor version are included in the header file since LLVM
version 3.1.0. Since the minimal required version is 3.3.0 we can
remove the workaround if no values for major/minor were found in the
header.
Since LLVM 3.6.0 the patch version is inside the header file of LLVM.
Only radeon driver
A function with the LLVM version checked is moved to the top.
The function is called where the old code was.
No functional change.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 144 ++-
1 file changed, 74 insert
This way LLVM_CONFIG can bet set from an env variable if it's outside
the $llvm_prefix.
This is not a must, but it helps testing.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/configure.ac
This moves the LLVM version check to the helper function
gallium_require_llvm() and uses the llvm_check_version_for() helper
instead of open conding the LLVM version check.
gallium_require_llvm is functionally the same as before, because
"enable_gallium_llvm" is only set to "yes" if the host cpu i
This makes sure that we handle LLVM minor version >= 10 correctly.
Signed-off-by: Tobias Droste
---
configure.ac | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index 5fc2c87..4cdd0cc 100644
--- a/configure.ac
+++ b/configure.ac
@@ -990,7 +990
This just moves code around so that all LLVM related stuff is at the
top of the file in the correct order.
No functional change.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 151 +--
1 file changed, 74 insertio
Consolidate the required LLVM versions at the top where the other
versions for dependencies are listed.
v5:
Splitted out separate changes (see patch 19 and 20)
Signed-off-by: Tobias Droste
---
configure.ac | 68 +++-
1 file changed, 54 ins
There are no non gallium r300 and r600 drivers anymore.
No need to explicilty mention gallium here.
Just cosmetics, no functional change.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/configure
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/configure.ac b/configure.ac
index 0c955bf..4f98c19 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2242,11 +2242,7 @@ if test "x$enable_gallium_llvm"
Use the new helper function to add LLVM targets and components.
The components are added one by one to later find out which component
is missing in case there is one.
Signed-off-by: Tobias Droste
Reviewed-by: Emil Velikov
---
configure.ac | 25 ++---
1 file changed, 18 inser
On Sat, Nov 19, 2016 at 1:13 AM, Tom Stellard wrote:
> On Sat, Nov 19, 2016 at 01:09:00AM +0100, Marek Olšák wrote:
>> On Wed, Nov 16, 2016 at 4:38 PM, Tom Stellard wrote:
>> > On Wed, Nov 16, 2016 at 11:13:45AM +0100, Nicolai Hähnle wrote:
>> >> Have you looked at the shader-db impact?
>> >>
>>
On Sat, 2016-11-19 at 11:14 +1100, Timothy Arceri wrote:
> This fixes a regression with the standalone compiler caused by
> 9d96d3803ab5dc
I forgot to add this:
Note that we change standalone_compiler_cleanup() to no longer
explicitly free the linked shaders as the will be freed when
we free the
https://bugs.freedesktop.org/show_bug.cgi?id=98774
--- Comment #1 from Timothy Arceri ---
Thanks, fix sent for review:
https://lists.freedesktop.org/archives/mesa-dev/2016-November/135829.html
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for
On Sat, Nov 19, 2016 at 01:09:00AM +0100, Marek Olšák wrote:
> On Wed, Nov 16, 2016 at 4:38 PM, Tom Stellard wrote:
> > On Wed, Nov 16, 2016 at 11:13:45AM +0100, Nicolai Hähnle wrote:
> >> Have you looked at the shader-db impact?
> >>
> >
> > shader-db is mostly unchanged. There are a few decreas
This fixes a regression with the standalone compiler caused by
9d96d3803ab5dc
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98774
---
src/compiler/glsl/standalone.cpp | 61
1 file changed, 43 insertions(+), 18 deletions(-)
diff --git a/src/compil
On Wed, Nov 16, 2016 at 4:38 PM, Tom Stellard wrote:
> On Wed, Nov 16, 2016 at 11:13:45AM +0100, Nicolai Hähnle wrote:
>> Have you looked at the shader-db impact?
>>
>
> shader-db is mostly unchanged. There are a few decreases in SGPR usage and
> code size, and a 4 byte increase in code size for
https://bugs.freedesktop.org/show_bug.cgi?id=98740
Alejandro Vilicic changed:
What|Removed |Added
CC||alejandro.vilicich@inacapma
On Wed, Nov 16, 2016 at 10:23 PM, Tom Stellard wrote:
> cd724208d3e1e3307f84a794f2c1fc83b69ccf8a added a call to
> _mesa_lock_debug_state(ctx) but wasn't unlocking the debug state.
>
> This fixes a hang in glsl-fs-loop piglit test with MESA_DEBUG=context.
> ---
> src/gallium/drivers/radeonsi/si_p
Reviewed-by: Marek Olšák
Marek
On Thu, Nov 17, 2016 at 11:00 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Mark variables and static functions that only occur in assert()s as
> MAYBE_UNUSED.
> ---
> src/mesa/state_tracker/st_sampler_view.c | 4 ++--
> 1 file changed, 2 insertions(+), 2
Reviewed-by: Marek Olšák
Marek
On Fri, Nov 18, 2016 at 8:22 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> For compute shaders, we free the selector after the shader has been
> compiled, so we need to save this bit somewhere else. Also, make sure that
> this type of bug cannot re-appear
Reviewed-by: Marek Olšák
Marek
On Wed, Nov 16, 2016 at 7:46 PM, Emil Velikov wrote:
> From: Emil Velikov
>
> Currently things are a bit buried within the text, making it harder to
> find out. Move at the top and be clear what is _not_ a good idea.
>
> We had some people consistently using the
Mesa master builds again against LLVM 3.6.
Tested-by: Pierre Moreau
On 07:57 pm - Nov 18 2016, Vedran Miletić wrote:
> The commit 8e430ff8b060b4e8e922bae24b3c57837da6ea77 support for LLVM
> 3.9 and older versionsin Clover. This patch restores it and refactors
> the support using Clover compatib
On Fri, Nov 18, 2016 at 11:17 PM, Rob Clark wrote:
> On Fri, Nov 18, 2016 at 5:07 PM, Marek Olšák wrote:
>> On Fri, Nov 18, 2016 at 2:39 PM, Rob Clark wrote:
>>> From: Rob Clark
>>>
>>> This will be needed for explicit synchronization with devices outside
>>> the gpu, ie. EGL_ANDROID_native_fen
On Fri, Nov 18, 2016 at 5:07 PM, Marek Olšák wrote:
> On Fri, Nov 18, 2016 at 2:39 PM, Rob Clark wrote:
>> From: Rob Clark
>>
>> This will be needed for explicit synchronization with devices outside
>> the gpu, ie. EGL_ANDROID_native_fence_sync.
>>
>> Signed-off-by: Rob Clark
>> Reviewed-by: Ma
https://bugs.freedesktop.org/show_bug.cgi?id=98774
Bug ID: 98774
Summary: glsl/tests/warnings-test regression
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: All
Status: NEW
Keywords: bisected, hav
On Fri, Nov 18, 2016 at 2:39 PM, Rob Clark wrote:
> From: Rob Clark
>
> This will be needed for explicit synchronization with devices outside
> the gpu, ie. EGL_ANDROID_native_fence_sync.
>
> Signed-off-by: Rob Clark
> Reviewed-by: Marek Olšák
> ---
> src/gallium/include/pipe/p_context.h | 6
On Fri, 2016-11-18 at 20:35 +, Emil Velikov wrote:
> On 11 November 2016 at 00:45, Timothy Arceri
> wrote:
> >
> > This will be used to share data between gl_program and
> > gl_shader_program
> > allowing for greater code simplification as we can remove a number
> > of
> > awkward uses of gl_
On Fri, Nov 18, 2016 at 10:57 AM, Vedran Miletić wrote:
> The commit 8e430ff8b060b4e8e922bae24b3c57837da6ea77 support for LLVM
> 3.9 and older versionsin Clover. This patch restores it and refactors
> the support using Clover compatibility layer for LLVM.
>
> Signed-off-by: Vedran Miletić
> ---
From: Boyuan Zhang
When using dual instance and rate control, driver needs to submit jobs either
in dual submissions or 2 consecutive single submissions to keep the pattern
constant for rate control
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=98005
Signed-off-by: Boyuan Zhang
---
src/
On Fri, Nov 18, 2016 at 11:54 AM Jason Ekstrand
wrote:
> From: Jason Ekstrand
>
> The original aubinator that Kristian wrote had a bug in the handling of
> MI_BATCH_BUFFER_START that propagated into the version in upstream mesa.
> In particular, it ignored the "2nd level" bit which tells you whe
From: Boyuan Zhang
The gop_size in rate control is the budget window for internal rate
control calculation, and shouldn't always equal to idr period. Define
a coefficient to let budget window contains a number of idr period for
proper rate control calculation. Adjust the number of i/p frame rema
Hi Tim,
In general we could use the odd local variable to make things shorter
(and cut down the number of derefs).
That can (should?) be done once we're finished with the big churn.
On 11 November 2016 at 00:45, Timothy Arceri
wrote:
> @@ -296,6 +296,8 @@ init_shader_program(struct gl_shader_pr
v2: reworded commit message
Reviewed-by: Emil Velikov
---
scons/llvm.py | 21 +++--
1 file changed, 19 insertions(+), 2 deletions(-)
diff --git a/scons/llvm.py b/scons/llvm.py
index 1fc8a3f..977e47a 100644
--- a/scons/llvm.py
+++ b/scons/llvm.py
@@ -106,7 +106,24 @@ def generate
To buils The SWR driver (currently optional, not compiled by default)
v3: add option as opposed to target
Reviewed-by: Emil Velikov
---
common.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/common.py b/common.py
index fb0884e..704ad2e 100644
--- a/common.py
+++ b/common.py
@@ -110,5 +11
Drivers that contain C++ .hpp files need to ignore them too, along
with .h files, when building source file lists.
Reviewed-by: Emil Velikov
---
scons/custom.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scons/custom.py b/scons/custom.py
index bdb4039..544b15d 100644
---
- Handle dynamic library loading for windows
- Implement swap for gdi
- fix prototypes
- update include paths on configure-based build for swr_loader.cpp
v2: split to multiple patches
v3: split and reshuffle some more; renamed title
v4: move Makefile.am changes to other commit. Modify header files
Modify gen_knobs.py so that each invocation creates a single generated
file. This is more similar to how the other generators behave.
v5: remove Scoscript edits from this commit; moved to commit that first
adds SConscript
Acked-by: Emil Velikov
---
src/gallium/drivers/swr/Makefile.am
Reviewed-by: Emil Velikov
---
src/util/macros.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/util/macros.h b/src/util/macros.h
index 733bf42..6f55ac6 100644
--- a/src/util/macros.h
+++ b/src/util/macros.h
@@ -178,7 +178,6 @@ do { \
# elif defined(_MSC_VER) && !d
Include swr library and include -DHAVE_SWR in the compile line.
v3: split to a separate commit
Reviewed-by: Emil Velikov
---
src/gallium/targets/libgl-gdi/SConscript | 4
src/gallium/targets/libgl-xlib/SConscript | 4
src/gallium/targets/osmesa/SConscript | 4
3 files change
v4: Add windows-specific gen_knobs.{cpp|h} changes
v5: remove aggresive squashing of gen_knobs.py to this commit
Reviewed-by: Emil Velikov
---
src/gallium/SConscript | 1 +
src/gallium/drivers/swr/SConscript | 216 +
2 files changed, 217 insertio
There are 2 swr_create_screen() functions. One in swr_loader.cpp, which
is used during driver init, and the other is hiding in swr_screen.cpp,
which ends up in the arch-specific .dll/.so.
Rename the second one to swr_create_screen_internal(), to avoid confusion
in header files.
---
src/gallium/d
Changes to support Windows scons builds for OpenSWR driver, since scons
is the only supported build system for windows.
Scons swr build will not work at this point. Also, windows scons swr build
requires llvm version 3.9 (and above).
Build on windows using the following command line:
scons swr=
Reorder header files so that we have a chance to defined NOMINMAX before
mesa include files include windows.h
v3: split from bigger patch
Reviewed-by: Emil Velikov
---
src/gallium/drivers/swr/swr_context.cpp | 16
src/gallium/drivers/swr/swr_context.h | 2 ++
src/gallium/dri
Added hooks for screen creation and swap. Still keep llvmpipe the default
software renderer.
v2: split from bigger patch
v3: reword commit message
Reviewed-by: Emil Velikov
---
src/gallium/targets/libgl-gdi/libgl_gdi.c | 28 +++-
1 file changed, 23 insertions(+), 5 dele
On 11 November 2016 at 00:45, Timothy Arceri
wrote:
> This will be used to share data between gl_program and gl_shader_program
> allowing for greater code simplification as we can remove a number of
> awkward uses of gl_shader_program.
> ---
> src/mesa/main/mtypes.h| 25 ++
On Thu, Nov 17, 2016 at 10:06:12PM -0800, Jason Ekstrand wrote:
> On Wed, Oct 19, 2016 at 10:47 AM, Nanley Chery
> wrote:
>
> > The HZ sequence modifies less state than the blorp path and requires
> > less CPU time to generate the necessary packets.
> >
> > Signed-off-by: Nanley Chery
> > ---
>
https://bugs.freedesktop.org/show_bug.cgi?id=77662
Nanley Chery changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
The helper automatically handles masking for us so we don't have to worry
about whether or not something is in the bottom bits.
---
src/intel/tools/aubinator.c | 29 +++--
1 file changed, 19 insertions(+), 10 deletions(-)
diff --git a/src/intel/tools/aubinator.c b/src/inte
From: Jason Ekstrand
The original aubinator that Kristian wrote had a bug in the handling of
MI_BATCH_BUFFER_START that propagated into the version in upstream mesa.
In particular, it ignored the "2nd level" bit which tells you whether this
MI_BATCH_BUFFER_START is a subroutine call (2nd level) o
This new helper is automatically handles 32 vs. 48-bit GTT issues. It also
handles 48-bit canonical addresses on Broadwell and above.
---
src/intel/tools/aubinator.c | 47 ++---
1 file changed, 31 insertions(+), 16 deletions(-)
diff --git a/src/intel/tools
---
src/intel/tools/aubinator.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c
index 0da01f4..f5e5167 100644
--- a/src/intel/tools/aubinator.c
+++ b/src/intel/tools/aubinator.c
@@ -442,9 +442,9 @@ handle_3dstate_hs(
On 16 November 2016 at 21:47, Timothy Arceri
wrote:
> On Wed, 2016-11-16 at 21:17 +, Emil Velikov wrote:
>> On 11 November 2016 at 00:45, Timothy Arceri
>> wrote:
>> >
>> > This will allow us to directly store metadata we want to retain in
>> > gl_program this metadata is currently stored in
> -Original Message-
> From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On
> Behalf Of Emil Velikov
> Sent: Friday, November 18, 2016 5:24 AM
> To: Kyriazis, George
> Cc: ML mesa-dev
> Subject: Re: [Mesa-dev] [PATCH v4 09/10] gallium: swr: Added swr build for
> windows
>
From: Nicolai Hähnle
For compute shaders, we free the selector after the shader has been
compiled, so we need to save this bit somewhere else. Also, make sure that
this type of bug cannot re-appear, by NULL-ing the selector pointer after
we're done with it.
This bug has been there since the fea
On Fri, Nov 18, 2016 at 7:45 PM, Kai Wasserbäch
wrote:
> Hi everybody,
> Nicolai Hähnle wrote on 18.11.2016 17:48:
>> On 18.11.2016 16:56, Emil Velikov wrote:
>>> On 18 November 2016 at 12:34, Marek Olšák wrote:
On Fri, Nov 18, 2016 at 12:49 PM, Emil Velikov
wrote:
[...]
> Sp
On Fri, Nov 18, 2016 at 4:56 PM, Emil Velikov wrote:
> On 18 November 2016 at 12:34, Marek Olšák wrote:
>> On Fri, Nov 18, 2016 at 12:49 PM, Emil Velikov
>> wrote:
>>> On 17 November 2016 at 23:42, Marek Olšák wrote:
On Thu, Nov 17, 2016 at 4:06 PM, Emil Velikov
wrote:
> On 15
The commit 8e430ff8b060b4e8e922bae24b3c57837da6ea77 support for LLVM
3.9 and older versionsin Clover. This patch restores it and refactors
the support using Clover compatibility layer for LLVM.
Signed-off-by: Vedran Miletić
---
.../state_trackers/clover/llvm/codegen/bitcode.cpp | 9 ++
src
The commit 8e430ff8b060b4e8e922bae24b3c57837da6ea77 support for LLVM
3.9 and older versionsin Clover. This patch restores it and refactors
the support using Clover compatibility layer for LLVM.
Signed-off-by: Vedran Miletić
---
.../state_trackers/clover/llvm/codegen/bitcode.cpp | 9 ++
src
Thanks!
Reviewed-by: Jason Ekstrand
On Fri, Nov 18, 2016 at 4:44 AM, Iago Toral Quiroga
wrote:
> This patch improves pass rate of dEQP-VK.texture.explicit_lod.2d.sizes.*
> from 68.0% (98/144) to 83.3% (120/144) by enabling sampler address
> rounding mode when the selected filter is not nearest
Hi everybody,
Nicolai Hähnle wrote on 18.11.2016 17:48:
> On 18.11.2016 16:56, Emil Velikov wrote:
>> On 18 November 2016 at 12:34, Marek Olšák wrote:
>>> On Fri, Nov 18, 2016 at 12:49 PM, Emil Velikov
>>> wrote:
>>> [...]
Speaking of patchwork, mostly I'm fine with it. There are some
The first 4 patches of this series at least (including this one) work
well on i915 when combined with the pending i915 patches for mesa,
libdrm and kernel.
Tested-by: Rafael Antognolli
On Fri, Nov 18, 2016 at 08:39:33AM -0500, Rob Clark wrote:
> From: Rob Clark
>
> With fixes from Chad squashe
On 18.11.2016 16:56, Emil Velikov wrote:
On 18 November 2016 at 12:34, Marek Olšák wrote:
On Fri, Nov 18, 2016 at 12:49 PM, Emil Velikov wrote:
On 17 November 2016 at 23:42, Marek Olšák wrote:
On Thu, Nov 17, 2016 at 4:06 PM, Emil Velikov wrote:
On 15 November 2016 at 16:57, Marek Olšák
On 18 November 2016 at 12:34, Marek Olšák wrote:
> On Fri, Nov 18, 2016 at 12:49 PM, Emil Velikov
> wrote:
>> On 17 November 2016 at 23:42, Marek Olšák wrote:
>>> On Thu, Nov 17, 2016 at 4:06 PM, Emil Velikov
>>> wrote:
On 15 November 2016 at 16:57, Marek Olšák wrote:
> On Tue, Nov
Thanks for upstreaming this, this patch has been tested and confirmed
working on a qemu setup.
Tested-by: Robert Foss
On Fri, 2016-11-18 at 08:39 -0500, Rob Clark wrote:
> From: Gustavo Padovan
>
> ---
> src/gallium/drivers/virgl/virgl_context.c | 47 +++-
> -
> src/gallium/d
On 18 November 2016 at 15:26, Marek Olšák wrote:
> On Nov 18, 2016 2:55 PM, "Emil Velikov" wrote:
>>
>> [Pardon for dropping in uninvited]
>>
>> On 15 November 2016 at 18:04, Marek Olšák wrote:
>>
>> > Immutable metadata (modifiers) stored in the kernel is the only
>> > scalable (and thus usable
Hi Emil,
On 18 November 2016 at 15:24, Emil Velikov wrote:
> On 18 November 2016 at 15:17, Daniel Stone wrote:
>> Actually, present-and-zero modifier has a very well-defined meaning:
>> it _forces_ linear interpretation of the buffer, whereas a non-present
>> modifier may cause a kernel query (e
On Nov 18, 2016 2:55 PM, "Emil Velikov" wrote:
>
> [Pardon for dropping in uninvited]
>
> On 15 November 2016 at 18:04, Marek Olšák wrote:
>
> > Immutable metadata (modifiers) stored in the kernel is the only
> > scalable (and thus usable) solution here. There was an argument
> > against _mutable
On 18 November 2016 at 15:17, Daniel Stone wrote:
> Hi,
>
> On 18 November 2016 at 14:50, Emil Velikov wrote:
>> On 16 November 2016 at 09:28, Varad Gautam wrote:
>>> + if (nonzero_modifier_found && dri2_dpy->image->createImageFromDmaBufs2)
>>> {
>>> + dri_image =
>>> + dri2_dpy-
Hi Gustavo,
On 17 November 2016 at 18:57, Gustavo Zacarias wrote:
> When cross-compiling the .pc file might point to the wrong
> wayland-scanner binary (target rather than host) resulting in a
> non-executable and wrong scanner.
> Try searching the PATH first, and if that fails fall back into
> p
Hi,
On 18 November 2016 at 14:50, Emil Velikov wrote:
> On 16 November 2016 at 09:28, Varad Gautam wrote:
>> + if (nonzero_modifier_found && dri2_dpy->image->createImageFromDmaBufs2) {
>> + dri_image =
>> + dri2_dpy->image->createImageFromDmaBufs2(dri2_dpy->dri_screen,
>> +
On 15 November 2016 at 14:24, Varad Gautam wrote:
> From: Varad Gautam
>
> allow egl clients to query the dmabuf formats supported on this platform.
>
> Signed-off-by: Louis-Francis Ratté-Boulianne
> Signed-off-by: Varad Gautam
> ---
> src/egl/drivers/dri2/egl_dri2.c | 87
> ++
On 16 November 2016 at 09:28, Varad Gautam wrote:
> From: Pekka Paalanen
>
> allow creating EGLImages with dmabuf format modifiers when target is
> EGL_LINUX_DMA_BUF_EXT for EGL_EXT_image_dma_buf_import_modifiers.
>
> v2: clear modifier assembling and error label name (Eric Engestrom)
>
> Signed-
On Fri, Nov 18, 2016 at 9:20 AM, Chris Wilson wrote:
> On Fri, Nov 18, 2016 at 08:39:37AM -0500, Rob Clark wrote:
>> +void fd_fence_server_sync(struct pipe_context *pctx,
>> + struct pipe_fence_handle *fence)
>> +{
>> + struct fd_context *ctx = fd_context(pctx);
>> + struct fd_
On 15 November 2016 at 14:24, Varad Gautam wrote:
> From: Varad Gautam
>
> support importing dmabufs into DRIimage taking format modifiers in
> account, as per DRIimage extension version 14.
>
With the following discussion in mind [1] I'm wondering if we don't
want to rework things to pass/store
On Fri, Nov 18, 2016 at 08:39:37AM -0500, Rob Clark wrote:
> +void fd_fence_server_sync(struct pipe_context *pctx,
> + struct pipe_fence_handle *fence)
> +{
> + struct fd_context *ctx = fd_context(pctx);
> + struct fd_batch *batch = ctx->batch;
> +
> + if (sync_accumulate("f
On 15 November 2016 at 14:24, Varad Gautam wrote:
> From: Pekka Paalanen
>
> The EGL_EXT_dma_buf_import_modifiers extension adds support for a
> fourth plane, just like DRM KMS API does.
>
> Bump maximum dma_buf plane count to four.
>
> Signed-off-by: Pekka Paalanen
> Signed-off-by: Varad Gautam
On 15 November 2016 at 14:24, Varad Gautam wrote:
> +#ifndef EGL_MESA_platform_surfaceless
> +#define EGL_MESA_platform_surfaceless 1
> +#define EGL_PLATFORM_SURFACELESS_MESA 0x31DD
> +#endif /* EGL_MESA_platform_surfaceless */
> +
Thinking out loud: is dropping the similar hunk from
include/
[Pardon for dropping in uninvited]
On 15 November 2016 at 18:04, Marek Olšák wrote:
> Immutable metadata (modifiers) stored in the kernel is the only
> scalable (and thus usable) solution here. There was an argument
> against _mutable_ metadata attached to BOs and the synchronization
> hell it c
On Thu, 2016-11-17 at 08:56 -0800, Jason Ekstrand wrote:
> On Thu, Nov 17, 2016 at 4:43 AM, Iago Toral
> wrote:
> > Ah, I had missed this, ignore my comment then :)
> >
> I just sent out a v2 of patch 7 and force-pushed my review branch if
> you want to test things out.
Yep, that fixes the prob
On 17 November 2016 at 20:15, poma wrote:
>
> Airlie solved everything concerning the kernel,
> so it seems, now it's user space turn.
>
> = mesa-libgbm-12.0.3 - works OK
> ...
> [ 714.429] (II) Loading sub module "glamoregl"
> [ 714.429] (II) LoadModule: "glamoregl"
> [ 714.430] (II) Loadin
Requires newer libdrm, and the support only is advertised with a
sufficiently new kernel (v4.9 and later) as it depends on fence fd
support in the submit ioctl.
Signed-off-by: Rob Clark
---
configure.ac | 2 +-
src/gallium/drivers/freedreno/freedreno_batch.c
From: Gustavo Padovan
---
src/gallium/drivers/virgl/virgl_context.c | 47 +++--
src/gallium/drivers/virgl/virgl_screen.c | 12 +++-
src/gallium/drivers/virgl/virgl_winsys.h | 16 -
src/gallium/winsys/virgl/drm/virgl_drm_winsys.c| 78 ++
If app tries to create a fence but there is no rendering to submit, we
need a dummy/no-op submit. Use a string-marker for the purpose.. mostly
since it avoids needing to realize that the packet format changes in
later gen's (so one less place to fixup for a5xx).
Signed-off-by: Rob Clark
---
src
Prep-work for next patch, mostly move to tracking last_fence as a
pipe_fence_handle (created now only in fd_gmem_render_tiles()), and a
bit of superficial renaming.
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/freedreno_batch.c | 2 --
src/gallium/drivers/freedreno/freedreno
From: Rob Clark
This enables gallium support for EGL_ANDROID_native_fence_sync, for
drivers which support PIPE_CAP_NATIVE_FENCE_FD.
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/freedreno_screen.c | 2 +
src/gallium/drivers/i915/i915_screen.c | 1 +
src/gallium/drivers
From: Rob Clark
This will be needed for explicit synchronization with devices outside
the gpu, ie. EGL_ANDROID_native_fence_sync.
Signed-off-by: Rob Clark
Reviewed-by: Marek Olšák
---
src/gallium/include/pipe/p_context.h | 6 ++
src/gallium/state_trackers/dri/dri2.c | 6 +-
2 files c
1 - 100 of 115 matches
Mail list logo
