On Wed, 1 Sep 2010, Neil Brown wrote: > On Mon, 30 Aug 2010 21:40:27 +1000 > And here is a patch. It isn't really complete, but I have done enough for > today. It at least shows what I am trying to do. Thanks. Okay, I see what it's trying to do. And I can accept the part which validates that the underlying dentry trees still match the union tree (wouldn't it be simpler to just d_lookup() and check if the child matches?) However redoing the lookup and changing the upperpath and lowerpath for directories is no good. Upperpath and lowerpath are constant, changing them would be like changing dentry->d_inode, the dentry would suddenly become something else. That's crazy. Making sure that evething is in a sane state and erroring out if not sounds a workable alternative to enforcing no change blindly. Although no change should probably be the default (e.g. unless a "-o dont_enforce_no_change" mount option is given). Thanks, Miklos -- 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
