From: Ross Zwisler <ross.zwisler@linux.intel.com>
To: Dan Williams <dan.j.williams@intel.com>
Cc: Ross Zwisler <ross.zwisler@linux.intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Theodore Ts'o <tytso@mit.edu>,
Alexander Viro <viro@zeniv.linux.org.uk>,
Andreas Dilger <adilger.kernel@dilger.ca>,
Andrew Morton <akpm@linux-foundation.org>,
Dave Chinner <david@fromorbit.com>, Jan Kara <jack@suse.com>,
linux-ext4 <linux-ext4@vger.kernel.org>,
linux-fsdevel <linux-fsdevel@vger.kernel.org>,
Linux MM <linux-mm@kvack.org>,
"linux-nvdimm@lists.01.org" <linux-nvdimm@lists.01.org>
Subject: Re: [PATCH 0/7] re-enable DAX PMD support
Date: Mon, 15 Aug 2016 15:11:06 -0600 [thread overview]
Message-ID: <20160815211106.GA31566@linux.intel.com> (raw)
In-Reply-To: <CAPcyv4j_eh8Rcozb40JeiPwvbPoMY2sCt+yTewZ-MZzUkBbj-Q@mail.gmail.com>
On Mon, Aug 15, 2016 at 01:21:47PM -0700, Dan Williams wrote:
> On Mon, Aug 15, 2016 at 12:09 PM, Ross Zwisler
> <ross.zwisler@linux.intel.com> wrote:
> > DAX PMDs have been disabled since Jan Kara introduced DAX radix tree based
> > locking. This series allows DAX PMDs to participate in the DAX radix tree
> > based locking scheme so that they can be re-enabled.
>
> Looks good to me.
>
> > This series restores DAX PMD functionality back to what it was before it
> > was disabled. There is still a known issue between DAX PMDs and hole
> > punch, which I am currently working on and which I plan to address with a
> > separate series.
>
> Perhaps we should hold off on applying patch 6 and 7 until after the
> hole-punch fix is ready?
Sure, I'm cool with holding off on patch 7 (the Kconfig change) until after
the hole punch fix is ready.
I don't see a reason to hold off on patch 6, though? It stands on it's own,
implements the correct locking, and doesn't break anything.
--
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: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
WARNING: multiple messages have this Message-ID (diff)
From: Ross Zwisler <ross.zwisler@linux.intel.com>
To: Dan Williams <dan.j.williams@intel.com>
Cc: Theodore Ts'o <tytso@mit.edu>,
"linux-nvdimm@lists.01.org" <linux-nvdimm@lists.01.org>,
Dave Chinner <david@fromorbit.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Linux MM <linux-mm@kvack.org>,
Andreas Dilger <adilger.kernel@dilger.ca>,
Alexander Viro <viro@zeniv.linux.org.uk>,
Jan Kara <jack@suse.com>,
linux-fsdevel <linux-fsdevel@vger.kernel.org>,
linux-ext4 <linux-ext4@vger.kernel.org>,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH 0/7] re-enable DAX PMD support
Date: Mon, 15 Aug 2016 15:11:06 -0600 [thread overview]
Message-ID: <20160815211106.GA31566@linux.intel.com> (raw)
In-Reply-To: <CAPcyv4j_eh8Rcozb40JeiPwvbPoMY2sCt+yTewZ-MZzUkBbj-Q@mail.gmail.com>
On Mon, Aug 15, 2016 at 01:21:47PM -0700, Dan Williams wrote:
> On Mon, Aug 15, 2016 at 12:09 PM, Ross Zwisler
> <ross.zwisler@linux.intel.com> wrote:
> > DAX PMDs have been disabled since Jan Kara introduced DAX radix tree based
> > locking. This series allows DAX PMDs to participate in the DAX radix tree
> > based locking scheme so that they can be re-enabled.
>
> Looks good to me.
>
> > This series restores DAX PMD functionality back to what it was before it
> > was disabled. There is still a known issue between DAX PMDs and hole
> > punch, which I am currently working on and which I plan to address with a
> > separate series.
>
> Perhaps we should hold off on applying patch 6 and 7 until after the
> hole-punch fix is ready?
Sure, I'm cool with holding off on patch 7 (the Kconfig change) until after
the hole punch fix is ready.
I don't see a reason to hold off on patch 6, though? It stands on it's own,
implements the correct locking, and doesn't break anything.
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm
WARNING: multiple messages have this Message-ID (diff)
From: Ross Zwisler <ross.zwisler@linux.intel.com>
To: Dan Williams <dan.j.williams@intel.com>
Cc: Ross Zwisler <ross.zwisler@linux.intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"Theodore Ts'o" <tytso@mit.edu>,
Alexander Viro <viro@zeniv.linux.org.uk>,
Andreas Dilger <adilger.kernel@dilger.ca>,
Andrew Morton <akpm@linux-foundation.org>,
Dave Chinner <david@fromorbit.com>, Jan Kara <jack@suse.com>,
linux-ext4 <linux-ext4@vger.kernel.org>,
linux-fsdevel <linux-fsdevel@vger.kernel.org>,
Linux MM <linux-mm@kvack.org>,
"linux-nvdimm@lists.01.org" <linux-nvdimm@ml01.01.org>
Subject: Re: [PATCH 0/7] re-enable DAX PMD support
Date: Mon, 15 Aug 2016 15:11:06 -0600 [thread overview]
Message-ID: <20160815211106.GA31566@linux.intel.com> (raw)
In-Reply-To: <CAPcyv4j_eh8Rcozb40JeiPwvbPoMY2sCt+yTewZ-MZzUkBbj-Q@mail.gmail.com>
On Mon, Aug 15, 2016 at 01:21:47PM -0700, Dan Williams wrote:
> On Mon, Aug 15, 2016 at 12:09 PM, Ross Zwisler
> <ross.zwisler@linux.intel.com> wrote:
> > DAX PMDs have been disabled since Jan Kara introduced DAX radix tree based
> > locking. This series allows DAX PMDs to participate in the DAX radix tree
> > based locking scheme so that they can be re-enabled.
>
> Looks good to me.
>
> > This series restores DAX PMD functionality back to what it was before it
> > was disabled. There is still a known issue between DAX PMDs and hole
> > punch, which I am currently working on and which I plan to address with a
> > separate series.
>
> Perhaps we should hold off on applying patch 6 and 7 until after the
> hole-punch fix is ready?
Sure, I'm cool with holding off on patch 7 (the Kconfig change) until after
the hole punch fix is ready.
I don't see a reason to hold off on patch 6, though? It stands on it's own,
implements the correct locking, and doesn't break anything.
next prev parent reply other threads:[~2016-08-15 21:11 UTC|newest]
Thread overview: 72+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-15 19:09 [PATCH 0/7] re-enable DAX PMD support Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
[not found] ` <20160815190918.20672-1-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-08-15 19:09 ` [PATCH 1/7] ext2: tell DAX the size of allocation holes Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-16 9:10 ` Jan Kara
2016-08-16 9:10 ` Jan Kara
2016-08-16 9:10 ` Jan Kara
[not found] ` <20160816091025.GA27284-4I4JzKEfoa/jFM9bn6wA6Q@public.gmane.org>
2016-08-16 22:52 ` Ross Zwisler
2016-08-16 22:52 ` Ross Zwisler
2016-08-16 22:52 ` Ross Zwisler
2016-08-16 22:52 ` Ross Zwisler
2016-08-15 19:09 ` [PATCH 2/7] ext4: " Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-16 9:12 ` Jan Kara
2016-08-16 9:12 ` Jan Kara
2016-08-16 9:12 ` Jan Kara
2016-08-15 19:09 ` [PATCH 3/7] dax: remove buffer_size_valid() Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-16 9:13 ` Jan Kara
2016-08-16 9:13 ` Jan Kara
2016-08-15 19:09 ` [PATCH 4/7] dax: rename 'ret' to 'entry' in grab_mapping_entry Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-16 9:14 ` Jan Kara
2016-08-16 9:14 ` Jan Kara
2016-08-15 19:09 ` [PATCH 5/7] dax: lock based on slot instead of [mapping, index] Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-16 9:28 ` Jan Kara
2016-08-16 9:28 ` Jan Kara
2016-08-16 9:28 ` Jan Kara
2016-08-17 20:25 ` Ross Zwisler
2016-08-17 20:25 ` Ross Zwisler
2016-08-17 20:25 ` Ross Zwisler
[not found] ` <20160817202556.GA13009-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-08-18 14:15 ` Jan Kara
2016-08-18 14:15 ` Jan Kara
2016-08-18 14:15 ` Jan Kara
2016-08-18 14:15 ` Jan Kara
2016-08-15 19:09 ` [PATCH 6/7] dax: re-enable DAX PMD support Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` [PATCH 7/7] dax: remove "depends on BROKEN" from FS_DAX_PMD Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 19:09 ` Ross Zwisler
2016-08-15 20:21 ` [PATCH 0/7] re-enable DAX PMD support Dan Williams
2016-08-15 20:21 ` Dan Williams
2016-08-15 20:21 ` Dan Williams
2016-08-15 20:21 ` Dan Williams
2016-08-15 21:11 ` Ross Zwisler [this message]
2016-08-15 21:11 ` Ross Zwisler
2016-08-15 21:11 ` Ross Zwisler
2016-08-15 21:14 ` Dan Williams
2016-08-15 21:14 ` Dan Williams
[not found] ` <CAPcyv4i+XHZSN_3T_vcrv+sOkEMQzuTKRo4WBFcPxN=TzSk9iw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-08-17 16:21 ` Ross Zwisler
2016-08-17 16:21 ` Ross Zwisler
2016-08-17 16:21 ` Ross Zwisler
[not found] ` <20160817162124.GA16779-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-08-17 17:21 ` Jan Kara
2016-08-17 17:21 ` Jan Kara
2016-08-17 17:21 ` Jan Kara
2016-08-17 17:21 ` Jan Kara
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=20160815211106.GA31566@linux.intel.com \
--to=ross.zwisler@linux.intel.com \
--cc=adilger.kernel@dilger.ca \
--cc=akpm@linux-foundation.org \
--cc=dan.j.williams@intel.com \
--cc=david@fromorbit.com \
--cc=jack@suse.com \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linux-nvdimm@lists.01.org \
--cc=tytso@mit.edu \
--cc=viro@zeniv.linux.org.uk \
/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.