linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <andrewm@uow.edu.au>
To: Marcelo Tosatti <marcelo@conectiva.com.br>
Cc: Rik van Riel <riel@conectiva.com.br>, linux-mm@kvack.org
Subject: Re: [RFC] VM statistics to gather
Date: Tue, 26 Jun 2001 17:59:09 +1000	[thread overview]
Message-ID: <3B3840CD.B60448EB@uow.edu.au> (raw)
In-Reply-To: <Pine.LNX.4.21.0106260206240.1676-100000@freak.distro.conectiva>

Marcelo Tosatti wrote:
> 
> On Tue, 26 Jun 2001, Andrew Morton wrote:
> 
> > Rik van Riel wrote:
> > >
> > > Hi,
> > >
> > > I am starting the process of adding more detailed instrumentation
> > > to the VM subsystem and am wondering which statistics to add.
> > > A quick start of things to measure are below, but I've probably
> > > missed some things. Comments are welcome ...
> >
> > Neat.
> >
> > - bdflush wakeups
> > - pages written via page_launder's writepage by kswapd
> > - pages written via page_launder's writepage by non-PF_MEMALLOC
> >   tasks.  (ext3 has an interest in this because of nasty cross-fs
> >   reentrancy and journal overflow problems with writepage)
> 
> Does ext3 call page_launder() with __GFP_IO ?
> 
> If it does not (which I believe so), page_launder() without PF_MEMALLOC
> never happens.

OK, I was using PF_MEMALLOC as shorthand for `kswapd', with
unsuccessful accuracy.  I think it's OK to block non-kswapd
tasks in writepage() while we open a new transaction, but it's
perhaps not so good to block kswapd there.

At present, if the caller is PF_MEMALLOC we make a non-blocking
attempt to open a transaction handle.  If it fails, redirty the
page and return.  It usually succeeds.  This may be excessively
paranoid.

I haven't played with it a lot recently, but it may turn out to
be useful to know whether the caller is kswapd or someone else,
and it'd be nice to know the calling context by means other than
inferring it from PF_MEMALLOC.  What happened to the writepage
`priority' patch you had, BTW?
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/

  parent reply	other threads:[~2001-06-26  7:59 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-06-25 23:04 Rik van Riel
2001-06-25 23:35 ` Roger Larsson
2001-06-25 23:59   ` Rik van Riel
2001-06-26  1:11     ` Roger Larsson
2001-06-26  7:42     ` Stephen C. Tweedie
2001-06-26  1:45 ` Marcelo Tosatti
2001-06-26  7:13   ` Marcelo Tosatti
2001-06-26  4:24 ` Andrew Morton
2001-06-26  5:07   ` Marcelo Tosatti
2001-06-26  5:10     ` Marcelo Tosatti
2001-06-26  7:59     ` Andrew Morton [this message]
2001-06-27 10:30       ` Marcelo Tosatti

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3B3840CD.B60448EB@uow.edu.au \
    --to=andrewm@uow.edu.au \
    --cc=linux-mm@kvack.org \
    --cc=marcelo@conectiva.com.br \
    --cc=riel@conectiva.com.br \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox