On 5/1/24 12:07, Benjamin Marzinski wrote:
> On Wed, May 01, 2024 at 09:09:22AM +0900, Damien Le Moal wrote:
>> +static int dm_check_zoned(struct mapped_device *md, struct dm_table *t)
>> +{
>> + struct gendisk *disk = md->disk;
>> + unsigned int nr_conv_zones = 0;
>> + int ret;
>> +
>> + /* Revalidate only if something changed. */
>> + md->zone_revalidate_map = t;
>> + ret = dm_blk_report_zones(disk, 0, UINT_MAX,
>> + dm_check_zoned_cb, &nr_conv_zones);
>
> Aside from not really understanding what that comment is getting at, it
> looks good.
Oops. Too much copy-paste :)
I fixed that. Sending V3 shortly.
> Reviewed-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx>
Thanks.
--
Damien Le Moal
Western Digital Research
