Hello, On Wed, Mar 20, 2024 at 07:02:22PM +0800, Kemeng Shi wrote: > We never use gdtc->dom set with GDTC_INIT_NO_WB, just remove unneeded > GDTC_INIT_NO_WB > > Signed-off-by: Kemeng Shi <shikemeng@xxxxxxxxxxxxxxx> ... > void global_dirty_limits(unsigned long *pbackground, unsigned long *pdirty) > { > - struct dirty_throttle_control gdtc = { GDTC_INIT_NO_WB }; > + struct dirty_throttle_control gdtc = { }; Even if it's currently not referenced, wouldn't it still be better to always guarantee that a dtc's dom is always initialized? I'm not sure what we get by removing this. Thanks. -- tejun