Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753053Ab3G2TFR (ORCPT ); Mon, 29 Jul 2013 15:05:17 -0400 Received: from mail.linux-iscsi.org ([67.23.28.174]:35421 "EHLO linux-iscsi.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751171Ab3G2TFO (ORCPT ); Mon, 29 Jul 2013 15:05:14 -0400 Message-ID: <1375125066.5171.10.camel@haakon3.risingtidesystems.com> Subject: Re: [PATCH RESEND 0/1] AHCI: Optimize interrupt processing From: "Nicholas A. Bellinger" To: Tejun Heo Cc: Jens Axboe , Alexander Gordeev , James Bottomley , Mike Christie , linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org, Jeff Garzik , linux-scsi Date: Mon, 29 Jul 2013 12:11:06 -0700 In-Reply-To: <20130729115011.GC20951@mtj.dyndns.org> References: <1374248000.2266.20.camel@dabdike> <1374267684.7397.1058.camel@haakon3.risingtidesystems.com> <1374296162.7397.1098.camel@haakon3.risingtidesystems.com> <20130722150359.GA16564@dhcp-26-207.brq.redhat.com> <1374527436.7397.1145.camel@haakon3.risingtidesystems.com> <20130725101641.GB31994@dhcp-26-207.brq.redhat.com> <1374790082.7397.1411.camel@haakon3.risingtidesystems.com> <20130726020928.GL29296@kernel.dk> <1374873276.7397.1512.camel@haakon3.risingtidesystems.com> <1374885793.7397.1537.camel@haakon3.risingtidesystems.com> <20130729115011.GC20951@mtj.dyndns.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.4.4-1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1403 Lines: 33 On Mon, 2013-07-29 at 07:50 -0400, Tejun Heo wrote: > Yo, > > On Fri, Jul 26, 2013 at 05:43:13PM -0700, Nicholas A. Bellinger wrote: > > Considering there can be more than a single ata_device hanging off each > > ata_port, the '*sdev = ap->link.device[0].sdev' in __ata_qc_from_tag() > > is definitely bogus, but I'm not sure how else to correlate > > blk-mq/scsi-mq per device descriptors to existing code expecting > > ata_port->qcmd[] descriptors to be shared across multiple devices.. > > > > Tejun..? > > I have no idea. Let's please just do simpler conversion and worry > about embedding qc's into scsi_cmnds later. libata isn't a normal > SCSI driver and has a rather its own thick midlayer doing the > impedance matching inbetween && I really don't think there is too much > benefit to be reaped from embedding qc's into scsi_cmnds. > That is essentially the same conclusion that I came to, but wanted to at least give you a chance to comment here. ;) So that said, I'll include a simple conversion for libata into the scsi-mq WIP branch, and folks who are interested in more detailed conversions can pursue them as separate items. --nab -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/