Re: [PATCH 10/10] mm: per device dirty threshold

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Miklos Szeredi <miklos@...>
Cc: <a.p.zijlstra@...>, <linux-mm@...>, <linux-kernel@...>, <neilb@...>, <dgc@...>, <tomoki.sekiyama.qu@...>, <nikita@...>, <trond.myklebust@...>, <yingchao.zhou@...>
Date: Saturday, April 21, 2007 - 6:54 am

On Sat, 21 Apr 2007 12:38:45 +0200 Miklos Szeredi <miklos@szeredi.hu> wrote:


Let's go back to the original changelog:

Author: marcelo.tosatti <marcelo.tosatti>
Date:   Tue Mar 8 17:25:19 2005 +0000

    [PATCH] vm: pageout throttling
    
    With silly pageout testcases it is possible to place huge amounts of memory
    under I/O.  With a large request queue (CFQ uses 8192 requests) it is
    possible to place _all_ memory under I/O at the same time.
    
    This means that all memory is pinned and unreclaimable and the VM gets
    upset and goes oom.
    
    The patch limits the amount of memory which is under pageout writeout to be
    a little more than the amount of memory at which balance_dirty_pages()
    callers will synchronously throttle.
    
    This means that heavy pageout activity can starve heavy writeback activity
    completely, but heavy writeback activity will not cause starvation of
    pageout.  Because we don't want a simple `dd' to be causing excessive
    latencies in page reclaim.
    
    Signed-off-by: Andrew Morton <akpm@osdl.org>
    Signed-off-by: Linus Torvalds <torvalds@osdl.org>

(A good one!  I wrote it ;))


I believe that the combination of dirty-page-tracking and its calls to
balance_dirty_pages() mean that we can now never get more than dirty_ratio
of memory into the dirty-or-writeback condition.

The vm scanner can convert dirty pages into clean, under-writeback pages,
but it cannot increase the total of dirty+writeback.

Hence I assert that the problem which throttle_vm_writeout() was designed
to address can no longer happen, so we can simply remove it.

(There might be problems with ZONE_DMA or ZONE_NORMAL 100% full of
dirty+writeback pages, but throttle_vm_writeout() wont help in this case
anyway)
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 10/10] mm: per device dirty threshold, Peter Zijlstra, (Fri Apr 20, 11:52 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Andrew Morton, (Sat Apr 21, 5:55 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Peter Zijlstra, (Sat Apr 21, 8:01 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Andrew Morton, (Sun Apr 22, 3:26 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Peter Zijlstra, (Sat Apr 21, 8:15 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Christoph Lameter, (Mon Apr 23, 11:48 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Peter Zijlstra, (Mon Apr 23, 11:58 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Christoph Lameter, (Mon Apr 23, 12:08 pm)
Re: [PATCH 10/10] mm: per device dirty threshold, Peter Zijlstra, (Sat Apr 21, 3:50 pm)
Re: [PATCH 10/10] mm: per device dirty threshold, Miklos Szeredi, (Sat Apr 21, 6:38 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Andrew Morton, (Sat Apr 21, 6:54 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Miklos Szeredi, (Sat Apr 21, 4:25 pm)
Re: [PATCH 10/10] mm: per device dirty threshold, Peter Zijlstra, (Mon Apr 23, 2:14 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Miklos Szeredi, (Mon Apr 23, 2:29 am)
Re: [PATCH 10/10] mm: per device dirty threshold, Andrew Morton, (Mon Apr 23, 2:39 am)