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