From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH] libata: revert "libata: use blk taging" et al. Date: Thu, 12 Mar 2015 08:11:57 -0400 Message-ID: <20150312121157.GH25944@htj.duckdns.org> References: <5500863D.4070807@cybernetics.com> <5500B783.20106@fb.com> <5500BF6F.6080000@cybernetics.com> <20150312023148.GA1960870@devbig257.prn2.facebook.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Dan Williams Cc: Shaohua Li , Tony Battersby , Jens Axboe , IDE/ATA development list , Christoph Hellwig , "linux-kernel@vger.kernel.org" List-Id: linux-ide@vger.kernel.org On Thu, Mar 12, 2015 at 06:14:24AM -0400, Dan Williams wrote: > > @@ -572,7 +572,6 @@ int sas_ata_init(struct domain_device *found_dev) > > > > ap->private_data = found_dev; > > ap->cbl = ATA_CBL_SATA; > > - ap->scsi_host = shost; > > rc = ata_sas_port_init(ap); > > if (rc) { > > ata_sas_port_destroy(ap); > > We need a scsi_host for error recovery, see: > > ata_std_sched_eh() IOW, let's just add a flag bit to identify SAS hosts. Thanks. -- tejun