[PATCH] mtd: fix race in cfi_cmdset_0001 driver

Joakim Tjernlund joakim.tjernlund at transmode.se
Fri Feb 11 09:54:13 EST 2011


Artem Bityutskiy <dedekind1 at gmail.com> wrote on 2011/02/11 15:50:54:
>
> On Fri, 2011-02-11 at 15:46 +0100, Joakim Tjernlund wrote:
> > Artem Bityutskiy <dedekind1 at gmail.com> wrote on 2011/02/11 15:02:16:
> > >
> > > On Mon, 2011-02-07 at 17:07 +0100, Joakim Tjernlund wrote:
> > > > As inval_cache_and_wait_for_operation() drop and reclaim the lock
> > > > to invalidate the cache, some other thread may suspend the operation
> > > > before reaching the for(;;) loop. Therefore the loop must start with
> > > > checking the chip->state before reading status from the chip.
> > > >
> > > > Signed-off-by: Joakim Tjernlund <Joakim.Tjernlund at transmode.se>
> > >
> > > Am I right that this patch should be ignored and a new patch will be
> > > created?
> >
> > No, this is the real thing. Should go to Linus ASAP.
> >
> > There is a hunt for another flash HW related bug hunt ongoing too
> > that may result in another patch touching other areas.
>
> OK, all I can do is pushing to the l2 tree, the Linus story is up to
> dwmw2 as usually, you should bug him. All I can do for you is letting
> him know about this in the mtd chat.

Please do, I have mailed him directly too but he seems very busy.

 Jocke




More information about the linux-mtd mailing list