From: Matthew Wilcox <willy@linux.intel.com>
To: Jeff Moyer <jmoyer@redhat.com>
Cc: rob.gittins@linux.intel.com, linux-pmfs@lists.infradead.org,
linux-fsdevel@veger.org, linux-kernel@vger.kernel.org
Subject: Re: RFC Block Layer Extensions to Support NV-DIMMs
Date: Thu, 5 Sep 2013 11:34:47 -0400 [thread overview]
Message-ID: <20130905153447.GB20931@linux.intel.com> (raw)
In-Reply-To: <x49y57ba2ne.fsf@segfault.boston.devel.redhat.com>
On Thu, Sep 05, 2013 at 08:12:05AM -0400, Jeff Moyer wrote:
> If the memory is available to be mapped into the address space of the
> kernel or a user process, then I don't see why we should have a block
> device at all. I think it would make more sense to have a different
> driver class for these persistent memory devices.
We already have at least two block devices in the tree that provide
this kind of functionality (arch/powerpc/sysdev/axonram.c and
drivers/s390/block/dcssblk.c). Looking at how they're written, it
seems like implementing either of them as a block device on top of a
character device that extended their functionality in the direction we
want would be a pretty major bloating factor for no real benefit (not
even a particularly cleaner architecture).
> > Different applications, filesystem and drivers may wish to share
> > ranges of PMEM. This is analogous to partitioning a disk that is
> > using multiple and different filesystems. Since PMEM is addressed
> > on a byte basis rather than a block basis the existing partitioning
> > model does not fit well. As a result there needs to be a way to
> > describe PMEM ranges.
> >
> > struct pmem_layout *(*getpmem)(struct block_device *bdev);
>
> If existing partitioning doesn't work well, then it sounds like a block
> device isn't the right fit (again). Ignoring that detail, what about
> requesting and releasing ranges of persistent memory, as in your
> "partitioning" example? Would that not also be a function of the
> driver?
"existing partitioning" doesn't even work well for existing drives!
Nobody actually builds a drive with fixed C/H/S any more.
next prev parent reply other threads:[~2013-09-05 15:35 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-04 21:54 RFC Block Layer Extensions to Support NV-DIMMs Rob Gittins
2013-09-05 12:12 ` Jeff Moyer
2013-09-05 15:34 ` Matthew Wilcox [this message]
2013-09-05 17:15 ` Jeff Moyer
2013-09-05 17:57 ` Matthew Wilcox
2013-09-05 19:46 ` Zuckerman, Boris
2013-09-05 20:43 ` Gittins, Rob
2013-09-05 21:02 ` Zuckerman, Boris
2013-09-05 16:33 ` Rob Gittins
2013-09-05 17:19 ` Jeff Moyer
2013-09-07 5:12 ` Vladislav Bolkhovitin
2013-09-23 22:51 ` Rob Gittins
2013-09-26 6:58 ` Vladislav Bolkhovitin
2013-09-26 14:56 ` Zuckerman, Boris
2013-09-26 17:56 ` Matthew Wilcox
2013-09-26 19:36 ` Zuckerman, Boris
2013-09-28 7:44 ` Vladislav Bolkhovitin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130905153447.GB20931@linux.intel.com \
--to=willy@linux.intel.com \
--cc=jmoyer@redhat.com \
--cc=linux-fsdevel@veger.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pmfs@lists.infradead.org \
--cc=rob.gittins@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.