[Mesa-dev] [PATCH] st/mesa: disable L3 thread pinning

[Marek Olšák]

Thanks, pushed.

Marek

On Mon, Nov 12, 2018 at 4:14 PM Edmondo Tommasina <
edmondo.tommasina at gmail.com> wrote:

>
> Patch is:
> Reviewed-by: Edmondo Tommasina <edmondo.tommasina at gmail.com>
>
>
> On Mon, Nov 12, 2018 at 9:45 PM Marek Olšák <maraeo at gmail.com> wrote:
>
>> From: Marek Olšák <marek.olsak at amd.com>
>>
>> This implementation can have massive drawbacks.
>>
>> Cc: 18.3 <mesa-stable at lists.freedesktop.org>
>> ---
>>  src/mesa/state_tracker/st_manager.c | 9 ---------
>>  1 file changed, 9 deletions(-)
>>
>> diff --git a/src/mesa/state_tracker/st_manager.c
>> b/src/mesa/state_tracker/st_manager.c
>> index 690d5bc2313..076ad42646d 100644
>> --- a/src/mesa/state_tracker/st_manager.c
>> +++ b/src/mesa/state_tracker/st_manager.c
>> @@ -1065,29 +1065,20 @@ st_api_make_current(struct st_api *stapi, struct
>> st_context_iface *stctxi,
>>           ret = _mesa_make_current(st->ctx, incomplete, incomplete);
>>        }
>>
>>        st_framebuffer_reference(&stdraw, NULL);
>>        st_framebuffer_reference(&stread, NULL);
>>
>>        /* Purge the context's winsys_buffers list in case any
>>         * of the referenced drawables no longer exist.
>>         */
>>        st_framebuffers_purge(st);
>> -
>> -      /* Notify the driver that the context thread may have been changed.
>> -       * This should pin all driver threads to a specific L3 cache for
>> optimal
>> -       * performance on AMD Zen CPUs.
>> -       */
>> -      struct glthread_state *glthread = st->ctx->GLThread;
>> -      thrd_t *upper_thread = glthread ? &glthread->queue.threads[0] :
>> NULL;
>> -
>> -      util_context_thread_changed(st->pipe, upper_thread);
>>     }
>>     else {
>>        ret = _mesa_make_current(NULL, NULL, NULL);
>>     }
>>
>>     return ret;
>>  }
>>
>>
>>  static void
>> --
>> 2.17.1
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20181112/db02f13c/attachment.html>