Re: [libvirt] [PATCH 1/2] Refactor virMutexInit virRWLockInit and virCondInit

2014-07-23 Thread Daniel P. Berrange
On Wed, Jul 23, 2014 at 07:27:24AM +0200, Martin Kletzander wrote: > On Fri, Jul 18, 2014 at 07:30:38AM -0600, Eric Blake wrote: > >On 07/17/2014 10:49 PM, Jincheng Miao wrote: > >>Implement InitInternal functions for Mutex, RWLock and Cond, these functions > >>contain error report using virReportS

Re: [libvirt] [PATCH 1/2] Refactor virMutexInit virRWLockInit and virCondInit

2014-07-22 Thread Martin Kletzander
On Fri, Jul 18, 2014 at 07:30:38AM -0600, Eric Blake wrote: On 07/17/2014 10:49 PM, Jincheng Miao wrote: Implement InitInternal functions for Mutex, RWLock and Cond, these functions contain error report using virReportSystemErrorFull, it is controlled by an argument 'quite'. The related macros a

Re: [libvirt] [PATCH 1/2] Refactor virMutexInit virRWLockInit and virCondInit

2014-07-19 Thread Jincheng Miao
- Original Message - > After your patch 2, do we really have any callers that use the quiet > version? This seems like the sort of patch where ALL callers should be > noisy (especially since the failure is so rare, but also means we are > quite hosed if it happens). What are the callers th

Re: [libvirt] [PATCH 1/2] Refactor virMutexInit virRWLockInit and virCondInit

2014-07-18 Thread Eric Blake
On 07/17/2014 10:49 PM, Jincheng Miao wrote: > Implement InitInternal functions for Mutex, RWLock and Cond, these functions > contain error report using virReportSystemErrorFull, it is controlled by > an argument 'quite'. > The related macros are Init and InitQuite, they call InitInternal function

Re: [libvirt] [PATCH 1/2] Refactor virMutexInit virRWLockInit and virCondInit

2014-07-18 Thread Eric Blake
On 07/17/2014 10:49 PM, Jincheng Miao wrote: > Implement InitInternal functions for Mutex, RWLock and Cond, these functions > contain error report using virReportSystemErrorFull, it is controlled by > an argument 'quite'. s/quite/quiet/ > The related macros are Init and InitQuite, they call InitI

[libvirt] [PATCH 1/2] Refactor virMutexInit virRWLockInit and virCondInit

2014-07-17 Thread Jincheng Miao
Implement InitInternal functions for Mutex, RWLock and Cond, these functions contain error report using virReportSystemErrorFull, it is controlled by an argument 'quite'. The related macros are Init and InitQuite, they call InitInternal function by passing 'false' or 'true' to quite argument. Sign