On Wed 09-10-24 23:17:27, Tang Yizhou wrote: > From: Tang Yizhou <yizhou.tang@xxxxxxxxxx> > > The name of the BANDWIDTH_INTERVAL macro is misleading, as it is not > only used in the bandwidth update functions wb_update_bandwidth() and > __wb_update_bandwidth(), but also in the dirty limit update function > domain_update_dirty_limit(). > > Currently, we haven't found an ideal name, so update the comment only. > > Signed-off-by: Tang Yizhou <yizhou.tang@xxxxxxxxxx> Looks good. Feel free to add: Reviewed-by: Jan Kara <jack@xxxxxxx> Honza > --- > mm/page-writeback.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/page-writeback.c b/mm/page-writeback.c > index fcd4c1439cb9..c7c6b58a8461 100644 > --- a/mm/page-writeback.c > +++ b/mm/page-writeback.c > @@ -54,7 +54,7 @@ > #define DIRTY_POLL_THRESH (128 >> (PAGE_SHIFT - 10)) > > /* > - * Estimate write bandwidth at 200ms intervals. > + * Estimate write bandwidth or update dirty limit at 200ms intervals. > */ > #define BANDWIDTH_INTERVAL max(HZ/5, 1) > > -- > 2.25.1 > -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR
