Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758710AbZIPSlH (ORCPT ); Wed, 16 Sep 2009 14:41:07 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755743AbZIPSlF (ORCPT ); Wed, 16 Sep 2009 14:41:05 -0400 Received: from brick.kernel.dk ([93.163.65.50]:46216 "EHLO kernel.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754424AbZIPSlD (ORCPT ); Wed, 16 Sep 2009 14:41:03 -0400 Date: Wed, 16 Sep 2009 20:41:06 +0200 From: Jens Axboe To: Jan Kara Cc: LKML , Wu Fengguang Subject: Re: [PATCH] fs: Fix busyloop in wb_writeback() Message-ID: <20090916184106.GT23126@kernel.dk> References: <1253121768-20673-1-git-send-email-jack@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1253121768-20673-1-git-send-email-jack@suse.cz> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 805 Lines: 21 On Wed, Sep 16 2009, Jan Kara wrote: > If all inodes are under writeback (e.g. in case when there's only one inode > with dirty pages), wb_writeback() with WB_SYNC_NONE work basically degrades > to busylooping until I_SYNC flags of the inode is cleared. Fix the problem by > waiting on I_SYNC flags of an inode on b_more_io list in case we failed to > write anything. Interesting, so this will happen if the dirtier and flush thread end up "fighting" each other over the same inode. I'll throw this into the testing mix. How did you notice? -- Jens Axboe -- 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/