From: Andrew Morton <akpm@linux-foundation.org>
To: "Joakim Tjernlund" <Joakim.Tjernlund@transmode.se>
Cc: haokexin@gmail.com, akorolev@infradead.org, nico@cam.org,
joern@logfs.org, linux-mtd@lists.infradead.org,
dwmw2@infradead.org
Subject: Re: [PATCH] Fix of broken state in CFI driver caused by FL_SHUTDOWN
Date: Wed, 9 Apr 2008 16:01:05 -0700 [thread overview]
Message-ID: <20080409160105.cd5b9d50.akpm@linux-foundation.org> (raw)
In-Reply-To: <009801c89a95$09ae4680$1d0ad380$@Tjernlund@transmode.se>
On Thu, 10 Apr 2008 00:57:07 +0200
"Joakim Tjernlund" <Joakim.Tjernlund@transmode.se> wrote:
> > -----Original Message-----
> > From: linux-mtd-bounces@lists.infradead.org [mailto:linux-mtd-bounces@lists.infradead.org] On Behalf
> > Of Alexey Korolev
> > Sent: den 4 april 2008 14:22
> > To: J__rn Engel
> > Cc: nico@cam.org; haokexin@gmail.com; dwmw2@infradead.org; akpm@linux-foundation.org; linux-
> > mtd@lists.infradead.org
> > Subject: Re: [PATCH] Fix of broken state in CFI driver caused by FL_SHUTDOWN
> >
> > Hi
> >
> >
> > CFI driver in 2.6.24 kernel is broken. Not so intensive read/write operations cause incomplete writes
> > which lead to kernel panics in JFFS2.
> > We investigated the issue - it is caused by bug in FL_SHUTDOWN parsing code. Sometimes chip returns -
> > EIO as if it is in FL_SHUTDOWN state when it should wait in FL_PONT (error in order of conditions).
> >
> > The following patch fixes the bug in state parsing code of CFI.
> > Also I've added comments to notify developers if they want to add new case in future.
> > Please include it.
> >
> > Signed-off-by: Alexey Korolev <akorolev@infradead.org>
> > Reviewed-By: Joern Engel <joern@logfs.org>
>
> Acked-By: Joakim Tjernlund <Joakim.Tjenrlund@transmode.se>
>
> Guys, I think you should sent this directly to Linus as David seems
> unavailable ATM. It should really go into 2.6.25
>
I don't have a copy of the patch.
next prev parent reply other threads:[~2008-04-09 23:01 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-04 9:06 [PATCH] Fix of broken state in CFI driver caused by FL_SHUTDOWN Alexey Korolev
2008-04-04 9:34 ` Jörn Engel
2008-04-04 12:10 ` Alexey Korolev
2008-04-04 12:21 ` Alexey Korolev
2008-04-05 3:46 ` Kevin Hao
2008-04-09 22:57 ` Joakim Tjernlund
2008-04-09 23:01 ` Andrew Morton [this message]
[not found] ` <009801c89a95$09ae4680$1d0ad380$%Tjernlund@transmode.se>
2008-04-09 23:02 ` Nicolas Pitre
2008-04-09 23:08 ` Joakim Tjernlund
2008-04-10 16:57 ` Alexey Korolev
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=20080409160105.cd5b9d50.akpm@linux-foundation.org \
--to=akpm@linux-foundation.org \
--cc=Joakim.Tjernlund@transmode.se \
--cc=akorolev@infradead.org \
--cc=dwmw2@infradead.org \
--cc=haokexin@gmail.com \
--cc=joern@logfs.org \
--cc=linux-mtd@lists.infradead.org \
--cc=nico@cam.org \
/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.