On Fri, May 20, 2011 at 01:44:30PM -0400, Josef Bacik wrote: > + if (unlikely(d_need_lookup(dentry))) { > + if (nameidata_dentry_drop_rcu(nd, dentry)) > + return -ECHILD; > + dput(dentry); > + dentry = NULL; > + goto retry; Yecchhh... How about simple goto unlazy; here instead and doing the rest there? Especially since you have the same kind of thing elsewhere in the same sucker. It had been bloody painful to untangle that thing; let's not add to the rat's nest that still remains... -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html