Hi, and thanks for the patch =)
Have you done any performance testing on this to verify it
gives us a speedup of any kind? I'm asking because it seems like
this might be something that a decent compiler should be able to do.
Performance related patches, at least in core mesa, usually have
some
Reviewed-By: George Kyriazis
>
On Apr 12, 2018, at 8:49 PM, Vlad Golovkin
> wrote:
Zeroing memory after calloc is not necessary. This also allows to avoid
possible
Thanks a lot Juan, it looks all good to me :)
On 16/04/18 04:09, Juan A. Suarez Romero wrote:
On Thu, 2018-04-12 at 14:54 -0700, Lionel Landwerlin wrote:
We're not counting correctly with depth & stencil images.
Additionally we need to move an assert that is meant just for color
attachments.
https://bugs.freedesktop.org/show_bug.cgi?id=106074
--- Comment #1 from Philip Rebohle ---
Created attachment 138867
--> https://bugs.freedesktop.org/attachment.cgi?id=138867=edit
Proposed patch
The attached patch fixes the issue in FF XIV. I'm not sure if that
Yes, please.
Fixes clover platform and device detection when installed in
/usr/local/ using ocl-icd for me when building with meson. I think
that was the last thing blocking me from switching over from autotools
on the rest of my machines.
Not familiar enough with the versioning/linking in
On 15/04/18 08:55, Matt Turner wrote:
> On Wed, Apr 11, 2018 at 7:30 PM, Jose Maria Casanova Crespo
> wrote:
>> Implement at brw_eu_validate the restriction from Intel Broadwell PRM, vol
>> 07,
>> section "Instruction Set Reference", subsection "EUISA Instructions", Send
On Mon, Apr 16, 2018, 4:52 AM Christian König <
ckoenig.leichtzumer...@gmail.com> wrote:
> Am 15.04.2018 um 20:46 schrieb Nicolai Hähnle:
> > On 07.04.2018 04:31, Marek Olšák wrote:
> >> From: Marek Olšák
> >>
> >> (This patch doesn't enable the behavior. It will be enabled
You never check EXT_texture_norm16 anywhere... is that right? Should
probably respect that in the _mesa_is_es3_* helpers. Also what about
shader support? I see no mention in the spec that these should be
usable as images, except with NV_image_formats is enabled. You're just
exposing them no matter
On Mon, Apr 16, 2018 at 6:45 AM, Erico Nunes wrote:
> On Sun, Apr 15, 2018 at 2:30 AM, Jason Ekstrand
> wrote:
> > On April 14, 2018 12:43:35 Connor Abbott wrote:
> > I think that it's probably impractical to use this path, and
Signed-off-by: Tapani Pälli
---
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 73a6c73f53..88a66e9e26 100644
---
Patch enables use of short and unsigned short data for texture uploads,
rendering and reading of framebuffers within the restrictions specified
in GL_EXT_texture_norm16 spec.
Patch also enables those 16bit format layout qualifiers listed in
GL_NV_image_formats that depend on EXT_texture_norm16.
Acked-by: Samuel Pitoiset
On 04/16/2018 04:38 PM, Jason Ekstrand wrote:
---
include/vulkan/vulkan_core.h | 2 +-
src/vulkan/registry/vk.xml | 10 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/vulkan/vulkan_core.h
Reviewed-by: Samuel Pitoiset
On 04/16/2018 04:38 PM, Jason Ekstrand wrote:
---
src/amd/vulkan/radv_extensions.py | 7 +--
src/intel/vulkan/anv_extensions_gen.py | 7 +--
2 files changed, 2 insertions(+), 12 deletions(-)
diff --git
https://bugs.freedesktop.org/show_bug.cgi?id=106074
gloriouseggr...@gmail.com changed:
What|Removed |Added
CC||gloriouseggr...@gmail.com
On Thu, 2018-04-12 at 14:54 -0700, Lionel Landwerlin wrote:
> We're not counting correctly with depth & stencil images.
>
> Additionally we need to move an assert that is meant just for color
> attachments.
>
> v2: Move an assert() (Reported by Craig)
> Change aspect mask checks (Francesco)
On 16/04/18 18:19, Juan A. Suarez Romero wrote:
On Fri, 2018-04-13 at 08:17 -0600, Brian Paul wrote:
Hi Brain, are you happy if we push this in the meantime? It seems to
impact a number of apps running on Wine?
I don't have time to look at it myself right now. So if it looks good
to you and
On Mon, 2018-04-16 at 22:17 +1000, Timothy Arceri wrote:
> >
Thanks for the feedback.
J.A.
> On 16/04/18 18:19, Juan A. Suarez Romero wrote:
>
> > On Fri, 2018-04-13 at 08:17 -0600, Brian Paul wrote:
> > > > Hi Brain, are you happy if we push this in the meantime? It seems to
> > >
v2: add constant_folding
v6: print non final NIR only for verbose debugging
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 39 ++
1 file changed, 39 insertions(+)
diff --git
a lot of those fields are not valid for a lot of tex ops. Not quite sure if
it's worth the effort to check for those or just keep it like that. It seems
to kind of work.
v2: reworked offset handling
add tex support with indirect R/S arguments
handle GLSL_SAMPLER_DIM_EXTERNAL
drop
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
v4: use smarter getIndirect helper
use new getSlotAddress helper
v5: use loadFrom helper
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 23 ++
1 file changed, 23 insertions(+)
diff --git
v2: support more sys values
fixed a bug where for multi component reads all values ended up in x
v3: add load_patch_vertices_in
v4: add subgroup stuff
v5: add helper invocation
v6: fix loading 64 bit system values
Signed-off-by: Karol Herbst
---
v3: and load_output
v4: use smarter getIndirect helper
use new getSlotAddress helper
v5: don't use const_offset directly
fix for indirects
v6: add support for interpolateAt
v7: fix compiler warnings
add load_barycentric_sample
handle load_output for fragment shaders
Signed-off-by:
v4: use loadFrom helper
v5: support indirect buffer access
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 90 ++
1 file changed, 90 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
v2: use new getIndirect helper
fixes symbols for 64 bit types
v4: use smarter getIndirect helper
simplify address calculation
use loadFrom helper
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 10 ++
1 file
Hello (ping) Thomas and Daniel,
any more suggestions? As i can see Eero has done a test-run and v2 also
looks good.
As alternative solution we also may try to call '*dri3_fence_await(draw->conn,
draw, buffer)'* with '*draw*'-parameter as '*NULL'.*
What do you think?
On Tue, Apr 10, 2018 at 11:10
https://bugs.freedesktop.org/show_bug.cgi?id=106074
Bug ID: 106074
Summary: radv: si_scissor_from_viewport returns incorrect
result when using half-pixel viewport offset
Product: Mesa
Version: git
Hardware: Other
v2: add vote_eq support
use the new subop intrinsic helper
add ballot
v3: add read_(first_)invocation
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 42 ++
1 file changed, 42 insertions(+)
diff --git
v3: add workaround for RA issues
indirects have to be multiplied by 0x10
fix indirect access
v4: use smarter getIndirect helper
use storeTo helper
v5: don't use const_offset directly
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp
v7: don't assert in default case
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 22 ++
1 file changed, 22 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
v2: user bitfield_insert instead of bfi
rework switch helper macros
remove some lowering code (LoweringHelper is now used for this)
v3: add pack_half_2x16_split
add unpack_half_2x16_split_x/y
v5: replace first argument with nullptr in loadImm calls
prefer getSSA over getScratch
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 8
1 file changed, 8 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
index
v3: fix compiler warnings
v4: use loadFrom helper
v5: fix signed min/max
v6: set tex mask
add support for indirect image access
set cache mode
v7: make compatible with 884d27bcf688d36c3bbe01bceca525595add3b33
rework the whole deref thing to prepare for bindless
Signed-off-by: Karol
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp| 17 +
1 file changed, 17 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
we store those arrays in local memory and reserve some space for each of the
arrays. The arrays are stored in a packed format, because we know quite easily
the context of each index. We don't do that in TGSI so far.
This causes various issues to come up in the MemoryOpt pass, because ld/st with
v4: use smarter getIndirect helper
use new getSlotAddress helper
use loadFrom helper
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 27 ++
1 file changed, 27 insertions(+)
diff --git
v2: use mkOp
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 13 +
1 file changed, 13 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
v4: use loadFrom helper
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
v5: add more barrier intrinsics
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 19 +++
1 file changed, 19 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
---
include/vulkan/vulkan_core.h | 2 +-
src/vulkan/registry/vk.xml | 10 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/vulkan/vulkan_core.h b/include/vulkan/vulkan_core.h
index ed0d596..2cafcdd 100644
--- a/include/vulkan/vulkan_core.h
+++
v2: remove TGSI related bits
Signed-off-by: Karol Herbst
Reviewed-by: Pierre Moreau
---
src/gallium/drivers/nouveau/Makefile.sources | 2 +
.../nouveau/codegen/nv50_ir_from_common.cpp| 107 +
not all those nir options are actually required, it just made the work a
little easier.
v2: fix asserts
parse compute shaders
don't lower bitfield_insert
v3: fix memory leak
v4: don't lower fmod32
v5: set lower_all_io_to_temps to false
fix memory leak because we take over ownership of
this makes debugging the shader header a little easier
Signed-off-by: Karol Herbst
Acked-by: Pierre Moreau
---
src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 1 +
1 file changed, 1 insertion(+)
diff --git
Changes to v6:
* fixed automake build
* fixed shader cache collision with TGSI shaders
* fixed handling of inot
* fixed location of images
* fixed load_output of fragment shaders (FBFETCH)
* added load_barycentric_sample handling
* some preparation for bindless_texture support
There will be some
From: Connor Abbott
In order to reduce moves when coalescing multiple registers into a
larger register, RA will try to coalesce MERGE instructions with their
definitions. For example, for something like this in GLSL:
uint a = ...;
uint b = ...;
uint64 x = packUint2x32(a,
v2: add support for geometry shaders
set idx
add some missing mappings
fix for 64bit inputs/outputs
fix up some FP color output index messup
parse centroid flag
v3: fix arrays in outputs as well
fix input/ouput size calculation for tessellation shaders
v4: add
v6: fix loops with blocks at the end nothing points to
skip blocks with no instructions and no predecessors
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 270 -
1 file changed, 268 insertions(+), 2 deletions(-)
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 18 ++
1 file changed, 18 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
v2: parse a few more fields
v3: add special handling for GL_ISOLINES
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 60 ++
1 file changed, 60 insertions(+)
diff --git
this is mostly usefull for lazy IR converters not wanting to deal with 64 bit
lowering and other illegal stuff
v5: also handle SAT
v6: rename type variables
fixed lowering of NEG
add lowering of NOT
Signed-off-by: Karol Herbst
---
v4: treat imul as unsigned
v5: remove pointless !!
v7: inot is unsigned as well
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 116 +
1 file changed, 116 insertions(+)
diff --git
v2: add helper function for indirects
v4: add new getIndirect overload for easier use
v5: use getSSA for ssa values
we can just create the values for unassigned registers in getSrc
v6: always create at least 32 bit values
Signed-off-by: Karol Herbst
---
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/nouveau_screen.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/nouveau_screen.c
b/src/gallium/drivers/nouveau/nouveau_screen.c
index 2598c78a45b..655d2d090f6
Signed-off-by: Karol Herbst
---
.../drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 72 ++
1 file changed, 72 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp
https://bugs.freedesktop.org/show_bug.cgi?id=106074
soredake changed:
What|Removed |Added
CC||fds...@krutt.org
--
You
this is required for Drivers which don't allow reading from outputs.
Reviewed-by: Timothy Arceri
Signed-off-by: Karol Herbst
---
src/mesa/state_tracker/st_glsl_to_nir.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git
v2: allow for non debug builds as well
v3: move reading out env var more global
disable tg4 with multiple offsets with nir
disable caps for 64 bit types
v6: nv50 support
disable MS images
disable bindless textures
Signed-off-by: Karol Herbst
---
---
src/amd/vulkan/radv_extensions.py | 7 +--
src/intel/vulkan/anv_extensions_gen.py | 7 +--
2 files changed, 2 insertions(+), 12 deletions(-)
diff --git a/src/amd/vulkan/radv_extensions.py
b/src/amd/vulkan/radv_extensions.py
index db37d61..099cae7 100644
---
https://bugs.freedesktop.org/show_bug.cgi?id=106090
Bug ID: 106090
Summary: Compiling compute shader crashes RADV
Product: Mesa
Version: 18.0
Hardware: Other
OS: All
Status: NEW
Severity: normal
On Thu, Apr 5, 2018 at 12:38 PM, Lepton Wu wrote:
> If users are running mesa under old version of qemu or have turned off
> GL at runtime, virtio gpu driver actually doesn't work. Adds a detection
> here so mesa can fall back to software rendering.
>
> v2:
> - move
Pushed. Thanks and welcome to Mesa!
On April 14, 2018 12:26:18 Jason Ekstrand wrote:
Reviewed-by: Jason Ekstrand
What driver is hitting this path? The !supports_ints path isn't used to my
knowledge so if some driver has started using it, they're
On Sun, Apr 15, 2018 at 2:47 PM, Nicolai Hähnle wrote:
> How much testing have you done with the radeon drm? It may be safer to
> just skip that part of the changes.
>
Piglit on CIK.
Marek
___
mesa-dev mailing list
Both the availability and result paths shared a bit of code so they were
marged.
Signed-off-by: Rhys Perry
---
src/gallium/drivers/nouveau/nvc0/mme/com9097.mme | 45
src/gallium/drivers/nouveau/nvc0/mme/com9097.mme.h | 34 ++
This patch set applies improvements related to the query buffer object
functionality of the nvc0 driver.
Changes in v2:
- Increase space requirement in patch 3 to ensure there is room for fence
emission.
Rhys Perry (3):
nvc0: set current bufctx to nvc0->bufctx at the end of nvc0_draw_vbo()
This can save one instruction since bitcount doesn't care about specific
bits' positions.
---
src/gallium/drivers/nouveau/nv50/nv50_screen.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/nv50/nv50_screen.c
Quoting Jan Alexander Steffens (heftig) (2018-04-14 10:23:22)
> This is for parity with autotools. It names the library
> libMesaOpenCL.so.1.0.0 and points mesa.icd to the .1 symlink.
>
> opencl_version now matches configure.ac's OPENCL_VERSION.
>
> Signed-off-by: Jan Alexander Steffens (heftig)
From: Marek Olšák
Cc: 18.0
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index
Quoting Dylan Baker (2018-04-13 08:46:46)
> Quoting Dylan Baker (2018-04-09 14:02:51)
> > This fixes -Ddri-drivers-path, -Dvdpau-libs-path, etc. with DESTDIR when
> > those paths are absolute. Currently due to the way python's os.path.join
> > handles absolute paths these will ignore DESTDIR,
Quoting Jan Alexander Steffens (heftig) (2018-04-14 10:23:20)
> This is for parity with autotools. We were suddenly getting assertion
> failures after switching to meson, see [1].
>
> [1]: https://bugs.archlinux.org/task/58218
>
> Signed-off-by: Jan Alexander Steffens (heftig)
Aaron Watry writes:
> From CL 1.2 Section 5.2.1:
> CL_INVALID_VALUE if buffer was created with CL_MEM_HOST_WRITE_ONLY and
> flags specify CL_MEM_HOST_READ_ONLY , or if buffer was created with
> CL_MEM_HOST_READ_ONLY and flags specify CL_MEM_HOST_WRITE_ONLY , or if
On 16.04.2018 20:44, Marek Olšák wrote:
On Sun, Apr 15, 2018 at 2:47 PM, Nicolai Hähnle > wrote:
How much testing have you done with the radeon drm? It may be safer
to just skip that part of the changes.
Piglit on CIK.
Probably good
Quoting Jan Alexander Steffens (heftig) (2018-04-14 10:23:21)
> This is for parity with autotools.
>
> Signed-off-by: Jan Alexander Steffens (heftig)
> ---
> src/gallium/drivers/swr/meson.build | 4
> 1 file changed, 4 insertions(+)
>
> diff --git
Quoting Jason Ekstrand (2018-04-16 07:38:53)
> ---
> src/amd/vulkan/radv_extensions.py | 7 +--
> src/intel/vulkan/anv_extensions_gen.py | 7 +--
> 2 files changed, 2 insertions(+), 12 deletions(-)
>
> diff --git a/src/amd/vulkan/radv_extensions.py
>
On Monday, March 19, 2018 4:41:44 AM PDT Tapani Pälli wrote:
> Add format definition and required plumbing to create images.
> Note that there is no match to drm_fourcc definition, just like
> with existing _DRI_IMAGE_FOURCC_SARGB.
>
> Signed-off-by: Tapani Pälli
>
When GLmatrix elements and its inverse are stored contiguously in memory it is
possible to
allocate, free and copy these fields with 1 function call instead of 2.
---
src/mesa/math/m_matrix.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git
On Fri, 2018-04-13 at 08:17 -0600, Brian Paul wrote:
> > Hi Brain, are you happy if we push this in the meantime? It seems to
> > impact a number of apps running on Wine?
>
> I don't have time to look at it myself right now. So if it looks good
> to you and you can check it in, go for it.
I
On 04/15/2018 08:52 PM, Juan A. Suarez Romero wrote:
On Fri, 2018-04-13 at 13:55 -0700, Mark Janes wrote:
Hi Samuel,
When this patch is applied directly to the 18.0 branch, I get a mesa
compile error:
radv_image.c:1175:9: error: implicit declaration of function
‘radv_image_has_dcc’; did you
This is for parity with autotools. We were suddenly getting assertion
failures after switching to meson, see [1].
[1]: https://bugs.archlinux.org/task/58218
Signed-off-by: Jan Alexander Steffens (heftig)
---
meson.build | 2 ++
1 file changed, 2 insertions(+)
diff
Reviewed-by: Nicolai Hähnle
On 16.04.2018 00:09, Bas Nieuwenhuizen wrote:
No clue how I missed those ...
Fixes: 4503ff760c "ac/nir: Add workaround for GFX9 buffer views."
CC:
Bugzilla:
This is for parity with autotools.
Signed-off-by: Jan Alexander Steffens (heftig)
---
src/gallium/drivers/swr/meson.build | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/drivers/swr/meson.build
b/src/gallium/drivers/swr/meson.build
index
This is for parity with autotools. It names the library
libMesaOpenCL.so.1.0.0 and points mesa.icd to the .1 symlink.
opencl_version now matches configure.ac's OPENCL_VERSION.
Signed-off-by: Jan Alexander Steffens (heftig)
---
src/gallium/targets/opencl/meson.build | 4
Hi,
these patches were added to mesa 18.0.0 by Arch Linux in order to make the
meson build product more
like the autotools one. They are in addition to two patches already in master:
- 6ac87c17 (meson: fix megadriver symlinking)
- 19dbed64 (meson: Set .so version for xa like autotools does)
On Mon, Apr 16, 2018 at 1:17 PM, Juan A. Suarez Romero
wrote:
> On Mon, 2018-04-16 at 00:09 +0200, Bas Nieuwenhuizen wrote:
>> No clue how I missed those ...
>>
>> Fixes: 4503ff760c "ac/nir: Add workaround for GFX9 buffer views."
>> CC:
>>
Aaron Watry writes:
> On Mon, Apr 16, 2018, 5:24 PM Francisco Jerez wrote:
>
>> Aaron Watry writes:
>>
>> > From CL 1.2 Section 5.2.1:
>> > CL_INVALID_VALUE if buffer was created with CL_MEM_HOST_WRITE_ONLY
>> and
>> > flags
From: Marek Olšák
It will contain more variables.
---
src/gallium/drivers/radeonsi/si_compute.c | 8 +--
src/gallium/drivers/radeonsi/si_pipe.c| 52 ---
src/gallium/drivers/radeonsi/si_pipe.h| 6 +-
src/gallium/drivers/radeonsi/si_shader.c
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_pipe.c| 30
src/gallium/drivers/radeonsi/si_pipe.h| 7
src/gallium/drivers/radeonsi/si_shader.h | 1 +
.../drivers/radeonsi/si_shader_tgsi_setup.c | 34 +--
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_pipe.c | 9 +
src/gallium/drivers/radeonsi/si_shader.h| 1 +
src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c | 7 +--
3 files changed, 11 insertions(+), 6 deletions(-)
diff --git
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_pipe.c | 10 ++
src/gallium/drivers/radeonsi/si_shader.h| 1 +
src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c | 7 ++-
3 files changed, 13 insertions(+), 5 deletions(-)
diff
Hi,
This cleanup is motivated by a Mesa/LLVM crash on Ubuntu 18.04.
It happens inside gallivm_dispose_target_library_info and only
LLVM 5.0 and 6.0 seem to be affected.
The workaround is to move target_library_info into the context
(1 instance) and the screen (1 instance per compiler thread) to
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c
b/src/gallium/drivers/radeonsi/si_shader_tgsi_setup.c
index
Series:
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Mon, Apr 16, 2018 at 7:36 PM, Tapani Pälli wrote:
> Hi;
>
> On 16.04.2018 18:19, Ilia Mirkin wrote:
>>
>> You never check EXT_texture_norm16 anywhere... is that right? Should
>> probably respect that in the _mesa_is_es3_* helpers. Also what about
>
>
> Oops that is very
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_blit.c| 12 +--
src/gallium/drivers/radeonsi/si_clear.c | 14 +--
src/gallium/drivers/radeonsi/si_compute.c | 8 +-
src/gallium/drivers/radeonsi/si_cp_dma.c | 2 +-
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_pipe.h | 21 ++--
src/gallium/drivers/radeonsi/si_state.c | 14 -
src/gallium/drivers/radeonsi/si_state.h | 6 +++---
src/gallium/drivers/radeonsi/si_state_draw.c | 2 +-
4
From: Marek Olšák
r600_resource could be renamed to si_buffer.
---
src/gallium/drivers/radeon/radeon_vcn_dec.c | 2 +-
src/gallium/drivers/radeonsi/cik_sdma.c | 22 +--
src/gallium/drivers/radeonsi/si_blit.c| 50 +++---
From: Marek Olšák
This reverts commit b650fc09c3a35ca624aad5fe4b5c34867708f116.
There is a 100%-reproducible deadlock if I increase the number of compiler
queues.
Also:
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105379
Cc: 18.0
From: Marek Olšák
---
src/amd/common/ac_llvm_util.c | 6 +-
src/amd/common/ac_llvm_util.h | 4 +++-
src/amd/vulkan/radv_shader.c| 2 +-
src/gallium/drivers/radeonsi/si_pipe.c | 3 ++-
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_pipe.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
b/src/gallium/drivers/radeonsi/si_pipe.c
index 09b9f588a6f..490a090da87 100644
---
From: Marek Olšák
The compiler queue was limited to 3 threads, so shader-db running
on a 16-thread CPU would have a bottleneck on the 3-thread queue.
---
src/gallium/drivers/radeonsi/si_pipe.c | 39 +-
src/gallium/drivers/radeonsi/si_pipe.h | 6 ++--
Hi;
On 16.04.2018 18:19, Ilia Mirkin wrote:
You never check EXT_texture_norm16 anywhere... is that right? Should
probably respect that in the _mesa_is_es3_* helpers. Also what about
Oops that is very correct, will add the checks.
shader support? I see no mention in the spec that these
1 - 100 of 130 matches
Mail list logo