![]() | |||||||||||||
Messages in this thread |
On Thu, Mar 09, Andrew Morton wrote: > > Hmm, right. Andrew, if you want a rediff against -mm just tell me. I'm > > actually diff'ing against lates linux-2.6.git. > > I'll work it out. > Ok, so I'll just send an updated patch against linux-2.6.git adressing your issues later. > Cosmetically, I don't think wait_on_prunes() should be concerned about > whether or not it "slept". That action is not significant and preemptible > kernels can "sleep" at just about any stage. So I think the concept of > "slept" in there should be replaced with, say, "prunes_remaining" or > something like that. Consequently the all-important comment over > wait_on_prunes() should be updated to provide a bit more information about > the significance of its return value, please. Ok, will do. > Also I think there should be some explanation somewhere which describes why > we can continue to assume that there aren't any prunes left to do after > wait_on_prunes() has dropped dcache_lock. I mean, once you've dropped the > lock it's usually the case that anything which you examined while holding > that lock now becomes out-of-date and invalid. I assume the thinking is > that because there's an unmount in progress, nothing can come in and add > new dentries? > > IOW: why isn't there a race between wait_on_prunes() and prune_one_dentry()? Because outside dcache_lock, either the refcount on the parent dentry is wrong (therefore select_parent() might return 0) and sb_prunes != 0 or the refcount is correct and the sb_prunes == 0. Since sb_root == NULL when unmounting the filesystem there are no new dentries coming in. > Are we all happy with this patch now? I'll update the comments and come back to you with an updated patch. Then I'm fine with the patch. Regards, Jan -- Jan Blunck jblunck@suse.de SuSE LINUX AG - A Novell company Maxfeldstr. 5 +49-911-74053-608 D-90409 Nürnberg http://www.suse.de - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | ||||||||||||
| Last update: 2006-03-09 12:01 [W:0.123 / U:0.260 seconds] ©2003-2008 Jasper Spaans | |||||||||||||