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:     Arthur Simchaev <Arthur.Simchaev@wdc.com>
Cc:     "martin.petersen@oracle.com" <martin.petersen@oracle.com>,
        "beanhuo@micron.com" <beanhuo@micron.com>,
        "linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v4 0/4] ufs: core: Always read the descriptors with max
 length
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
Organization: Oracle Corporation
Message-ID: <yq1wn73hoa8.fsf@ca-mkp.ca.oracle.com>
References: <1669550910-9672-1-git-send-email-Arthur.Simchaev@wdc.com>
        <BY5PR04MB6327A3C457A16BDFC5C13ADAED199@BY5PR04MB6327.namprd04.prod.outlook.com>
Date:   Tue, 06 Dec 2022 22:48:22 -0500
In-Reply-To: <BY5PR04MB6327A3C457A16BDFC5C13ADAED199@BY5PR04MB6327.namprd04.prod.outlook.com>
        (Arthur Simchaev's message of "Sun, 4 Dec 2022 13:22:43 +0000")
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?eSd49x6P9gUWRZjrWre7l4/rXTY4ZJNKzas/4go7iKAZBoA/4v4WfhJ0s44w?=
 =?us-ascii?Q?6NHaU4VA67xspgflVkpkfUBJzjxPVQV1phTmfthcxJ1i4WmOhOLzj0zXmJwy?=
 =?us-ascii?Q?41cpIF1Io7ILTw8xiMTj0eZnZmiKuZxFX/sIH+1niyV9c4TIITCYUJmKIp1R?=
 =?us-ascii?Q?Iw0V+Q+L2OcPl90PYrJGueT1IZdVUsPRMZRymWIIXQJqmlqXKUnL4E/iBqax?=
 =?us-ascii?Q?CVzRuor4MFN09JH8d6grNKPDdmej8aqYwg1rJLlUBHupVZBC7Oup13JITHz1?=
 =?us-ascii?Q?8XeCbTmFP/wwJwB497JE/BSucqeuq67yd9C0gEfL3WnUIKXLScpRhmjoLIk2?=
 =?us-ascii?Q?wNE9i5bnNGUON2Uu/70hxnmiVqV2z8nA6nspU7qFqn0/q/ipO5pWU/cLVlDY?=
 =?us-ascii?Q?H6lJ+SBQGsOEW8ycqGDvhX+Q5JdGaZt/AUUSlx8/DRziiAK//V0mBeHqWuw3?=
 =?us-ascii?Q?SajDctBLGKSbP0Lf5J8UZueZgXshOCS+JC73nDKPzHPO/sMF0Yx6eoE+PbDq?=
 =?us-ascii?Q?h4KJ2ukUsKH9+OAcQTfpoWxle+BISgN+u4kwkCv7kNEWbFiIM/OjiCWmn+V2?=
 =?us-ascii?Q?L4D8WxCTj6Q2lX+pjj7GRVDOUVNOj968hRL0EX3eeyNCdGYkl+hVElPCUK6V?=
 =?us-ascii?Q?UO4jQR75AQd4xGxqwpwnda7KOVOO1cgZclXl/M7uKEEXfXH9ItG42AgcnMLL?=
 =?us-ascii?Q?bUQkKTXfF29SKsJrmOHlKiLchrmafWGhsASjsOMj66Gtgon59Swa2DI4CgFl?=
 =?us-ascii?Q?YIZhHqXcXSp/Hz+KmUhAnuFTdhgcVVc84e3lO631sRI2QUXnw2svrOQiwnDx?=
 =?us-ascii?Q?M24sAc5gw9TOMDb15S1GRoxj/chyk8sn1r+k7ZT16dZK1++72zTyr/Vv6wGE?=
 =?us-ascii?Q?4Xpvs+IxrAW4mAjw370fBTcMp4XBnOsD5Dud/bXRtqHZL26Q1sQQbdnB23xz?=
 =?us-ascii?Q?AyrbPn5VSeJTSUmEW1F2J2fcAd0Lly6+vaC+p+z7BZ65yo/rP3I+yGwWoEEV?=
 =?us-ascii?Q?S2eNBrt97kkHiy22x/afLA1X4n0pHJ/h85VHgMADtRbL6gwZAXqqSfaqyXpp?=
 =?us-ascii?Q?VUnXYoVv07vDWdVK4hXDfHD3v7UT4Xo+3Uk5TBQ095PZdjoIvQdoOZypLIe/?=
 =?us-ascii?Q?Uro4CgsiQsMXshxY1mfenUzipWZyNeaXhfhVCLXDIYUjcOIA87n9yuRStKHT?=
 =?us-ascii?Q?jQIr/jNn9b7tixsWTqU59CiG1xLQMhaUKUtcL93+F5ozFrDO3e3/iMMTRs4b?=
 =?us-ascii?Q?DZeFnu8Zb1ZDlqX6AbD8cj3iDRlV+w6olS/1K/jfmLieGrVN0vlLstfH3r32?=
 =?us-ascii?Q?Mo+FekLZHwvPAjdimHsh8ukqahqhqaWEc4mj8b+HVhbwebW1xbbFPqLNmE85?=
 =?us-ascii?Q?/8O/Gky3TjAQBrsNgn6DDlgkVoeRQLvHtTigo/jaUrcrrhPYFIlS3QF35cW1?=
 =?us-ascii?Q?Z3a5y0uV+S8QLjkBYkEV4Y0iJQHTEUrL4hhKOE2mqMh+uQRH4CuE1jLhhLsW?=
 =?us-ascii?Q?L7tSLQPzT/heobez3G12WYL5GyrvgVxyIC7v7gpuIKL5qJsN9q+ipS2ZH5Rs?=
 =?us-ascii?Q?vY1ooEjmQiGn0g04rCQ+cr7xSPTwfzpbjOga+UL1mAIuXnTNANVR1Kj91RlK?=
 =?us-ascii?Q?PQ=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: MALBS85HcBn79kAigQMJt5CTzuvlsrO3mbTyE+omBQg8xKLMyGWFx7gKNQhoRYvzX2UfOHYNSXwwYcCpMDFZ8UfwbNCeRbGhqMB9TBGeYv9hQdI8MUQFTsWuZBCJ/+Ng/4tYlsZieD745tIWBOcNcA++wqzatw4s0M8RlkiVbz5laft40+c+Bk5cbVtC0zJy1U0m1gXPXE1lc9Ul8OqJlR8tibZ4ExGFxRWSsgl64t8oMoW4WLzxu+S6J5HBs06uT+1xHWUktdK/79W9rHF9n72IR3W/GL6EsHbvzl4KoxGOGGXZBmTZJYioSx8ZxRIqPNx3uz2XT8hewNHrA/JbeznoM6fzjE/OTzAaW1irJ4UAq7BlX1e5BSxBm+LOk+3ybWueUlznu6C+z7irIbXA9SCUVcX7PP0cUbPuyF3bOQvZ0f+jpPcJUYPyyJS1fBlp4ylwsIm2T+vtIOC2K3pjfOBQbcoiv6gkfcjaH9vj8iVbFoqFA4p7ngXXp1MKlK5f2W1CAQHGE8FZu+7KFi7lg8WVDcfdZFWBYy8JVBiZX5gKXqVSB9X0mfkXB7pdkXaMUbk99rL2lWUW4E7qmtGu8gh9UZBd/2BJbLo8mTmPgM175i5/cQf6r3DfG5hqgLgt2PIX6arfYsev62/bkXdwtuXgCVhx57k2az3aooWa2c9cUIQSro/B+PvPyzRJ1g5W+B5n6+bBp1ul71nnTUJPPNbgvzxCfTTsV44B31hFrrBZGwsco3YAZaxxGheWZtu/MWZqD84m2Bk3VTkNfFFw2dlG3gus3zIZmzdv2CkLB1E+FSFHrclVAH6Pd2lkL51jZwdlJC0XxcQ7wIyZ+Z9D+C+XFNSRk5jj6Q/qtW8+zqA=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b52c4cac-c4de-407f-4269-08dad805e50a
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Dec 2022 03:48:25.7312
 (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: lHnBq30AXLxUPotlbB7tavtqgHq/4LOfQ1wHvNyIaPlt7bc7hF2SICYtK1p3nG6DFSK3sZXV033UxWLMfd0z6oh1iw0yQ4aWFLVKN+T2qpM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLAPR10MB5156
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.923,Hydra:6.0.545,FMLib:17.11.122.1
 definitions=2022-12-06_12,2022-12-06_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 suspectscore=0
 malwarescore=0 phishscore=0 mlxlogscore=810 spamscore=0 adultscore=0
 mlxscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2210170000 definitions=main-2212070026
X-Proofpoint-ORIG-GUID: IAe2hB8NKox20X-bzHIEzOQq81mfgBJ8
X-Proofpoint-GUID: IAe2hB8NKox20X-bzHIEzOQq81mfgBJ8
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,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE 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


Hi Arthur!

> Gentle reminder

We're just a few days away from release, not adding new code this late
in the cycle. I would also like to see an additional review given that
this is a core change.

-- 
Martin K. Petersen	Oracle Linux Engineering