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, beanhuo@micron.com,
        linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v5 0/4] ufs: core: Always read the descriptors with max
 length
From:   "Martin K. Petersen" <martin.petersen@oracle.com>
Organization: Oracle Corporation
Message-ID: <yq1sfgw4l3u.fsf@ca-mkp.ca.oracle.com>
References: <1670763911-8695-1-git-send-email-Arthur.Simchaev@wdc.com>
Date:   Fri, 30 Dec 2022 16:50:43 -0500
In-Reply-To: <1670763911-8695-1-git-send-email-Arthur.Simchaev@wdc.com>
        (Arthur Simchaev's message of "Sun, 11 Dec 2022 15:05:07 +0200")
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?TyhU09naw9ppeZXgaGECBiRHbforqyAZHkn1tKMKsL2xdJEDOa45LxDz2fHA?=
 =?us-ascii?Q?X6/vOlgfe7Fm33zZ9SR62xKf+jPslv7rHLiPjQ/OmNqnEKdfm4VQ7J65OkWk?=
 =?us-ascii?Q?R5Vbsg4ILM1/Qtiw9FkKdPqJYFCG9eXavr9RcK824JWAs5PSU4xrPE7E7VEy?=
 =?us-ascii?Q?AXYac5G9xtB166GN7XU2T/vXzVKR1gtZ8kYJH+WtQwIa8fH+VXrMSCICyLBG?=
 =?us-ascii?Q?c+LvoM8MsEHuj/cLl8YFq+XOGyNQqE4xAGE3hE1BeoF8P3p+iITYLyrq4WMf?=
 =?us-ascii?Q?cMtpDah4zIIQ+BNZguFSYoyuful1Tc1ETdywf0+GiausSToR0CkvknHiAVY2?=
 =?us-ascii?Q?E34NTrbcrSZAj+vbT5Mro8hjA7n9VKdQHJ5OZajhdPB+tcXceBN80jjtCdEd?=
 =?us-ascii?Q?B7gkfH+lgh0yTxRwFpDp71skw2gUnffbN608Hq/O8wltI8Bs0Fb6oW9fA201?=
 =?us-ascii?Q?cLrfVcSIrpBvx00vG24cUDUw9SWilFCCiRelPu0W79mhADBOMuHvI0cUPG5c?=
 =?us-ascii?Q?XNLia2zRxGNQX9tWeh4Uwgc/IcEFfglebN++vDv+pYfB9JI2gOy69cMk6W/5?=
 =?us-ascii?Q?XRg2V6BfSRxxUMstD1NKEAaxKHqwweYkqZ6cd4YXjzAS++jTsh3AxQMGG1nJ?=
 =?us-ascii?Q?8LICKZKhMecLz4ANxs4nywbQ4rTDS5L0riZfaJJvAdMSKnMVcnTZS0egnR0N?=
 =?us-ascii?Q?3E72APun6l3HGEpWIW1i4VnvoYNlZmvH8rIeMZObZOro2/FcSBHvraSozwVq?=
 =?us-ascii?Q?vM17UmooKqFmJb60RhHl7HkVfJDlLRjRUVKjKFXTsa3WcAcOe+LhITP82Oqo?=
 =?us-ascii?Q?oY7rb4NEBleeloVqm3ms7fj2yrl8OLXhJ3jkvB2Hv0jr0VF5p0j3352s9PY8?=
 =?us-ascii?Q?dQ+NUPIeO6+vmFc/bfI7w1jL2HXiD43pWjyceqyZv2sXBszNLcnGx2FVZQIr?=
 =?us-ascii?Q?W6B7WAiLPevyupPlofoJ6bMc6pLz28TOz9fxpH5aQehBCvR8hyC8ko5LhR3/?=
 =?us-ascii?Q?B9m+R1insCSgtwj9+7WPRtFXH0xU3JB5IIFWEqUNWTfn2HbM08BjJPtWXDWb?=
 =?us-ascii?Q?GPFb1NXuEaYEM04QVGLweDPqeB2uZbx7aOjei+yZIwiVZhODW+3v7Adf6w+Y?=
 =?us-ascii?Q?ZGJ/i2tJDxe8YFTl8nymw46A5CPein0Kh4TH1bxQq5xsz/61pdpan+b9ZYI7?=
 =?us-ascii?Q?Zdp5c45sW32FQ1qMRyI3cH32R7DPdQFPTMNN7UKW7ah0M87I/vPDu74kFg5i?=
 =?us-ascii?Q?lMfMc9WIvdx/BY7X8KYe1//T7J7Aqq6FUAAJEeNIgozoELMYf8ecF7wzzXZt?=
 =?us-ascii?Q?ExjPChPHnKtNndO8UoOs0eHe/uZZx6hz5pOwRqZxScFKHhuAHOTRE8BwQyYK?=
 =?us-ascii?Q?JajeREhRmWzewFr8StBT/l83xMT2vmTsvcs/qMft9juHLwkNaLEYzprnyogC?=
 =?us-ascii?Q?RpV/P3N3Xa6JYuRS5fjtQZ6woIvexoShKnttvH+CO34Hukdl5ZJ9t5kl1+14?=
 =?us-ascii?Q?032OGLjNf/INAYKbxQf2EIdN91bs7J3JWTUPZdFLGusFY2yNtDdzpuvLN8km?=
 =?us-ascii?Q?tldHtJZDnj1D0AN9E5UMwyOhvp8fGhvSxhRb2xOemNyn34HCLg5y9At7AVYb?=
 =?us-ascii?Q?Eg=3D=3D?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 939f062f-7809-4323-f430-08daeaafe7a5
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB4759.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Dec 2022 21:50:45.4633
 (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: cAul//8mrWMo57HRpAjLofxnsD7vudRpSoDWCxzZ6YXY0NnRvFHO1pxmEKI2kdQzH8PLym0oU7WXETHiXPM2lD1qihVuvPAxAtk6R9uBJmc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR10MB6603
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-30_15,2022-12-30_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 phishscore=0 mlxscore=0
 bulkscore=0 malwarescore=0 adultscore=0 mlxlogscore=999 suspectscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2212070000
 definitions=main-2212300193
X-Proofpoint-GUID: fouWWYWSFsxdOIYPVY0vHzhZUBNQN9HD
X-Proofpoint-ORIG-GUID: fouWWYWSFsxdOIYPVY0vHzhZUBNQN9HD
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


Arthur,

> Read any descriptor with a maximum size of QUERY_DESC_MAX_SIZE.
> According to the spec the device rerurns the actual size.  Thus can
> improve code readability and save CPU cycles.  While at it, cleanup
> few leftovers around the descriptor size parameter.

Applied to 6.3/scsi-staging, thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering