Re: [PATCH 3/3] scsi: sd: remove some redundant initialization code

From: Haoqian He
Date: Wed Jul 03 2024 - 23:02:02 EST



> 2024年7月2日 11:33,Damien Le Moal <dlemoal@xxxxxxxxxx> 写道:
>
> On 7/2/24 12:01, Haoqian He wrote:
>> Since the memory allocated by kzalloc for sdkp has been
>> initialized to 0, the code that initializes some sdkp
>> fields to 0 is no longer needed.
>>
>> Signed-off-by: Haoqian He <haoqian.he@xxxxxxxxxx>
>> Signed-off-by: Li Feng <fengli@xxxxxxxxxx>
>
> Looks OK to me.
>
> Reviewed-by: Damien Le Moal <dlemoal@xxxxxxxxxx>
>
> --
> Damien Le Moal
> Western Digital Research
>

Hi Martin,

According to the SBC-3 SPEC:
"The device server in a logical unit that supports logical block provisioning
management shall set the LBPME bit to one in the parameter data returned for
a READ CAPACITY (16) command."

So we can use lbpme bit instead of lbpvpd to indicate if the device is thin
provisioned, which was implemented in patch 2 ("scsi: sd: remove scsi_disk
field lbpvpd").

Thanks,
Haoqian