Rb for the series. (I can't type my surname on the phone)
Marek
On Thu, Apr 25, 2019, 3:03 AM Timothy Arceri wrote:
> Some of the opts are not called in the general optimastion loop
> in the state trackers glsl -> nir conversion. We need to call
> the radeonsi specific optimisation once before
https://bugs.freedesktop.org/show_bug.cgi?id=110439
Juan A. Suarez changed:
What|Removed |Added
Depends on|110501 |
Referenced Bugs:
https://bugs.freedesktop.org/show_bug.cgi?id=110439
Bug 110439 depends on bug 110562, which changed state.
Bug 110562 Summary: Land VK_EXT_memory_budget
https://bugs.freedesktop.org/show_bug.cgi?id=110562
What|Removed |Added
Let me check the new tests and see if where the problem is.
Thanks for letting us know!
-Lionel
On 30/04/2019 13:43, Iago Toral Quiroga wrote:
Specifically, vkCmdCopyQueryPoolResults is required to see the effect
of a previous vkCmdResetQueryPool. This may not work currently when
query
Why can we not use tbuffer loads?
Marek
On Thu, Apr 25, 2019 at 7:18 AM Nicolai Hähnle wrote:
> Hi all,
>
> the following patches contain code to implement all vertex fetches
> using plain, non-format loads plus explicit shader arithmetic for
> format conversion.
>
> This allows us to remove
https://bugs.freedesktop.org/show_bug.cgi?id=110552
--- Comment #2 from Axel Davy ---
I think we can merge it asap.
The question on the GPU vendor name hasn't been resolved. I can remove that
patch of the serie and push in half an hour from now.
--
You are receiving this mail because:
You are
I have never understood why Mesa has gl_context::Visual and
gl_framebuffer::Visual. The Mesa code also allows a non-matching
doublebuffer setting between the two. libEGL also always uses a
singlebuffered config for pixmaps and pbuffers even if the context config
is doublebuffered.
Marek
On Tue,
https://bugs.freedesktop.org/show_bug.cgi?id=110552
--- Comment #1 from Juan A. Suarez ---
This is blocking 19.1 release.
Any chance to merge it asap?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the
On Mon, 2019-04-29 at 20:35 -0400, Marek Olšák wrote:
> Yeah that's possible. The thing is that a GL context can have a
> doublebuffered config and begin with GL_BACK as the draw buffer, but
> MakeCurrent can set a pbuffer with a singlebuffered config and then
> nothing is rendered with Mesa
https://bugs.freedesktop.org/show_bug.cgi?id=110565
--- Comment #2 from Volker Wegert ---
If you need any more info to help diagnose/reproduce the issue, please let me
know.
# python --version
Python 3.6.5
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA
On Tue, Apr 30, 2019 at 5:59 AM Mathias Fröhlich
wrote:
>
> Adam, Marek,
>
> On Monday, 29 April 2019 18:28:21 CEST Adam Jackson wrote:
> > On Fri, 2019-04-26 at 23:31 -0400, Marek Olšák wrote:
> >
> > I don't claim to know what this series is trying to fix, but:
> >
> > > +* 2) Pbuffers are
It would be better to add a pipe cap for this. I don't think you need
uint64_t here.
Marek
On Tue, Apr 30, 2019, 6:45 AM Sahu, Satyajit wrote:
> Query the uvd handles info. If the used uvd handles are equals to
> max possible handles then return error.
>
> Signed-off-by: Satyajit Sahu
> ---
>
https://bugs.freedesktop.org/show_bug.cgi?id=110439
Bug 110439 depends on bug 110552, which changed state.
Bug 110552 Summary: Land Gallium nine fixes for 19.1
https://bugs.freedesktop.org/show_bug.cgi?id=110552
What|Removed |Added
https://bugs.freedesktop.org/show_bug.cgi?id=110552
Axel Davy changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=110565
--- Comment #3 from Brian Paul ---
OK, the problem on my end was just a few missing package dependencies for X.
I also installed SCons 3.0.4 and that seemed to work too.
I have python3 version 3.7.3. I wonder if that's the difference. Can
On Tue, Apr 30, 2019 at 10:42 AM Ilia Mirkin wrote:
>
> On Tue, Apr 30, 2019 at 2:48 AM Dave Airlie wrote:
> >
> > On Tue, 30 Apr 2019 at 08:47, Dave Airlie wrote:
> > >
> > > On Tue, 30 Apr 2019 at 08:42, Ilia Mirkin wrote:
> > > >
> > > > Thing is, I think that would not be enough - with the
On Tue, Apr 30, 2019 at 4:36 AM Iago Toral wrote:
> Hi Jason,
>
> it seems that this was never addressed so I'll try to take it from here. A
> couple of comments regarding your feedback:
>
> We always setup a null render target if we don't have any, so I think that
> part of the patch is not
Signed-off-by: Alyssa Rosenzweig
---
src/gallium/drivers/panfrost/midgard/helpers.h | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/panfrost/midgard/helpers.h
b/src/gallium/drivers/panfrost/midgard/helpers.h
index 6d35f4de025..d2855cd89cd 100644
Signed-off-by: Alyssa Rosenzweig
---
src/gallium/drivers/panfrost/midgard/disassemble.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/panfrost/midgard/disassemble.c
b/src/gallium/drivers/panfrost/midgard/disassemble.c
index 0938486c883..4583e9e1584
Signed-off-by: Alyssa Rosenzweig
---
.../drivers/panfrost/pan_blend_shaders.c | 3 +--
src/gallium/drivers/panfrost/pan_context.c | 18 +-
2 files changed, 14 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/panfrost/pan_blend_shaders.c
Signed-off-by: Alyssa Rosenzweig
---
.../drivers/panfrost/include/panfrost-job.h | 6 --
src/gallium/drivers/panfrost/pandecode/decode.c | 15 ++-
2 files changed, 18 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/panfrost/include/panfrost-job.h
Redundant via the midgard shader dump.
Signed-off-by: Alyssa Rosenzweig
---
src/gallium/drivers/panfrost/pan_blend_shaders.c | 4
src/gallium/drivers/panfrost/pan_screen.c| 1 -
src/gallium/drivers/panfrost/pan_util.h | 1 -
src/gallium/drivers/panfrost/pan_wallpaper.c
We silently ignored certain bits of the mask, which causes issues when
disassembly 8/64-bit ops.
Signed-off-by: Alyssa Rosenzweig
---
src/gallium/drivers/panfrost/midgard/disassemble.c | 11 +++
1 file changed, 11 insertions(+)
diff --git
In preparation for 8-bit and 64-bit operands, let's not reinforce the
32-bit-centric biases in the ISA.
Signed-off-by: Alyssa Rosenzweig
---
.../drivers/panfrost/midgard/disassemble.c | 6 +++---
src/gallium/drivers/panfrost/midgard/midgard.h | 8
From: Dave Airlie
If no view is bound we still should reset the override to 0
and array mode.
This should fix misrendering in firefox WebRender since
the pbo sampler was removed.
Fixes: 1250383e36 (st/mesa: remove sampler associated with buffer texture in
pbo logic)
---
On Wed, 1 May 2019 at 12:34, Ilia Mirkin wrote:
>
> On Tue, Apr 30, 2019 at 10:42 AM Ilia Mirkin wrote:
> >
> > On Tue, Apr 30, 2019 at 2:48 AM Dave Airlie wrote:
> > >
> > > On Tue, 30 Apr 2019 at 08:47, Dave Airlie wrote:
> > > >
> > > > On Tue, 30 Apr 2019 at 08:42, Ilia Mirkin wrote:
> >
Meanwhile, we're forced to disable dest_override, since it's not yet
clear how this interacts with other bitnesses (it'll likely need to be
overhauled in any case).
Signed-off-by: Alyssa Rosenzweig
---
.../drivers/panfrost/midgard/disassemble.c| 39 ++-
1 file changed, 21
Signed-off-by: Alyssa Rosenzweig
---
.../drivers/panfrost/midgard/disassemble.c| 90 ++-
1 file changed, 68 insertions(+), 22 deletions(-)
diff --git a/src/gallium/drivers/panfrost/midgard/disassemble.c
b/src/gallium/drivers/panfrost/midgard/disassemble.c
index
Signed-off-by: Alyssa Rosenzweig
---
.../drivers/panfrost/midgard/disassemble.c| 20 ++-
1 file changed, 15 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/panfrost/midgard/disassemble.c
b/src/gallium/drivers/panfrost/midgard/disassemble.c
index
Signed-off-by: Alyssa Rosenzweig
---
.../drivers/panfrost/midgard/disassemble.c| 49 +--
1 file changed, 34 insertions(+), 15 deletions(-)
diff --git a/src/gallium/drivers/panfrost/midgard/disassemble.c
b/src/gallium/drivers/panfrost/midgard/disassemble.c
index
This change set implements support for disassembly 8-bit arithmetic in
Midgard, as exposed via the (u)char OpenCL types. This is far from
complete, but it does a decent job for some more basic 8-bit-using
shaders.
This is a prerequisite to a decent implementation of programmatic
blending, since
Midgard has only limited fixed-function blending support. For more
complex use cases, including many within ES2, a programmatic "blend
shader" is used instead, autogenerated by the driver and used as a
dedicated post-fragment shader stage. This patch set does some
preliminary work towards enabling
Per OpenCL.
Signed-off-by: Alyssa Rosenzweig
---
src/gallium/drivers/panfrost/midgard/helpers.h | 2 +-
src/gallium/drivers/panfrost/midgard/midgard.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/panfrost/midgard/helpers.h
This handles the usual case. 8-bit register access parallels 16-bit
access, but with one major caveat: in 8-bit mode, only half of the
register file is actually (directly) accessible as sources. In
particular, for each 16-bit integer register (hrN), we can only index a
*single* 8-bit integer
On Tue, Apr 30, 2019 at 2:48 AM Dave Airlie wrote:
>
> On Tue, 30 Apr 2019 at 08:47, Dave Airlie wrote:
> >
> > On Tue, 30 Apr 2019 at 08:42, Ilia Mirkin wrote:
> > >
> > > Thing is, I think that would not be enough - with the "recent" (like
> > > past 2 years) CSO/state change detection
Per the Vulkan spec 1.1.107, the predicate is a 32-bit value. Though
the AMD hardware treats it as a 64-bit value which means it might
fail to discard.
I don't know why this extension has been drafted like that but this
definitely not fit with AMD. The hardware doesn't seem to support
a 32-bit
https://bugs.freedesktop.org/show_bug.cgi?id=109323
Eric Engestrom changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=110439
Juan A. Suarez changed:
What|Removed |Added
Depends on||110562
Referenced Bugs:
From: Emil Velikov
To avoid pulling the latest libdrm we copy the helper locally. Document
when it was introduced, making it easier to purge in the future.
Cc: Eric Engestrom
Suggested-by: Eric Engestrom
Signed-off-by: Emil Velikov
---
Thanks for the suggestion Eric.
---
This error code typically indicated that a buffer object that was referenced
by the command stream was being used for CPU access by another client.
The correct action here is to retry after a while. Use usleep() until we
have proper kernel support for this wait.
Signed-off-by: Thomas Hellstrom
We didn't have the path using this command enabled as
typically we take an alternate path using DMA uploads.
Emable it so that we can exercise that code-path by turning off
the DMA path.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Brian Paul
---
src/gallium/winsys/svga/drm/vmw_screen.c | 1 +
The vmwgfx kernel module has a compatibility mode for user-space that is
not guest-backed resource aware. Add an environment variable to facilitate
testing of this mode on guest-backed aware kernels: if the environment
variable SVGA_FORCE_HOST_BACKED is defined, the driver will use host-backed
Some constant- and texture upload buffer data may bounce in malloced
buffers before being transferred to hardware buffers. In the case of
texture upload buffers this seems to be an oversight. In the case of
constant buffers, code comments indicate that we want to avoid mapping
hardware buffers for
The file vmwgfx_drm.h was a bit outdated. Update to a recent version,
including defines supporting coherent memory.
Signed-off-by: Thomas Hellstrom
Reviewed-by: Brian Paul
---
.../winsys/svga/drm/vmw_screen_ioctl.c| 4 +-
src/gallium/winsys/svga/drm/vmwgfx_drm.h | 358
Hi Jason,
it seems that this was never addressed so I'll try to take it from
here. A couple of comments regarding your feedback:
We always setup a null render target if we don't have any, so I think
that part of the patch is not necessary. I checked that simply making
sure that we don't remove the
Kindly Ignore this. Sent to wrong mailing list. Sent another email to
correct mailing list.
Regards,
Satyajit
On 4/30/2019 4:15 PM, Sahu, Satyajit wrote:
> [CAUTION: External Email]
>
> Query the max uvd handles and used uvd handles.
>
> Signed-off-by: Satyajit Sahu
> ---
> amdgpu/amdgpu.h
Adam, Marek,
On Monday, 29 April 2019 18:28:21 CEST Adam Jackson wrote:
> On Fri, 2019-04-26 at 23:31 -0400, Marek Olšák wrote:
>
> I don't claim to know what this series is trying to fix, but:
>
> > +* 2) Pbuffers are back buffers from the application point of view,
> > +*but they
Query the uvd handles info. If the used uvd handles are equals to
max possible handles then return error.
Signed-off-by: Satyajit Sahu
---
src/gallium/drivers/radeon/radeon_winsys.h| 1 +
src/gallium/drivers/radeonsi/si_get.c | 8
src/gallium/include/pipe/p_screen.h
Query the max uvd handles and used uvd handles.
Signed-off-by: Satyajit Sahu
---
amdgpu/amdgpu.h | 14 ++
amdgpu/amdgpu_gpu_info.c | 15 +++
2 files changed, 29 insertions(+)
diff --git a/amdgpu/amdgpu.h b/amdgpu/amdgpu.h
index c44a495a..407b5fae 100644
---
https://bugs.freedesktop.org/show_bug.cgi?id=110439
Bug 110439 depends on bug 110551, which changed state.
Bug 110551 Summary: Land patches for re-enabling fast color clears for GEN11
https://bugs.freedesktop.org/show_bug.cgi?id=110551
What|Removed |Added
https://bugs.freedesktop.org/show_bug.cgi?id=99781
Michel Dänzer changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
On Tue, 30 Apr 2019 at 08:47, Dave Airlie wrote:
>
> On Tue, 30 Apr 2019 at 08:42, Ilia Mirkin wrote:
> >
> > Thing is, I think that would not be enough - with the "recent" (like
> > past 2 years) CSO/state change detection changes, I think that you can
> > end up with no sampler set for a
On 2019-04-30 at 12:04, Emil Velikov wrote:
> From: Emil Velikov
>
> To avoid pulling the latest libdrm we copy the helper locally. Document
> when it was introduced, making it easier to purge in the future.
>
> Cc: Eric Engestrom
> Suggested-by: Eric Engestrom
> Signed-off-by: Emil Velikov
https://bugs.freedesktop.org/show_bug.cgi?id=110565
Bug ID: 110565
Summary: scons-3 breaks libgl-xlib build
Product: Mesa
Version: unspecified
Hardware: Other
OS: All
Status: NEW
Severity: normal
The series looks good to me.
Reviewed-by: Brian Paul
On 04/30/2019 05:04 AM, Thomas Hellstrom wrote:
The vmwgfx kernel module has a compatibility mode for user-space that is
not guest-backed resource aware. Add an environment variable to facilitate
testing of this mode on guest-backed aware
Specifically, vkCmdCopyQueryPoolResults is required to see the effect
of a previous vkCmdResetQueryPool. This may not work currently when
query execution is still on going, as some of the queries may become
available asynchronously after the reset.
Fixes new CTS tests:
On Mon, 29 Apr 2019 at 22:50, Marek Olšák wrote:
>
> On Mon, Apr 29, 2019 at 4:00 AM Pekka Paalanen wrote:
>>
>> On Sat, 27 Apr 2019 09:38:27 -0400
>> Marek Olšák wrote:
>>
>> > Those are all valid reasons, but I don't wanna expose swrast for AMD's
>> > customers.
>>
>> Hi Marek,
>>
>> is you
https://bugs.freedesktop.org/show_bug.cgi?id=110565
--- Comment #1 from Brian Paul ---
>From the Gentoo bug it looks like your command line was simply "scons
libgl-xlib".
I'm testing with Scons 3.0.1 from Ubuntu 19.04. I get something different:
$ scons libgl-xlib
scons: Reading SConscript
58 matches
Mail list logo