Re: [PATCH 1/15] ali14xx: fix deadlock on error handling

[Alan Cox <alan@xxxxxxxxxxxxxxxxxxx>]

On Tue, 02 Oct 2007 16:46:28 +0400
Sergei Shtylyov <sshtylyov@xxxxxxxxxxxxx> wrote:

> Bartlomiej Zolnierkiewicz wrote:
> 
> > Stop abusing ide_lock lock by switching to a private locking.
> 
> > Fixes same issue as fixed by Alan Cox in atiixp host driver with
> 
>     Has been also fixed in the piix driver.
> 
> > commit 6c5f8cc33eb2e10b6ab788bbe259fc142a068627.
> 
>     Heh, I've looked hard at the code trying to understand how this can 
> happen, and was unable to figure out. Probably was not hard enough...

The old IDE code timer, error handling and interrupt paths all race each
other. Anything can happen including this.

The proper fix is to rewrite the error handling but it was easier to port
the drivers to working error handling instead - hence libata PATA

Alan
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html