update tsan suppression

improve comments and add a gio function

Change-Id: I4f3d57786a280093390db966cb64abb23246ebe1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138136
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

tsan-suppress.txt

@@ -13,11 +13,11 @@
 deadlock:cppuhelper::ServiceManager::disposing()
 deadlock:cppuhelper::ServiceManager::loadImplementation
 
-# Ignore stuff in external DBUS library
-# Some kind of dbus lock issue when we call it from psp::CUPSManager
+# Ignore stuff in external DBUS library.
+# Some kind of dbus lock issue when we call it from psp::CUPSManager.
 deadlock:_dbus_lock
 
-# inside an assert so I don't care
+# inside an assert so I don't care.
 race:cppu::OWeakConnectionPoint::acquire
 race:AffineBridge::v_enter
 race:__vsnprintf_chk
@@ -35,16 +35,16 @@ race:rtl_uString_release
 race:rtl_uString_assign
 
 # I've convinced myself this is a false+, caused by ping-ponging the buffer between two
-# threads, but I might be wrong
+# threads, but I might be wrong.
 race:XBufferedThreadedStream::getNextBlock
 
-# not introduced in stuff the embedded JVM does
+# I am not interested in stuff the embedded JVM does.
 race:libjvm.so
 
 # I think this is OK, because at this point we are doing
 #  if (nRefCount > 1)
 # and we know from our callers that the refcount must be at least one
-# so there is no failure mode
+# so there is no failure mode.
 race:ireallocSequence
 
 # TODO There appears to be a race here, initialising the
@@ -53,10 +53,11 @@ race:ireallocSequence
 race:cppu::getTypeFavourUnsigned
 
 
-# This is all inside GIO/Glib, no idea what it is doing
+# This is all inside GIO/Glib, no idea what it is doing.
 #
 race:slab_allocator_alloc_chunk
 race:g_source_destroy_internal
 race:g_source_unref_internal
 race:g_task_finalize
 race:g_socket_send_message_with_timeout
+race:g_idle_source_new