Re: [PATCH v3 v3 2/2] ext4: allow clearing mballoc stats through mb_stats

[liubaolin <liubaolin12138@163.com>]

Dear Ted,
    Thank you and Baokun for your review and suggestions.
    I will incorporate your suggestions and submit the v4 patch as soon 
as possible.

Thanks,
Baolin

在 2026/4/24 20:07, Theodore Tso 写道:
> On Fri, Apr 24, 2026 at 04:09:31PM +0800, liubaolin wrote:
>>
>>     2. Do not delete the `/sys/fs/.../mb_stats` node for now; implement the
>> same write control logic.
>> 	* Write 0 to `/sys/fs/.../mb_stats` to disable statistics collection.
>> 	* Write 1 to `/sys/fs/.../mb_stats` to enable statistics collection.
>> 	* Write 2 to `/sys/fs/.../mb_stats` to clear statistics counters.
> 
> We could do that, but note that currently writing to
> /sys/fs/.../mb_stats just sets an unsigned integer in
> EXT4(sb)->s_mb_stats.  There is no ext4-specific function that runs
> when /sys/fs/.../mb_stats is updated.
> 
> So either you have to add some check in fs/ext4/mballoc.c which gets
> called every single time a block allocation happens --- and consider
> the race condition where two CPU's are checking s_mb_stats at the same
> time, and the desireability of adding a spinlock that would need to be
> taken every single time a block allocation happens ---- or you have
> add an ext4-specific function in fs/ext4/sysfs.c.
> 
>>     Compared to your suggestion, I recommend using the value 2 for the clear
>> operation because s_mb_stats is an unsigned int variable, and using -1
>> requires changing the variable type.
> 
> Well, since you have introduced an ext4-specific function which gets
> called when writing to the procfs file, that function can clear the
> statistics counter when -1 is written to the file --- and then set
> s_mbi_stats to 1.
> 
> Cheers,
> 
> 						- Ted