Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755408AbbHNPUF (ORCPT ); Fri, 14 Aug 2015 11:20:05 -0400 Received: from mailhost-k7-m1.nerim-networks.com ([78.40.49.172]:26497 "EHLO smtp-delay2.nerim.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755117AbbHNPUD (ORCPT ); Fri, 14 Aug 2015 11:20:03 -0400 X-Greylist: delayed 357 seconds by postgrey-1.27 at vger.kernel.org; Fri, 14 Aug 2015 11:20:02 EDT Date: Fri, 14 Aug 2015 17:14:01 +0200 From: Damien Wyart To: Jan Kara Cc: Tejun Heo , Jens Axboe , Eryu Guan , linux-kernel@vger.kernel.org, xfs@oss.sgi.com, axboe@fb.com, Jan Kara , linux-fsdevel@vger.kernel.org, kernel-team@fb.com Subject: Re: [PATCH block/for-linus] writeback: fix syncing of I_DIRTY_TIME inodes Message-ID: <20150814151401.GA6906@brouette> References: <20150812101204.GE17933@dhcp-13-216.nay.redhat.com> <20150813004435.GN3902@dastard> <20150813224415.GG4496@mtj.duckdns.org> <20150814111408.GB8710@quack.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150814111408.GB8710@quack.suse.cz> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2344 Lines: 55 > On Thu 13-08-15 18:44:15, Tejun Heo wrote: > > e79729123f63 ("writeback: don't issue wb_writeback_work if clean") > > updated writeback path to avoid kicking writeback work items if there > > are no inodes to be written out; unfortunately, the avoidance logic > > was too aggressive and made sync_inodes_sb() skip I_DIRTY_TIME inodes. > > This patch fixes the breakage by > > * Removing bdi_has_dirty_io() shortcut from bdi_split_work_to_wbs(). > > The callers are already testing the condition. > > * Removing bdi_has_dirty_io() shortcut from sync_inodes_sb() so that > > it always calls into bdi_split_work_to_wbs(). > > * Making bdi_split_work_to_wbs() consider the b_dirty_time list for > > WB_SYNC_ALL writebacks. > > Signed-off-by: Tejun Heo > > Fixes: e79729123f63 ("writeback: don't issue wb_writeback_work if clean") > > Cc: Ted Ts'o > > Cc: Jan Kara * Jan Kara [2015-08-14 13:14]: > So the patch looks good to me. But the fact that is fixes Eryu's problem > means there is something fishy going on. Either inodes get wrongly attached > to b_dirty_time list or bdi_has_dirty_io() somehow misbehaves only > temporarily and we don't catch it with the debug patch. > Can we add a test to wb_has_dirty_io() to also check whether it matches > bdi_has_dirty_io()? Since Eryu doesn't use lazytime (I assume, Eryu, please > speak up if you do), we could also warn if b_dirty_time lists get > non-empty. Hmm? Hi, I had an unstable system when running latest Linus tree with Tejun's patch applied on top. Nothing fishy in the logs after rebooting without the patch, but remote access with ssh when patch applied did not work (as if /home partition could not be read). This system has / as ext4 and other partitions (including /home) as XFS. Trying to login on tty instead of X resulted in hang of X. I could reboot with sysrq, but can't do further tests at the moment. Back to same tree without the patch resulted in normal system. So just a heads up the patch doesn't seem OK in its current state. Cheers Damien -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/