From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756564AbcBBQvU (ORCPT ); Tue, 2 Feb 2016 11:51:20 -0500 Received: from mail-yk0-f169.google.com ([209.85.160.169]:34658 "EHLO mail-yk0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753476AbcBBQvS (ORCPT ); Tue, 2 Feb 2016 11:51:18 -0500 MIME-Version: 1.0 In-Reply-To: References: <1454009704-25959-1-git-send-email-ross.zwisler@linux.intel.com> <1454009704-25959-2-git-send-email-ross.zwisler@linux.intel.com> <20160128213858.GA29114@infradead.org> <20160129182815.GB5224@linux.intel.com> <20160130052833.GY2948@linux.intel.com> <20160201145147.GD13740@quack.suse.cz> <20160201214730.GR20456@dastard> Date: Tue, 2 Feb 2016 08:51:17 -0800 Message-ID: Subject: Re: [PATCH 2/2] dax: fix bdev NULL pointer dereferences From: Dan Williams To: Jared Hulbert Cc: Dave Chinner , Jan Kara , Matthew Wilcox , Ross Zwisler , Christoph Hellwig , LKML , Alexander Viro , Andrew Morton , Jan Kara , Linux FS Devel , linux-nvdimm Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Feb 2, 2016 at 12:05 AM, Jared Hulbert wrote: [..] > Well... as CONFIG_BLOCK was not required with filemap_xip.c for a > decade. This CONFIG_BLOCK dependency is a result of an incremental > feature from a certain point of view ;) > > The obvious 'driver' is physical RAM without a particular driver. > Remember please I'm talking about embedded. RAM measured in MiB and > funky one off hardware etc. In the embedded world there are lots of > ways that persistent memory has been supported in device specific ways > without the new fancypants NFIT and Intel instructions,so frankly > they don't fit in the PMEM stuff. Maybe they could be supported in > PMEM but not without effort to bring embedded players to the table. Not sure what you're trying to say here. An ACPI NFIT only feeds the generic libnvdimm device model. You don't need NFIT to get pmem. > The other drivers are the MTD drivers, probably as read-only for now. > But the paradigm there isn't so different from what PMEM looks like > with asymmetric read/write capabilities. > > The filesystem I'm concerned with is AXFS > (https://www.kernel.org/doc/ols/2008/ols2008v1-pages-211-218.pdf). > Which I've been planning on trying to merge again due to a recent > resurgence of interest. The device model for AXFS is... weird. It > can use one or two devices at a time of any mix of NOR MTD, NAND MTD, > block, and unmanaged physical memory. It's a terribly useful model > for embedded. Anyway AXFS is readonly so hacking in a read only > dax_fault_nodev() and dax_file_read() would work fine, looks easy > enough. But... it would be cool if similar small embedded focused RW > filesystems were enabled. Are those also out of tree? > I don't expect you to taint DAX with design requirements for this > stuff that it wasn't built for, nobody ends up happy in that case. > However, if enabling the filesystem to manage the bdev_direct_access() > interactions solves some of the "alternate device" problems you are > discussing here, then there is a chance we can accommodate both. > Sometimes that works. > > So... Forget CONFIG_BLOCK=n entirely I didn't want that to be the > focus anyway. Does it help to support the weirder XFS and btrfs > device models to enable the filesystem to handle the > bdev_direct_access() stuff? It's not clear that it does. We just clarified with xfs and ext4 that we can really on get_blocks(). That solves the immediate concern with multi-device filesystems.