From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Chinner Subject: Re: [PATCH 1/3 v2] mm: Stop background writeback if there is other work queued for the thread Date: Sun, 8 Aug 2010 14:12:30 +1000 Message-ID: <20100808041230.GF26402@dastard> References: <1281034399-13055-1-git-send-email-jack@suse.cz> <1281034399-13055-2-git-send-email-jack@suse.cz> <20100805164535.f28d8807.akpm@linux-foundation.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Jan Kara , linux-fsdevel@vger.kernel.org, hch@infradead.org, jaxboe@fusionio.com, Wu Fengguang To: Andrew Morton Return-path: Received: from bld-mail13.adl6.internode.on.net ([150.101.137.98]:55090 "EHLO mail.internode.on.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750710Ab0HHENM (ORCPT ); Sun, 8 Aug 2010 00:13:12 -0400 Content-Disposition: inline In-Reply-To: <20100805164535.f28d8807.akpm@linux-foundation.org> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Thu, Aug 05, 2010 at 04:45:35PM -0700, Andrew Morton wrote: > On Thu, 5 Aug 2010 20:53:17 +0200 > Jan Kara wrote: > > --- > > fs/fs-writeback.c | 8 ++++++++ > > 1 files changed, 8 insertions(+), 0 deletions(-) > > > > diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c > > index d5be169..542471e 100644 > > --- a/fs/fs-writeback.c > > +++ b/fs/fs-writeback.c > > @@ -633,6 +633,14 @@ static long wb_writeback(struct bdi_writeback *wb, > > break; > > > > /* > > + * Background writeout and kupdate-style writeback are > > + * easily livelockable. Stop them if there is other work > > + * to do so that e.g. sync can proceed. > > + */ > > + if ((work->for_background || work->for_kupdate) && > > + !list_empty(&wb->bdi->work_list)) > > + break; > > + /* > > So what happens if an application sits in a loop doing write&fsync to a > file? The vm's call for help gets ignored and your data doesn't get > written back for three days?? To avoid the possibility of any such occurrence, perhaps requeuing the work rather than cancelling it would be better? i.e. stop, put it behind whatever work just came in and so when the new work completes, we restart the background/expiry based writeback? Cheers, Dave. -- Dave Chinner david@fromorbit.com