Messages in this thread | | | From | Arthur Simchaev <> | Subject | RE: [PATCH] scsi: ufs-bsg: Remove ufs_bsg_get_query_desc_size function | Date | Wed, 28 Sep 2022 17:01:46 +0000 |
| |
Hi Bean.
I think in any case we need remove the redundant ufs_bsg_get_query_desc_size function from ufs_bsg. As done in this patch and submit the another one in order to remove the desc size check in ufshcd.c entirely. Are you agree?
Regards Arthur
> -----Original Message----- > From: Arthur Simchaev <Arthur.Simchaev@wdc.com> > Sent: Wednesday, September 28, 2022 5:42 PM > To: Bean Huo <huobean@gmail.com>; martin.petersen@oracle.com; > beanhuo@micron.com > Cc: linux-scsi@vger.kernel.org; linux-kernel@vger.kernel.org; Daniil Lunev > <dlunev@chromium.org>; Avri Altman <Avri.Altman@wdc.com>; Avi > Shchislowski <Avi.Shchislowski@wdc.com> > Subject: RE: [PATCH] scsi: ufs-bsg: Remove ufs_bsg_get_query_desc_size > function > > CAUTION: This email originated from outside of Western Digital. Do not click > on links or open attachments unless you recognize the sender and know that > the content is safe. > > > Agree with you. Will change & send the patch. > > Regards > Arthur > > > -----Original Message----- > > From: Bean Huo <huobean@gmail.com> > > Sent: Wednesday, September 28, 2022 1:36 PM > > To: Arthur Simchaev <Arthur.Simchaev@wdc.com>; > > martin.petersen@oracle.com; beanhuo@micron.com > > Cc: linux-scsi@vger.kernel.org; linux-kernel@vger.kernel.org; Daniil Lunev > > <dlunev@chromium.org>; Avri Altman <Avri.Altman@wdc.com>; Avi > > Shchislowski <Avi.Shchislowski@wdc.com> > > Subject: Re: [PATCH] scsi: ufs-bsg: Remove ufs_bsg_get_query_desc_size > > function > > > > CAUTION: This email originated from outside of Western Digital. Do not click > > on links or open attachments unless you recognize the sender and know > that > > the content is safe. > > > > > > On Wed, 2022-09-28 at 08:33 +0000, Arthur Simchaev wrote: > > > Hi Bean > > > > > > In case you don't have any comments I will appreciate if you will add > > > "reviewed by" to the patch. > > > > > > Regards > > > Arthur > > > > > > Hi Arthur, > > > > I'm thinking we should remove the desc size check in ufshcd.c entirely. > > Just read any descriptor with a maximum size of QUERY_DESC_MAX_SIZE . > > For user space queries, ufs_bsg reads data of the maximum length and > > returns the requested length data. Thus can improve code readability > > and save CPU cycles, also can fix your concern. > > > > I don't know how about others' opinion? > > > > Kind regards, > > Bean > > > >
| |