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 and utf8 do things
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 che
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
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:
> > len(functions) ==
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" wrote:
> On Sat, Sep
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
> ---
> src/egl/drivers/dri2/platform_x11_dri3.c | 2 +-
> src/glx/dri3_glx.c | 2 +-
> s
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 wayland and x11).
>
> Note: in
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 Velikov
> ---
> src/egl/drive
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
> ---
>
> It makes things clearer the way on this end, but if peop
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
>
> src/egl/driv
https://bugs.freedesktop.org/show_bug.cgi?id=77449
Vedran Miletić changed:
What|Removed |Added
Depends on||97917
Referenced Bugs:
https://bugs.f
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 always be done.
>
> Add t
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
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 easier
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 receiving this mail
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&action=edit
Profile of a few seconds in the middle of the game (only radeonsi)
--
You are receiving this mail because:
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&action=edit
Profile of a few seconds in the middle of the game (all components)
--
You are receiving this mail because:
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 the attached
scre
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&action=edit
Profile of the the first few seconds of a game (all components)
--
You are receiving this mail because:
You
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&action=edit
Profile of the the first few seconds of a game (only radeonsi)
--
You are receiving this mail because:
You
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 things and as we do that we can r
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 one to
> explicitly "request"
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 bug.
23 matches
Mail list logo