[PATCH 1/2] winegstreamer: Use affinity to determine 'max-threads' value.

[Brendan McGrath]

From: Brendan McGrath

GStreamer uses _SC_NPROCESSORS_CONF to determine 'max-threads'. On the
Steam Deck, this is configured to be 16 (which is double its number
of logical cores).

_SC_NPROCESSORS_CONF also disregards a process's CPU affinity, thus it
can create more threads than is useful, which ultimately wastes memory
resources.

Using affinity to set 'max-threads' addresses both these problems.
---
 dlls/winegstreamer/wg_parser.c | 35 ++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

diff --git a/dlls/winegstreamer/wg_parser.c b/dlls/winegstreamer/wg_parser.c
index 7253013b6a3..dc820df2d24 100644
--- a/dlls/winegstreamer/wg_parser.c
+++ b/dlls/winegstreamer/wg_parser.c
@@ -567,6 +567,40 @@ static void no_more_pads_cb(GstElement *element, gpointer user)
     pthread_cond_signal(&parser->init_cond);
 }

+static void deep_element_added_cb(GstBin *self, GstBin *sub_bin, GstElement *element, gpointer user)
+{
+    GstElementFactory *factory = NULL;
+    const char *name = NULL;
+
+    if (element)
+        factory = gst_element_get_factory(element);
+
+    if (factory)
+        name = gst_element_factory_get_longname(factory);
+
+    if (name && strstr(name, "libav H.264"))
+    {
+        const gint32 MAX_THREADS = 16;
+        gint32 max_threads;
+
+#ifdef HAVE_SCHED_SETAFFINITY
+        gint32 thread_count;
+        cpu_set_t p_aff;
+        memset( &p_aff, 0, sizeof(p_aff) );
+        if( sched_getaffinity( 0, sizeof(p_aff), &p_aff ) )
+            thread_count = 0;
+        else
+            thread_count = CPU_COUNT(&p_aff);
+#else
+        /* A thread_count of 0 is the default value for gstreamer and means 'auto' */
+        gint32 thread_count = 0;
+#endif
+        max_threads = MIN(thread_count, MAX_THREADS);
+        GST_DEBUG("%s found, setting max-threads to %d.", name, max_threads);
+        g_object_set(element, "max-threads", max_threads, NULL);
+    }
+}
+
 static gboolean sink_event_cb(GstPad *pad, GstObject *parent, GstEvent *event)
 {
     struct wg_parser_stream *stream = gst_pad_get_element_private(pad);
@@ -1797,6 +1831,7 @@ static BOOL decodebin_parser_init_gst(struct wg_parser *parser)
     g_signal_connect(element, "autoplug-continue", G_CALLBACK(autoplug_continue_cb), parser);
     g_signal_connect(element, "autoplug-select", G_CALLBACK(autoplug_select_cb), parser);
     g_signal_connect(element, "no-more-pads", G_CALLBACK(no_more_pads_cb), parser);
+    g_signal_connect(element, "deep-element-added", G_CALLBACK(deep_element_added_cb), parser);

pthread_mutex_lock(&parser->mutex);
     parser->no_more_pads = false;