https://bugs.freedesktop.org/show_bug.cgi?id=106972
Bug ID: 106972
Summary: vulkan.h:75:35: fatal error: X11/extensions/Xrandr.h:
No such file or directory
Product: Mesa
Version: git
Hardware: All
OS: Linux
Ooops, right.
Reviewed-by: Samuel Pitoiset
On 06/20/2018 12:29 AM, Bas Nieuwenhuizen wrote:
A case of making things worse while trying to fix something minor ...
Fixes: ef79457004e "radv: Merge the flush bits of CMASK & DCC clear."
---
src/amd/vulkan/radv_meta_clear.c | 2 +-
1 file change
https://bugs.freedesktop.org/show_bug.cgi?id=106151
--- Comment #19 from Samuel Pitoiset ---
Thanks, what preset do you use?
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.___
mesa-dev mail
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:
> This warning is misleading: When a struct is partially initiali
On Wednesday, 2018-06-20 10:24:51 +0200, Gert Wollny wrote:
> 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.
Looks good to me, thanks!
Reviewed-by: Eric Engestrom
>
> many thanks,
> Ger
Reviewed-by: Bas Nieuwenhuizen
for the series.
On Tue, Jun 19, 2018 at 4:57 PM, Samuel Pitoiset
wrote:
> Similar to the clear color values.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/radv_cmd_buffer.c | 35 ++--
> src/amd/vulkan/radv_meta_clear.c |
Reviewed-by: Bas Nieuwenhuizen
for the series.
On Tue, Jun 19, 2018 at 2:25 PM, Samuel Pitoiset
wrote:
> We know the number of samples at compile time.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/radv_nir_to_llvm.c | 40 +++
> src/amd/vulkan/radv_pipe
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 o
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 mapp
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 us
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 usage
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 c
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&action=edit
possible fix
Does this patch help?
--
You are receiving this mail because:
You are the assignee for the
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
>
> Th
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 a/docs/release-ca
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 include
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
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
b/src/gallium/aux
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. Th
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.
Signed-off-b
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:
https://gitlab.collabo
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
Signed
https://bugs.freedesktop.org/show_bug.cgi?id=106976
Danylo changed:
What|Removed |Added
CC||danylo.pilia...@gmail.com
--- Comment #1 from
https://bugs.freedesktop.org/show_bug.cgi?id=106151
--- Comment #21 from Samuel Pitoiset ---
If my patch doesn't help, can you try master with "export
RADV_DEBUG=nocompute"?
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
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
>
> Sign
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 have the same issue. Maybe we should just init
> ctx->blend to
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 ha
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 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 2 +-
src/amd/vulkan/radv_query.c | 9 ++---
src/amd/vulkan/si_cmd_buffer.c | 7 +--
3 files changed, 12 insertions(+), 6 deletions(-)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buff
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 +
> src
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 dele
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 c8d79ae377d68285bd1e..e4fb8adada354253143
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 a
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 +---
src/vul
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, ca
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 +
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:
> > T
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 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
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 deletion(-
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 file
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 Cont
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 SIGSEGV,
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
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 for
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 insertions(+)
On Tue, Jun 19, 2018 at 10:31 PM, Keith Packard wrote:
> Jason Ekstrand writes:
>
> >> + if (allocator)
> >> + alloc = allocator;
> >> + else
> >> + alloc = &device->instance->alloc;
> >>
> >
> > This is what vk_alloc2 is for. :-)
> ...
> > And vk_free2
> ...
> > This isn't needed if
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 dur
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 Landw
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 +
src/intel
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
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
--- a/src/mesa/dr
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 a/src/mesa/drivers/dri/i965/brw_
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 the
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 f
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
> ---
> src/i
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 thi
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
> > > >
>
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 change
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 +
src/gallium/drivers/radeons
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 bug.___
> 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 min
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 tests
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.
Signed-off-by
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
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 io
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 b/src/inte
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 = drm_connector->props
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
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
> ---
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
> Signed-off-by
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 we
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
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
Pr
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
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, wh
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
https://lists.f
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, d
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
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 NULL
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 be
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 vary
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 thi
https://bugs.freedesktop.org/show_bug.cgi?id=106980
Bas Nieuwenhuizen changed:
What|Removed |Added
Component|Drivers/Vulkan/radeon |Drivers/Vulkan/Common
QA Co
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&action=edit
Miscompiled shader
Attached is the failing shader. spirv2nir is enough to trigger if you add a
nir_valid
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 +
---
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
---
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
--- a/src/mapi/glapi/gen/apiexec.
---
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
@@ -1
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
T
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 ge
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
+++ b/src/amd/common/ac_s
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 a/src/gallium/drivers/radeonsi/si_bl
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
--- a/src/gallium/drivers/radeonsi/si_te
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
--- a/src/gallium/drivers/radeonsi/
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 @@ _mesa_GetInteger64v(
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:
1 - 100 of 111 matches
Mail list logo
