From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx165.postini.com [74.125.245.165]) by kanga.kvack.org (Postfix) with SMTP id 2841B6B005C for ; Sun, 10 Jun 2012 00:36:48 -0400 (EDT) Date: Sun, 10 Jun 2012 12:36:41 +0800 From: Fengguang Wu Subject: Re: [PATCH] page-writeback.c: fix update bandwidth time judgment error Message-ID: <20120610043641.GA10355@localhost> References: <1339302005-366-1-git-send-email-liwp.linux@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1339302005-366-1-git-send-email-liwp.linux@gmail.com> Sender: owner-linux-mm@kvack.org List-ID: To: Wanpeng Li Cc: LKML , Jan Kara , Andrew Morton , Peter Zijlstra , Johannes Weiner , linux-mm@kvack.org, Gavin Shan , Wanpeng Li Wanpeng, Sorry this I won't take this: it don't really improve anything. Even with the changed test, the real intervals are still some random values above (and not far away from) 200ms.. We are saying about 200ms intervals just for convenience. Thanks, Fengguang On Sun, Jun 10, 2012 at 12:20:05PM +0800, Wanpeng Li wrote: > From: Wanpneg Li > > Since bdi_update_bandwidth function should estimate write bandwidth at 200ms intervals, > so the time is bdi->bw_time_stamp + BANDWIDTH_INTERVAL == jiffies, but > if use time_is_after_eq_jiffies intervals will be bdi->bw_time_stamp + > BANDWIDTH_INTERVAL + 1. > > Signed-off-by: Wanpeng Li > --- > 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 c833bf0..099e225 100644 > --- a/mm/page-writeback.c > +++ b/mm/page-writeback.c > @@ -1032,7 +1032,7 @@ static void bdi_update_bandwidth(struct backing_dev_info *bdi, > unsigned long bdi_dirty, > unsigned long start_time) > { > - if (time_is_after_eq_jiffies(bdi->bw_time_stamp + BANDWIDTH_INTERVAL)) > + if (time_is_after_jiffies(bdi->bw_time_stamp + BANDWIDTH_INTERVAL)) > return; > spin_lock(&bdi->wb.list_lock); > __bdi_update_bandwidth(bdi, thresh, bg_thresh, dirty, > -- > 1.7.9.5 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org