From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 70F0DC433EF for ; Wed, 16 Mar 2022 23:23:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237664AbiCPXY2 (ORCPT ); Wed, 16 Mar 2022 19:24:28 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59922 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346600AbiCPXYX (ORCPT ); Wed, 16 Mar 2022 19:24:23 -0400 Received: from esa4.hgst.iphmx.com (esa4.hgst.iphmx.com [216.71.154.42]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4135C6579 for ; Wed, 16 Mar 2022 16:23:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1647472988; x=1679008988; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=YG3c9enGf73FZQYfT2r6IIW+DLqVkhyrPxUwVJug/IQ=; b=WJbCkY2Kldbk3/WcYZQF6dRc5c7T+tyARLZDDb2sBFDMOd4njifG41CP LFcwB/uRxuC2tL5iWaTMxUHqKQ4V/CCV8e9ngj9pKs+Ocp9l+1envD8jW 39FugdPsySRA5nptiRlvdEOmOH2X8GFq+YCT+OuWdzpXtxmYZZ8LxJuPv RqbgAfdeXsKOsMEsukdQEhXpiUkLQOoXQPEoFJtMvx73FZpVU1AiMJbk9 IVRe910S//pZs0UNb7G/UpZj1A8415ZxXTLCPqqlY2wwAwAq/apkZ6DOH oGmLDsMC+36lMBjNZic1kno9Zx8LRDsnHX16iBR2/yJ2k2veCNTHJVZEo g==; X-IronPort-AV: E=Sophos;i="5.90,187,1643644800"; d="scan'208";a="194466685" Received: from h199-255-45-15.hgst.com (HELO uls-op-cesaep02.wdc.com) ([199.255.45.15]) by ob1.hgst.iphmx.com with ESMTP; 17 Mar 2022 07:23:07 +0800 IronPort-SDR: B7hn/T5p2pedSEtyK/usSMVU6g86dl+FyA2YOiD3nwvQZ8gb+okhoYae3kSqURWw1IKcnvxok/ HZNTZPJxFjZInc9nGZC0Iow4u34FiMpD4QOs8nQE9WJ7zouHBzupF2kMfCNb04Crw9zJJtwAhi zMka9sOsjjqm1/hsMlj8zFciorQ+xmOuOe7z+BJSh1Q7N6eGcTZ75EPwnx7Z/5lWjNwaJAcySv qFb33RVsuVOM+n8ITTfHGY+lS7OrXMRouOwfcCbPQmpyIVVaZsLkpfRHuMcHsIbgOi5HuIhS+j EkvhOqUhtqKfB/vjSb0j4BC6 Received: from uls-op-cesaip02.wdc.com ([10.248.3.37]) by uls-op-cesaep02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Mar 2022 15:54:13 -0700 IronPort-SDR: e7CObk+rnYXK1I16xxa7H7XiLe8tjEeqpU/aCi9G4ZFMhOK7nYQUpFBPyTGsWOwaCHJqCO0RXg SiYW57B5GnjhQ/qCbCZYNsSHk8CM9YbcJsO8EQLTZAZYHXEVdUbjmAc3+TOPxpoXMobF1LwUAx DdhuxxWctMJDlTqebFFdjz5iROCSq2HSP9edEzk68PZwKcAT+PEAUwwDqww4qRN0hgmIUOxvSi 2AvpkuVIw0ouRLl4d5Chvl9POY0AkA7jUf7qbTmBoJViEvM2ITkuPJMJ5jFzMb57dI2XPpgd8c iQY= WDCIronportException: Internal Received: from usg-ed-osssrv.wdc.com ([10.3.10.180]) by uls-op-cesaip02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Mar 2022 16:23:08 -0700 Received: from usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTP id 4KJmXq3ylDz1SVp2 for ; Wed, 16 Mar 2022 16:23:07 -0700 (PDT) Authentication-Results: usg-ed-osssrv.wdc.com (amavisd-new); dkim=pass reason="pass (just generated, assumed good)" header.d=opensource.wdc.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d= opensource.wdc.com; h=content-transfer-encoding:content-type :in-reply-to:organization:from:references:to:content-language :subject:user-agent:mime-version:date:message-id; s=dkim; t= 1647472986; x=1650064987; bh=YG3c9enGf73FZQYfT2r6IIW+DLqVkhyrPxU wVJug/IQ=; b=ZlcFFOQcJwkYtdSKzLDONIb2R/8b/na7A+bzhx2GgA5SDvB/32c W3MDmSMa/WPnB99MS3E1rylqGWPLP1zOl6AnqutB5uKozKsB4PmK2QNbGPcRF3Ne VWaa5kPeuuTkmecfDDpAYrVMwCHAIINOokN6PGxDb1+HuyD1Pzodt5wlyLmRi6+v zC79Vpvs1jEVY/NM+9ApnR3tamSuJf1Eo7T9pd8ibyFhW+akqTRpdS2FjvVKKUdv twzp+zrY/IWyri/yAc+DmYYqIU+POmlNaRym8nAiFfJKXewULvxLpHsX514orlaE blxs+zkHba7Gr9YGe8TqBYT7QvhOGG3TsWg== X-Virus-Scanned: amavisd-new at usg-ed-osssrv.wdc.com Received: from usg-ed-osssrv.wdc.com ([127.0.0.1]) by usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id Q_of3AX9frW1 for ; Wed, 16 Mar 2022 16:23:06 -0700 (PDT) Received: from [10.225.163.105] (unknown [10.225.163.105]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTPSA id 4KJmXn10xpz1Rvlx; Wed, 16 Mar 2022 16:23:04 -0700 (PDT) Message-ID: Date: Thu, 17 Mar 2022 08:23:03 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.6.2 Subject: Re: [PATCH RFC 2/2] libata: Use scsi cmnd budget token for qc tag for SAS host Content-Language: en-US To: John Garry , Christoph Hellwig Cc: jejb@linux.ibm.com, martin.petersen@oracle.com, bvanassche@acm.org, ming.lei@redhat.com, hare@suse.de, linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, martin.wilck@suse.com References: <1647340746-17600-1-git-send-email-john.garry@huawei.com> <1647340746-17600-3-git-send-email-john.garry@huawei.com> <99541f2d-2aea-6bd7-e3b6-21dbc355875d@opensource.wdc.com> <650c667f-ca55-821d-4e0f-29fce69a68fc@huawei.com> <20220316083439.GA6701@lst.de> <4dfcddff-7736-e01e-7fcf-804de77ac623@huawei.com> From: Damien Le Moal Organization: Western Digital Research In-Reply-To: <4dfcddff-7736-e01e-7fcf-804de77ac623@huawei.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-scsi@vger.kernel.org On 3/16/22 19:46, John Garry wrote: > On 16/03/2022 08:34, Christoph Hellwig wrote: >> In addition to the comments from Damien: I think we should kill >> ata_qc_new_init as well. It is a bit pointless and having it in >> libata-core.c when it pokes into scsi internals also doesn't make sense. >> >> So maybe something like: > > Seems reasonable to me at least. Yep, that will be a nice cleanup. > > But I'd send these as 2x separate patches. > >> * sata_async_notification - SATA async notification handler >> * @ap: ATA port where async notification is received >> diff --git a/drivers/ata/libata-scsi.c b/drivers/ata/libata-scsi.c >> index ed8be585a98f7c..5e0bc7b05a107e 100644 >> --- a/drivers/ata/libata-scsi.c >> +++ b/drivers/ata/libata-scsi.c >> @@ -638,24 +638,44 @@ EXPORT_SYMBOL_GPL(ata_scsi_ioctl); >> static struct ata_queued_cmd *ata_scsi_qc_new(struct ata_device *dev, >> struct scsi_cmnd *cmd) >> { > ... > >> >> - if (scsi_cmd_to_rq(cmd)->rq_flags & RQF_QUIET) >> - qc->flags |= ATA_QCFLAG_QUIET; >> + if (ap->flags & ATA_FLAG_SAS_HOST) { >> + /* XXX: add a comment here why SAS is different */ > > I think this is simply because SAS hosts have shost.can_queue > 32 > >> + if (WARN_ON_ONCE(cmd->budget_token >= ATA_MAX_QUEUE)) >> + goto fail; >> + tag = cmd->budget_token; > > thanks, > John -- Damien Le Moal Western Digital Research