From: Andrew Morton <akpm@osdl.org>
To: Nikita Danilov <nikita@clusterfs.com>
Cc: linux-mm@kvack.org
Subject: Re: [PATCH]: VM 7/8 cluster pageout
Date: Tue, 26 Apr 2005 12:39:31 -0700 [thread overview]
Message-ID: <20050426123931.545f0f9c.akpm@osdl.org> (raw)
In-Reply-To: <17006.27177.240316.526941@gargle.gargle.HOWL>
Nikita Danilov <nikita@clusterfs.com> wrote:
>
> Andrew Morton writes:
> > Nikita Danilov <nikita@clusterfs.com> wrote:
> > >
> > > Andrew Morton writes:
> > > > Nikita Danilov <nikita@clusterfs.com> wrote:
> > > > >
> > > > > Implement pageout clustering at the VM level.
> > > >
> > > > I dunno...
> > > >
> > > > Once __mpage_writepages() has started I/O against the pivot page, I don't
> > > > see that we have any guarantees that some other CPU cannot come in,
> > > > truncated or reclaim all the inode's pages and then reclaimed the inode
> > > > altogether. While __mpage_writepages() is still dinking with it all.
> > >
> > > Ah, silly me. Will __iget(page->mapping->host) in pageout_cluster() be
> > > enough? We risk truncate on matching iput(), but VM scanner calls iput()
> > > on inodes with ->i_nlink == 0 already (from shrink_dcache()).
> >
> > I have vague memories about iput() in page reclaim causing deadlocks or
> > some other nastiness. Maybe not.
>
> Aren't you talking about
>
> http://marc.theaimsgroup.com/?t=108272583200001&r=1&w=2
>
> by any chance?
Nope, this all happened in the early 2002 timeframe.
--
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/ .
Don't email: <a href=mailto:"aart@kvack.org"> aart@kvack.org </a>
next prev parent reply other threads:[~2005-04-26 19:39 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-04-17 17:38 Nikita Danilov
2005-04-26 4:15 ` Andrew Morton
2005-04-26 9:16 ` Nikita Danilov
2005-04-26 9:36 ` Andrew Morton
2005-04-26 16:19 ` Nikita Danilov
2005-04-26 19:39 ` Andrew Morton [this message]
2005-05-02 4:12 ` William Lee Irwin III
2005-05-02 5:51 ` Rik van Riel
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=20050426123931.545f0f9c.akpm@osdl.org \
--to=akpm@osdl.org \
--cc=linux-mm@kvack.org \
--cc=nikita@clusterfs.com \
/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