Christoph Hellwig wrote: > On Tue, Feb 07, 2006 at 08:52:50AM +0100, Hannes Reinecke wrote: >> This patch converts platform_data->qfrozen to atomic_t. >> This way we can get rid of ahd_lock / ahd_unlock for the >> accessor functions; it also fixes some deadlocks in the >> recovery code (again). > > Is ahd_freeze_simq called recursively at all? If not it would be much > better to just kill it completely and just rely on the block layer, as > we do in all the other drivers. > As in void ahd_freeze_simq(struct ahd_softc *ahd) { scsi_block_requests(ahd->platform_data->host); } ? Hmm. Looks as if this could work ... Lemme check. Cheers, Hannes -- Dr. Hannes Reinecke hare@xxxxxxx SuSE Linux Products GmbH S390 & zSeries Maxfeldstraße 5 +49 911 74053 688 90409 Nürnberg http://www.suse.de - : send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html