Reviewed-by: Marek Olšák
Marek
On Fri, Sep 22, 2017 at 1:49 AM, Timothy Arceri wrote:
> c7affbf6875622a enabled GLSLOptimizeConservatively on some
> drivers. The idea was to speed up compile times by running
> the GLSL IR passes only once each time
Didn't you send this already? It looks familiar, even in the v2.
Anyway, both patches:
Reviewed-by: Nicolai Hähnle
On 27.09.2017 16:01, Brian Paul wrote:
---
src/gallium/auxiliary/util/u_log.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
Cc: Eric Anholt
Fixes: 5b102160ae ("broadcom/genxml: Introduce a V3D packet/struct decoder.")
---
src/broadcom/Makefile.cle.am| 4 +++-
src/gallium/drivers/vc4/Makefile.am | 2 ++
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/broadcom/Makefile.cle.am
On Wed, Sep 27, 2017 at 12:38 PM, Eric Anholt wrote:
> Dylan Baker writes:
>
>> [ Unknown signature status ]
>> I've gone ahead and pushed the Vulkan drivers meson builds.
>>
>> For those interested in helping finish the conversion, I have a branch
>>
On Wed, 2017-09-27 at 16:18 +0100, Chris Wilson wrote:
> Mention to the debug log if the kernel scheduler is enabled; and in
> particular if it has preemption enabled.
>
> Signed-off-by: Chris Wilson
> Cc: Joonas Lahtinen
> Cc: Ben
From: Emil Velikov
Shared glapi (libglapi.so) has been a requirement for years, in order
to build EGL.
Remove the no longer necessary dlopen/dlsym dance and link to the
library directly.
This allows us to remove a handful of platform specific workarounds, due
to the
When writing to set > 0, we were just wrongly writing to set 0. This
commit fixes this by allocating each set from the dynamic state stream
as we write to them.
Cc: "17.2 17.1"
Fixes: 9f60ed98e501 ("anv: add VK_KHR_push_descriptor support")
Reported-by: Daniel
On Tue, 2017-09-26 at 14:51 -0700, Francisco Jerez wrote:
> Jan Vesely writes:
>
> > On Wed, 2017-09-20 at 19:10 -0500, Aaron Watry wrote:
> > > [SNIP]
> > >
> > > Not trying to rain on your parade, but I've been thinking that we
> > > might need to be able to query the
Dylan Baker writes:
> [ Unknown signature status ]
> I've gone ahead and pushed the Vulkan drivers meson builds.
>
> For those interested in helping finish the conversion, I have a branch
> "wip/meson-4" on my github (github.com/dcbaker/mesa) that has i965 and most of
> the
On 27 September 2017 at 16:00, Daniel Stone wrote:
> Hi Marek,
>
> On 27 September 2017 at 15:55, Marek Olšák wrote:
>> if (dmabuf_ret && dmabuf_ret->val.val_bool) {
>>uint64_t cap;
>>
>>if (drmGetCap(sPriv->fd, DRM_CAP_PRIME, ) == 0 &&
On Wed, Sep 27, 2017 at 6:22 PM, Emil Velikov wrote:
> On 27 September 2017 at 16:00, Daniel Stone wrote:
>> Hi Marek,
>>
>> On 27 September 2017 at 15:55, Marek Olšák wrote:
>>> if (dmabuf_ret && dmabuf_ret->val.val_bool) {
Hi,
In general, I think the organization was great. I agree having
everything happen in a single room was a good point. Here's some of my
personal feedback:
* I didn't like the tables layout at all. I found it to be extremely
uncomfortable to have to look sideways in order to see the screens
Quoting Eric Engestrom (2017-09-27 04:10:37)
> On Tuesday, 2017-09-26 23:38:11 +, Dylan Baker wrote:
> > It is possible to have DEBUG disabled but asserts on (NDEBUG(, which
>
> parentheses typo
>
> other than that:
> Reviewed-by: Eric Engestrom
>
> and I think
Cc: Eric Anholt
Fixes: 5b102160ae ("broadcom/genxml: Introduce a V3D packet/struct decoder.")
---
src/broadcom/Makefile.cle.am | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/broadcom/Makefile.cle.am b/src/broadcom/Makefile.cle.am
index
Build mesa 5633 completed
Commit a8fd58eae5 by Eric Anholt on 5/12/2017 11:05 PM:
vc4: Add labels to BOs for debug builds or with VC4_DEBUG=surf set.\n\nThis has proven to be incredibly useful for debugging CMA allocation\nfailures and driving memory
On 27 September 2017 at 17:28, Marek Olšák wrote:
> On Wed, Sep 27, 2017 at 6:22 PM, Emil Velikov
> wrote:
>> On 27 September 2017 at 16:00, Daniel Stone wrote:
>>> Hi Marek,
>>>
>>> On 27 September 2017 at 15:55, Marek Olšák
Mention to the debug log if the kernel scheduler is enabled; and in
particular if it has preemption enabled.
Signed-off-by: Chris Wilson
Cc: Joonas Lahtinen
Cc: Ben Widawsky
---
On Wed, Sep 27, 2017 at 1:52 AM, Daniel Stone wrote:
> Hi,
>
> On 26 September 2017 at 23:55, Jason Ekstrand
> wrote:
> > @@ -833,24 +816,19 @@ wsi_wl_surface_create_swapchain(VkIcdSurfaceBase
> *icd_surface,
> > chain->vk_format =
Tested-by: Dieter Nützel
Dieter
Am 26.09.2017 17:11, schrieb Nicolai Hähnle:
From: Nicolai Hähnle
It leads to surprising states with integer inputs and outputs on
vertex processing stages (e.g. geometry stages). Instead, rely on the
driver to
For the series:
Tested-by: Dieter Nützel
Dieter
Am 26.09.2017 16:43, schrieb Nicolai Hähnle:
From: Nicolai Hähnle
Not that those are super common or useful, but hey! Fun corner cases
of the API...
Fixes
For the series:
Tested-by: Dieter Nützel
Dieter
Am 26.09.2017 16:46, schrieb Nicolai Hähnle:
From: Nicolai Hähnle
This fixes the extremely unlikely case that an application uses
0x8000 or 0x3f80 as border color for an integer texture
On Wed, Sep 27, 2017 at 1:55 AM, Daniel Stone wrote:
> Good stuff; thanks for cleaning that up. Modulo my comments on 2 and
> 3, the series is:
> Reviewed-by: Daniel Stone
>
Thanks!
___
mesa-dev mailing
On Wed, Sep 27, 2017 at 1:53 AM, Daniel Stone wrote:
> Hi,
>
> On 26 September 2017 at 23:55, Jason Ekstrand
> wrote:
> > static void
> > -wsi_wl_display_destroy(struct wsi_wl_display *display)
> > +wsi_wl_display_ref(struct wsi_wl_display *display)
On Wed, 2017-09-27 at 16:13 +0100, Emil Velikov wrote:
> Hi Juan,
>
> On 26 September 2017 at 18:00, Juan A. Suarez Romero
> wrote:
> > On Tue, 2017-08-22 at 09:20 +0100, Daniel Stone wrote:
> > > Hi,
> > >
> > > On 21 August 2017 at 18:30, Emil Velikov
On Wed, 2017-09-27 at 16:15 +0100, Emil Velikov wrote:
> On 26 September 2017 at 18:08, Juan A. Suarez Romero
> wrote:
> > On Wed, 2017-09-06 at 15:07 +0100, Emil Velikov wrote:
> > > On 5 August 2017 at 00:25, Emil Velikov wrote:
> > > > From: Emil
From: Marek Olšák
---
src/mesa/main/texstate.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/texstate.c b/src/mesa/main/texstate.c
index 269e291..edd2253 100644
--- a/src/mesa/main/texstate.c
+++ b/src/mesa/main/texstate.c
@@
Boris Brezillon writes:
> On Wed, 27 Sep 2017 15:24:16 +0100
> Chris Wilson wrote:
>
>> Quoting Boris Brezillon (2017-09-27 15:06:53)
>> > On Wed, 27 Sep 2017 14:50:10 +0100
>> > Chris Wilson wrote:
>> >
Giving the same name to two different types violates the C++ One Definition
Rule and gcc will complain about it in LTO builds.
Signed-off-by: Nicholas Miell
---
src/amd/addrlib/gfx9/gfx9addrlib.cpp| 2 +-
src/amd/addrlib/inc/chip/gfx9/gfx9_gb_reg.h | 2 +-
On Fri, Aug 25, 2017 at 6:25 PM, Matt Turner wrote:
> Cannonlake (Gen10) adds align1 access mode to ternary instructions. In align1
> mode, instructions can use more (and mixed) datatypes and a single 16-bit
> immediate value. This series adds the infrastructure to emit and
Jan Vesely writes:
> On Tue, 2017-09-26 at 14:51 -0700, Francisco Jerez wrote:
>> Jan Vesely writes:
>>
>> > On Wed, 2017-09-20 at 19:10 -0500, Aaron Watry wrote:
>> > > [SNIP]
>> > >
>> > > Not trying to rain on your parade, but I've been
On Wed, Sep 27, 2017 at 11:36 AM, Emil Velikov wrote:
> From: Emil Velikov
>
> Shared glapi (libglapi.so) has been a requirement for years, in order
> to build EGL.
>
> Remove the no longer necessary dlopen/dlsym dance and link to the
>
On Wed, 27 Sep 2017 12:41:52 -0700
Eric Anholt wrote:
> Boris Brezillon writes:
>
> > On Wed, 27 Sep 2017 10:15:23 -0700
> > Eric Anholt wrote:
> >
> >> Boris Brezillon writes:
> >>
https://bugs.freedesktop.org/show_bug.cgi?id=102980
Andy Furniss changed:
What|Removed |Added
Resolution|--- |FIXED
On Wednesday, September 27, 2017 5:21:49 AM PDT Iago Toral Quiroga wrote:
> Triggering the push model when 64-bit inputs are involved is not easy due to
> the constrains on the maximum number of registers that we allow for this mode,
> however, for GS with 'points' primitive type and just a couple
On 2017-09-27 01:49 PM, Emil Velikov wrote:
From: Emil Velikov
Now that wayland-drm (correctly) keeps a local copy of the callbacks,
this should not longer cause explosions.
After all the symbol is a local, constant data.
Cc: Daniel Stone
On Wed, 27 Sep 2017 10:15:23 -0700
Eric Anholt wrote:
> Boris Brezillon writes:
>
> > On Wed, 27 Sep 2017 15:24:16 +0100
> > Chris Wilson wrote:
> >
> >> Quoting Boris Brezillon (2017-09-27 15:06:53)
> >> > On
From: Emil Velikov
The callbacks may be called even when they are no longer valid.
Say, the user is dlclose(ing) libEGL while the buffers are being
destroyed.
Cc: Daniel Stone
Cc: Derek Foreman
Cc:
From: Emil Velikov
Now that wayland-drm (correctly) keeps a local copy of the callbacks,
this should not longer cause explosions.
After all the symbol is a local, constant data.
Cc: Daniel Stone
Cc: Derek Foreman
On 27.09.2017 20:42, Nicholas Miell wrote:
Giving the same name to two different types violates the C++ One Definition
Rule and gcc will complain about it in LTO builds.
Oh my. What does the gcc warning look like? (I assume it's just a warning.)
Since these are auto-generated headers which
On Wed, 2017-09-27 at 18:09 +0100, Emil Velikov wrote:
> On 27 September 2017 at 17:28, Marek Olšák wrote:
> > On Wed, Sep 27, 2017 at 6:22 PM, Emil Velikov
> > wrote:
> > > On 27 September 2017 at 16:00, Daniel Stone wrote:
> >
Boris Brezillon writes:
> On Wed, 27 Sep 2017 10:15:23 -0700
> Eric Anholt wrote:
>
>> Boris Brezillon writes:
>>
>> > On Wed, 27 Sep 2017 15:24:16 +0100
>> > Chris Wilson
On Wed, 27 Sep 2017 22:03:15 +0200
Boris Brezillon wrote:
> On Wed, 27 Sep 2017 12:41:52 -0700
> Eric Anholt wrote:
>
> > Boris Brezillon writes:
> >
> > > On Wed, 27 Sep 2017 10:15:23 -0700
> > > Eric
string_buffer_test.cpp:43: error: ISO C++ forbids initialization of
member ‘str1’
string_buffer_test.cpp:43: error: making ‘str1’ static
string_buffer_test.cpp:43: error: invalid in-class initialization of
static data member of non-integral type ‘const char*’
Bugzilla:
On 09/27/2017 11:53 AM, Nicolai Hähnle wrote:
> On 27.09.2017 20:42, Nicholas Miell wrote:
>> Giving the same name to two different types violates the C++ One
>> Definition
>> Rule and gcc will complain about it in LTO builds.
>
> Oh my. What does the gcc warning look like? (I assume it's just a
Marek Olšák wrote:
Sorry too late, I pushed it.
I don't know if stable is affected.
It regresses things starting on radeonsi using weston eg.
mpv -
[vo/opengl/wayland] error occurred on the display fd: closing file
descriptor
kodi -
terminate called after throwing an instance of
On Wed, Sep 27, 2017 at 12:36 PM, Thomas Helland
wrote:
> string_buffer_test.cpp:43: error: ISO C++ forbids initialization of
> member ‘str1’
> string_buffer_test.cpp:43: error: making ‘str1’ static
> string_buffer_test.cpp:43: error: invalid in-class initialization of
This was missing.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_meta_decompress.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/amd/vulkan/radv_meta_decompress.c
b/src/amd/vulkan/radv_meta_decompress.c
index 4ff4e41ea7..fedcfad3ae 100644
---
On 09/27/2017 05:06 PM, Marek Olšák wrote:
> It fixes a crash with an apitrace that I have. Other than that, no
> tests. Yes, it should be tagged for stable.
Ok. Adding a test would be cool, but I won't bust your chops. :)
Reviewed-by: Ian Romanick
> Marek
>
> On
From: Roland Scheidegger
The operation performed is all the same as LODQ, but with the usual
differences between dx10 and GL texture opcodes, that is separate resource
and sampler indices (plus result swizzling, and setting z/w channels
to zero).
---
This implementation is correct (afaict), but takes two shortcuts
regarding the import/export of Android sync fds.
Shortcut 1. When Android calls vkAcquireImageANDROID to import a sync
fd into a VkSemaphore or VkFence, the driver instead simply blocks on
the sync fd, then puts the
On 09/27/2017 04:55 PM, Rob Clark wrote:
> On Wed, Sep 27, 2017 at 7:25 PM, Ian Romanick wrote:
>> On 09/26/2017 09:57 AM, Daniel Vetter wrote:
>>> Hi all,
>>>
>>> First again big thanks to Stéphane and Jennifer for organizing a great XDC.
>>>
>>> Like last year we'd like to
Build mesa 5632 failed
Commit e48445a757 by Dylan Baker on 9/22/2017 7:55 PM:
meson: build the loader
Configure your notification preferences
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
I've gone ahead and pushed the Vulkan drivers meson builds.
For those interested in helping finish the conversion, I have a branch
"wip/meson-4" on my github (github.com/dcbaker/mesa) that has i965 and most of
the core classic mesa stack building. I'm working on GLX, then EGL to start
testing
On 09/27/2017 11:18 AM, Nicolai Hähnle wrote:
Didn't you send this already? It looks familiar, even in the v2.
Yeah, but I neglected to cc my coworkers.
Anyway, both patches:
Reviewed-by: Nicolai Hähnle
Thanks!
-Brian
On 27.09.2017 16:01, Brian Paul wrote:
Reviewed-by: Lionel Landwerlin
On 28/09/17 01:02, Jason Ekstrand wrote:
This is just legacy cruft. We don't push these values; we pass them in
as vertex attributes.
---
src/intel/vulkan/anv_private.h | 7 ---
1 file changed, 7 deletions(-)
diff --git
On 09/26/2017 09:57 AM, Daniel Vetter wrote:
> Hi all,
>
> First again big thanks to Stéphane and Jennifer for organizing a great XDC.
>
> Like last year we'd like to hear feedback on how this year's XDC went,
> both the good (and what you'd like to see more of) and the not so
> good. Talk
Boris Brezillon writes:
> On Wed, 27 Sep 2017 12:41:52 -0700
> Eric Anholt wrote:
>
>> Boris Brezillon writes:
>>
>> > On Wed, 27 Sep 2017 10:15:23 -0700
>> > Eric Anholt wrote:
>> >
This looks right... at least it makes ATI_fragment_shader behave like
ARB_fragment_program. I don't suppose there are any tests that exercise
this problem? Should this be tagged for stable?
On 09/27/2017 08:39 AM, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
>
On Wed, Sep 27, 2017 at 7:25 PM, Ian Romanick wrote:
> On 09/26/2017 09:57 AM, Daniel Vetter wrote:
>> Hi all,
>>
>> First again big thanks to Stéphane and Jennifer for organizing a great XDC.
>>
>> Like last year we'd like to hear feedback on how this year's XDC went,
>>
This is just legacy cruft. We don't push these values; we pass them in
as vertex attributes.
---
src/intel/vulkan/anv_private.h | 7 ---
1 file changed, 7 deletions(-)
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h
index 3ba623a..b58c803 100644
---
It fixes a crash with an apitrace that I have. Other than that, no
tests. Yes, it should be tagged for stable.
Marek
On Thu, Sep 28, 2017 at 1:34 AM, Ian Romanick wrote:
> This looks right... at least it makes ATI_fragment_shader behave like
> ARB_fragment_program. I
This series adds Android support to Anvil. And Android requires
VK_ANDROID_native_buffer.
I tested the series on 64-bit ARC++ on a Skylake Chromebook with a 3.18
kernel. (ARC++ is the Android container in Chrome OS). (Yes, I said 3.18.
That's not a typo).
Here's my test results:
- A little,
From: Tapani Pälli
chadv: I made this patch by extracting the hunk from Tapani's patch in
https://lists.freedesktop.org/archives/mesa-dev/2017-September/169602.html.
Signed-off-by: Chad Versace
---
src/intel/Android.vulkan.mk | 2 +-
1 file
From: Tapani Pälli
Now that anvil fully implements the Vulkan HAL interface, we can install
it as the vendor HAL module at /vendor/lib/hw/vulkan.${board}.so. To do
so:
- Rename LOCAL_MODULE to vulkan.$(TARGET_BOARD_PLATFORM).
- Use LOCAL_PROPRIETARY_MODULE to install
This implementation is correct (afaict), but takes two shortcuts
regarding the import/export of Android sync fds.
Shortcut 1. When Android calls vkAcquireImageANDROID to import a sync
fd into a VkSemaphore or VkFence, the driver instead simply blocks on
the sync fd, then puts the
This will allow us to implement VK_ANDROID_native_buffer without dup'ing
the fd. We must close the fd in VK_KHR_external_memory_fd, but we should
not in VK_ANDROID_native_buffer.
---
src/intel/vulkan/anv_allocator.c | 12
src/intel/vulkan/anv_device.c| 11 +++
Creation of hiz, ccs, and mcs surfaces was encoded in a large, deep 'if'
tree at the tail of make_surface(). This patch extracts that 'if' tree
into one function per aux type:
try_make_hiz_surface()
try_make_ccs_surface()
try_make_mcs_surface()
For clarity, also rename make_surface()
In src/intel/vulkan/*, redirect all instances of printf, vk_error,
anv_loge, anv_debug, anv_finishme, anv_perf_warn, anv_assert, and their
many variants to the new intel_log functions. I believe I caught them
all.
The other subdirs of src/intel are left for a future exercise.
v2:
- Rebase onto
This patch prevents compilation failures in upcoming Android Vulkan
patches, failures due to redefinition of LOG_TAG in Android system
headers.
This patch does not change the value of LOG_TAG. It remains
"INTEL-MESA". (I don't like it, though. The all-caps smells like
FORTRAN).
Only one Intel
A first step to supporting Vulkan on ARC++. Mesa on ARC++ uses
Autotools, not Android.mk.
Doing this now, even before VK_ANDROID_native_buffer is implemented,
allows us to incrementally add Android support to the Autotools build.
---
src/intel/Makefile.vulkan.am | 5 +
1 file changed, 5
If this flag is set, then the image and it's bo have the same
lifetime. vkDestroyImage will release the bo.
We need this for VK_ANDROID_native_buffer, because that extension
creates the VkImage and imports its memory in the same call,
vkCreateImage.
---
src/intel/vulkan/anv_image.c | 9
The code that restricts the VkImage's tiling flags, extract it into
a new function named choose_isl_tiling_flags(). This reduces the diff
in upcoming patches for VK_ANDROID_native_buffer.
v2:
- Rebase onto 'needs_shadow' changes on master.
- Assert that choose_isl_tiling_flags() chooses at
Will use in VK_ANDROID_native_buffer.
---
src/intel/vulkan/anv_gem.c | 16
src/intel/vulkan/anv_private.h | 1 +
2 files changed, 17 insertions(+)
diff --git a/src/intel/vulkan/anv_gem.c b/src/intel/vulkan/anv_gem.c
index 3994c6b66c2..34c09891086 100644
---
Upcoming patches teach vkCreateImage to understand
VkNativeBufferANDROID. And much later patches will do the same for
whatever VkImageCreateInfo-chained struct imports dma-bufs. And then
again for the struct that will be introduced by the inevitable extension
VK_ANDROID_external_memory_whatever.
I'm bringing up Vulkan in the Android container of Chrome OS (ARC++).
On Android, stdio goes to /dev/null. On Android, remote gdb is even more
painful than the usual remote gdb. On Android, nothing works like you
expect and debugging is hell. I need logging.
This patch introduces a small, simple
The patch renames anv_bo_cache_import() to
anv_bo_cache_import_with_size(); and adds a new variant,
anv_bo_cache_import(), that lacks the 'size' parameter. Same as
pre-patch, anv_bo_cache_import_with_size() continues to validate the
imported size.
The patch is essentially a refactor patch. It
Reviewed-by: Bas Nieuwenhuizen
On Wed, Sep 27, 2017 at 2:09 PM, Samuel Pitoiset
wrote:
> This was missing.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/radv_meta_decompress.c | 1 +
> 1 file changed,
Signed-off-by: Dylan Baker
CC: Kristian Høgsberg
---
src/intel/vulkan/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/meson.build b/src/intel/vulkan/meson.build
index 9f0ee558e8a..a0be95c747c 100644
It was a great time! Next year I should pay more attention that this was
happening so I don't sign up at the last moment causing me to miss a day,
and having a fever on the last day.
Looking forward to next year's!
On Wed, Sep 27, 2017 at 4:25 PM, Ian Romanick wrote:
> On
On Wed, Sep 27, 2017 at 10:49 PM, Ian Romanick wrote:
> On 09/27/2017 04:55 PM, Rob Clark wrote:
>> On Wed, Sep 27, 2017 at 7:25 PM, Ian Romanick wrote:
>>> On 09/26/2017 09:57 AM, Daniel Vetter wrote:
Hi all,
First again big thanks to
On Wed, Sep 27, 2017 at 10:07 PM, Rob Clark wrote:
> If you had known of the khr dates, and brought it up in Feb (or really
> somewhat earlier, given that XDC is roughly same time each year +/-
> few weeks), that *might* have been early enough to move things.
That's unfair.
Reviewed-by: Tapani Pälli
On 09/28/2017 02:26 AM, Dylan Baker wrote:
Signed-off-by: Dylan Baker
CC: Kristian Høgsberg
---
src/intel/vulkan/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On Thu, Sep 28, 2017 at 7:36 AM, Matt Turner wrote:
> On Wed, Sep 27, 2017 at 10:07 PM, Rob Clark wrote:
>> If you had known of the khr dates, and brought it up in Feb (or really
>> somewhat earlier, given that XDC is roughly same time each year +/-
>>
On 09/27/2017 10:07 PM, Rob Clark wrote:
> On Wed, Sep 27, 2017 at 10:49 PM, Ian Romanick wrote:
>> On 09/27/2017 04:55 PM, Rob Clark wrote:
>>> On Wed, Sep 27, 2017 at 7:25 PM, Ian Romanick wrote:
On 09/26/2017 09:57 AM, Daniel Vetter wrote:
I will review these eventually
On September 27, 2017 8:14:25 PM Chad Versace wrote:
This implementation is correct (afaict), but takes two shortcuts
regarding the import/export of Android sync fds.
Shortcut 1. When Android calls vkAcquireImageANDROID to import a sync
This implementation is correct (afaict), but takes two shortcuts
regarding the import/export of Android sync fds.
Shortcut 1. When Android calls vkAcquireImageANDROID to import a sync
fd into a VkSemaphore or VkFence, the driver instead simply blocks on
the sync fd, then puts the
On Wednesday, September 27, 2017 8:39:27 AM PDT Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/mesa/main/texstate.c | 8 +---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/main/texstate.c b/src/mesa/main/texstate.c
> index
Sure, I can do that.
Iago
On Tue, 2017-09-26 at 06:32 -0400, Ilia Mirkin wrote:
> Perhaps a debug message would be warranted in such a situation? I
> suspect it would be difficult to debug, esp if it came up in a
> regular application.
> On Sep 26, 2017 3:50 AM, "Iago Toral Quiroga"
Hi,
Right now the way the thing works is that it walks the batchbuffer just after
the kernel returns from the ioctl and updates its internal view of the GPU
state as it walks and emits to the log file the data. The log on a single
batchbuffer is (essentially) just a list of call ID's from the
Software rasterizer and LLVM contain code to enable clipping as soon as
a vertex shader writes to gl_ClipDistance, even if the corresponding
clip planes are disabled.
GLSL specification states that "Values written into gl_ClipDistance for
planes that are not enabled have no effect."
The actual
For the series:
Reviewed-by: Samuel Pitoiset
On 09/26/2017 04:39 PM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
It has to happen after descriptor uploads since otherwise we'll print out
the wrong GPU list / incorrectly claim descriptor
For patches 2-11:
Reviewed-by: Marek Olšák
Patch 1:
Acked-by: Marek Olšák
Marek
On Sat, Sep 16, 2017 at 1:23 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Note: this causes spurious regressions
Pbuffers are problematic because it's an ancient feature that nobody
cares about anymore.
I think Miklos made KOTOR work on radeonsi or r600.
Marek
On Mon, Sep 25, 2017 at 6:50 PM, Federico Dossena wrote:
> Hello everyone,
> you may remember that a few months ago I was
Reviewed-by: Marek Olšák
If a 3D or array texture can't occur here, it would be better to set
box.depth = 1. If a 3D texture can't occur here but an array texture
can, it needs to be box.depth = resource->array_size.
Marek
On Tue, Sep 26, 2017 at 5:04 PM, Leo Liu
Reviewed-by: Marek Olšák
Marek
On Tue, Sep 26, 2017 at 5:11 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> It leads to surprising states with integer inputs and outputs on
> vertex processing stages (e.g. geometry
On Wed, Sep 27, 2017 at 10:31 AM, Federico Dossena wrote:
> Yes his work was on radeonsi, but the patch that he says fixes the crash
> with framebuffer effects and soft shadows was in the state tracker. Link:
> https://patchwork.freedesktop.org/patch/68298/
>
> The patch
On Wed, Sep 6, 2017 at 12:38 PM, Emil Velikov wrote:
> On 4 September 2017 at 21:36, Marek Olšák wrote:
>> From: Marek Olšák
>>
>> Building gallium is faster by 7.5 seconds on a 4core/8thread 3GHz CPU.
>> (gallium build time is
On Wed, Sep 6, 2017 at 12:35 PM, Nicolai Hähnle wrote:
> https://travis-ci.org/nhaehnle/mesa-1/jobs/272422053
>
> I suspect the se of $(top_srcdir) is the problem. Are you testing
> out-of-tree builds?
No, I'm not testing out-of-tree builds. I don't know how to test them.
Hi,
On 26 September 2017 at 23:55, Jason Ekstrand wrote:
> @@ -833,24 +816,19 @@ wsi_wl_surface_create_swapchain(VkIcdSurfaceBase
> *icd_surface,
> chain->vk_format = pCreateInfo->imageFormat;
> chain->drm_format = wl_drm_format_for_vk_format(chain->vk_format,
Hi,
On 26 September 2017 at 23:55, Jason Ekstrand wrote:
> static void
> -wsi_wl_display_destroy(struct wsi_wl_display *display)
> +wsi_wl_display_ref(struct wsi_wl_display *display)
> +{
> + display->refcount++;
> +}
Better:
static struct wsi_wl_display *
1 - 100 of 142 matches
Mail list logo