Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
To:     Wenchao Hao <haowenchao2@huawei.com>
Cc:     "Martin K . Petersen" <martin.petersen@oracle.com>,
        Bart Van Assche <bvanassche@acm.org>,
        "James E . J . Bottomley" <jejb@linux.ibm.com>,
        <linux-scsi@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <louhongxiang@huawei.com>
Subject: Re: [PATCH v3 0/2] cleanup patch
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
Organization: Oracle Corporation
Message-ID: <yq1r0lje8lp.fsf@ca-mkp.ca.oracle.com>
References: <20231018113746.1940197-1-haowenchao2@huawei.com>
Date:   Tue, 24 Oct 2023 22:31:26 -0400
In-Reply-To: <20231018113746.1940197-1-haowenchao2@huawei.com> (Wenchao Hao's
        message of "Wed, 18 Oct 2023 19:37:44 +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?lgBjfsCelMeAe2EBfdjAEqdi0Ml9T5RzF1CWh4gtX6bgtwsmItrcs8N59Aq6?=
 =?us-ascii?Q?AWb3Gc9lKZtEOFrzyiMdK34s/8xbDcK6uzTxcFD7hel3+1m8RLSkYFkRl84r?=
 =?us-ascii?Q?d4X2cEkoaHzuHnyjZ+q/ZragnHrDAmE5YU6mx7VOrwu92231jdsGSCse/heq?=
 =?us-ascii?Q?cr4wZw0Lsw+8zvGwmhHfx+Cm1gsu2bSx/e/5n4vIR1Vgjyv1e6v8ECglX7vX?=
 =?us-ascii?Q?bVPCmPzmTYdfJinZNbHkl0iQR6oWSz8TBPZquaKJAjvCWCRGRr1zpXCZJ/v6?=
 =?us-ascii?Q?MLNpwyTn0jhN+lA9EyIBvOclVwzIsSTw5vqBXLAodddFj+O3BsjbGCR7xjp1?=
 =?us-ascii?Q?oNGthns2ubQwa63wpT6hQjjanDIoEcnPt+QrkaoMhpmr+EyM3QQP8N57WLjm?=
 =?us-ascii?Q?q5mgvgToUvBA2aFINPpOgffR5lEjM3v26Et0M4uwZVtMl777KjzVcQ2BiKSl?=
 =?us-ascii?Q?QmkDXD8zNW38OESCMypwS215QXKYNlt0eb6E1SMBifgIwb6xcs09aZvUjVJ0?=
 =?us-ascii?Q?XC4AcRy7nN4s6LUd2fWSPtrkNL7U3pJrqAwYokc7PXBoSUoi9mjbT6ffPnas?=
 =?us-ascii?Q?ZQ4VUdsAPT1hgkLAgrkfnYOAzV19GyMmobBu8Bkh5vJ2YjHge/1l8t9uSdlY?=
 =?us-ascii?Q?w5iElfAf6gQovBO/2MTnJXvgfo6aXTqs8bIvgLqkDa73Tf4I3A/fByf4/jsL?=
 =?us-ascii?Q?TRRp8iltcXNFl6w7NavMxJQDD51tr8Dps6ifyArYAFwu8YDKI5yLSvAqi4+q?=
 =?us-ascii?Q?Ne/veS75GijsKhHIu/4Af2QlELLjbCHJe0tzG+gmTlA7dzhWjCwtWvY6iuKe?=
 =?us-ascii?Q?U0VSkYMAdOzo+fuKMuYLBEZzhvcx2IrCsYYZYCvRnSahGFX+G9F39E7BMORm?=
 =?us-ascii?Q?RylxsVk3Y8ekM6xOlxGahJdmoJ4jbwBzQ0OWRRUrB9OVK0y/4tVBKmrgaqr2?=
 =?us-ascii?Q?BgtFOkGolGLE3RG+WP2dqrcZq9EdniusMEWwSXncolqtPq9d+ENZT7o8OL71?=
 =?us-ascii?Q?oEfZCA7PkV0vylAxYuAsgmR76aLXXyySEmT2dBNrmgdOXCumoQIa75s3neDr?=
 =?us-ascii?Q?T/pllZK6h8A8Xg18EmLjIGAkX93pcGtvz/2zqHB6q4pj2Bd0qy1dCvCt+dle?=
 =?us-ascii?Q?5itF9JPjoZiA+qhiWvuMLztsCnMf45UmsLkZbDE/ONKFYLLYVsccCaGlDpx8?=
 =?us-ascii?Q?DJWVqqFaZNUj868rXZnEoFU/NBSMLijh/vE+nta/bzNkPjEiNgBqQBrZNoMr?=
 =?us-ascii?Q?aqtTZDdVVZQm3WP0Xdrtiizorqe/wyG38R5mma1j7SAGC5Zo81BgKkflLYyx?=
 =?us-ascii?Q?nt0mCfFnm8jFBTHrwAf1bpr6jAdO65RnDA9UVCzEApa45R/+3BmJSLPuSK7J?=
 =?us-ascii?Q?DWxedBrbyy0s6dma/0h6yI8iWvTEqKPPXBz95ZS2vGl8ohJlFZdoYWpa8ja5?=
 =?us-ascii?Q?XKdl/XXhaSWv3n992WYJYkpTbEAt24JGMZI1Q/vH8vaSDmRen6Y6cKJEaCHf?=
 =?us-ascii?Q?ik3ctXdQqGmH/GMWadus0fMnJkC0CMw7gVEkq+OxCERtV2K3nvW0dtdzdRy/?=
 =?us-ascii?Q?8tdL0eN8WLwVnAFzz7pRnB55LCYSJmoNOMUFeNSVJGAHQuxUi/6x+cwvhvr0?=
 =?us-ascii?Q?sw=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: iovzvw58SRM5Dh5oWIPGMQjAD/IRa+Wy7f4gumDRTjpPeo+goG1nr6dGgZnpZeSLqpcDbJuk8sLsoqH28Z3MyJjahC6toyh+95+lobHeKHclAYvYixAa3Hqi4CSHBFXJ9ttVDnTWazhI6vgT+jLhAAyVLIlsMQgDPPbKWHRZBe2QniwF0HL54IpayKkf/j8eCV0Y7yQ88glgJZdxiVNQWIFnS7YFWv2MfbovnX4Ud3SMPChbTu2HxbhDKphAqzQnuzmdtplvi0kUasU5Q4xJJlioWlBmzEYVqaE3aPVmrYl4v6lBL3Nb27AmLa2Bs3PFTILONOosWa/nKmKJoZlrc0/DAFEsl1nZoYo/oJXuB2teyP6tLCdG7fgzBv3xWxzQUGBJ6g3meNJErq6Z3w5NcminMe8osnt0Wwss2uDeGnSj4hT9vjSNsxckWehHOt+w5ESlNVCDT+TWG8EEHrsTIMcOVafRb42OtQEQvxuPieYgFV3ubgEYxgtp6B0tsx/LR1/Ex71JDS80+cuHEUbB5UNjdDmv/LKIvViOT71S8HCQ2GUSwbKj3GsF0sV5cPlmoTn3PgWf/qDOjjkZ9i5ccUYmtyNIHfa8a1FG+hDNW+nfvG7oG6VAdThMqOf+5PxOWnkEKGskZo5rkqfT+6WTRvtWFJIzzVBuA/W0+yrDUHEV2O3ReAWQyf/MWWA0veVb4uxNlcAOixxwM149Alwhh71HdeAVA8UVgAM/2t9iUviaJoB1L1Kmoj6NPZXiwx2I4lT0ovLoil6lV9P+f3lL9JVp9QQLEC3zRjztAoCfxAeC7ttFYAK7tOE/Vdq0GLlddK3FuAhjadeYrdjETatZdxwH8SEgDWQUOzuWSHnwNRs=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2dca7e05-a5b7-4112-5c2e-08dbd5027e48
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Oct 2023 02:31:29.3058
 (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: oMEFQuc8cB545dt04Vo74uYHCaTkQsT7DgP78Zvo+oNo097gyBtt4Qs71W/SqAw1jEMHLp6LGn/SXHtGbMvnSaST9iOns0t8YhuVgh+HTVs=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR10MB7268
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.980,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2023-10-25_01,2023-10-24_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 malwarescore=0 bulkscore=0
 mlxscore=0 suspectscore=0 phishscore=0 adultscore=0 mlxlogscore=760
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2310170001
 definitions=main-2310250019
X-Proofpoint-ORIG-GUID: Abh1Bhzf5l-9YlPVIrDB4CFNUEJV8MOM
X-Proofpoint-GUID: Abh1Bhzf5l-9YlPVIrDB4CFNUEJV8MOM
X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Tue, 24 Oct 2023 19:31:56 -0700 (PDT)


Wenchao,

> This is a cleanup patchset, no logic changed.
>
> The first patch cleanup scsi_dev_queue_ready();
> The second patch add comment to tell callback target_destroy of
> scsi_host_template must not sleep.

Applied to 6.7/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering