Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp453320ybz; Fri, 17 Apr 2020 04:18:33 -0700 (PDT) X-Google-Smtp-Source: APiQypIg8mIwZd6dyIUgJ0ckcYqBUHi5/hq8A7JRnuUy0OA+aeA+0rS6tgGAdl9YlnsNEt3AXde7 X-Received: by 2002:aa7:d0d6:: with SMTP id u22mr2485869edo.262.1587122313519; Fri, 17 Apr 2020 04:18:33 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1587122313; cv=pass; d=google.com; s=arc-20160816; b=uebNo0DiLd4GY747Y2R2mc0/zp54tY3J6xEeAtMDKzb0iA1QGqt93Gt1AyItOl9Wa5 +EEpgiCqriXSZHQLusEo7mnFdttvAkfX1mCSUm9YH7y+gESnZASk+voCdJOwRMzt7gJZ qFLauiRqK4FPqVfjiwQ3d/076Iuy9iY0s2mOXBdbfG2sNBmtRsBQe/ceCHueR3qwTM21 SJJBeM1zIS+dVze42Gyq4+2Hf+kR3ABwfdVUIR84tFvbpxVzJiszqZnxpMc16yfo47no T028w4bEm3rUowpkZll4sdTveSRxgqOpMc07BmWe6/LMpaCRmdQthWp9D092lZlZlTem A6mQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :wdcipoutbound:content-language:accept-language:in-reply-to :references:message-id:date:thread-index:thread-topic:subject:cc:to :from:dkim-signature:ironport-sdr:dkim-signature; bh=/gbcRDLUK/CWo+dnZdqMol9+5Qf+EmUyu4JbUzB/MHQ=; b=f0KuuHkU5mLDliJD3P6527RDgJy4gzOZSm2dGsTHJE/FEaTOtMTGEK8WD7YO4nY0Ir l+RfCInQpNhjtUo3aRO05tHPHeErBYojE2CyIxpPlFQCwT5Vl9iiwa1E7AAlCyaq2D+u Ij1tQWSqsPdfD9mHakcu8REbyrHmiIKLB5oaejplt+s5Fj0CRsSqXgAdHekmhzJQ0AM/ bNiqrq4N9lA26z3S9JjP2Irccud0l+QDYVme80Gb/PnIGDQGVWtAjDZcEn1o6KLmHCF+ DTKc8fE7hyZhMPGAzllYmBM53idd2m7crnhePyfFWP+yNlaWnMSUMDRjIG56RhLa1N93 3gwQ== ARC-Authentication-Results: i=2; mx.google.com; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b=Xw4sYBrK; dkim=pass header.i=@sharedspace.onmicrosoft.com header.s=selector2-sharedspace-onmicrosoft-com header.b=Lmxxw0DS; arc=pass (i=1 spf=pass spfdomain=wdc.com dkim=pass dkdomain=wdc.com dmarc=pass fromdomain=wdc.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=wdc.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id j21si13305999ejv.161.2020.04.17.04.18.10; Fri, 17 Apr 2020 04:18:33 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b=Xw4sYBrK; dkim=pass header.i=@sharedspace.onmicrosoft.com header.s=selector2-sharedspace-onmicrosoft-com header.b=Lmxxw0DS; arc=pass (i=1 spf=pass spfdomain=wdc.com dkim=pass dkdomain=wdc.com dmarc=pass fromdomain=wdc.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=wdc.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730266AbgDQLO7 (ORCPT + 99 others); Fri, 17 Apr 2020 07:14:59 -0400 Received: from esa2.hgst.iphmx.com ([68.232.143.124]:40391 "EHLO esa2.hgst.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730091AbgDQLO6 (ORCPT ); Fri, 17 Apr 2020 07:14:58 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1587122109; x=1618658109; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=uwDTJvQF+ggoZxXhliuBBdeLUlaT0MzSk4RdsQ7roSw=; b=Xw4sYBrKzT1iU2wPCMc7FLdqGlV1vfQkbsPNDOyOLlIshhW4KK8HNqfe cFvj8/gUUSzPYC+umyhsvysuxRhqBDOqJ6HeRd4kiNf+GadDfcj07YFPE xBcobbrrEo2z+sZ65rMhu3cne2WZUMtmu0jYp6KjfqkB2jivT7WUwcPvS 665HYARw3qHwn5QfS5t0lG7X4KCmrHe/5Po1xD+Y19N1yVSm89n3gDsiV x7PLdFEVRPVFZjzBMlaAsGZjRLfDFxvfpILg0AyP4Y5CzReVsdCeJqfdz KM42XGBZTRLwdnKJVLqcMiZP47Ck1CprkWOe+h1K/RT2U1smkolHOfSi8 g==; IronPort-SDR: rc/HFN0ZDTThghrsf1Vi/CRIdSx9F6GnOBuHEpGAnyghoYf0+G3R46KW+i1Vkow29j2kH/uue7 uEvB0NIyfV42ovKRm5wPA6+o+xYuyToTP5aSyFuanZCDwvWbm2hzX+iJC22kWvZZx1g1UeJOUs MPonszxaH6Dlshqhp7mo5gRv3nLYAhcZytXTPQ9udHgHYS1bo+tJt131GiwriVIM3k+CDCd+Ru 1lDdnVzb7ABUPyaN4v2dUvvKIV05eXTosg62L9a+RpHr7VgDQ3uNaTg6jFSb76UlBaHlexd/lm n9E= X-IronPort-AV: E=Sophos;i="5.72,394,1580745600"; d="scan'208";a="237986144" Received: from mail-sn1nam04lp2054.outbound.protection.outlook.com (HELO NAM04-SN1-obe.outbound.protection.outlook.com) ([104.47.44.54]) by ob1.hgst.iphmx.com with ESMTP; 17 Apr 2020 19:15:08 +0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=UeZRcIF/7+81DX65qo1gkl/KFaESYQNlHjQCW1/C3IeEMOoQU7HjaTfiuhpHBfWzZFPQ3m+D8CA8aX6vyTW7sgv8BMEI20KZgnOq/QDsGCTE8b2rhm3uhxMn5dtzqFiUYnBJwt0q/33Wjr8JH2KBykl55CX+Baj0HmVkUgoteobNC6msFy00xT4+5TLJZ9a3do0IElaGPd1i7FL2MIu4pubh0qAGnNy7VbgihRity0FLFnlof+xvHeT6B3C81TKSc0Z+vIkovYDPyHdZGkHQTY8Vgbrzu19AD6OOvZ9yvEvpBlggbuX84lN47ORgSRhtNYvcQSnijvEMZPipDgiHjg== 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-SenderADCheck; bh=/gbcRDLUK/CWo+dnZdqMol9+5Qf+EmUyu4JbUzB/MHQ=; b=F9EvrBsO5fgJzkEk4mOaGTDeIl00wOPe++hB4apgQ5JC8Qgu8yWtSld/Q/jqo2ZTej2BFiwlTKI2//ZN51StM2YgVEUOWfPCZh4fCKfdB0pnfGPr62+vRB5n3sCNx1ZqU04bZMIgzkweIEaYhAyOr5Y7m7Ea6Ic0JAL7c78roO+/0tBCG8vvfR7selxTNTPKwhIdOc7Ge0KM9kEVt4QaDfwam1kyLs8+7TkVnYtsSagcLRP5mLyXspqjCZUS19ADNweX2jvVhM3PTJ3Wz5wpuxHJMrIF1BXVAZbgo3GSbX1mf1NT1LlacV0YrEQYfKP9beQm1pyPsQzUPICRxKp5Tg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=wdc.com; dmarc=pass action=none header.from=wdc.com; dkim=pass header.d=wdc.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sharedspace.onmicrosoft.com; s=selector2-sharedspace-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=/gbcRDLUK/CWo+dnZdqMol9+5Qf+EmUyu4JbUzB/MHQ=; b=Lmxxw0DSPH+kPfTv14SzXH+WmYCXpZobRDm4XSzbX47hXYd/FVH+3764newITtRQusS46TPeD13hpv3qu/TUYxa7/lj8eXV6U5TqBAl+4fIhRN34886X4zYR7zRtO4EueZ8Bcc4YmkwI9vHF3uB6LnJTVtyPbOb4E5pY4mLePlQ= Received: from SN6PR04MB4640.namprd04.prod.outlook.com (2603:10b6:805:a4::19) by SN6PR04MB4447.namprd04.prod.outlook.com (2603:10b6:805:2b::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2921.26; Fri, 17 Apr 2020 11:14:56 +0000 Received: from SN6PR04MB4640.namprd04.prod.outlook.com ([fe80::3877:5e49:6cdd:c8b]) by SN6PR04MB4640.namprd04.prod.outlook.com ([fe80::3877:5e49:6cdd:c8b%5]) with mapi id 15.20.2921.027; Fri, 17 Apr 2020 11:14:56 +0000 From: Avri Altman To: "Martin K. Petersen" , Alim Akhtar CC: "asutoshd@codeaurora.org" , "nguyenb@codeaurora.org" , "hongwus@codeaurora.org" , "rnayak@codeaurora.org" , "linux-scsi@vger.kernel.org" , "kernel-team@android.com" , "saravanak@google.com" , "salyzyn@google.com" , "James E.J. Bottomley" , Stanley Chu , Bean Huo , Bart Van Assche , Venkat Gopalakrishnan , Tomas Winkler , open list , Can Guo Subject: RE: [PATCH v1 1/1] scsi: ufs: full reinit upon resume if link was off Thread-Topic: [PATCH v1 1/1] scsi: ufs: full reinit upon resume if link was off Thread-Index: AQHWEfSTyhgt9a0PxkGvKkn0dRVeQKh9Lncw Date: Fri, 17 Apr 2020 11:14:55 +0000 Message-ID: References: <1585362454-5413-1-git-send-email-cang@codeaurora.org> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=Avri.Altman@wdc.com; x-originating-ip: [77.138.4.172] x-ms-publictraffictype: Email x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: 7773aa92-5311-40eb-be54-08d7e2c08f42 x-ms-traffictypediagnostic: SN6PR04MB4447: x-microsoft-antispam-prvs: wdcipoutbound: EOP-TRUE x-ms-oob-tlc-oobclassifiers: OLM:7691; x-forefront-prvs: 0376ECF4DD x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SN6PR04MB4640.namprd04.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(10019020)(4636009)(376002)(136003)(39860400002)(346002)(396003)(366004)(76116006)(66446008)(186003)(66946007)(64756008)(66556008)(4326008)(478600001)(4744005)(66476007)(71200400001)(26005)(52536014)(7416002)(6506007)(81156014)(33656002)(8676002)(8936002)(316002)(110136005)(55016002)(86362001)(54906003)(5660300002)(7696005)(2906002)(9686003);DIR:OUT;SFP:1102; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 3tma5WZ08GrrM4v3Fi3GrPvn/TF1Xka2A/ZZvBOeBXofzZJEr5xLw2Y2eXrU6p95jcMcx87oFbWYJXV2PFCOLUkxsuj3ikdCNuR13uZayhsHZ7P4Fa1jfZ9kQERkqMvcVT6ZIzvF4VBiinJQyRbaniCeiGSQsSj5RZ0545Bhg6HsEIhSX5kTPsvSR26SBgCf7XZjP8/8/f4HIigJIvPWsyf4v2++GgGcigTV5J7d9BpTyM+V6LA9xi3NZjDe6Yhg9eM4xPH6s3e/CFEJlvXzrYNi6tpScVNong1awU/qAW2ZwJzjtIf4/piEtnZ3x7nMniH9TGHgG0wmnRS143eFEgzXforpV0H0M+UdbASkV1bsIaxPb5axtFqr8n0s/4dmOfKEiS/UMCezI7n/mpzFixFXvEjatbqirzTtWhBu1rWC5KwyDCwczBnvese4hpSL x-ms-exchange-antispam-messagedata: hjIHUThXkxYJjl8AKd3TaHT0BaDn2n/t2DA+k0hrZojHOdUjOSxTstl1hao/KkomZeQv3qDajBTWJgrtTjcPp9nPuh17y/eQRHWQTRJ/u2u/gVOlo7DQSOoEUjQpOCFN9S5Iil8WfjnzT2pv3Whkwg== x-ms-exchange-transport-forked: True Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: wdc.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7773aa92-5311-40eb-be54-08d7e2c08f42 X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Apr 2020 11:14:55.9389 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: GtezYHn+ON/5P+SBffoTUcTtAPzq3ZxfI20sZ8SVSzz0dva0xVWuCQLNtP95kmSlJpQU3Io0NGP1TJlfDo68pQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR04MB4447 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, >=20 >=20 > > During suspend, if the link is put to off, it would require a full > > initialization during resume. This patch resets and restores both the > > hba and the card during initialization. >=20 > Avri, Alim: Any opinions on this change in behavior wrt. your > controllers? Would a quirk or callback be preferred to changing this for > everyone? You have a v2 of this patch and a Reviewed-by tag by Alim. It looks fine to me as well. Thanks, Avri >=20 > -- > Martin K. Petersen Oracle Linux Engineering