From patchwork Fri Apr 15 12:42:24 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chesnokov Gleb X-Patchwork-Id: 562140 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 7BABBC433FE for ; Fri, 15 Apr 2022 12:43:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1353755AbiDOMqK (ORCPT ); Fri, 15 Apr 2022 08:46:10 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47686 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1353762AbiDOMqH (ORCPT ); Fri, 15 Apr 2022 08:46:07 -0400 X-Greylist: delayed 67 seconds by postgrey-1.37 at lindbergh.monkeyblade.net; Fri, 15 Apr 2022 05:43:36 PDT Received: from smtp.digdes.com (smtp.digdes.com [85.114.5.13]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DE27CC7496 for ; Fri, 15 Apr 2022 05:43:36 -0700 (PDT) Received: from DDSM-MAIL01.digdes.com (172.16.100.67) by relay.digdes.com (172.16.96.24) with Microsoft SMTP Server (TLS) id 14.3.498.0; Fri, 15 Apr 2022 15:42:26 +0300 Received: from DDSM-MAIL01.digdes.com (172.16.100.67) by DDSM-MAIL01.digdes.com (172.16.100.67) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.12; Fri, 15 Apr 2022 15:42:26 +0300 Received: from smtp.digdes.com (172.16.96.56) by DDSM-MAIL01.digdes.com (172.16.100.67) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.2375.12 via Frontend Transport; Fri, 15 Apr 2022 15:42:26 +0300 Received: from EUR05-VI1-obe.outbound.protection.outlook.com (104.47.17.174) by relay2.digdes.com (172.16.96.56) with Microsoft SMTP Server (TLS) id 14.3.498.0; Fri, 15 Apr 2022 15:42:26 +0300 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=TJhLoPWIQhiwDVIyrZjBs6/pyPB1sDmJkcsMbdmKyQwlmU3OktO18VqhOtSyKL3MeK3w0xfB8tj6YkLn8cgcwF8/TaO/KF/1ycZ1orEb1Mqn4ZGiFl3sCTNumXm1KYV75G3tMCpnDGdXthpmld43Lj90TU+Hf2URIZpOkV63AjxsOgX1c2/cb00JNnc1jKUglTtUYscp1gCsBJXgLvWnnRgWEeKFq16t9JpHpERp2ochYoqPpgR9RY9OJGhDXiOjpXKuzDJF/WGIT90LsaS0BW/hRtXPaGFrnUa0htMuMI9PmTH0zAtsH1fDdOTJofX6E9JHUmI8kugDwIdVDMkVpw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=NBwVL/WQw9H7kOeiROa5jfGWN+5d/fkEQyChIU+mQ/o=; b=aPFFw1D2LNDs34C331Djia49PyMgApSx0okW6x1NFGTkii86X+wxeHz8QlazjeqEWJX3wuUMQukUq8BPH7DxBQ+3EeFsXukLOcn1Nqxtp+3vMinSRAGhROerAUY7Ws8jugWfwxP3hgLMt8f/7pLxCga88+ATS9D2738FbR1Oq1+E2o18KsWgDV+m4KB3lIKm7GoPoFLcs6W8M6ABulh7d/XTUueKEp4+O80oHgRARHOkedDn6CxWwCpMqOahnYt/pN9rqtqiaFhM74GNbKNLAaynL1dfPp6RBYeOPEDL1HDlYzb5PPpG7Rj76sOhYjWYQEWsnpv+LmbAArm9NkJLDQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=raidix.com; dmarc=pass action=none header.from=raidix.com; dkim=pass header.d=raidix.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=digdes.onmicrosoft.com; s=selector2-digdes-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NBwVL/WQw9H7kOeiROa5jfGWN+5d/fkEQyChIU+mQ/o=; b=Cpm8szQ0O0LH/nHxrRhO6fMAx+TYFGKTrksEK28aVrDexUecFru39Orn8k4JyHM7xNi4w/tsdCBd5OQrv16N1hLIlRDkW3Z1aFIjrFzBTk7JaOwOfD8YjDA8DWRsGaw5TBuZP3ASIuhVql92xufjgHAElQDc4iUdnTUbrStn1tA= Received: from AS8PR10MB4952.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:403::6) by AS8PR10MB4519.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:2e2::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5144.22; Fri, 15 Apr 2022 12:42:25 +0000 Received: from AS8PR10MB4952.EURPRD10.PROD.OUTLOOK.COM ([fe80::74ca:aa66:a112:d987]) by AS8PR10MB4952.EURPRD10.PROD.OUTLOOK.COM ([fe80::74ca:aa66:a112:d987%3]) with mapi id 15.20.5164.020; Fri, 15 Apr 2022 12:42:25 +0000 From: Chesnokov Gleb To: "linux-scsi@vger.kernel.org" Subject: [PATCH 1/2] qla2xxx: Remove free_sg command flag Thread-Topic: [PATCH 1/2] qla2xxx: Remove free_sg command flag Thread-Index: AQHYUMYWYkt1LwLtU0eGmzdW21PURA== Date: Fri, 15 Apr 2022 12:42:24 +0000 Message-ID: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: suggested_attachment_session_id: 8ad1690c-d667-7a20-e5e8-149fb4163c2c authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=raidix.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 85e54aa4-5398-425a-e153-08da1edd6497 x-ms-traffictypediagnostic: AS8PR10MB4519:EE_ x-microsoft-antispam-prvs: x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: K5tkARijj1a416AU0Km1mUAjsb0lDqmqnfrK2FduzZc0Epxzgu1g0IPQhucF0Y5O5aek3Es4nWImNjdbYw/Cv8aRx7OqSUhDe+Kb9m3Bd+kzB/4wn1rDDAeHyg18wdKoKniREwS59kqypPsg5fqWQTo9u69e9WfD/UDE1No3NKLjGrnt1s3fcX6AUfQ9Gofb2fHqbRDFk8KlRjxd+id3aPALPh6RuNvE5iKZgGYZRor0Emdl3U/NdVGIhMM/lOz0JBHf94vVTvwJ8ONZxOrX7fcbNhRqC+0ayUQTd/zffGaiwdNjFgnp47BZD1bFJ8N7q1Q5/wyunxHR1N/xjMUA4kLHnFdd/C9e0FngYiJeuMxHQhSQ2YFZyjldLwwXosAfbvoDK5DHRWaXRhXcbb+KkdsZiAY9Hi1XEV/n8pujH/q3/g7LMpy4W3gz4lB53o7MKvhphQwS6k33zBST6QgwkkfOAh/tZWYAhFGXHx99+1kOKxWnImK7wYA3V9cwTaGGemak1vHWxqZBIquesnYfU7Bva6uh34BZYKs0C0KZahs1QMIYkmt3qhXBg9SUvclNQH7iXE+3Hs+4S1ix9/AEOZFhhzNq0lAUVGMklHTF/yI3Ka4FJqb1+jTCeoWvfdfG2P3fGPFF3vejPcDVO84VX/zeuy3RB+kdR+CD2rPSEAW5xuUqXLDxvMx6J9jHe8b6pgXPFu14926e6nnDQeraEw== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AS8PR10MB4952.EURPRD10.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(13230001)(4636009)(366004)(64756008)(66946007)(76116006)(9686003)(2906002)(38070700005)(122000001)(71200400001)(33656002)(55016003)(8936002)(316002)(186003)(5660300002)(52536014)(7696005)(66446008)(26005)(66476007)(8676002)(66556008)(83380400001)(6506007)(91956017)(6916009)(38100700002)(508600001)(86362001); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?q?/VEjBCUOSyrroHev21IAhXi?= =?iso-8859-1?q?WMQC1jVHOdpycKIWLt/sQZANHaTKMZOQTMrv1KbQadYSAHx6pcGaLEOYHBrH?= =?iso-8859-1?q?MljPXeBvYI3vOSVA7FV1f5KfWoPGrzNgVKkcFt/kTwFLdXg6OOwNd53oHB4q?= =?iso-8859-1?q?e+tQWlXcRDZm36piWJSa65AaiQpTLg30Zfi17gndBd3FOy6FsYScB/UP821x?= =?iso-8859-1?q?gIoCiKCZQIYyz1FzrjPgC6FyUoVV9Xbt7/rmZsH28NTnREDtBJDd6BPL+/VP?= =?iso-8859-1?q?qp2UIHYoFeCVthd+uAMZDjmw6OlbWnjNf7RQOj/Cq/6DuWeKzSzb0DATPNEk?= =?iso-8859-1?q?tPjqfo8+o9t3YwCm6D4SEHTZjmQj3N6KkBbWivLUU5U6e6Sh8b2q7dSwK/yM?= =?iso-8859-1?q?bxBYQ4IicEDtNoMCr+F41F3W0v81lU+gvzAVgJIwrtA/SMB5PP5C6HJZQtOB?= =?iso-8859-1?q?HsYEPK3z1X5deZSa7ZaOsnw6xevTFqWtbR7NQhClgpLP9kos8qK7bU4WaKx6?= =?iso-8859-1?q?oS7D6p7RuLZTCjWB5HoX2L2Ya38SehlyRW5IGACEAYaD5UuqXwT6TITvBizy?= =?iso-8859-1?q?fTlIhkpkrQLzzhZNGoqAlTtARJ25A/7ViRVhJkBYzLPE3lDXcaA8wE25JBsa?= =?iso-8859-1?q?ylW2LMuM5232AVpSsbvme7td7cXc8tMCTT3ewtwzxE5/Xexm25FVQYOjLKoq?= =?iso-8859-1?q?EFZgj7uveHhfWmEWBKRaV5RPku5DazN7oe7R6xzJ3i3qbDFKzZ21RBXrgcCf?= =?iso-8859-1?q?9LpRA90AB/k7Kwpv1HrTx4Ggl+xNY58yITTYmAg1kWUpltdtwaZLpuuTtIsQ?= =?iso-8859-1?q?l/q4omT1aI20QLbXUZFjmU6noBCYUroOMQTkxRejNmTnd+V3lJ0t0mfzxqjg?= =?iso-8859-1?q?eq2uSElyGfd3sv1GON6dVB7VdCqJrztayuMkRiNN8DE2SAI+OppMtAqyCmvv?= =?iso-8859-1?q?AifeSv5gS6uzOayF5ezF2nMs5+ZEIUBJFjZc6PdxMP15ssbrEFA7neQECYW4?= =?iso-8859-1?q?rkS+/PAMBTtu49Rp4f6IJ+8145ffNR6gn9wej+e0bcmBfB4+U3VE7F2VVOlF?= =?iso-8859-1?q?Slb9vXK5BYRswkCyq01mBNKPDeFtg2/RMmKyyBFPkyyslPLbbCSVc+FWM9hT?= =?iso-8859-1?q?10WUvmVSSulqouLNLJWlKKs+9T35t1AlpzTcU5La+uo9DlxuOKIckTqhAz7O?= =?iso-8859-1?q?fNCI9u5MUBE47/E/whGIUJSogYvVS/WQ0kRtVjB8hjqDnhWftBDnj06IFvqI?= =?iso-8859-1?q?Ww8LCg2BmmUMGwGcU6m6nnWky32uCY8gcEHJWQmqf6MBFm4Na0gbwCTlmZ9q?= =?iso-8859-1?q?+98T6XHB8SIJGS3m5k2JOXFm+8ABUSsOtJCxW7Ec8kGXPMq4FGd3j/dSGcmQ?= =?iso-8859-1?q?huPEhbfEidG5AiaiRJHWQW0VvupEl+1KsetE46J4jQUfcRREtTog0lWm4OWJ?= =?iso-8859-1?q?NJQDA7LnVLWWd02ROaBe3K7hUwas3YYpTm3yKzkbgjRR/wqQmey79OYCmn3Q?= =?iso-8859-1?q?98TxVa+9n/feLeMCl+yFBuvaLWmY6T88f/EMzdvtkkXSCCYGZCMxu+NZNsZC?= =?iso-8859-1?q?3r+jj/pM5mdkXQl3ntgH5IkvDUOXkgqGX3yI54rudoRe0eIt4hXhqHZSO/9I?= =?iso-8859-1?q?KV7APgRiuSFWr1ZwphhCmituJq/XWNmlYXwZvsOgI38Fx8rYDsXEkNQgXXxV?= =?iso-8859-1?q?TlRZl+igelSX3aAMRoQQ5ETjid6OcdR2VKfstjewSsxqSZ0Vpz0xbBhJBbNk?= =?iso-8859-1?q?WmpvcL9AIKVRDlEfFTGf3IsqzXi2Nn8JbWSgqzMVh1Pptrg=3D=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: AS8PR10MB4952.EURPRD10.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-Network-Message-Id: 85e54aa4-5398-425a-e153-08da1edd6497 X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Apr 2022 12:42:24.9065 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 70c55e28-9cd7-4753-937e-c751128a9d38 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: mYVPy39gxBg5quE7Jqm0Dn1+rn+q/1vHqZXAOToWmDgDFu3/eZwtkQlyROClxhZVqzxtUAEomIHxhkl4WPQ4pw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR10MB4519 X-OriginatorOrg: raidix.com Precedence: bulk List-ID: X-Mailing-List: linux-scsi@vger.kernel.org The use of the free_sg command flag was dropped in 2c39b5ca2a8c ("qla2xxx: Remove SRR code"). Hence remove this flag and its check. Signed-off-by: Gleb Chesnokov Reviewed-by: Himanshu Madhani --- drivers/scsi/qla2xxx/qla_target.c | 2 -- drivers/scsi/qla2xxx/qla_target.h | 1 - 2 files changed, 3 deletions(-) diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c index 85dbf81f3204..2d30578aebcf 100644 --- a/drivers/scsi/qla2xxx/qla_target.c +++ b/drivers/scsi/qla2xxx/qla_target.c @@ -3863,8 +3863,6 @@ void qlt_free_cmd(struct qla_tgt_cmd *cmd) BUG_ON(cmd->sg_mapped); cmd->jiffies_at_free = get_jiffies_64(); - if (unlikely(cmd->free_sg)) - kfree(cmd->sg); if (!sess || !sess->se_sess) { WARN_ON(1); diff --git a/drivers/scsi/qla2xxx/qla_target.h b/drivers/scsi/qla2xxx/qla_target.h index 156b950ca7e7..de3942b8efc4 100644 --- a/drivers/scsi/qla2xxx/qla_target.h +++ b/drivers/scsi/qla2xxx/qla_target.h @@ -883,7 +883,6 @@ struct qla_tgt_cmd { /* to save extra sess dereferences */ unsigned int conf_compl_supported:1; unsigned int sg_mapped:1; - unsigned int free_sg:1; unsigned int write_data_transferred:1; unsigned int q_full:1; unsigned int term_exchg:1;