https://bugs.freedesktop.org/show_bug.cgi?id=99956
Vinson Lee changed:
What|Removed |Added
Version|git |17.1
--
You are receiving this mail becaus
https://bugs.freedesktop.org/show_bug.cgi?id=97957
Michel Dänzer changed:
What|Removed |Added
Attachment #132276|0 |1
is obsolete|
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
--
You are receiving this mail because:
You are the QA Conta
https://bugs.freedesktop.org/show_bug.cgi?id=101560
programmerj...@gmail.com changed:
What|Removed |Added
Component|Drivers/Vulkan/intel|Drivers/Vulkan/radeon
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
> > reference to the EGLImage's texture in stimg.texture.
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 the llvm build a clean one? What is the va
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 Velikov wrote:
> >>
> >> On 3 July 2017 at 16:31,
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 in
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 ---
1 file changed,
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 a full CTS run later
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]
https://github.com/freebsd/freebsd-ports-graphics/blob/master/gra
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 partia
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 analogous to st/dri.
Can you sp
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
- Add GALLIUM_TARGET_DRIVERS addition
Changes since v2:
- Rebased on upstream/master
- Added commit message
Cha
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 +++--
src/gallium
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 GALLIUM_TARGET_DRIVERS addition
Changes since v2:
- Rebased on upstream/master
Changes since v1:
Tapani Pälli
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 mor
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 PM Andres Gomez wrot
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 Turner
- Switch from util_str
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
- Fix comment to mention the correct string copy func
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 +
1 file changed, 9 insertions(+)
diff --git a/src/util/u_strin
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 Paul
- Patch added
src/util/u_string.h | 9 +
1 file changed, 9
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 +++
src/mesa/state_tracker/st_c
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 changed, 4 insertions(+), 3 deleti
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 a/src/gallium/drivers/radeonsi/si_descriptors.c
b/sr
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%.
Signed-off-by:
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 | 9 +++--
src/galli
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 a/src/gallium/drivers/radeonsi/si_state.h
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
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 changed, 88 insertions(+)
diff --git a/src/gall
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 a/src/gallium/include/pipe/p_screen.h
b/src/gallium/i
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 | 82
+++
2 file
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 f5c0bd27c35..287ec24b18
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
index abcc53965ad..ee614653
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
---
src/gallium/drivers/freedreno/a5xx/fd5_format.c | 138 ---
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 previously had set
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 differen
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 swapbuffers before we
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 re
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 a/src/mesa/state_tracker/tests/test_glsl_to_tgsi_lifetim
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 s
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
src/
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 like dumping resource
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 to
my original
qu
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 cannot rely on that.
>> Doesn't seem like LLVM is giving us
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 Laan
---
src/gallium/d
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 h
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 +-
src/gallium/drivers/etnaviv/etnaviv_translate.h | 13 +
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 | 95 +
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
---
src/galliu
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: Lucas
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
---
v2: don't advertise
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 +++
1 file changed, 19 insert
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
Reviewed-by: Wladimir J. van der Laan
---
src/gallium/drivers/etnaviv
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 -
src/gallium/drivers/etnaviv/etnaviv_screen.c | 19 --
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 insertions(+)
diff --git a/s
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 the version is FOOgit-$SHA while the DSO uses, FOOsvn. At the same
time at least 3.9.0/3.9.1 has broken ll
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 ++
src/gallium/drivers/radeon/r600_texture
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
---
Tim, this does _not_ quite work, hence I'm sending it as RFC.
The current build failures can be seen here
Autotools
https://travis-ci.org/evelikov/Mesa/jo
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.
___
mesa
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
signa
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
src/mesa/state_tracker/st_cb_memoryobjec
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
b/src/gallium/winsys/radeon/drm/radeon_drm_
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 instead of an explicit padding member variable
---
s
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 +
> 1 file cha
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(-)
diff --git a/src/a
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)
---
src/compiler/glsl/linker.cpp | 120
++
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 has broken llvm-config --libnames/libfiles,
>>> so we cannot rely on that.
>>> Do
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.
>
> LLVM users/devs (Rafael),
Add local strlcpy implementation.
Signed-off-by: Robert Foss
---
Changes since v4:
Gustaw Smolarczyk
- Make util_strlcpy have the same behaviour as strlcpy
Changes since v3:
Matt Turner
- Change name of util_strncpy to util_strlcpy
Changes since v2:
Brian Paul
- Patch added
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
- Fix comment to mention the correct string copy func
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 Turner
- Switch from util_str
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 Turner
- Switch from util_str
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
- Fix comment to mention the correct string copy func
Add local strlcpy implementation.
Signed-off-by: Robert Foss
---
Changes since v5:
Actually include changes from v5 in patch
Changes since v4:
Gustaw Smolarczyk
- Make util_strlcpy have the same behaviour as strlcpy
Changes since v3:
Matt Turner
- Change name of util_strncpy to ut
77 matches
Mail list logo
