Reviewed-by: Marek Olšák
Marek
On Wed, Jun 20, 2018 at 3:25 PM, Grazvydas Ignotas wrote:
> This allows to avoid having to see garbage in Dying Light loading screen
> at least, which probably expects Windows/NV behavior of all allocations
> being zeroed by default.
>
> Analogous to radv flag
Reviewed-by: Marek Olšák
Marek
On Wed, Jun 20, 2018 at 3:25 PM, Grazvydas Ignotas wrote:
> Avoids a branch and reduces code size a tiny bit:
> text data bss dechex filename
> 10804563 398653 2070368 13273584 ca89f0 /tmp/radeonsi_dri.so.old
> 10804499 398653 2070368 13273520
https://bugs.freedesktop.org/show_bug.cgi?id=106907
--- Comment #10 from Timothy Arceri ---
(In reply to xinghua from comment #9)
> (In reply to Jordan Justen from comment #1)
> > Any chance you might be able to write a small piglit test
> > that shows the bug? For example:
> >
> >
Two more below. I had them in the previous email by my phone app decided to
hide them. :-(
On June 20, 2018 15:53:04 Keith Packard wrote:
This extension provides fences and frame count information to direct
display contexts. It uses new kernel ioctls to provide 64-bits of
vblank sequence
Jason Ekstrand writes:
>> + fence = vk_alloc2(>instance->alloc, allocator, sizeof (*fence),
>> 8,
>> + VK_SYSTEM_ALLOCATION_SCOPE_OBJECT);
>
> zalloc?
Yes, definitely. Thanks for catching this. Updated and pushed the series.
--
-keith
signature.asc
Description: PGP
Jason Ekstrand writes:
>> + /* Look for a DPMS property if we haven't already found one */
>> + for (int p = 0; connector->dpms_property == 0 && p <
>> drm_connector->count_props; p++) {
>
> I'm guessing this is well over 80 characters
Sorry I missed this -- I haven't managed to configure
https://bugs.freedesktop.org/show_bug.cgi?id=106907
--- Comment #9 from xinghua ---
(In reply to Jordan Justen from comment #1)
> Any chance you might be able to write a small piglit test
> that shows the bug? For example:
>
> https://cgit.freedesktop.org/piglit/commit/?id=f1dc46ddf8c1
Hi,
On June 20, 2018 15:53:04 Keith Packard wrote:
This extension provides fences and frame count information to direct
display contexts. It uses new kernel ioctls to provide 64-bits of
vblank sequence and nanosecond resolution.
v2: Remove DRM_CRTC_SEQUENCE_FIRST_PIXEL_OUT flag. This has
been
On June 20, 2018 15:53:00 Keith Packard wrote:
This extension provides fences and frame count information to direct
display contexts. It uses new kernel ioctls to provide 64-bits of
vblank sequence and nanosecond resolution.
v2: Adopt Jason Ekstrand's coding conventions
Declare variables
Reviewed-by: Timothy Arceri
On 21/06/18 11:02, Marek Olšák wrote:
From: Marek Olšák
Cc: 18.1
---
src/mesa/main/get.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index 772ca00da1f..db0079beb51 100644
---
On June 20, 2018 17:04:25 Jordan Justen wrote:
features.txt? It looks like some others have been missed recently.
Yeah, I keep forgetting that's a thing. Should probably update that one of
these days...
--Jason
-Jordan
On 2018-06-20 08:07:20, Jason Ekstrand wrote:
Cc: Keith Packard
Reviewed-by: Marek Olšák
Marek
On Wed, Jun 20, 2018 at 8:55 PM, Robert Tarasov
wrote:
> From: "Joe M. Kniss"
>
> Add support for glsl 'invariant' modifier for output data declarations.
> Gallium drivers that use TGSI serialization currently loose invariant
> modifiers in glsl shaders.
>
> v2:
From: Marek Olšák
Cc: 18.1
---
src/mesa/main/get.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index 772ca00da1f..db0079beb51 100644
--- a/src/mesa/main/get.c
+++ b/src/mesa/main/get.c
@@ -1996,21 +1996,21 @@
From: Marek Olšák
---
src/amd/common/ac_surface.c | 21 +++--
1 file changed, 19 insertions(+), 2 deletions(-)
diff --git a/src/amd/common/ac_surface.c b/src/amd/common/ac_surface.c
index 618b755afc7..6a335111314 100644
--- a/src/amd/common/ac_surface.c
+++
From: Marek Olšák
This is reproducible on Stoney, but other chips may be affected too.
Cc 18.1
---
src/gallium/drivers/radeonsi/si_blit.c | 5 +
src/gallium/drivers/radeonsi/si_clear.c | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git
These were all discovered on Stoney except for the last patch, which is a
precaution.
Please review.
Thanks,
Marek
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Marek Olšák
Cc: 18.1
---
src/gallium/drivers/radeonsi/si_texture.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_texture.c
b/src/gallium/drivers/radeonsi/si_texture.c
index b46208be252..05d5d1b8a6d 100644
---
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_state.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/radeonsi/si_state.c
index cfe32bc7f5e..a7377f38745 100644
---
On 20/06/18 15:16, Mathias Fröhlich wrote:
Hi,
It seems that dlist.c is getting popular again, but can't we generate
most of dlist.c using the api files?
Probably saves a lot of typing, copying and pasting in the long run.
Should also be less error prone in the long run.
Not that I think code
From: "Joe M. Kniss"
Add support for glsl 'invariant' modifier for output data declarations.
Gallium drivers that use TGSI serialization currently loose invariant
modifiers in glsl shaders.
v2: use boolean for invariant instead of unsigned.
Change-Id: Ieac8639116def45233513b6867a847cf7fda2f55
---
src/mesa/main/extensions_table.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index bc1f21a2926..b1ee0214d6c 100644
--- a/src/mesa/main/extensions_table.h
+++ b/src/mesa/main/extensions_table.h
@@
---
src/mapi/glapi/gen/apiexec.py| 16
src/mesa/main/extensions_table.h | 2 +-
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/mapi/glapi/gen/apiexec.py b/src/mapi/glapi/gen/apiexec.py
index e69c6b4df16..1a91785d375 100644
---
---
src/mesa/main/dlist.c | 211 ++
1 file changed, 211 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 57ece6c30e0..8b1ddb05038 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -290,6 +290,15 @@ typedef enum
features.txt? It looks like some others have been missed recently.
-Jordan
On 2018-06-20 08:07:20, Jason Ekstrand wrote:
> Cc: Keith Packard
> ---
> src/amd/vulkan/radv_extensions.py | 1 +
> src/amd/vulkan/radv_wsi_display.c | 57 +
> src/intel/vulkan/anv_extensions.py | 1 +
https://bugs.freedesktop.org/show_bug.cgi?id=106980
--- Comment #2 from Bas Nieuwenhuizen ---
Created attachment 140254
--> https://bugs.freedesktop.org/attachment.cgi?id=140254=edit
Miscompiled shader
Attached is the failing shader. spirv2nir is enough to trigger if you add a
https://bugs.freedesktop.org/show_bug.cgi?id=106980
Bas Nieuwenhuizen changed:
What|Removed |Added
Component|Drivers/Vulkan/radeon |Drivers/Vulkan/Common
QA
https://bugs.freedesktop.org/show_bug.cgi?id=106980
--- Comment #1 from Bas Nieuwenhuizen ---
with some print debugging, it looks like the failing validate is here:
https://gitlab.freedesktop.org/mesa/mesa/blob/master/src/amd/vulkan/radv_shader.c#L233
which is directly after spirv_to_nir. I
nir_compact_varyings() is meant to skip over varyings used by xfb:
/* We can't repack xfb varyings. */
if (var->data.always_active_io)
continue;
Any idea why that isn't working in this case?
On 21/06/18 06:30, Eric Anholt wrote:
If one of the TF variables got
This extension provides fences and frame count information to direct
display contexts. It uses new kernel ioctls to provide 64-bits of
vblank sequence and nanosecond resolution.
v2: Adopt Jason Ekstrand's coding conventions
Declare variables at first use, eliminate extra whitespace
This extension provides fences and frame count information to direct
display contexts. It uses new kernel ioctls to provide 64-bits of
vblank sequence and nanosecond resolution.
v2: Remove DRM_CRTC_SEQUENCE_FIRST_PIXEL_OUT flag. This has
been removed from the proposed kernel API.
Add
This extension provides fences and frame count information to direct
display contexts. It uses new kernel ioctls to provide 64-bits of
vblank sequence and nanosecond resolution.
v2:
Rework fence integration into the driver so that waiting for
any of a mixture of fence types (wsi,
Handle the case where the set of fences to wait for is not all of the
same type by either waiting for them sequentially (waitAll), or
polling them until the timer has expired (!waitAll). We hope the
latter case is not common.
While the current code makes sure that it always has fences of only
one
Here's the latest version of this series with only a few minor
changes. It adapts to the API changes for surface_get_capabilities and
changes how the allocator is found for fences.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
Jason Ekstrand writes:
> That seems good to me. Unless, of course, DPMS is something we expect to
> change over time somehow. Then again, we don't handle that at all right
> now so meh. Let's go with what you wrote above for now.
It's not even the dpms value, it's the dpms property itself,
Jason Ekstrand writes:
> I believe that the WSI common code should be capable of fishing the
> instance allocator out of the wsi_display so we need only pass the
> allocator argument unmodified through to the core WSI code. Make sense?
Thanks, I think I've sorted it out. I've pushed an updated
https://bugs.freedesktop.org/show_bug.cgi?id=106980
Bug ID: 106980
Summary: Basemark GPU vulkan benchmark fails.
Product: Mesa
Version: git
Hardware: Other
OS: All
Status: NEW
Severity: normal
Reviewed-by: Timothy Arceri
On 12/05/18 19:40, Alejandro Piñeiro wrote:
From: Nicolai Hähnle
They are supported by SPIR-V for ARB_gl_spirv.
v2 (changes on top of Nicolai's original patch):
* Handle UniformConstant storage class for uniforms other than
samplers and images. (Eduardo
Reviewed-by: Timothy Arceri
On 20/06/18 22:27, Alejandro Piñeiro wrote:
From: Neil Roberts
Sets var->driver.location if the uniform was found from a previous
stage.
---
Hi Timothy,
thanks for the review of the original patch. Recently we found that we
missed a little thing (one line). As
On Tue, Jun 19, 2018 at 02:45:28PM +0100, Lionel Landwerlin wrote:
> From: Scott D Phillips
>
> For gen8+, write out PPGTT tables in aub files so that full 48-bit
> addresses can be serialized.
>
> v2: Fix handling of `end` index in map_ppgtt
>
> Signed-off-by: Scott D Phillips
>
diff -u --ignore-all-space shows that this and the original file are
roughly the same, except for some macros, some includes and how we check
for hardware gen.
Acked-by: Rafael Antognolli
On Tue, Jun 19, 2018 at 02:45:27PM +0100, Lionel Landwerlin wrote:
> Signed-off-by: Lionel Landwerlin
>
https://bugs.freedesktop.org/show_bug.cgi?id=106910
Daniel Serpell changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop. |intel-3d-bugs@lists.freedes
On Tue, Jun 19, 2018 at 9:44 PM, Keith Packard wrote:
> Jason Ekstrand writes:
>
>
> >> + if (!prop)
> >> + continue;
> >> + if (prop->flags & DRM_MODE_PROP_ENUM) {
> >> + if (!strcmp(prop->name, "DPMS"))
> >> +connector->dpms_property =
On Tue, Jun 19, 2018 at 02:45:26PM +0100, Lionel Landwerlin wrote:
> Scott added new stuff in IGT.
>
> Signed-off-by: Lionel Landwerlin
> ---
> src/intel/tools/intel_aub.h | 26 ++
> 1 file changed, 26 insertions(+)
>
> diff --git a/src/intel/tools/intel_aub.h
Jason Ekstrand writes:
> I just sent a v2 which allocates a temporary array, calls properties2, and
> then copies it back over. It doesn't duplicate the iteration code and
> instead just leverages propertie2. On the down side, it's a bit more
> allocation and data motion but, compared to the
Jason Ekstrand writes:
All looks good to me.
Reviewed-by: Keith Packard
--
-keith
signature.asc
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 20/06/18 19:19, Rafael Antognolli wrote:
On Tue, Jun 19, 2018 at 02:45:21PM +0100, Lionel Landwerlin wrote:
We use memfd to store physical pages as they get read/written to and
the GGTT entries translating virtual address to physical pages.
Based on a commit by Scott Phillips.
If one of the TF variables got varying-packed, it would end up with a new
driver_location, despite the pipe_stream_output already being set up for
the old driver location. This left the gallium driver with no way to work
back to what varying was referenced by pipe_stream_output.
Fixes these
> On Jun 18, 2018, at 9:23 AM, Juan A. Suarez Romero
> wrote:
>
> RADV now requires LLVM 5.0 or greater, and thus we can't build dist
> tarball because swr requires LLVM 4.0.
>
> Let's bump required LLVM to 5.0 in swr too.
Sorry, I didn't see this sooner. We are fine with bumping the swr
https://bugs.freedesktop.org/show_bug.cgi?id=105699
--- Comment #3 from Nanley Chery ---
Is this still an issue? I'm not able to reproduce this on my SKL.
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the
This allows to avoid having to see garbage in Dying Light loading screen
at least, which probably expects Windows/NV behavior of all allocations
being zeroed by default.
Analogous to radv flag with the same name.
---
src/gallium/drivers/radeonsi/si_pipe.c| 1 +
Avoids a branch and reduces code size a tiny bit:
text data bss dechex filename
10804563 398653 2070368 13273584 ca89f0 /tmp/radeonsi_dri.so.old
10804499 398653 2070368 13273520 ca89b0 /tmp/radeonsi_dri.so
---
src/gallium/drivers/radeonsi/si_descriptors.c | 4 ++--
1 file
On Wed, Jun 20, 2018 at 12:01:28PM -0700, Rafael Antognolli wrote:
> On Wed, Jun 20, 2018 at 11:03:32AM +0100, Lionel Landwerlin wrote:
> > On 20/06/18 01:00, Rafael Antognolli wrote:
> > > On Tue, Jun 19, 2018 at 02:45:22PM +0100, Lionel Landwerlin wrote:
> > > > From: Scott D Phillips
> > > >
On Wed, Jun 20, 2018 at 11:03:32AM +0100, Lionel Landwerlin wrote:
> On 20/06/18 01:00, Rafael Antognolli wrote:
> > On Tue, Jun 19, 2018 at 02:45:22PM +0100, Lionel Landwerlin wrote:
> > > From: Scott D Phillips
> > >
> > > v2: by Lionel
> > > Fix memfd_create compilation issue
I guess
On Tue, Jun 19, 2018 at 02:45:21PM +0100, Lionel Landwerlin wrote:
> We use memfd to store physical pages as they get read/written to and
> the GGTT entries translating virtual address to physical pages.
>
> Based on a commit by Scott Phillips.
>
> Signed-off-by: Lionel Landwerlin
> ---
>
https://bugs.freedesktop.org/show_bug.cgi?id=106976
--- Comment #3 from Danylo ---
https://patchwork.freedesktop.org/patch/230449/ fixes meson build and only adds
dependency on xrandr to autoconf. Meanwhile I fixed autoconf build.
--
You are receiving this mail because:
You are the QA Contact
Eric Engestrom writes:
> On Tuesday, 2018-06-19 16:06:14 -0700, Keith Packard wrote:
>> When VK_USE_PLATFORM_XLIB_XRANDR_EXT is defined, vulkan.h includes
>> X11/extensions/Xrandr.h for the RROutput typedef which is used in
>> the vkGetRandROutputDisplayEXT interface.
>>
>> Make sure we have
v2: Implement missing Enable/Disable (Emil)
v3: Drop unused NewIntelBlackholeRender (Ken)
v4: Bring back NewIntelBlackholeRender as i965 implementation uses it
again (Lionel)
Signed-off-by: Lionel Landwerlin
Reviewed-by: Kenneth Graunke
---
src/mesa/main/enable.c | 14
Inspired by what is already in the kernel.
Signed-off-by: Lionel Landwerlin
---
src/mesa/drivers/dri/i965/brw_context.h | 2 ++
src/mesa/drivers/dri/i965/brw_defines.h | 1 +
src/mesa/drivers/dri/i965/intel_batchbuffer.c | 13 +
3 files changed, 16 insertions(+)
diff
Signed-off-by: Lionel Landwerlin
---
src/mesa/drivers/dri/i965/brw_pipe_control.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_pipe_control.c
b/src/mesa/drivers/dri/i965/brw_pipe_control.c
index 122ac260703..1b89e55c396 100644
---
When doing a LRI Post Sync operation, you can put the register offset
in the lower 32bits of the address but won't need a BO.
Signed-off-by: Lionel Landwerlin
---
src/mesa/drivers/dri/i965/brw_pipe_control.c | 6 ++
1 file changed, 6 insertions(+)
diff --git
v2: condition the extension on context isolation support from the
kernel (Chris)
v3: (Lionel)
The initial version of this change used a feature of the Gen7+
command parser to turn the primitive instructions into no-ops.
Unfortunately this doesn't play well with how we're using
The kernel uses it. It's not recommended to use it in the batchbuffer,
but the hardware doesn't seem to complain.
Signed-off-by: Lionel Landwerlin
---
src/intel/genxml/gen10.xml | 1 +
src/intel/genxml/gen11.xml | 1 +
src/intel/genxml/gen7.xml | 1 +
src/intel/genxml/gen75.xml | 1 +
Hi all,
This is a respin of the blackhole render extension with Haswell
disabled. After digging a bit I found out that the kernel command
parser drops our batch when it contains a write to INSTPM.
Unfortunately I have to land those patches to be able to land the
kernel ones.
Cheers,
Lionel
On Tuesday, 2018-06-19 16:06:14 -0700, Keith Packard wrote:
> When VK_USE_PLATFORM_XLIB_XRANDR_EXT is defined, vulkan.h includes
> X11/extensions/Xrandr.h for the RROutput typedef which is used in
> the vkGetRandROutputDisplayEXT interface.
>
> Make sure we have the required header by checking
On Tue, Jun 19, 2018 at 10:31 PM, Keith Packard wrote:
> Jason Ekstrand writes:
>
> >> + if (allocator)
> >> + alloc = allocator;
> >> + else
> >> + alloc = >instance->alloc;
> >>
> >
> > This is what vk_alloc2 is for. :-)
> ...
> > And vk_free2
> ...
> > This isn't needed if you're
Reviewed-by: Marek Olšák
Marek
On Wed, Jun 20, 2018 at 9:26 AM, Robert Foss wrote:
> Signed-off-by: Robert Foss
> Reviewed-by: Tomasz Figa
> ---
> Changes since v3:
> - Added r-b from Tomasz
>
> src/gallium/auxiliary/util/u_debug_stack_android.cpp | 4 ++--
> 1 file changed, 2
It looks good. Thanks!
Marek
On Wed, Jun 20, 2018 at 12:47 PM, Dylan Baker wrote:
> Quoting Marek Olšák (2018-06-20 08:58:05)
>> On Wed, Jun 20, 2018 at 11:26 AM, Dylan Baker wrote:
>> > Quoting Marek Olšák (2018-06-08 20:16:54)
>> >> From: Marek Olšák
>> >>
>> >> This improves performance
Quoting Marek Olšák (2018-06-20 08:58:05)
> On Wed, Jun 20, 2018 at 11:26 AM, Dylan Baker wrote:
> > Quoting Marek Olšák (2018-06-08 20:16:54)
> >> From: Marek Olšák
> >>
> >> This improves performance for certain games.
> >>
> >> Cc: 18.1
> >> ---
> >> src/gallium/drivers/radeonsi/si_buffer.c
https://bugs.freedesktop.org/show_bug.cgi?id=106910
--- Comment #2 from Daniel Serpell ---
There is a backtrace in
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=901701:
Thread 2.3 "glxgears" received signal
https://bugs.freedesktop.org/show_bug.cgi?id=106976
--- Comment #2 from Bas Nieuwenhuizen ---
I think this patch already on the list should also fix it?
https://patchwork.freedesktop.org/patch/230449/
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA
Reviewed-by: Bas Nieuwenhuizen
for the series.
On Wed, Jun 20, 2018 at 4:10 PM, Samuel Pitoiset
wrote:
> Ported from RadeonSI.
> Not sure why this is needed but AMDVLK does something similar.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/si_cmd_buffer.c | 12 +---
> 1
On Wed, Jun 20, 2018 at 11:26 AM, Dylan Baker wrote:
> Quoting Marek Olšák (2018-06-08 20:16:54)
>> From: Marek Olšák
>>
>> This improves performance for certain games.
>>
>> Cc: 18.1
>> ---
>> src/gallium/drivers/radeonsi/si_buffer.c | 6 +-
>> 1 file changed, 5 insertions(+), 1
Quoting Marek Olšák (2018-06-08 20:16:54)
> From: Marek Olšák
>
> This improves performance for certain games.
>
> Cc: 18.1
> ---
> src/gallium/drivers/radeonsi/si_buffer.c | 6 +-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_buffer.c
Hello, Thanks for your feedback.
We don't implement GS before gen6, and I don't think there are plans
for it at this point, so I think we can just simplify the patch by
assuming that devinfo->gen is always going to be 6 here (later gens use
a different implementation of GS).
Got it. I will fix
Quoting Gert Wollny (2018-06-20 01:24:51)
> Hi Dylan & Eric,
>
> are you fine with how the flag is added now in meson? If you don't have
> any complaints I'm going to push the changes later today.
>
> many thanks,
> Gert
>
> Am Dienstag, den 19.06.2018, 10:07 +0200 schrieb Gert Wollny:
> >
Quoting Emil Velikov (2018-06-20 05:33:48)
> From: Emil Velikov
>
> Earlier commit removed the column, instead of adjusting it's height.
>
> Cc: Dylan Baker
> Fixes: 0d4f338a116 ("docs: Update release-notes and calendar")
> Signed-off-by: Emil Velikov
> ---
> docs/release-calendar.html | 1 +
On Tue, Jun 19, 2018 at 10:05 PM, Keith Packard wrote:
> Jason Ekstrand writes:
>
> > Thoughts?
>
> You've looked at the code more closely than I have; please feel free to
> leave it if you think it would seem worse as separate functions.
>
I just sent a v2 which allocates a temporary array,
Cc: Keith Packard
---
src/amd/vulkan/radv_extensions.py | 1 +
src/amd/vulkan/radv_wsi_display.c | 57 +
src/intel/vulkan/anv_extensions.py | 1 +
src/intel/vulkan/anv_wsi_display.c | 56 +
src/vulkan/wsi/wsi_common_display.c | 175 +---
https://bugs.freedesktop.org/show_bug.cgi?id=106151
--- Comment #22 from pritzl3...@gmail.com ---
I am away travelling and I wont be able to try the patch until late next week.
I will try the patch when I'm back and get back with the results.
--
You are receiving this mail because:
You are the
Reviewed-by: Lionel Landwerlin
On 19/06/18 16:05, Eric Engestrom wrote:
Signed-off-by: Eric Engestrom
---
src/intel/tools/aubinator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c
index
It's always false.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 3 +--
src/amd/vulkan/radv_private.h| 2 --
src/amd/vulkan/radv_query.c | 7 ++-
src/amd/vulkan/si_cmd_buffer.c | 28
4 files changed, 15 insertions(+), 25
Looks good to me
Reviewed-By: Gert Wollny
Am Freitag, den 08.06.2018, 11:20 +1000 schrieb Dave Airlie:
> From: Dave Airlie
>
> ---
> docs/features.txt| 2 +-
> src/gallium/drivers/virgl/virgl_encode.c | 7 +--
> src/gallium/drivers/virgl/virgl_hw.h | 1 +
>
On Wed, Jun 20, 2018 at 10:01 AM, Ilia Mirkin wrote:
> On Wed, Jun 20, 2018 at 9:56 AM, Rob Clark wrote:
>> On Tue, Jun 19, 2018 at 9:02 PM, Jonathan Marek wrote:
>>> blend can be NULL, so check for that
>>
>> hmm, which case are you hitting that? From a quick look I think
>> a3xx-a5xx would
On Tue, Jun 19, 2018 at 9:02 PM, Jonathan Marek wrote:
> blend can be NULL, so check for that
hmm, which case are you hitting that? From a quick look I think
a3xx-a5xx would have the same issue. Maybe we should just init
ctx->blend to a dummy state obj when the ctx is created?
BR,
-R
>
>
https://bugs.freedesktop.org/show_bug.cgi?id=106976
Danylo changed:
What|Removed |Added
CC||danylo.pilia...@gmail.com
--- Comment #1 from
VK_USE_PLATFORM_XLIB_XRANDR_EXT requires xlib leasing which requires
xcb-randr >= 1.13. Also xrandr header is required for this extension.
The extension should not be automatically enabled if these dependencies
aren't satisfied.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106976
This series replaces the dependency on
GRALLOC_MODULE_PERFORM_GET_DRM_FD with DRM node
probing and disables the support for drm_gralloc.
The series has been tested on Qemu+AOSP, where a
virtio gpu was successfully probed for and
opened.
The current branch can be found here:
Signed-off-by: Robert Foss
Reviewed-by: Tomasz Figa
---
Changes since v3:
- Added r-b from Tomasz
src/gallium/auxiliary/util/u_debug_stack_android.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_debug_stack_android.cpp
This patch both adds support for probing & filtering DRM nodes
and switches away from using the GRALLOC_MODULE_PERFORM_GET_DRM_FD
gralloc call.
Currently the filtering is based just on the driver name,
and the desired name is supplied using the "drm.gpu.vendor_name"
Android property.
From: Rob Herring
Maintaining both flink names and prime fd support which are provided by
2 different gralloc implementations is problematic because we have a
dependency on a specific gralloc implementation header.
This mostly disables the dependency on the gralloc implementation and
headers.
https://bugs.freedesktop.org/show_bug.cgi?id=106976
Bug ID: 106976
Summary: Compilation failure due to missing xcb_randr_lease_t
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Hi Tomeu,
On Wed, May 23, 2018 at 10:54:06AM +0200, Tomeu Vizoso wrote:
> --- a/src/mesa/main/texcompress_etc.h
> +++ b/src/mesa/main/texcompress_etc.h
> @@ -28,6 +28,7 @@
> #include "glheader.h"
> #include "texcompress.h"
> #include "texstore.h"
> +#include "pipe/p_format.h"
>
Gallium
From: Emil Velikov
Earlier commit removed the column, instead of adjusting it's height.
Cc: Dylan Baker
Fixes: 0d4f338a116 ("docs: Update release-notes and calendar")
Signed-off-by: Emil Velikov
---
docs/release-calendar.html | 1 +
1 file changed, 1 insertion(+)
diff --git
Hi Timothy,
thanks for the review of the patches of this series. This patch and the
squash I have just sent are the only patches pending to be reviewed.
Would you mind to take a look to this patch too?
Thanks in advance
On 12/05/18 11:40, Alejandro Piñeiro wrote:
> From: Nicolai Hähnle
>
>
https://bugs.freedesktop.org/show_bug.cgi?id=106151
--- Comment #20 from Samuel Pitoiset ---
Created attachment 140246
--> https://bugs.freedesktop.org/attachment.cgi?id=140246=edit
possible fix
Does this patch help?
--
You are receiving this mail because:
You are the assignee for the bug.
From: Neil Roberts
Sets var->driver.location if the uniform was found from a previous
stage.
---
Hi Timothy,
thanks for the review of the original patch. Recently we found that we
missed a little thing (one line). As we didn't push it to master yet,
could you take a look to this change, so we
On Tue, 2018-06-19 at 17:06 +0300, Andrii Simiklit wrote:
> We can not use the VUE Dereference flags combination for EOT
> message under ILK and SNB because the threads are not initialized
> there with initial VUE handle unlike Pre-IL.
> So to avoid GPU hangs on SNB and ILK we need
> to avoid
Hey Tomasz,
Thanks for the quick feedback.
On 2018-06-14 08:30, Tomasz Figa wrote:
Hi Rob,
Thanks for sending v3. Please see few more comments inline.
On Sun, Jun 10, 2018 at 2:28 AM Robert Foss wrote:
This patch both adds support for probing & filtering DRM nodes
and switches away from
On 20/06/18 01:00, Rafael Antognolli wrote:
On Tue, Jun 19, 2018 at 02:45:22PM +0100, Lionel Landwerlin wrote:
From: Scott D Phillips
v2: by Lionel
Fix memfd_create compilation issue
Fix pml4 address stored on 32 instead of 64bits
Return no buffer if first ppgtt page is not
On 19/06/18 23:56, Rafael Antognolli wrote:
On Tue, Jun 19, 2018 at 11:40:30AM -0700, Rafael Antognolli wrote:
On Tue, Jun 19, 2018 at 02:45:17PM +0100, Lionel Landwerlin wrote:
Now that we rely on mmap of the data to parse, we can't process the
standard input anymore.
Didn't we rely on mmap
1 - 100 of 107 matches
Mail list logo