[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-09-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #13 from hofmann.zach...@gmail.com --- I can also confirm this problem with Radeon R7 250 (radeonsi), Mesa 12.0.2, LLVM 3.8.1 and kernel version 4.6.0. -- You are receiving this mail because: You are the assignee for the

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-09-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #16 from Tobias Droste --- I tried to do some profiling but with callgrind it's impossible because after half an hour I'm not even in the game creation menu. I tried zoom as an alternative (never used it before) and

Re: [Mesa-dev] [RFC 1/7] eglplatform.h: introduce and use EGL_USE_PLATFORM_*_KHR

2016-09-24 Thread Eric Engestrom
On Thu, Sep 22, 2016 at 09:38:06AM +0100, Emil Velikov wrote: > From: Emil Velikov > > In order to avoid the current, somewhat fragile detection in > eglplatform.h introduce explicit platform selection. > > The approach is based on the one used in Vulkan and allows

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-09-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #15 from Tobias Droste --- Created attachment 126771 --> https://bugs.freedesktop.org/attachment.cgi?id=126771=edit Profile of the the first few seconds of a game (only radeonsi) -- You are receiving this mail

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-09-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #14 from Tobias Droste --- Created attachment 126770 --> https://bugs.freedesktop.org/attachment.cgi?id=126770=edit Profile of the the first few seconds of a game (all components) -- You are receiving this mail

Re: [Mesa-dev] [PATCH v2 6/7] egl/x11: attribute for dri2_add_config failure

2016-09-24 Thread Eric Engestrom
On Fri, Sep 16, 2016 at 06:23:05PM +0100, Emil Velikov wrote: > From: Emil Velikov > > ... in dri2_x11_add_configs_for_visuals(). > > Currently the latter does not consider that, thus in such cases it adds > "empty" configs in the list. > > Properly account for

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-09-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #18 from Tobias Droste --- Created attachment 126773 --> https://bugs.freedesktop.org/attachment.cgi?id=126773=edit Profile of a few seconds in the middle of the game (only radeonsi) -- You are receiving this mail

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-09-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #19 from Tobias Droste --- And I added screenshots of the profile for a few seconds where the game is running fine. I figured that it might be interesting where the difference between good and bad is. -- You are

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-09-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #17 from Tobias Droste --- Created attachment 126772 --> https://bugs.freedesktop.org/attachment.cgi?id=126772=edit Profile of a few seconds in the middle of the game (all components) -- You are receiving this

[Mesa-dev] [PATCH mesa 2/2] mapi: add asserts to enforce ordering and uniqueness

2016-09-24 Thread Eric Engestrom
Dylan Baker recently added functions to that list and had to try a couple times to avoid duplicates. He said [1] he ended up testing it using: len(functions) == len(set(functions)) which I thought should always be done. Add this and a couple other tests using asserts to enforce the ordering and

Re: [Mesa-dev] [PATCH 28/30] loader/dri3: constify the loader_dri3_vtable

2016-09-24 Thread Eric Engestrom
On Thu, Aug 25, 2016 at 05:18:50PM +0100, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov Oops, forgot this one, which is also: Reviewed-by: Eric Engestrom > --- >

Re: [Mesa-dev] [PATCH mesa 2/2] mapi: add asserts to enforce ordering and uniqueness

2016-09-24 Thread Ilia Mirkin
On Sat, Sep 24, 2016 at 10:17 PM, Eric Engestrom wrote: > Dylan Baker recently added functions to that list and had to try a couple > times > to avoid duplicates. He said [1] he ended up testing it using: > len(functions) == len(set(functions)) > which I thought should

[Mesa-dev] [Bug 77449] Tracker bug for all bugs related to Steam titles

2016-09-24 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=77449 Vedran Miletić changed: What|Removed |Added Depends on||97917 Referenced

Re: [Mesa-dev] [PATCH 22/30] egl/dri2: coding style cleanup

2016-09-24 Thread Eric Engestrom
On Thu, Aug 25, 2016 at 05:18:44PM +0100, Emil Velikov wrote: > From: Emil Velikov > > Consistently indent with space rather than a mix of tab and > spaces. > > Signed-off-by: Emil Velikov > --- > src/egl/drivers/dri2/egl_dri2.c | 286 >

Re: [Mesa-dev] [PATCH 10/30] egl/dri2: rework dri2_make_current code flow

2016-09-24 Thread Eric Engestrom
On Thu, Aug 25, 2016 at 05:18:32PM +0100, Emil Velikov wrote: > From: Emil Velikov > > Duplicate a few lines at the expense of making the code-flow clearer > while adding a few extra comments ;-) > > Signed-off-by: Emil Velikov > --- > >

[Mesa-dev] [PATCH mesa 1/2] mapi: sort functions list

2016-09-24 Thread Eric Engestrom
An interesting side-effect of sorting the list is that is makes lowercase come after uppercase, which makes case-typo such as s/TexBufferRange/TexbufferRange/ evident right away. Such a typo was spotted in a recent patch to this function list. The next patch enforces this ordering, making it

Re: [Mesa-dev] [PATCH 11/30] egl/android: don't populate dri2_dpy->dri2_loader_extension

2016-09-24 Thread Eric Engestrom
On Thu, Aug 25, 2016 at 05:18:33PM +0100, Emil Velikov wrote: > From: Emil Velikov > > The extension stored in struct dri2_egl_display isn't used, thus we can > create a static const instance of the extension and point extensions[] > to it. > > Signed-off-by: Emil

Re: [Mesa-dev] [PATCH mesa 2/2] mapi: add asserts to enforce ordering and uniqueness

2016-09-24 Thread Ilia Mirkin
Also note... the list is already sorted. You just picked a different sort order. One that I'm pretty sure is subject to locale settings. As I recall C and utf8 do things differently in a way relevant to this list but I haven't checked. On Sep 24, 2016 10:27 PM, "Ilia Mirkin"

Re: [Mesa-dev] [PATCH mesa 2/2] mapi: add asserts to enforce ordering and uniqueness

2016-09-24 Thread Eric Engestrom
On Sat, Sep 24, 2016 at 11:39:39PM -0400, Ilia Mirkin wrote: > Also note... the list is already sorted. You just picked a different sort > order. One that I'm pretty sure is subject to locale settings. As I recall > C and utf8 do things differently in a way relevant to this list but I > haven't

Re: [Mesa-dev] [PATCH mesa 2/2] mapi: add asserts to enforce ordering and uniqueness

2016-09-24 Thread Ilia Mirkin
On Sun, Sep 25, 2016 at 12:27 AM, Eric Engestrom wrote: > On Sat, Sep 24, 2016 at 11:39:39PM -0400, Ilia Mirkin wrote: >> Also note... the list is already sorted. You just picked a different sort >> order. One that I'm pretty sure is subject to locale settings. As I recall >> C

Re: [Mesa-dev] [PATCH v2 30/30] egl/dri2: set WL_bind_wayland_display in a consistent way

2016-09-24 Thread Eric Engestrom
On Fri, Sep 16, 2016 at 06:02:11PM +0100, Emil Velikov wrote: > From: Emil Velikov > > Introduce a helper and use it throughout the platform code. This allows > us to reduce the amount of ifdef(s) and (potentially) use > kms_swrast_dri.so for !drm platforms (namely

Re: [Mesa-dev] [PATCH mesa 2/2] mapi: add asserts to enforce ordering and uniqueness

2016-09-24 Thread Eric Engestrom
On Sat, Sep 24, 2016 at 10:27:52PM -0400, Ilia Mirkin wrote: > On Sat, Sep 24, 2016 at 10:17 PM, Eric Engestrom wrote: > > Dylan Baker recently added functions to that list and had to try a couple > > times > > to avoid duplicates. He said [1] he ended up testing it using: > >

[Mesa-dev] [PATCH mesa v2 2/2] mapi: add asserts to enforce ordering and uniqueness

2016-09-24 Thread Eric Engestrom
Dylan Baker recently added functions to that list and had to try a couple times to avoid duplicates. He said [1] he ended up testing it using: len(functions) == len(set(functions)) which I thought should always be done. Add this and a couple other tests using asserts to enforce the ordering and