All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nick Piggin <npiggin@suse.de>
To: Trond Myklebust <trond.myklebust@fys.uio.no>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Sage Weil <sage@newdream.net>,
	linux-fsdevel@vger.kernel.org, linux-mm@kvack.org
Subject: Re: [patch] mm: close page_mkwrite races
Date: Thu, 2 Apr 2009 11:00:09 +0200	[thread overview]
Message-ID: <20090402090009.GA22256@wotan.suse.de> (raw)
In-Reply-To: <1238628811.18376.4.camel@heimdal.trondhjem.org>

On Wed, Apr 01, 2009 at 07:33:31PM -0400, Trond Myklebust wrote:
> On Wed, 2009-04-01 at 16:02 -0700, Andrew Morton wrote:
> > What is "the problem"?  Can we get "the problem"'s description included
> > in the changelog?
> > 
> > The patch is fairly ugly, somewhat costly and makes things (even) more
> > complex.    Sigh.
> 
> The problem is that currently, pages can be marked as dirty after they
> have been written out, or even during writeout.
> 
> IOW: the filesystem and the mm no longer agree on the state of the page,
> which again triggers issues such as
>   http://bugzilla.kernel.org/show_bug.cgi?id=12913

Thanks, that would go nicely at the top of the changelog.



WARNING: multiple messages have this Message-ID (diff)
From: Nick Piggin <npiggin@suse.de>
To: Trond Myklebust <trond.myklebust@fys.uio.no>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Sage Weil <sage@newdream.net>,
	linux-fsdevel@vger.kernel.org, linux-mm@kvack.org
Subject: Re: [patch] mm: close page_mkwrite races
Date: Thu, 2 Apr 2009 11:00:09 +0200	[thread overview]
Message-ID: <20090402090009.GA22256@wotan.suse.de> (raw)
In-Reply-To: <1238628811.18376.4.camel@heimdal.trondhjem.org>

On Wed, Apr 01, 2009 at 07:33:31PM -0400, Trond Myklebust wrote:
> On Wed, 2009-04-01 at 16:02 -0700, Andrew Morton wrote:
> > What is "the problem"?  Can we get "the problem"'s description included
> > in the changelog?
> > 
> > The patch is fairly ugly, somewhat costly and makes things (even) more
> > complex.    Sigh.
> 
> The problem is that currently, pages can be marked as dirty after they
> have been written out, or even during writeout.
> 
> IOW: the filesystem and the mm no longer agree on the state of the page,
> which again triggers issues such as
>   http://bugzilla.kernel.org/show_bug.cgi?id=12913

Thanks, that would go nicely at the top of the changelog.


--
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:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2009-04-02  9:00 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-30 13:53 [patch] mm: close page_mkwrite races Nick Piggin
2009-03-30 13:56 ` Nick Piggin
2009-03-30 13:56   ` Nick Piggin
2009-03-31 19:55   ` Sage Weil
2009-04-01 23:02     ` Andrew Morton
2009-04-01 23:33       ` Trond Myklebust
2009-04-01 23:33         ` Trond Myklebust
2009-04-02  9:00         ` Nick Piggin [this message]
2009-04-02  9:00           ` Nick Piggin
2009-04-02  9:04       ` Nick Piggin
2009-04-02  9:04         ` Nick Piggin

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=20090402090009.GA22256@wotan.suse.de \
    --to=npiggin@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=sage@newdream.net \
    --cc=trond.myklebust@fys.uio.no \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.