Re: [Mesa-dev] [PATCH] st/mesa: don't do L3 thread pinning for Blender

2018-11-13 Thread Edmondo Tommasina
Hi Marek Sure. Thanks for writing these patches. The looks good. I've done some small testing: drawoverhead numbers looks great in my eyes: 29: DrawElements ( 1 VBO, 8 UBO, 8 Tex) w/ sample mask enable change: 6.63 million (94.7%) Hitman benchmark runs nicely, even slightly bit faster tha

Re: [Mesa-dev] [PATCH] st/mesa: don't do L3 thread pinning for Blender

2018-11-12 Thread Marek Olšák
Hi Edmondo, can you test the two attached patches? They re-enable and rework the thread pinning. Thanks, Marek On Mon, Nov 12, 2018 at 4:31 PM Edmondo Tommasina < edmondo.tommas...@gmail.com> wrote: > On Mon, Nov 12, 2018 at 6:43 PM Michel Dänzer wrote: > >> On 2018-11-08 6:23 a.m., Marek Olšá

Re: [Mesa-dev] [PATCH] st/mesa: don't do L3 thread pinning for Blender

2018-11-12 Thread Edmondo Tommasina
On Mon, Nov 12, 2018 at 6:43 PM Michel Dänzer wrote: > On 2018-11-08 6:23 a.m., Marek Olšák wrote: > > Thanks a lot man. I'll reconsider this depending on the results I > receive. > > > > I may also just pin the Mesa threads and keep the app thread intact. It > > should perform OK with glthread,

Re: [Mesa-dev] [PATCH] st/mesa: don't do L3 thread pinning for Blender

2018-11-12 Thread Michel Dänzer
On 2018-11-08 6:23 a.m., Marek Olšák wrote: > Thanks a lot man. I'll reconsider this depending on the results I receive. > > I may also just pin the Mesa threads and keep the app thread intact. It > should perform OK with glthread, but not without glthread. > > Another option is to have the galli

Re: [Mesa-dev] [PATCH] st/mesa: don't do L3 thread pinning for Blender

2018-11-07 Thread Marek Olšák
Thanks a lot man. I'll reconsider this depending on the results I receive. I may also just pin the Mesa threads and keep the app thread intact. It should perform OK with glthread, but not without glthread. Another option is to have the gallium and winsys threads "chase" the main thread within the

Re: [Mesa-dev] [PATCH] st/mesa: don't do L3 thread pinning for Blender

2018-11-06 Thread Edmondo Tommasina
Hi Marek It would be nice to have the driconf part of this patch committed in master to make it easy to test with and without the L3 pinning, so this patch is: Reviewed-by: Edmondo Tommasina Now with this patch in place I'm starting to collect some numbers with and without the CCX affinity on m

[Mesa-dev] [PATCH] st/mesa: don't do L3 thread pinning for Blender

2018-10-30 Thread Marek Olšák
From: Marek Olšák so that all Blender threads are not forced to be on 1 CCX. Fixes: 8d473f555a0 --- src/gallium/auxiliary/pipe-loader/driinfo_gallium.h | 1 + src/gallium/include/state_tracker/st_api.h | 1 + src/gallium/state_trackers/dri/dri_screen.c | 2 ++ src/mesa/state_tr