Hello, On 04/01/2010 02:54 PM, Herbert Xu wrote: > On Thu, Apr 01, 2010 at 01:56:53PM +0900, Tejun Heo wrote: >> >> OIC. It's also cleared in ide_timer_expiry() too. Asymmetry among >> different failure paths worries me. e.g. looking at the code, I can't >> find how ide_error() would requeue the request either. It looks like >> each hwif->rq = NULL in failure path should be investigated and the >> affected ones should be replaced with a function which requeues and >> clears hwif->rq. Hmmm.... am I misunderstanding something? > > I had a look at the rest of them and they seemed to be fine. In ide_timer_expiry() if drive->waiting_for_dma is false, ide_error() is called, which in turn calls __ide_error() for fs requests. ide_ata_error() will be called if the device is a disk. If the request hasn't reached the retry limit and reset is not necessary, ide_ata_error() will return ide_stopped without requeueing the request. ide_timer_expiry() will clear hwif->rq without requeueing the request and the request will be lost. No? > So are you OK for this patch to go in? Yeah yeah, I think those patches are okay by themselves and am just trying to find out whether anything similar is missing, in which case the requeue might fit better somewhere higher in the call chain. Thanks. -- tejun -- 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