On Tue, Jul 17, 2012 at 08:10:14AM +0800, Fengguang Wu wrote: > Hi David, > > I don't know enough to tell whether this if false alarm. Would you > help confirm it? Thank you. Yes, I just changed the locking there. I've checked it again and it's correct; the kind of lock taken depends on which list the structure was found on. Thanks > tree: git://git.kernel.org/pub/scm/linux/kernel/git/teigland/linux-dlm.git next > head: 96006ea6d4eea73466e90ef353bf34e507724e77 > commit: c04fecb4d9f7753e0cbff7edd03ec68f8721cdce [1/6] dlm: use rsbtbl as resource directory > > One new sparse warning is triggered (and there were no old warnings before patch): > > + fs/dlm/lock.c:1049:9: sparse: context imbalance in 'dlm_master_lookup' - different lock contexts for basic block > > vim +1049 fs/dlm/lock.c > 1046 unlock_rsb(r); > 1047 put_rsb(r); > 1048 } > > 1049 return 0; > 1050 > 1051 not_found: > 1052 error = get_rsb_struct(ls, name, len, &r); > > --- > 0-DAY kernel build testing backend Open Source Technology Centre > Fengguang Wu <wfg@xxxxxxxxxxxxxxx> Intel Corporation -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html