All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kara <jack-AlSwsSmVLrQ@public.gmane.org>
To: "Aneesh Kumar K.V"
	<aneesh.kumar-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
Cc: Theodore Ts'o <tytso-3s7WtUTddSA@public.gmane.org>,
	Matthew Wilcox <mawilcox-0li6OtcxBFHby3iVrkZq2A@public.gmane.org>,
	Dave Chinner <david-FqsqvQoI3Ljby3iVrkZq2A@public.gmane.org>,
	linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Christoph Hellwig <hch-jcswGhMUV9g@public.gmane.org>,
	linux-xfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org,
	Andreas Dilger
	<adilger.kernel-m1MBpc4rdrD3fQ9qLvQP4Q@public.gmane.org>,
	Alexander Viro
	<viro-RmSDqhL/yNMiFSDQTTA3OLVCufUGDwFn@public.gmane.org>,
	linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Jan Kara <jack-AlSwsSmVLrQ@public.gmane.org>,
	linux-ext4-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Andrew Morton
	<akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
Subject: Re: [PATCH v6 17/17] dax: remove "depends on BROKEN" from FS_DAX_PMD
Date: Mon, 17 Oct 2016 11:09:37 +0200	[thread overview]
Message-ID: <20161017090937.GG3359@quack2.suse.cz> (raw)
In-Reply-To: <87eg3ftt4r.fsf-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>

On Mon 17-10-16 11:27:24, Aneesh Kumar K.V wrote:
> Ross Zwisler <ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> writes:
> 
> > Now that DAX PMD faults are once again working and are now participating in
> > DAX's radix tree locking scheme, allow their config option to be enabled.
> >
> > Signed-off-by: Ross Zwisler <ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> > ---
> >  fs/Kconfig | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/fs/Kconfig b/fs/Kconfig
> > index 2bc7ad7..b6f0fce 100644
> > --- a/fs/Kconfig
> > +++ b/fs/Kconfig
> > @@ -55,7 +55,6 @@ config FS_DAX_PMD
> >  	depends on FS_DAX
> >  	depends on ZONE_DEVICE
> >  	depends on TRANSPARENT_HUGEPAGE
> > -	depends on BROKEN
> >  
> >  endif # BLOCK
> 
> 
> This series made dax_pmd_fault return VM_FAULT_FALLBACK in [PATCH v6 09/17] dax: remove dax_pmd_fault(). 
> That essentially disable dax pmd for ext4. Any plans to fix that ?

I'm working on converting ext4 to the new iomap infrastructure. Once that
is done, it can use new DAX helpers for PMD faults easily.

								Honza
-- 
Jan Kara <jack-IBi9RG/b67k@public.gmane.org>
SUSE Labs, CR

WARNING: multiple messages have this Message-ID (diff)
From: Jan Kara <jack@suse.cz>
To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
Cc: Ross Zwisler <ross.zwisler@linux.intel.com>,
	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>,
	Christoph Hellwig <hch@lst.de>,
	Dan Williams <dan.j.williams@intel.com>,
	Dave Chinner <david@fromorbit.com>, Jan Kara <jack@suse.cz>,
	Matthew Wilcox <mawilcox@microsoft.com>,
	linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	linux-mm@kvack.org, linux-nvdimm@lists.01.org,
	linux-xfs@vger.kernel.org
Subject: Re: [PATCH v6 17/17] dax: remove "depends on BROKEN" from FS_DAX_PMD
Date: Mon, 17 Oct 2016 11:09:37 +0200	[thread overview]
Message-ID: <20161017090937.GG3359@quack2.suse.cz> (raw)
In-Reply-To: <87eg3ftt4r.fsf@linux.vnet.ibm.com>

On Mon 17-10-16 11:27:24, Aneesh Kumar K.V wrote:
> Ross Zwisler <ross.zwisler@linux.intel.com> writes:
> 
> > Now that DAX PMD faults are once again working and are now participating in
> > DAX's radix tree locking scheme, allow their config option to be enabled.
> >
> > Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
> > ---
> >  fs/Kconfig | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/fs/Kconfig b/fs/Kconfig
> > index 2bc7ad7..b6f0fce 100644
> > --- a/fs/Kconfig
> > +++ b/fs/Kconfig
> > @@ -55,7 +55,6 @@ config FS_DAX_PMD
> >  	depends on FS_DAX
> >  	depends on ZONE_DEVICE
> >  	depends on TRANSPARENT_HUGEPAGE
> > -	depends on BROKEN
> >  
> >  endif # BLOCK
> 
> 
> This series made dax_pmd_fault return VM_FAULT_FALLBACK in [PATCH v6 09/17] dax: remove dax_pmd_fault(). 
> That essentially disable dax pmd for ext4. Any plans to fix that ?

I'm working on converting ext4 to the new iomap infrastructure. Once that
is done, it can use new DAX helpers for PMD faults easily.

								Honza
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR

--
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: Jan Kara <jack@suse.cz>
To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
Cc: Ross Zwisler <ross.zwisler@linux.intel.com>,
	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>,
	Christoph Hellwig <hch@lst.de>,
	Dan Williams <dan.j.williams@intel.com>,
	Dave Chinner <david@fromorbit.com>, Jan Kara <jack@suse.cz>,
	Matthew Wilcox <mawilcox@microsoft.com>,
	linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	linux-mm@kvack.org, linux-nvdimm@lists.01.org,
	linux-xfs@vger.kernel.org
Subject: Re: [PATCH v6 17/17] dax: remove "depends on BROKEN" from FS_DAX_PMD
Date: Mon, 17 Oct 2016 11:09:37 +0200	[thread overview]
Message-ID: <20161017090937.GG3359@quack2.suse.cz> (raw)
In-Reply-To: <87eg3ftt4r.fsf@linux.vnet.ibm.com>

On Mon 17-10-16 11:27:24, Aneesh Kumar K.V wrote:
> Ross Zwisler <ross.zwisler@linux.intel.com> writes:
> 
> > Now that DAX PMD faults are once again working and are now participating in
> > DAX's radix tree locking scheme, allow their config option to be enabled.
> >
> > Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
> > ---
> >  fs/Kconfig | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/fs/Kconfig b/fs/Kconfig
> > index 2bc7ad7..b6f0fce 100644
> > --- a/fs/Kconfig
> > +++ b/fs/Kconfig
> > @@ -55,7 +55,6 @@ config FS_DAX_PMD
> >  	depends on FS_DAX
> >  	depends on ZONE_DEVICE
> >  	depends on TRANSPARENT_HUGEPAGE
> > -	depends on BROKEN
> >  
> >  endif # BLOCK
> 
> 
> This series made dax_pmd_fault return VM_FAULT_FALLBACK in [PATCH v6 09/17] dax: remove dax_pmd_fault(). 
> That essentially disable dax pmd for ext4. Any plans to fix that ?

I'm working on converting ext4 to the new iomap infrastructure. Once that
is done, it can use new DAX helpers for PMD faults easily.

								Honza
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR

WARNING: multiple messages have this Message-ID (diff)
From: Jan Kara <jack@suse.cz>
To: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
Cc: Ross Zwisler <ross.zwisler@linux.intel.com>,
	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>,
	Christoph Hellwig <hch@lst.de>,
	Dan Williams <dan.j.williams@intel.com>,
	Dave Chinner <david@fromorbit.com>, Jan Kara <jack@suse.cz>,
	Matthew Wilcox <mawilcox@microsoft.com>,
	linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	linux-mm@kvack.org, linux-nvdimm@ml01.01.org,
	linux-xfs@vger.kernel.org
Subject: Re: [PATCH v6 17/17] dax: remove "depends on BROKEN" from FS_DAX_PMD
Date: Mon, 17 Oct 2016 11:09:37 +0200	[thread overview]
Message-ID: <20161017090937.GG3359@quack2.suse.cz> (raw)
In-Reply-To: <87eg3ftt4r.fsf@linux.vnet.ibm.com>

On Mon 17-10-16 11:27:24, Aneesh Kumar K.V wrote:
> Ross Zwisler <ross.zwisler@linux.intel.com> writes:
> 
> > Now that DAX PMD faults are once again working and are now participating in
> > DAX's radix tree locking scheme, allow their config option to be enabled.
> >
> > Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
> > ---
> >  fs/Kconfig | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/fs/Kconfig b/fs/Kconfig
> > index 2bc7ad7..b6f0fce 100644
> > --- a/fs/Kconfig
> > +++ b/fs/Kconfig
> > @@ -55,7 +55,6 @@ config FS_DAX_PMD
> >  	depends on FS_DAX
> >  	depends on ZONE_DEVICE
> >  	depends on TRANSPARENT_HUGEPAGE
> > -	depends on BROKEN
> >  
> >  endif # BLOCK
> 
> 
> This series made dax_pmd_fault return VM_FAULT_FALLBACK in [PATCH v6 09/17] dax: remove dax_pmd_fault(). 
> That essentially disable dax pmd for ext4. Any plans to fix that ?

I'm working on converting ext4 to the new iomap infrastructure. Once that
is done, it can use new DAX helpers for PMD faults easily.

								Honza
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR

  parent reply	other threads:[~2016-10-17  9:09 UTC|newest]

Thread overview: 126+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-12 22:50 [PATCH v6 00/17] re-enable DAX PMD support Ross Zwisler
2016-10-12 22:50 ` Ross Zwisler
2016-10-12 22:50 ` Ross Zwisler
2016-10-12 22:50 ` Ross Zwisler
2016-10-12 22:50 ` Ross Zwisler
     [not found] ` <20161012225022.15507-1-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-12 22:50   ` [PATCH v6 01/17] ext4: allow DAX writeback for hole punch Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 02/17] ext4: tell DAX the size of allocation holes Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 03/17] dax: remove buffer_size_valid() Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 04/17] ext2: remove support for DAX PMD faults Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 07/17] dax: consistent variable naming for DAX entries Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 08/17] dax: coordinate locking for offsets in PMD range Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 09/17] dax: remove dax_pmd_fault() Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 10/17] dax: correct dax iomap code namespace Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 11/17] dax: add dax_iomap_sector() helper function Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 12/17] dax: dax_iomap_fault() needs to call iomap_end() Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 13/17] dax: move RADIX_DAX_* defines to dax.h Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 14/17] dax: move put_(un)locked_mapping_entry() in dax.c Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-13 15:11     ` Jan Kara
2016-10-13 15:11       ` Jan Kara
2016-10-13 15:11       ` Jan Kara
2016-10-12 22:50   ` [PATCH v6 15/17] dax: add struct iomap based DAX PMD support Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
     [not found]     ` <20161012225022.15507-16-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-13 15:42       ` Jan Kara
2016-10-13 15:42         ` Jan Kara
2016-10-13 15:42         ` Jan Kara
2016-10-13 15:42         ` Jan Kara
2016-10-13 15:42         ` Jan Kara
2016-10-13 19:23         ` [PATCH v7 " Ross Zwisler
2016-10-13 19:23           ` Ross Zwisler
2016-10-13 19:23           ` Ross Zwisler
2016-10-13 19:23           ` Ross Zwisler
2016-10-17  6:06           ` Aneesh Kumar K.V
2016-10-17  6:06             ` Aneesh Kumar K.V
2016-10-17  6:06             ` Aneesh Kumar K.V
2016-10-17  6:06             ` Aneesh Kumar K.V
2016-10-17  6:06             ` Aneesh Kumar K.V
2016-10-17 14:55             ` Ross Zwisler
2016-10-17 14:55               ` Ross Zwisler
2016-10-17 14:55               ` Ross Zwisler
2016-10-17 14:55               ` Ross Zwisler
2016-10-13 19:25         ` [PATCH v6 " Ross Zwisler
2016-10-13 19:25           ` Ross Zwisler
2016-10-13 19:25           ` Ross Zwisler
2016-10-13 19:25           ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 16/17] xfs: use struct iomap based DAX PMD fault path Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50   ` [PATCH v6 17/17] dax: remove "depends on BROKEN" from FS_DAX_PMD Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
2016-10-12 22:50     ` Ross Zwisler
     [not found]     ` <20161012225022.15507-18-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-13 15:42       ` Jan Kara
2016-10-13 15:42         ` Jan Kara
2016-10-13 15:42         ` Jan Kara
2016-10-13 15:42         ` Jan Kara
2016-10-17  5:57       ` Aneesh Kumar K.V
2016-10-17  5:57         ` Aneesh Kumar K.V
2016-10-17  5:57         ` Aneesh Kumar K.V
2016-10-17  5:57         ` Aneesh Kumar K.V
2016-10-17  5:57         ` Aneesh Kumar K.V
2016-10-17  5:57         ` Aneesh Kumar K.V
     [not found]         ` <87eg3ftt4r.fsf-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2016-10-17  9:09           ` Jan Kara [this message]
2016-10-17  9:09             ` Jan Kara
2016-10-17  9:09             ` Jan Kara
2016-10-17  9:09             ` Jan Kara
2016-10-12 22:50 ` [PATCH v6 05/17] dax: make 'wait_table' global variable static Ross Zwisler
2016-10-12 22:50   ` Ross Zwisler
2016-10-12 22:50   ` Ross Zwisler
2016-10-12 22:50   ` Ross Zwisler
2016-10-12 22:50 ` [PATCH v6 06/17] dax: remove the last BUG_ON() from fs/dax.c Ross Zwisler
2016-10-12 22:50   ` Ross Zwisler
2016-10-12 22:50   ` Ross Zwisler
2016-10-12 22:50   ` Ross Zwisler

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=20161017090937.GG3359@quack2.suse.cz \
    --to=jack-alswssmvlrq@public.gmane.org \
    --cc=adilger.kernel-m1MBpc4rdrD3fQ9qLvQP4Q@public.gmane.org \
    --cc=akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
    --cc=aneesh.kumar-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org \
    --cc=david-FqsqvQoI3Ljby3iVrkZq2A@public.gmane.org \
    --cc=hch-jcswGhMUV9g@public.gmane.org \
    --cc=linux-ext4-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org \
    --cc=linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org \
    --cc=linux-xfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mawilcox-0li6OtcxBFHby3iVrkZq2A@public.gmane.org \
    --cc=tytso-3s7WtUTddSA@public.gmane.org \
    --cc=viro-RmSDqhL/yNMiFSDQTTA3OLVCufUGDwFn@public.gmane.org \
    /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.