Re: [PATCH] remove throttle_vm_writeout()

On Fri, 2007-10-05 at 20:32 +0200, Peter Zijlstra wrote:
> Well, the thing is, we throttle pageout in throttle_vm_writeout(). As it
> stand we can deadlock there because it just waits for the numbers to
> drop, and unstable pages don't automagically dissapear. Only
> write_inodes() - normally called from balance_dirty_pages() will call
> COMMIT.
> 
> So my thought was that calling pageout() on an unstable page would do
> the COMMIT - we're low on memory, otherwise we would not be paging, so
> getting rid of unstable pages seems to make sense to me.

Why not rather track which mappings have large numbers of outstanding
unstable writes at the VM level, and then add some form of callback to
allow it to notify the filesystem when it needs to flush them out?

Cheers
  Trond

-
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/

---

• Follow-Ups:
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Trond Myklebust <[email protected]>

• References:
  • [PATCH] remove throttle_vm_writeout()
    • From: Miklos Szeredi <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Miklos Szeredi <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Miklos Szeredi <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Miklos Szeredi <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Andrew Morton <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Peter Zijlstra <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Miklos Szeredi <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Peter Zijlstra <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Miklos Szeredi <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Peter Zijlstra <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Trond Myklebust <[email protected]>
  • Re: [PATCH] remove throttle_vm_writeout()
    • From: Peter Zijlstra <[email protected]>

• Prev by Date: Re: [PATCH] Cute feature: colored printk output
• Next by Date: Re: [PATCH] Cute feature: colored printk output
• Previous by thread: Re: [PATCH] remove throttle_vm_writeout()
• Next by thread: Re: [PATCH] remove throttle_vm_writeout()
• 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]