Re: [PATCH 4/8] util: Initialize virLogMutex statically

[Erik Skultety]

On Tue, Jan 04, 2022 at 02:47:08PM +0100, Martin Kletzander wrote:
> The only difference is that we are not going to be guaranteed that the mutex 
> is
> normal (as opposed to recursive, although there is no system known to me that
> would default to recursive mutexes), but that was done only to find occasional
> errors (during runtime, back in 2010, commit 336fd879c00b).  Functions using
> this mutex are mostly stable and unchanging, and it makes the virLogOnceInit()
> function only return 0 (or possibly abort in glib calls).  On top of that we 
> can
> assume that the virLogMutex is always initialized which enables us to be more
> consistent in some early error reporting.
> 
> Signed-off-by: Martin Kletzander <mklet...@redhat.com>
> ---
Reviewed-by: Erik Skultety <eskul...@redhat.com>