Re: [PATCH 02/43] bhyve: convert virMutex to GMutex

2020-04-15 Thread Pavel Mores
On Wed, Apr 15, 2020 at 01:59:46PM +0100, Daniel P. Berrangé wrote: > On Wed, Apr 15, 2020 at 02:15:35PM +0200, Pavel Mores wrote: > > On Tue, Apr 14, 2020 at 07:05:03PM +0200, Rafael Fonseca wrote: > > > On Tue, Apr 14, 2020 at 6:06 PM Pavel Mores wrote: > > > > > > > > By the way, the approach t

Re: [PATCH 02/43] bhyve: convert virMutex to GMutex

2020-04-15 Thread Daniel P . Berrangé
On Wed, Apr 15, 2020 at 02:15:35PM +0200, Pavel Mores wrote: > On Tue, Apr 14, 2020 at 07:05:03PM +0200, Rafael Fonseca wrote: > > On Tue, Apr 14, 2020 at 6:06 PM Pavel Mores wrote: > > > > > > By the way, the approach taken here with bhyveDriver{Lock,Unlock}() might > > > make > > > sense with t

Re: [PATCH 02/43] bhyve: convert virMutex to GMutex

2020-04-15 Thread Pavel Mores
On Tue, Apr 14, 2020 at 07:05:03PM +0200, Rafael Fonseca wrote: > On Tue, Apr 14, 2020 at 6:06 PM Pavel Mores wrote: > > > > By the way, the approach taken here with bhyveDriver{Lock,Unlock}() might > > make > > sense with the whole series - implement e.g. virMutexInit() in terms of > > g_mutex_i

Re: [PATCH 02/43] bhyve: convert virMutex to GMutex

2020-04-14 Thread Rafael Fonseca
On Tue, Apr 14, 2020 at 6:06 PM Pavel Mores wrote: > > By the way, the approach taken here with bhyveDriver{Lock,Unlock}() might make > sense with the whole series - implement e.g. virMutexInit() in terms of > g_mutex_init() in the first phase and only then replace the actual > virMutexInit() call

Re: [PATCH 02/43] bhyve: convert virMutex to GMutex

2020-04-14 Thread Pavel Mores
On Fri, Apr 10, 2020 at 03:54:29PM +0200, Rafael Fonseca wrote: > Signed-off-by: Rafael Fonseca > --- > src/bhyve/bhyve_driver.c | 11 --- > src/bhyve/bhyve_utils.h | 2 +- > 2 files changed, 5 insertions(+), 8 deletions(-) > > diff --git a/src/bhyve/bhyve_driver.c b/src/bhyve/bhyve_dr

[PATCH 02/43] bhyve: convert virMutex to GMutex

2020-04-10 Thread Rafael Fonseca
Signed-off-by: Rafael Fonseca --- src/bhyve/bhyve_driver.c | 11 --- src/bhyve/bhyve_utils.h | 2 +- 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/src/bhyve/bhyve_driver.c b/src/bhyve/bhyve_driver.c index b6204c7fb9..6e9a79cb52 100644 --- a/src/bhyve/bhyve_driver.c +++