Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
To:     John Garry <john.garry@huawei.com>
Cc:     <jejb@linux.ibm.com>, <martin.petersen@oracle.com>,
        <linux-scsi@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <linuxarm@huawei.com>, Xiang Chen <chenxiang66@hisilicon.com>
Subject: Re: [PATCH] scsi: hisi_sas: Undo RPM resume for failed notify phy
 event for v3 HW
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
Organization: Oracle Corporation
Message-ID: <yq1czgk2620.fsf@ca-mkp.ca.oracle.com>
References: <1651839939-101188-1-git-send-email-john.garry@huawei.com>
Date:   Tue, 10 May 2022 21:48:17 -0400
In-Reply-To: <1651839939-101188-1-git-send-email-john.garry@huawei.com> (John
        Garry's message of "Fri, 6 May 2022 20:25:39 +0800")
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?HYCN8eKYacwY9pVCGEXAnINWCbqEK1USQ5hPBpFQIYnimNj4nLB/gkah959a?=
 =?us-ascii?Q?gQvFBwKlZqSL31XAC0Q7oAT3K0Bom1j2Wt/kaN/EhR19BI2x45q+h63/t18C?=
 =?us-ascii?Q?+qQ+GjCJ3iyucIQFStoCCdORp6M7Yd3qXdisN47cQTI0liIlCulyz/hJLGHM?=
 =?us-ascii?Q?AFeld6PSlhnJdbX0EcvicA+yr5iKU80JgpgVNve9wtjJGmMrIFoaiQiYwPsf?=
 =?us-ascii?Q?U7oxL8tCVD6xVJilWoDWCT4suVym78W45Yxut5UJhc9Um4hvPRThgooSkLK6?=
 =?us-ascii?Q?Om/mClDFd0VPVGemE3Mm+i34OGI7im6lnQPDpOoNm6au09YmLwyw5KAgGq0R?=
 =?us-ascii?Q?e/irOWvM3Z0ktLHIvpwEv6ZHXFxNW9FFZTdEV6HD6yDLYV9A/FcSpKC7ceG0?=
 =?us-ascii?Q?jGAiGG9eMdlYbF9I6BucmHZa1+NUKM9TuMO611OredhrSasWLlcEzkiSsH/d?=
 =?us-ascii?Q?GrNeiLtat7nBQfcTvVyhCGRR5Tn9sKOtuMvBb0pSSP2AmxNS1DAI4+Or8SGW?=
 =?us-ascii?Q?Pkqu4LXJdMMy3YWCYKHT7kLQtFNrp7o6LPV75P5wLg6M2+PNJn4Afbkd1Xdc?=
 =?us-ascii?Q?Oj0bELWYq0mzw5aBsclRqsFsd+XOHXHOYHDq/Lx2jEQddgcwEbUGTABcfODO?=
 =?us-ascii?Q?9YiBhD6NsrH97ry/zEXPeiHaeJIPXcNEIQsluctO1RQED0Fma5jM2J+jd4wq?=
 =?us-ascii?Q?5ok5hpXO1vNuBvjV2JA9sWNEMEs+lcu+QhQxQ0JtO2M9F5FU9JrLEqTgN4l0?=
 =?us-ascii?Q?QF90NSkyhtGBQtXPPfByTVevNvstwskN8uPO6kj6h2Cb48PdabJBf9JcOEoy?=
 =?us-ascii?Q?0+42POdzqU7JgzpxkQ3rse5hPqhgq6WNc42UAuDB+zb85pneH3kVMCx8nxDu?=
 =?us-ascii?Q?GZQDW8xn1iVDb/7JCbfDlgBOxe2pvXa6x7sk06B5UhnppgrXcFZdd/Zrhu3N?=
 =?us-ascii?Q?VzbnQDFGqIJ7J09+fm0TxZGPZsNP2fNp5vyslviV0MWQ5P9ukJapdlctKHHx?=
 =?us-ascii?Q?XMAhnH1D5j7+Dl20XdO2udrgMCagt8WlzO/7k6YyyXzzbGpCHnqXqDH5p8qb?=
 =?us-ascii?Q?/phDTZiiGOUDGPHnlnXZbQ0XFhsX+mGoIRy/I3OkWYf+ueb6hhVsBPg9wEhQ?=
 =?us-ascii?Q?giOJfaYTVTA/5M6gTTaPjvW0vxJsEjOblaMQlOfm7rJ3mm3fMQute9zJrNby?=
 =?us-ascii?Q?KsGsf0+s4aNXD4cU7DsJ7XFu7XayWujTNrN41zEMlPdp2Q7L6cTWFZXf0M0f?=
 =?us-ascii?Q?TLkQRNT7BHW6t2/EJ7ChHs31JYc5XvHyJcGem95n6mpamfqlf2nf5q6cT+E6?=
 =?us-ascii?Q?VYR/XSpAAmTXlZxGRlv9kTUeSo5oj9IlAhg7Y3CDXIQlA0SInDsSRohIkG1E?=
 =?us-ascii?Q?jMprFnzU+QIRnXxOHmw1AXaDcDtOBfs+4V5jasdEyXoRIsS6FXizFV6yXkik?=
 =?us-ascii?Q?+eVra0gzFn12aCRnj6QxYQvT8SWMOaASu0//lf0JsW2IGkLFWLKhiKLN/aAN?=
 =?us-ascii?Q?iNgPufAM05R9xhiUr5+Mv8kdADPuOtyXVK1pAmFmGca86b09IPErrb0p2Grz?=
 =?us-ascii?Q?XAtGFs0EqazdnzMNmMIYsugiNBMmA2l6UylEQU4Fj7sn5hZyYe9hFCW/jzjT?=
 =?us-ascii?Q?7NoZzdfyGTToCTqRbD1pD4Op3HsUK3gMuWKCoiTGqCVZxcUSM7MDb1IrXxuq?=
 =?us-ascii?Q?N/eGHMSTnpYxHitwdhi9bc/alP998ZM4q7oJetln/b3HdGrDkob3rR2bclBs?=
 =?us-ascii?Q?VwF+Z3rj2ymHmMSv0PVRYeCuqxqjyM4=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8f95ecad-e2af-472b-565d-08da32f0531d
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 May 2022 01:48:19.9465
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ennd8Lqhdi877NaspK6hQoq4YWctuR7yOG4xcQXIGu2Hp+bq6O22VQPla1Fx8UmmXwNSnZOOUdSKTyXCKbqLznqpLw0dNCoR7gULLvP3e94=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR10MB3733
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.486,18.0.858
 definitions=2022-05-10_07:2022-05-09,2022-05-10 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxlogscore=583 malwarescore=0
 suspectscore=0 phishscore=0 spamscore=0 bulkscore=0 mlxscore=0
 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2202240000 definitions=main-2205110006
X-Proofpoint-ORIG-GUID: mF5sZNf4fGAsOs51FkYOdbnkgVtKwpFS
X-Proofpoint-GUID: mF5sZNf4fGAsOs51FkYOdbnkgVtKwpFS
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,
        SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


John,

> If we fail to notify the phy up event then undo the RPM resume, as the
> phy up notify event handling pairs with that RPM resume.

Applied to 5.19/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering