Re: [PATCH] shrink_dcache_parent() races against shrink_dcache_memory()

Hello Jan,

this is much cleaner now and looks more like my original patch and is smaller/more beautifull with counters usage. Thanks.

Yes, it is heavily inspired by you patch.

thanks again. BTW, out of curiosity why do you work on this?

However, with counters instead of list it is possible to create a live lock :( So I'm not sure it is really ok.

Hmm, I don't really get what you mean with "live lock".

By "live lock" I mean the situation when you are "locked" in shrink_dcache_parent() due to wait_on_prunes() always returns 1. We used shrinker list with a reference to dentry specially to avoid this as much as possible. I'm not sure how real such live lock can be created, but I can think it over.

BTW, what kernel is it for? 2.6.15 or 2.6.16-X?

http://www.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git from
today.

thanks!

Kirill

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH] shrink_dcache_parent() races against shrink_dcache_memory()
    • From: Jan Blunck <[email protected]>
  • Re: [PATCH] shrink_dcache_parent() races against shrink_dcache_memory()
    • From: Kirill Korotaev <[email protected]>
  • Re: [PATCH] shrink_dcache_parent() races against shrink_dcache_memory()
    • From: Jan Blunck <[email protected]>
  • Re: [PATCH] shrink_dcache_parent() races against shrink_dcache_memory()
    • From: Kirill Korotaev <[email protected]>
  • Re: [PATCH] shrink_dcache_parent() races against shrink_dcache_memory()
    • From: Jan Blunck <[email protected]>

• Prev by Date: Re: ALSA on MIPS platform
• Next by Date: Re: security capabilities on filesystems
• Previous by thread: Re: [PATCH] shrink_dcache_parent() races against shrink_dcache_memory()
• Next by thread: [PATCH 1/9] device-mapper snapshot: load metadata on creation
• Index(es):
  • Date
  • Thread

[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]