Re: [PATCH] sanlock: Don't spam logs with "target pid not found"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Mar 09, 2015 at 12:31:02 +0000, Daniel P. Berrange wrote:
> On Fri, Mar 06, 2015 at 04:58:22PM +0100, Jiri Denemark wrote:
> > Commit v1.2.4-52-gda879e5 fixed issues with domains started before
> > sanlock driver was enabled by checking whether a running domain is
> > registered with sanlock and if it's not, sanlock driver is basically
> > ignored for the domain.
> > 
> > However, it was checking this even for domain which has just been
> > started and no sanlock_* API was called for them yet. This results in
> > 
> >     cmd 9 target pid 2135544 not found
> > 
> > error messages to appear in sanlock.log whenever we start a new domain.
> > 
> > This patch avoids this useless check for freshly started domains.
> > 
> > Signed-off-by: Jiri Denemark <jdenemar@xxxxxxxxxx>
> > ---
> >  src/locking/domain_lock.c         | 22 ++++++++++++----------
> >  src/locking/lock_driver.h         | 12 +++++++++++-
> >  src/locking/lock_driver_lockd.c   |  2 +-
> >  src/locking/lock_driver_sanlock.c |  9 ++++++---
> >  src/locking/lock_manager.c        |  2 +-
> >  5 files changed, 31 insertions(+), 16 deletions(-)
> 
> ACK, this makes sense.

Thanks, I separated the change in flags into a new patch and pushed
both.

Jirka

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]