From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Axboe Subject: Re: [PATCH 10/11] writeback: splice dirty inode entries to default bdi on bdi_destroy() Date: Wed, 16 Sep 2009 15:21:08 +0200 Message-ID: <20090916132108.GP23126@kernel.dk> References: <1253038617-30204-1-git-send-email-jens.axboe@oracle.com> <1253038617-30204-11-git-send-email-jens.axboe@oracle.com> <20090916131249.GG26030@duck.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, chris.mason@oracle.com, hch@infradead.org, tytso@mit.edu, akpm@linux-foundation.org, trond.myklebust@fys.uio.no To: Jan Kara Return-path: Received: from brick.kernel.dk ([93.163.65.50]:35008 "EHLO kernel.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750696AbZIPNVF (ORCPT ); Wed, 16 Sep 2009 09:21:05 -0400 Content-Disposition: inline In-Reply-To: <20090916131249.GG26030@duck.suse.cz> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, Sep 16 2009, Jan Kara wrote: > On Tue 15-09-09 20:16:56, Jens Axboe wrote: > > We cannot safely ensure that the inodes are all gone at this point > > in time, and we must not destroy this bdi with inodes having off it. > ^^^ hanging > > > So just splice our entries to the default bdi since that one will > > always persist. > BTW: Why can't we make sure all inodes on the BDI are clean when we > destroy it? Common sence would suggest that we better should be able to do > it :). > Maybe it's because most users of private BDI do not call bdi_unregister > but rather directly bdi_destroy? Is this correct behavior? Not sure yet, it's on the TODO. This basically works around the problem for now at least. With dm at least, I'm seeing inodes still hanging off the bdi after we have done a sync_blockdev(bdev, 1);. -- Jens Axboe