Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752783Ab0K2O7i (ORCPT ); Mon, 29 Nov 2010 09:59:38 -0500 Received: from bombadil.infradead.org ([18.85.46.34]:39464 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750831Ab0K2O7h (ORCPT ); Mon, 29 Nov 2010 09:59:37 -0500 Date: Mon, 29 Nov 2010 09:59:37 -0500 From: Christoph Hellwig To: npiggin@kernel.dk Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [patch 4/7] fs: preserve inode dirty bits on failed metadata writeback Message-ID: <20101129145936.GC26076@infradead.org> References: <20101123140610.292941494@kernel.dk> <20101123140707.922192750@kernel.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101123140707.922192750@kernel.dk> User-Agent: Mutt/1.5.21 (2010-09-15) X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 524 Lines: 12 On Wed, Nov 24, 2010 at 01:06:14AM +1100, npiggin@kernel.dk wrote: > Otherwise we think the inode is clean even if syncing failed. The patch itself looks fine, but I'm not sure it's enough. If we do an synchronous writeout it could fail long after ->write_inode has returned. -- 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/