Re: [PATCH] Per-superblock unused dentry LRU lists.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


> per-sb, you mean? ;)


> > You can use trylock. Please see the patches in -mm to fix the umount
> > race.
> I'm not sure what unmount race you are talking about.
> AFAICT, there is no race here - we've got a reference on the superblock so  it
> can't go away and the lru list is protected by the dcache_lock, so there's
> nothing else we can race with. However, we can deadlock by taking the
> s_umount lock here. So why even bother to try to take the lock when we don't
> actually need it?

Please read the thread at

> > > +		if (__put_super_and_need_restart(sb) && count)
> > > +			goto restart;
> > 
> > Comment please.
> I'm not sure what a comment needs to explain that the code doesn't.
> Which bit do you think needs commenting?

I was referring to the __put_super_and_need_restart() part.

> > This should not be required with per super-block dentries. The only
> > reason, I think we moved dentries to the tail is to club all entries
> > from the sb together (to free them all at once).
> I think we still need to do that. We get called in contexts that aren't
> related to unmounting, so we want these dentries to be the first
> to be reclaimed from that superblock when we next call prune_dcache().

Is it? I quickly checked the callers of shrink_dcache_sb() and all of
them seem to be mount related. shrink_dcache_parent() is another story.
Am I missing something? Code reference will be particularly useful.

> No. Right now I just want to fix the problem that has been reported with
> shrink_dcache_sb().


	Balbir Singh,
	Linux Technology Center,
	IBM Software Labs
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at
Please read the FAQ at

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux