Thread (33 messages) 33 messages, 5 authors, 2021-02-10

Re: [PATCH v19 2/3] scsi: ufs: L2P map management for HPB read

From: Bean Huo <hidden>
Date: 2021-02-05 11:58:50
Also in: lkml

On Fri, 2021-02-05 at 11:42 +0000, Avri Altman wrote:
quoted
"If the requested field of the HPB Region or HPB Sub-Region is out
of
range, then the device shall terminate the command by sending
RESPONSE
UPIU with CHECK CONDITION status, with the SENSE KEY set to ILLEGAL
REQUEST, and the additional sense code set to INVALID FIELD IN CDB"
You don't need to worry about setting invalid ppn to HPB-READ command
- 
you'll never get a read request for those LBAs.

Say all subregions are 16MB and the last subregion of the last region
is 10MB.
Keep all sizes the same - 16MB, and the ppn of the last subregion
contain some invalid data.
But you'll never get a read request for those LBAs anyway - they
don't exist,
so you'll never get to use those invalid ppns.

Thanks,
Avri
Hi Avri
ah, I don't know if your above comments are inline with Spec.

Spec:
"A HPB Region is divided into HPB Sub-Regions. HPB Sub-Region size is
specified by the bHPBSubRegionSize parameter of Geometry Descriptor.
HPB Sub-Regions are equally sized except for the last one which is
smaller if the last HPB Region is not an interger multiple of
bHPBSubRegionSize."

It is not invalidate ppn concern, it is illegal request issue in sense
key.

Kind regards,
Bean


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help