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

[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.tommasina at gmail.com> wrote:

> On Mon, Nov 12, 2018 at 6:43 PM Michel Dänzer <michel at daenzer.net> 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, but not without glthread.
>> >
>> > Another option is to have the gallium and winsys threads "chase" the
>> main
>> > thread within the CPU by changing the thread affinity based on getcpu().
>>
>> While those are interesting ideas for the future, I'm afraid it's too
>> late for them for the 18.3.0 release (scheduled for November 21st IIRC).
>>
>> Please make sure the thread pinning code is disabled for the release, at
>> least by default.
>>
>
> I'm not sure what the best solution is, but pinning the threads to
> the L3 CCX has shown great potential on my Ryzen 5 2600 and it would
> be nice to explore the ideas presented by Marek or maybe understand,
> why the kernel scheduler prefers to put the threads on cores on
> different CCX.
>
> For example The Wicher 2 goes from 60 FPS to 70 FPS average and this
> is impressive. Tomb Raider just increases about 1 FPS (average 104 FPS)
> but this can be just noise and for sure not noticeable.
>
> Regards
> edmondo
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20181112/0a77e28b/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-st-mesa-regularly-re-pin-driver-threads-to-the-CCX-w.patch
Type: application/x-patch
Size: 2907 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20181112/0a77e28b/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-st-mesa-pin-driver-threads-to-a-fixed-CCX-when-glthr.patch
Type: application/x-patch
Size: 5710 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20181112/0a77e28b/attachment-0001.bin>