At 2025-08-15 13:23:50, "Damien Le Moal" <dlemoal@xxxxxxxxxx> wrote:
>On 8/15/25 13:30, Genjian wrote:
>> From: Genjian Zhang <zhanggenjian@xxxxxxxxxx>
>
>The patch title is a little strange. What about:
>
>null_blk: Fix the description of the cache_size module argument
>
>>
>> When executing modinfo null_blk, there is an error in the description
>> of module parameter mbps, and the output information of cache_size is
>> incomplete.The output of modinfo before and after applying this patch
>> is as follows:
>>
>> Before:
>> [...]
>> parm: cache_size:ulong
>> [...]
>> parm: mbps:Cache size in MiB for memory-backed device.
>> Default: 0 (none) (uint)
>> [...]
>>
>> After:
>> [...]
>> parm: cache_size:Cache size in MiB for memory-backed device.
>> Default: 0 (none) (ulong)
>> [...]
>> parm: mbps:Limit maximum bandwidth (in MiB/s).
>> Default: 0 (no limit) (uint)
>> [...]
>>
>> Fixes: 058efe000b31 ("null_blk: add module parameters for 4 options")
>>
>
>Remove the blank line here.
>
>> Signed-off-by: Genjian Zhang <zhanggenjian@xxxxxxxxxx>
>
>With these fixed,
>
>Reviewed-by: Damien Le Moal <dlemoal@xxxxxxxxxx>
>
>--
>Damien Le Moal
>Western Digital Research
Thank you for your suggestions. I will send a patch v2.
--
Genjian
