Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
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>
Subject: Re: [PATCH 0/3] hisi_sas: A device rescan and IT nexus reset fix
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
Organization: Oracle Corporation
Message-ID: <yq1tu9pq63u.fsf@ca-mkp.ca.oracle.com>
References: <1652354134-171343-1-git-send-email-john.garry@huawei.com>
Date:   Mon, 16 May 2022 21:51:27 -0400
In-Reply-To: <1652354134-171343-1-git-send-email-john.garry@huawei.com> (John
        Garry's message of "Thu, 12 May 2022 19:15:31 +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?HpjL/pwFDpIS0ZftX9/I6CW18ypDIMbR9n6sl4OtSkMvvZfakPg2mkQhw4Fx?=
 =?us-ascii?Q?eB9YaPlWK5n5tiFbSwuE3hvjbItSzVD5owF4uGGf4+RAgnOtf/EqgqUo1Xbd?=
 =?us-ascii?Q?anvjZAg+l2p6g9k7uFUGZ2C4zeudhAbkEgQaf3ea7FALCoB4SQ8F46k5y0cm?=
 =?us-ascii?Q?CysS+bxyPjX9bhjmSgOMCxQDq01G8fI6BfyKrbl2skOEKvvR8KQdxd3r3OJ6?=
 =?us-ascii?Q?EHqIHO7MaKk98EEvNdLkIcFMI6c7v34qaPr2bJ9l2IbxD09thfhAUtyIo8p/?=
 =?us-ascii?Q?UCWZBFD2fWcAlp943Mj2svJuX4L0ZSByk/51Bzc+A+tDx2B6Gp4Gw6gnhTEW?=
 =?us-ascii?Q?hsXK5M6+ytQJenQau0WFB++3IjVG21HlxWJt3O3b/Q9OHpYbTqV/ONFqcK9L?=
 =?us-ascii?Q?tvJB+T9J/XHNspJ9izBjmQNN3x/aSfX0FY+JMBeF4k71gzx7I8PtsTXfqWYs?=
 =?us-ascii?Q?phMhLPnbRjiYWFpQnjxZe/Tkf+ZrIsKbvXm/VPc89XVCenoWKfzlc1WtbcAB?=
 =?us-ascii?Q?uJVZ8GxWneP3JahXy8ZVTrW7Sru20T7yqmLngEEMzxrtaGJA1tPlfbszfHp/?=
 =?us-ascii?Q?ocAXQA6kBdVpYrlEAY9gQ7fWaEomPEgYE8EW+VkO8NPJbtTV73rtRC4NM81i?=
 =?us-ascii?Q?KYbMw1iYyCJYAceR1fe3j5Z+HkeiOfdwFClu4A8GAcQOOwMgewrvp0L8EN39?=
 =?us-ascii?Q?ustrMxyqLREg5NR/wiomox/IV/H7v8S2iQEuGcsSfYXmCjhahWAegN4uboPY?=
 =?us-ascii?Q?252YBk2qW9N8AGyd8B3IYPXvrSp7uxbx7xu0JyccXqK70TbEu9nQlAYRp6FN?=
 =?us-ascii?Q?0aVJrLesnIFeSgnfhUbth85RFyQrxGHlSzNwvrkkCa1DPqIf58LkPdMZq/OT?=
 =?us-ascii?Q?mRvd91o8gZmZWkIlDzMuTTLwUCjII4zC/hSGYEAgt4OGwxhXiABQDk0pOOR4?=
 =?us-ascii?Q?UEMHz1NGgeUSekF6lxFC0kkJMPH39DEInoY6O/2fPJIr4qFAZEj5Cb5qFTdx?=
 =?us-ascii?Q?8caUSPSSm694tRRRpzfwGRIhp6fke6mmYgHMBO8oAgQ0+Lnwj8KPpFSpnn1u?=
 =?us-ascii?Q?t8ua4o+IwgzEa3b/ITvRGWuuRqVcydphUiKmXwVyMys/NG8WO8/y8nwP59s2?=
 =?us-ascii?Q?9IXmd2tURBTRpdr5Q1zGvzms1qvENbGoVRP2PSBrO3sC89JlPAEWAlDNHhTn?=
 =?us-ascii?Q?JfxOCzYsjA4AN42+ri1iBnfIoM4tjsypA/uPeLXHz1FYCU/X9hCMszzSdCyA?=
 =?us-ascii?Q?FaZyR1zB5MZKTzPdRcVOqZC8IIiFEJmxPyBEBcyK18GvhFwuP/1JOtYlfQuZ?=
 =?us-ascii?Q?k2HNLhyfkn4j76omvKD3mB+OdalBq6ntTCgGMfny+zJkzLK1AIpdCUoCSuG4?=
 =?us-ascii?Q?EsgltMXI8Y404IgdkWlfE1d7n7h51fuoc/OJo4G0mkGiol5MY6SyHay16dwK?=
 =?us-ascii?Q?wtwkvpjYRi1q1Jbq6L1myq3s2AUT4PTCyDDZGsIExJlYfc0GvZ5O3mgQflRS?=
 =?us-ascii?Q?qm0VMuu++j96gK0QJ7JgYNy81E+fjmvICu1/JEHZzSxhX7ZVfhLmmybnVJ7O?=
 =?us-ascii?Q?FuKsy3vcUW63rTkgyaDdLu/yK39KDFGVsLmLpnGgOaN+djC3o9XjmHTnIhXL?=
 =?us-ascii?Q?PANFDNnJP5AV1kXgeNmTEgaajAJ/Io6lKzWxD1ypBzuiSXxx4pFqzQtW4YRf?=
 =?us-ascii?Q?lLNazwnPEkBdNuZJB0NUruVR4XKrJzgRpWB9l73SEM/W2mPbMT+skZyCpoSH?=
 =?us-ascii?Q?ymYmEcGBjteFEZ/EhKffgrNjCt8jeOk=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 12a70311-ecdc-4b39-9485-08da37a7c2f4
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 May 2022 01:51:29.7939
 (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: u7527qe+5UXuMfhfsHANChWT8wOc2uaCPI7DR/BQjiGNOk0SrK+H+RaEiARoV0M7Oe9Gcu6uHjdM0jYzBtrUGDCYIQ7NfOP+OU559vgwdjs=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR10MB3853
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.486,18.0.858
 definitions=2022-05-16_16:2022-05-16,2022-05-16 signatures=0
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 mlxscore=0 suspectscore=0
 phishscore=0 bulkscore=0 adultscore=0 mlxlogscore=923 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2202240000
 definitions=main-2205170007
X-Proofpoint-GUID: 3qGZtspyca9BLZkm-56qcaU2fEmrq2V_
X-Proofpoint-ORIG-GUID: 3qGZtspyca9BLZkm-56qcaU2fEmrq2V_
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE
	autolearn=no 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,

> This series includes 2x fixes for the hisi_sas driver:
> - In issuing an ATA softreset in the IT nexus handler the disk may become 
>   lost as we may issue the softreset before the phy is back up.
>
>   So export functionality from sas_ata_hard_reset() to wait for phy up to
>   synchronize.
>
> - For host rescan we issue a nexus reset to the disk which is unnecessary,
>   so drop it. In addition, usage of the device status flag needs to be
>   fixed.

Applied to 5.19/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering