From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nick Piggin Subject: Re: [patch][rfc] fs: move bdev code out of buffer.c Date: Wed, 25 Feb 2009 11:53:38 +0100 Message-ID: <20090225105338.GH22785@wotan.suse.de> References: <20090225094419.GF22785@wotan.suse.de> <49A51EFF.4020600@garzik.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-fsdevel@vger.kernel.org, LKML , Jens Axboe To: Jeff Garzik Return-path: Received: from mx2.suse.de ([195.135.220.15]:56575 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752211AbZBYKxl (ORCPT ); Wed, 25 Feb 2009 05:53:41 -0500 Content-Disposition: inline In-Reply-To: <49A51EFF.4020600@garzik.org> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, Feb 25, 2009 at 05:35:43AM -0500, Jeff Garzik wrote: > Nick Piggin wrote: > >Move some block device related code out from buffer.c and put it in > >block_dev.c. I'm trying to move non-buffer_head code out of buffer.c > > > >Is this OK? > > > >--- > > fs/block_dev.c | 146 > > ++++++++++++++++++++++++++++++++++++++++++++ > > fs/buffer.c | 145 > > ------------------------------------------- > > include/linux/buffer_head.h | 7 -- > > include/linux/fs.h | 7 ++ > > 4 files changed, 153 insertions(+), 152 deletions(-) > > If you are going to churn all this old code, maybe the blkdev stuff > better belongs in block/ dir? If you have a look at it, it's all dealing with buffercache and filesystem metadata I think. So I think fs/block_dev.c is the right place for it. block/ shouldn't know about filesystems or pagecache.