[Spice-commits] 3 commits - AUTHORS server/tests

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Mon Dec 7 11:13:59 UTC 2020


 AUTHORS                          |    1 
 server/tests/valgrind/glib.supp  |  118 ++++++++++++++++++++++++++++++++++-----
 server/tests/valgrind/spice.supp |   41 ++++++++++++-
 3 files changed, 144 insertions(+), 16 deletions(-)

New commits:
commit 7da855b6f391348748085f04d1507f5a013f64f9
Author: Julien Ropé <jrope at redhat.com>
Date:   Wed Dec 2 15:10:39 2020 +0100

    Add my name to the AUTHORS file to fix CI
    
    Signed-off-by: Julien Ropé <jrope at redhat.com>
    Acked-by: Frediano Ziglio <fziglio at redhat.com>

diff --git a/AUTHORS b/AUTHORS
index 05c39998..f3c04a00 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -46,6 +46,7 @@ Patches also contributed by
     Hans de Goede <hdegoede at redhat.com>
     Javier Celaya <javier.celaya at flexvm.es>
     Jeremy White <jwhite at codeweavers.com>
+    Julien Ropé <jrope at redhat.com>
     Jürg Billeter <j at bitron.ch>
     Li Zhijian <lizhijian at cn.fujitsu.com>
     Liang Guo <bluestonechina at gmail.com>
commit d8c6ff25b68a68c34bf50ee4784d707f8b7b0df8
Author: Julien Ropé <jrope at redhat.com>
Date:   Mon Dec 7 10:40:42 2020 +0100

    ci: suppress new possible loss coming with Fedora 33
    
    - remove entry for g-type-register-static: the "possible" flag has been
      re-introduced in glib.supp
    - add new entries for several calloc issues
    
    Signed-off-by: Julien Ropé <jrope at redhat.com>
    Acked-by: Frediano Ziglio <fziglio at redhat.com>

diff --git a/server/tests/valgrind/spice.supp b/server/tests/valgrind/spice.supp
index bc85445a..63203dcc 100644
--- a/server/tests/valgrind/spice.supp
+++ b/server/tests/valgrind/spice.supp
@@ -81,11 +81,46 @@
 	fun:fake_client_connect_tls
 }
 
+# suppressing several new "possible" loss coming with Fedora 33 on calloc
 {
-    g-type-register-static-possible
+    # this one is making most tests to fail (20 out of 21)
+    calloc-possibly-lost-from-dl_init
     Memcheck:Leak
     match-leak-kinds:possible
-    fun:malloc
+    fun:calloc
+    fun:g_malloc0
     ...
-    fun:g_type_register_static
+    fun:_dl_init
+}
+
+{
+    calloc-possibly-lost-from-fake_client_connect-fake_client_new
+    Memcheck:Leak
+    match-leak-kinds:possible
+    fun:calloc
+    fun:g_malloc0
+    ...
+    fun:g_object_new
+    fun:fake_client_*
+}
+
+{
+    calloc-possibly-lost-from-g_output_stream
+    Memcheck:Leak
+    match-leak-kinds:possible
+    fun:calloc
+    ...
+    fun:g_thread_pool_push
+    fun:g_task_run_in_thread
+    ...
+    fun:g_output_stream_write_all
+}
+
+{
+    calloc-possibly-lost-from-g_thread_new
+    Memcheck:Leak
+    match-leak-kinds:possible
+    fun:calloc
+    ...
+    fun:g_thread_new
 }
commit 5f06f847dd15a76c615cdfc44696569227311f17
Author: Julien Ropé <jrope at redhat.com>
Date:   Mon Dec 7 10:38:43 2020 +0100

    ci: update glib.supp file
    
    Sync with Glib master file.
    
    Signed-off-by: Julien Ropé <jrope at redhat.com>
    Acked-by: Frediano Ziglio <fziglio at redhat.com>

diff --git a/server/tests/valgrind/glib.supp b/server/tests/valgrind/glib.supp
index 3f7b53eb..06c5f542 100644
--- a/server/tests/valgrind/glib.supp
+++ b/server/tests/valgrind/glib.supp
@@ -73,8 +73,6 @@
 	fun:malloc
 	...
 	fun:g_quark_init
-	...
-	fun:glib_init_ctor
 }
 
 {
@@ -84,8 +82,6 @@
 	fun:calloc
 	...
 	fun:g_quark_init
-	...
-	fun:glib_init_ctor
 }
 
 {
@@ -127,7 +123,7 @@
 {
 	g-type-register-static
 	Memcheck:Leak
-	match-leak-kinds:reachable
+	match-leak-kinds:possible,reachable
 	fun:malloc
 	...
 	fun:g_type_register_static
@@ -145,7 +141,7 @@
 {
 	g-type-register-static-calloc
 	Memcheck:Leak
-	match-leak-kinds:reachable
+	match-leak-kinds:possible,reachable
 	fun:calloc
 	...
 	fun:g_type_register_static
@@ -218,7 +214,7 @@
 }
 
 {
-	g-test-rand-init2
+	g-rand-init2
 	Memcheck:Leak
 	match-leak-kinds:reachable
 	fun:calloc
@@ -226,8 +222,6 @@
 	fun:g_rand_new_with_seed_array
 	...
 	fun:get_global_random
-	...
-	fun:g_test_init
 }
 
 {
@@ -445,6 +439,33 @@
 	fun:g_get_language_names
 }
 
+{
+        g-get-language_names-with-category-malloc
+        Memcheck:Leak
+        match-leak-kinds:reachable
+        fun:malloc
+        ...
+        fun:g_get_language_names_with_category
+}
+
+{
+        g-get-language_names-with-category-calloc
+        Memcheck:Leak
+        match-leak-kinds:reachable
+        fun:calloc
+        ...
+        fun:g_get_language_names_with_category
+}
+
+{
+        g-get-language_names-with-category-realloc
+        Memcheck:Leak
+        match-leak-kinds:reachable
+        fun:realloc
+        ...
+        fun:g_get_language_names_with_category
+}
+
 {
 	g-static-mutex
 	Memcheck:Leak
@@ -463,6 +484,17 @@
 	fun:g_system_thread_new
 }
 
+{
+	g-task-thread-pool-init
+	Memcheck:Leak
+	match-leak-kinds:possible,reachable,definite
+	fun:malloc
+	...
+	fun:g_thread_new
+	...
+	fun:g_task_thread_pool_init
+}
+
 {
 	g-io-module-default-proxy-resolver-gnome
 	Memcheck:Leak
@@ -478,7 +510,7 @@
 {
 	g-threaded-resolver-getaddrinfo-config
 	Memcheck:Leak
-	match-leak-kinds:reachable
+	match-leak-kinds:reachable,definite
 	fun:malloc
 	...
 	fun:__resolv_conf_allocate
@@ -876,11 +908,21 @@
 	fun:g_dbus_error_quark
 }
 
+# g_win32_registry_get_os_dirs_w*() caches an array of strings that is allocated only once.
+{
+	g_win32_registry_get_os_dirs
+	Memcheck:Leak
+	match-leak-kinds:reachable,definite
+	fun:malloc
+	...
+	fun:g_win32_registry_get_os_dirs*
+}
+
 # Thread-private data allocated once per thread
 {
 	g_private_set_alloc0
 	Memcheck:Leak
-	match-leak-kinds:reachable
+	match-leak-kinds:definite,reachable
 	fun:malloc
 	...
 	fun:g_private_set_alloc0
@@ -888,17 +930,26 @@
 {
 	g_private_set_alloc0-calloc
 	Memcheck:Leak
-	match-leak-kinds:reachable
+	match-leak-kinds:definite,reachable
 	fun:calloc
 	...
 	fun:g_private_set_alloc0
 }
 
+# Keys for thread-private data
+{
+	g_private_key
+	Memcheck:Leak
+	match-leak-kinds:reachable
+	fun:malloc
+	fun:g_private_impl_new
+}
+
 # Thread-private GMainContext stack
 {
 	g_main_context_push_thread_default
 	Memcheck:Leak
-	match-leak-kinds:reachable
+	match-leak-kinds:definite,reachable
 	fun:malloc
 	...
 	fun:g_queue_new
@@ -946,3 +997,44 @@
 	...
 	fun:g_file_*
 }
+
+# Cached charset
+{
+	g_get_charset
+	Memcheck:Leak
+	match-leak-kinds:reachable
+	fun:malloc
+	...
+	fun:g_get_charset
+}
+
+{
+	g_get_charset_calloc
+	Memcheck:Leak
+	match-leak-kinds:reachable
+	fun:calloc
+	...
+	fun:g_get_charset
+}
+
+# Global unused thread queue
+{
+	g_thread_pool_unused_thread_queue
+	Memcheck:Leak
+	match-leak-kinds:reachable
+	fun:malloc
+	...
+	fun:g_async_queue_new_full
+	...
+	fun:g_thread_pool_new
+}
+
+# One-time program name storage
+{
+	g_set_prgname
+	Memcheck:Leak
+	match-leak-kinds:reachable
+	fun:malloc
+	...
+	fun:g_set_prgname
+}


More information about the Spice-commits mailing list