From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f52.google.com (mail-pa0-f52.google.com [209.85.220.52]) by kanga.kvack.org (Postfix) with ESMTP id 917176B0032 for ; Thu, 15 Jan 2015 23:06:33 -0500 (EST) Received: by mail-pa0-f52.google.com with SMTP id eu11so21651583pac.11 for ; Thu, 15 Jan 2015 20:06:33 -0800 (PST) Received: from mail-pa0-x22b.google.com (mail-pa0-x22b.google.com. [2607:f8b0:400e:c03::22b]) by mx.google.com with ESMTPS id nq15si3938172pdb.212.2015.01.15.20.06.31 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Thu, 15 Jan 2015 20:06:32 -0800 (PST) Received: by mail-pa0-f43.google.com with SMTP id kx10so21770567pab.2 for ; Thu, 15 Jan 2015 20:06:31 -0800 (PST) Date: Thu, 15 Jan 2015 20:06:28 -0800 From: Brian Norris Subject: Re: [PATCH 03/12] fs: introduce f_op->mmap_capabilities for nommu mmap support Message-ID: <20150116040628.GG9759@ld-irv-0074> References: <1421228561-16857-1-git-send-email-hch@lst.de> <1421228561-16857-4-git-send-email-hch@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1421228561-16857-4-git-send-email-hch@lst.de> Sender: owner-linux-mm@kvack.org List-ID: To: Christoph Hellwig Cc: Jens Axboe , linux-nfs@vger.kernel.org, linux-mm@kvack.org, David Howells , linux-fsdevel@vger.kernel.org, linux-mtd@lists.infradead.org, Tejun Heo , ceph-devel@vger.kernel.org, David Woodhouse + dwmw2 On Wed, Jan 14, 2015 at 10:42:32AM +0100, Christoph Hellwig wrote: > Since "BDI: Provide backing device capability information [try #3]" the > backing_dev_info structure also provides flags for the kind of mmap > operation available in a nommu environment, which is entirely unrelated > to it's original purpose. > > Introduce a new nommu-only file operation to provide this information to > the nommu mmap code instead. Splitting this from the backing_dev_info > structure allows to remove lots of backing_dev_info instance that aren't > otherwise needed, and entirely gets rid of the concept of providing a > backing_dev_info for a character device. It also removes the need for > the mtd_inodefs filesystem. > > Signed-off-by: Christoph Hellwig > Reviewed-by: Tejun Heo > --- > Documentation/nommu-mmap.txt | 8 +-- > block/blk-core.c | 2 +- > drivers/char/mem.c | 64 ++++++++++---------- > drivers/mtd/mtdchar.c | 72 ++++------------------ > drivers/mtd/mtdconcat.c | 10 ---- > drivers/mtd/mtdcore.c | 80 +++++++------------------ > drivers/mtd/mtdpart.c | 1 - There's a small conflict in mtdcore.c with some stuff I have queued up for MTD in linux-next. Should be trivial to resolve later. I don't have a test platform for nommu, and I'll admit I'm not too familiar with this code, but it looks OK to me. So FWIW, for the MTD parts: Acked-by: Brian Norris > drivers/staging/lustre/lustre/llite/llite_lib.c | 2 +- > fs/9p/v9fs.c | 2 +- > fs/afs/volume.c | 2 +- > fs/aio.c | 14 +---- > fs/btrfs/disk-io.c | 3 +- > fs/char_dev.c | 24 -------- > fs/cifs/connect.c | 2 +- > fs/coda/inode.c | 2 +- > fs/configfs/configfs_internal.h | 2 - > fs/configfs/inode.c | 18 +----- > fs/configfs/mount.c | 11 +--- > fs/ecryptfs/main.c | 2 +- > fs/exofs/super.c | 2 +- > fs/ncpfs/inode.c | 2 +- > fs/ramfs/file-nommu.c | 7 +++ > fs/ramfs/inode.c | 22 +------ > fs/romfs/mmap-nommu.c | 10 ++++ > fs/ubifs/super.c | 2 +- > include/linux/backing-dev.h | 33 ++-------- > include/linux/cdev.h | 2 - > include/linux/fs.h | 23 +++++++ > include/linux/mtd/mtd.h | 2 + > mm/backing-dev.c | 7 +-- > mm/nommu.c | 69 ++++++++++----------- > security/security.c | 13 ++-- > 32 files changed, 169 insertions(+), 346 deletions(-) [...] Brian -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org