[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090811123819.GB18881@localhost>
Date: Tue, 11 Aug 2009 20:38:19 +0800
From: Wu Fengguang <fengguang.wu@...el.com>
To: Hidehiro Kawai <hidehiro.kawai.ez@...achi.com>
Cc: Andi Kleen <andi@...stfloor.org>, "tytso@....edu" <tytso@....edu>,
"hch@...radead.org" <hch@...radead.org>,
"mfasheh@...e.com" <mfasheh@...e.com>,
"aia21@...tab.net" <aia21@...tab.net>,
"hugh.dickins@...cali.co.uk" <hugh.dickins@...cali.co.uk>,
"swhiteho@...hat.com" <swhiteho@...hat.com>,
"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
"npiggin@...e.de" <npiggin@...e.de>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-mm@...ck.org" <linux-mm@...ck.org>,
Satoshi OSHIMA <satoshi.oshima.fk@...achi.com>,
Taketoshi Sakuraba <taketoshi.sakuraba.hc@...achi.com>
Subject: Re: [PATCH] [16/19] HWPOISON: Enable .remove_error_page for
migration aware file systems
On Tue, Aug 11, 2009 at 11:48:36AM +0800, Hidehiro Kawai wrote:
> Wu Fengguang wrote:
> > In fact we proposed a patch for preventing the re-corruption case, see
> >
> > http://lkml.org/lkml/2009/6/11/294
> >
> > However it is hard to answer the (policy) question "How sticky should
> > the EIO bit remain?".
>
> It's a good approach! This approach may also solve my concern,
> the re-corruption issue caused by transient IO errors.
>
> But I also think it needs a bit more consideration. For example,
> if the application has the valid data in the user space buffer,
> it would try to re-write it after detecting an IO error from the
> previous write. In this case, we should clear the sticky error flag.
Yes, and maybe more than that. The IO error issue really deserves an
independent work, which will inevitably involve lots of discussions
with lots of people.
For the data re-corruption problem, "vm.memory_failure_recovery = 0"
should be the most clean workaround for now. Can we settle with that?
Our goal for this initial hwpoison implementation is to achieve good
coverage (not necessarily every possible case :).
Thanks,
Fengguang
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists