Emil Velikov writes:
> On 4 July 2017 at 16:04, Rafael Avila de Espindola
> wrote:
>> Emil Velikov writes:
>>
>>> So the version is FOOgit-$SHA while the DSO uses, FOOsvn. At the same
>>> time at least 3.9.0/3.9.1
Emil Velikov writes:
> So the version is FOOgit-$SHA while the DSO uses, FOOsvn. At the same
> time at least 3.9.0/3.9.1 has broken llvm-config --libnames/libfiles,
> so we cannot rely on that.
> Doesn't seem like LLVM is giving us something robust to work with here.
>
On 04/07/17 19:37, Nicolai Hähnle wrote:
On 03.07.2017 14:34, Nicolai Hähnle wrote:
From: Nicolai Hähnle
Save some passes over the IR.
v2: redesign to make the users of find_assignments more readable
Reviewed-by: Ian Romanick (v1)
---
According to Nicolai the SX can already start work when all
the position exports are done, so do those first.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/common/ac_nir_to_llvm.c | 109
1 file changed, 54 insertions(+), 55 deletions(-)
Note,
this change depends on a libdrm change, which is also under review.
Sam
On 2017-07-04 05:35 PM, Samuel Li wrote:
> Change-Id: I98d17f4af07c30f1739b43db1a6e1616a58332ef
> Signed-off-by: Samuel Li
> ---
> src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 17
Reviewed-by: Timothy Arceri
On 03/07/17 22:34, Nicolai Hähnle wrote:
From: Nicolai Hähnle
Otherwise, the padding bits remain undefined, which leads to valgrind
errors when storing the gl_shader_variable in the disk cache.
v2: use rzalloc
Change-Id: I98d17f4af07c30f1739b43db1a6e1616a58332ef
Signed-off-by: Samuel Li
---
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c
On 2017-07-04 09:04 AM, Nicolai Hähnle wrote:
On 01.07.2017 01:02, Andres Rodriguez wrote:
From: Dave Airlie
---
src/mesa/Makefile.sources| 2 +
src/mesa/state_tracker/st_cb_memoryobjects.c | 65
On Monday, July 3, 2017 1:06:04 PM PDT Andres Gomez wrote:
> It looks like we could want this whole series into -stable (?)
I probably wouldn't bother - it's a lot of churn, and isn't really known
to fix any real applications...in fact not even test cases.
Thanks for asking, though!
--Ken
On Monday, July 3, 2017 1:57:16 PM PDT Andres Gomez wrote:
> It looks like we could want these 2 into -stable (?)
That seems reasonable. Feel free to pick them over. Thanks!
signature.asc
Description: This is a digitally signed message part.
___
From: Emil Velikov
With ealier commit we relaxed the requirement from C++14 to C++11.
Update the build script so that it
Cc: Tim Rowley
On 2017-07-04 09:30 AM, Christian König wrote:
Am 04.07.2017 um 15:13 schrieb Nicolai Hähnle:
On 01.07.2017 01:03, Andres Rodriguez wrote:
From: Dave Airlie
Signed-off-by: Andres Rodriguez
---
src/gallium/drivers/radeon/r600_pipe_common.h | 6 ++
On 4 July 2017 at 16:47, Rafael Avila de Espindola
wrote:
> Emil Velikov writes:
>
>> On 4 July 2017 at 16:04, Rafael Avila de Espindola
>> wrote:
>>> Emil Velikov writes:
>>>
So
On 2017-07-04 09:19 AM, Nicolai Hähnle wrote:
On 01.07.2017 01:02, Andres Rodriguez wrote:
From: Dave Airlie
---
src/gallium/include/pipe/p_screen.h | 36
src/gallium/include/pipe/p_state.h | 2 ++
2 files changed, 38
There is no point in keeping this indirection. Makes the code easier to
follow.
Signed-off-by: Lucas Stach
Reviewed-by: Wladimir J. van der Laan (v1)
---
src/gallium/drivers/etnaviv/etnaviv_resource.c | 21 -
This fixes failures to import the scanout buffer with screen resolutions
that don't satisfy the RS alignment restrictions, like 1680x1050.
Signed-off-by: Lucas Stach
Reviewed-by: Christian Gmeiner
Reviewed-by: Philipp Zabel
If we import a dma-buf with a sampler/pixel pipe incompatible modifier,
the imported buffer will end up in an external resource view. As
resource_changed signals the change of the imported resource, we need
to update the external view seqno, instead of the base resource seqno.
Signed-off-by:
Simply advertise all supported modifiers, independent of the format.
Special formats, like compressed, which don't support all those modifiers
are already culled from the dmabuf format list, as we don't support
the render target binding for them.
Signed-off-by: Lucas Stach
This allows the state trackers to know the tiling layout of the
resource and pass this through the various userspace protocols.
Signed-off-by: Lucas Stach
Reviewed-by: Wladimir J. van der Laan
---
src/gallium/drivers/etnaviv/etnaviv_resource.c | 19
The minimum RS alignment calculation is needed in various places.
Extract a helper to avoid open-coding the calcuation at every site.
Signed-off-by: Lucas Stach
---
src/gallium/drivers/etnaviv/etnaviv_resource.c | 14 +-
The current way of importing the resource from renderonly after allocation
is opaque and is taking away control from the driver, which it needs in
order to implement more advanced scenarios, than the simple linear
scanout with matching stride alignments.
Signed-off-by: Lucas Stach
This allows to create buffers with a specific tiling layout, which is primarily
used by GBM to allocate the EGL back buffers with the correct tiling/modifier
for use with the scanout engines.
Signed-off-by: Lucas Stach
---
src/gallium/drivers/etnaviv/etnaviv_resource.c |
This reworks the logic in etna_update_sampler_source to select the
newest resource view for updating the texture view. This should make
the logic easier to follow and fixes texture updates from imported
dma-bufs.
Signed-off-by: Lucas Stach
Reviewed-by: Wladimir J. van der
This implements resource import with modifier, deriving the correct
internal layout from the modifier and constructing a render compatible
base resource if needed.
This removes the special cases for DDX and renderonly scanout allocated
buffers, as the linear modifier is enough to trigger correct
On 4 July 2017 at 16:04, Rafael Avila de Espindola
wrote:
> Emil Velikov writes:
>
>> So the version is FOOgit-$SHA while the DSO uses, FOOsvn. At the same
>> time at least 3.9.0/3.9.1 has broken llvm-config --libnames/libfiles,
>> so we
On 28/06/17 17:54, Cherniak, Bruce wrote:
On Jun 26, 2017, at 2:10 PM, Marek Olšák wrote:
In my opinion, dumping resources isn't very useful. I think it would
be better to remove that completely.
From Michel's response, sounds like dumping resources is useful, so... Back
On 29/06/17 11:02, Michel Dänzer wrote:
On 29/06/17 01:54 AM, Cherniak, Bruce wrote:
On Jun 26, 2017, at 2:10 PM, Marek Olšák wrote:
In my opinion, dumping resources isn't very useful. I think it would
be better to remove that completely.
From Michel's response, sounds
This patch replaces the old register lifetime estiamtion and
rename mapping evaluation with the new one.
Performance to compare between the current and the new implementation
were measured by running the shader-db in one thread.
---
This patch adds a class for tracking the life times of temporary registers
in the glsl to tgsi translation. The algorithm runs in three steps:
First, in order to minimize the number of needed memory allocations the
program is scanned to evaluate the number of scopes.
Then, the program is scanned
The remapping evaluator first sorts the temporary registers ascending
based on their first life time instruction, and then uses a binary search
to find merge canidates.
For the initial sorting it uses std::sort because qsort is quite slow in
comparison. By removing the define USE_STL_SORT in
This patch adds a set of unit tests for the new lifetime tracker.
---
configure.ac |1 +
src/mesa/Makefile.am |2 +-
src/mesa/state_tracker/tests/Makefile.am | 36 +
.../tests/test_glsl_to_tgsi_lifetime.cpp
The patch adds tests for the register rename mapping evaluation and
combined life time estimation and renaming.
---
.../tests/test_glsl_to_tgsi_lifetime.cpp | 192 +
1 file changed, 192 insertions(+)
diff --git
To prepare the implementation of a temp register lifetime tracker
some of the classes are moved into seperate header/implementation
files to make them accessible from other files.
Specifically these are:
class st_src_reg;
class st_dst_reg;
class glsl_to_tgsi_instruction;
struct
Dear Nicolai,
this new version of the patch set that should address all the comments you gave
for v5.
Changes are:
- the components are now tracked individually and the life time of a temporary
is evaluated by merging the life-times of their components,
- BRK/CONT are now handled
This series: Reviewed-by: Sinclair Yeh
On Thu, Jun 22, 2017 at 12:42:32PM +0200, Thomas Hellstrom wrote:
> A patch series that deals with dri3 correctness- and performance fixes.
>
> The corectness fixes attempts to deal with the fact that we need to wait for
> all pending
Reviewed-by: Sinclair Yeh
On Thu, Jun 22, 2017 at 01:00:39PM +0200, Thomas Hellstrom wrote:
> The mesa state tracker was needlessly flushing the front buffer even if it
> hadn't been drawn to since the last flush. This was happening during
> glXSwapBuffers if we at some point
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/freedreno/a5xx/a5xx.xml.h | 57 ++-
1 file changed, 47 insertions(+), 10 deletions(-)
diff --git a/src/gallium/drivers/freedreno/a5xx/a5xx.xml.h
b/src/gallium/drivers/freedreno/a5xx/a5xx.xml.h
This enables S3TC, BPTC, ETC2, and ASTC texture decoding. Additionally
this enables RGB32 texture buffer objects, as well as 11_11_10_FLOAT and
10_10_10_2 vertex formats (and related extensions).
Signed-off-by: Ilia Mirkin
---
This fixes support for BGRA vertex formats
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/freedreno/a5xx/fd5_emit.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/freedreno/a5xx/fd5_emit.c
b/src/gallium/drivers/freedreno/a5xx/fd5_emit.c
index
Am 04.07.2017 um 15:13 schrieb Nicolai Hähnle:
On 01.07.2017 01:03, Andres Rodriguez wrote:
From: Dave Airlie
Signed-off-by: Andres Rodriguez
---
src/gallium/drivers/radeon/r600_pipe_common.h | 6 ++
src/gallium/drivers/radeon/r600_texture.c |
On 01.07.2017 01:02, Andres Rodriguez wrote:
From: Dave Airlie
---
src/gallium/include/pipe/p_screen.h | 36
src/gallium/include/pipe/p_state.h | 2 ++
2 files changed, 38 insertions(+)
diff --git
On 01.07.2017 01:03, Andres Rodriguez wrote:
From: Dave Airlie
Signed-off-by: Andres Rodriguez
---
src/gallium/drivers/radeon/r600_pipe_common.h | 6 ++
src/gallium/drivers/radeon/r600_texture.c | 82 +++
2 files
The number of bindless descriptors is dynamic and we definitely
have to support more than 256 slots.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_state.h | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git
Currently, when the array is full it is resized but it can grow
over and over because we don't try to re-use descriptor slots.
The idea is to maintain two simple lists which keep track of
which slots is used and which ones have been deleted. When the
array is resized, previously deleted slots can
Using VRAM address as bindless handles is not a good idea because
we have to use LLVMIntToPTr and the LLVM CSE pass can't optimize
because it has no information about the pointer.
Instead, use slots indexes like the existing descriptors.
This improves performance with DOW3 by +7%.
A new pair of user SGPR is needed for loading the bindless
descriptors from shaders. Because the descriptors are global for
all stages, there is no need to add separate indices for GFX9.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_shader.c
To share common code between rw buffers and bindless descriptors.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 57 +++
1 file changed, 31 insertions(+), 26 deletions(-)
diff --git
Looks like it's useless to initialize that field when CE is
unused. This will also allow to declare more than 64 elements
for the array of bindless descriptors.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 7 ---
1 file
On 01.07.2017 01:02, Andres Rodriguez wrote:
From: Dave Airlie
---
src/mesa/Makefile.sources| 2 +
src/mesa/state_tracker/st_cb_memoryobjects.c | 65
src/mesa/state_tracker/st_cb_memoryobjects.h | 25 +++
4 lip 2017 14:25 "Robert Foss" napisał(a):
Add local strlcpy implementation.
Signed-off-by: Robert Foss
---
Changes since v3:
Matt Turner
- Change name of util_strncpy to util_strlcpy
Changes since v2:
Brian
Add local strlcpy implementation.
Signed-off-by: Robert Foss
---
Changes since v3:
Matt Turner
- Change name of util_strncpy to util_strlcpy
Changes since v2:
Brian Paul
- Patch added
src/util/u_string.h | 9
Switch to using util_strlcpy to avoid potential overflow of
name array in struct hud_graph.
Also extract the name element length into a define so that it can
be used for strncpy et al.
Coverity-id: 1413761
Signed-off-by: Robert Foss
---
Changes since v3:
Matt
Switch to using util_strlcpy to avoid potential overflow of
name array in struct hud_graph.
Coverity-id: 1413760
Signed-off-by: Robert Foss
---
Changes since v3:
Matt Turner
- Switch from util_strncpy to util_strlcpy
Brian Paul
On Mon, 2017-07-03 at 22:48 -0700, Jason Ekstrand wrote:
> On July 3, 2017 12:31:48 PM Andres Gomez wrote:
>
> > Never too late! ☺
> >
> > On Fri, 2017-06-30 at 16:45 -0700, Jason Ekstrand wrote:
> > > I know this is rather late but
> > >
> > > On June 26, 2017 1:15:16
Am Montag, den 26.06.2017, 12:37 +0200 schrieb Philipp Zabel:
> On Fri, 2017-06-23 at 17:50 +0200, Lucas Stach wrote:
> > The current way of importing the resource from renderonly after allocation
> > is opaque and is taking away control from the driver, which it needs in
> > order to implement
From: Rob Herring
Add etnaviv to Android makefiles.
Signed-off-by: Rob Herring
Reviewed-by: Tapani Pälli
Reviewed-by: Emil Velikov
---
Changes sicne v3:
Emil Velikov
- Add
From: Tomeu Vizoso
Add Android.mk for winsys/imx/drm.
Signed-off-by: Tomeu Vizoso
Reviewed-by: Tapani Pälli
Reviewed-by: Emil Velikov
---
Changes sicne v3:
Emil Velikov
This series enables etnaviv and imx for the android platform.
This is done through updating the Android build scripts.
Rob Herring (1):
android: add etnaviv driver build support
Tomeu Vizoso (1):
android: build imx-drm winsys
Android.mk| 6 +++--
https://bugs.freedesktop.org/show_bug.cgi?id=100627
--- Comment #1 from Emil Velikov ---
Rezny, nicely spotted. The EGL codebase is missing a few checks analogous to
the GLX one. At the same time the i915/i965 are missing a drmGetCap(sPriv->fd,
DRM_CAP_PRIME) call
On 07/03/2017 09:22 PM, Nicolai Hähnle wrote:
On 03.07.2017 18:33, Samuel Pitoiset wrote:
On 07/03/2017 06:16 PM, Samuel Pitoiset wrote:
On 07/03/2017 06:09 PM, Nicolai Hähnle wrote:
On 03.07.2017 18:03, Nicolai Hähnle wrote:
On 29.06.2017 21:59, Samuel Pitoiset wrote:
Only emit
https://bugs.freedesktop.org/show_bug.cgi?id=99956
--- Comment #4 from Emil Velikov ---
Vinson do check the said site. Alternatively you can see/work with the FreeBSD
people - they seems to have some partial solution there [1]
[1]
Hi Connor,
Can confirm this does work here, thanks!
Makes this patch redundant too, since that problem's also fixed by
your changes:
https://lists.freedesktop.org/archives/mesa-dev/2017-June/161559.html
Alex
On 4 July 2017 at 04:32, Connor Abbott wrote:
> I'm going to do
On 03.07.2017 14:34, Nicolai Hähnle wrote:
From: Nicolai Hähnle
Save some passes over the IR.
v2: redesign to make the users of find_assignments more readable
Reviewed-by: Ian Romanick (v1)
---
src/compiler/glsl/linker.cpp | 120
On Friday, 2017-06-30 15:06:36 -0700, Ian Romanick wrote:
> Has this been reported to upstream (i.e., Khronos)? I don't mind fixing
> these in our tree, but they really should get fixed at the source too.
I can't see any mention of this on neither the old Bugzilla (both
private and public) nor
Hi
I'm not seeing this issue on Gentoo. My llvm-config --version shows
"5.0.0git-79da0992d18"
I'm attaching the patch we use
Regards
Mike
On Tue, 4 Jul 2017 at 10:10 Emil Velikov wrote:
> On 3 July 2017 at 22:51, Andy Furniss wrote:
> > Emil
On 3 July 2017 at 22:51, Andy Furniss wrote:
> Emil Velikov wrote:
>>
>> On 3 July 2017 at 16:31, Andy Furniss wrote:
>>>
>>> Rafael Avila de Espindola wrote:
What check is configure doing?
>>>
>>>
>>>
>>> Not sure just a user.
>>>
Is
Am Montag, den 03.07.2017, 17:18 +0200 schrieb Marek Olšák:
> Reviewed-by: Marek Olšák
Thanks, pushed.
> On Fri, Jun 30, 2017 at 11:03 AM, Philipp Zabel
> wrote:
> > The smapi->get_egl_image() call in st_egl_image_get_surface() stores a
> >
https://bugs.freedesktop.org/show_bug.cgi?id=101560
programmerj...@gmail.com changed:
What|Removed |Added
Component|Drivers/Vulkan/intel|Drivers/Vulkan/radeon
https://bugs.freedesktop.org/show_bug.cgi?id=97957
--- Comment #19 from Thomas Hellström ---
Hi!
I'm on vacation and will be back on Monday July 31st. For vmware linux graphics
driver issues, please contact
Thanks,
Thomas Hellström
https://bugs.freedesktop.org/show_bug.cgi?id=97957
Michel Dänzer changed:
What|Removed |Added
Attachment #132276|0 |1
is
https://bugs.freedesktop.org/show_bug.cgi?id=99956
Vinson Lee changed:
What|Removed |Added
Version|git |17.1
--
You are
71 matches
Mail list logo