From: Dave Chinner <david-FqsqvQoI3Ljby3iVrkZq2A@public.gmane.org>
To: Ross Zwisler <ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Cc: Matthew Wilcox <mawilcox-0li6OtcxBFHby3iVrkZq2A@public.gmane.org>,
linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-xfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Alexander Viro
<viro-RmSDqhL/yNMiFSDQTTA3OLVCufUGDwFn@public.gmane.org>,
Jan Kara <jack-IBi9RG/b67k@public.gmane.org>,
linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Andrew Morton
<akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>,
linux-ext4-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Christoph Hellwig <hch-jcswGhMUV9g@public.gmane.org>
Subject: Re: [PATCH] dax: correct dax iomap code namespace
Date: Tue, 11 Oct 2016 07:19:58 +1100 [thread overview]
Message-ID: <20161010201958.GD23194@dastard> (raw)
In-Reply-To: <1476126244-27673-1-git-send-email-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
On Mon, Oct 10, 2016 at 01:04:04PM -0600, Ross Zwisler wrote:
> The recently added DAX functions that use the new struct iomap data
> structure were named iomap_dax_rw(), iomap_dax_fault() and
> iomap_dax_actor(). These are actually defined in fs/dax.c, though, so
> should be part of the "dax" namespace and not the "iomap" namespace.
> Rename them to dax_iomap_rw(), dax_iomap_fault() and dax_iomap_actor()
> respectively.
>
> Signed-off-by: Ross Zwisler <ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> Suggested-by: Dave Chinner <david-FqsqvQoI3Ljby3iVrkZq2A@public.gmane.org>
> Reviewed-by: Christoph Hellwig <hch-jcswGhMUV9g@public.gmane.org>
> Reviewed-by: Jan Kara <jack-AlSwsSmVLrQ@public.gmane.org>
> ---
>
> Dave, per Christoph's suggestion can you please send this to Linus for
> v4.9? This patch applies cleanly to both the current linux/master and the
> current xfs/for-next.
I'll plan to send this for 4.9-rc2 or 4.9-rc3 in the first batch of
fixes for the -rc1 merge.
Cheers,
Dave.
--
Dave Chinner
david-FqsqvQoI3Ljby3iVrkZq2A@public.gmane.org
WARNING: multiple messages have this Message-ID (diff)
From: Dave Chinner <david@fromorbit.com>
To: Ross Zwisler <ross.zwisler@linux.intel.com>
Cc: Matthew Wilcox <mawilcox@microsoft.com>,
linux-nvdimm@lists.01.org, linux-kernel@vger.kernel.org,
linux-xfs@vger.kernel.org,
Alexander Viro <viro@zeniv.linux.org.uk>,
Jan Kara <jack@suse.com>,
linux-fsdevel@vger.kernel.org,
Andrew Morton <akpm@linux-foundation.org>,
linux-ext4@vger.kernel.org, Christoph Hellwig <hch@lst.de>
Subject: Re: [PATCH] dax: correct dax iomap code namespace
Date: Tue, 11 Oct 2016 07:19:58 +1100 [thread overview]
Message-ID: <20161010201958.GD23194@dastard> (raw)
In-Reply-To: <1476126244-27673-1-git-send-email-ross.zwisler@linux.intel.com>
On Mon, Oct 10, 2016 at 01:04:04PM -0600, Ross Zwisler wrote:
> The recently added DAX functions that use the new struct iomap data
> structure were named iomap_dax_rw(), iomap_dax_fault() and
> iomap_dax_actor(). These are actually defined in fs/dax.c, though, so
> should be part of the "dax" namespace and not the "iomap" namespace.
> Rename them to dax_iomap_rw(), dax_iomap_fault() and dax_iomap_actor()
> respectively.
>
> Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
> Suggested-by: Dave Chinner <david@fromorbit.com>
> Reviewed-by: Christoph Hellwig <hch@lst.de>
> Reviewed-by: Jan Kara <jack@suse.cz>
> ---
>
> Dave, per Christoph's suggestion can you please send this to Linus for
> v4.9? This patch applies cleanly to both the current linux/master and the
> current xfs/for-next.
I'll plan to send this for 4.9-rc2 or 4.9-rc3 in the first batch of
fixes for the -rc1 merge.
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
_______________________________________________
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: Dave Chinner <david@fromorbit.com>
To: Ross Zwisler <ross.zwisler@linux.intel.com>
Cc: linux-kernel@vger.kernel.org,
Alexander Viro <viro@zeniv.linux.org.uk>,
Andrew Morton <akpm@linux-foundation.org>,
Christoph Hellwig <hch@lst.de>,
Dan Williams <dan.j.williams@intel.com>, Jan Kara <jack@suse.com>,
Matthew Wilcox <mawilcox@microsoft.com>,
linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org,
linux-nvdimm@lists.01.org, linux-xfs@vger.kernel.org
Subject: Re: [PATCH] dax: correct dax iomap code namespace
Date: Tue, 11 Oct 2016 07:19:58 +1100 [thread overview]
Message-ID: <20161010201958.GD23194@dastard> (raw)
In-Reply-To: <1476126244-27673-1-git-send-email-ross.zwisler@linux.intel.com>
On Mon, Oct 10, 2016 at 01:04:04PM -0600, Ross Zwisler wrote:
> The recently added DAX functions that use the new struct iomap data
> structure were named iomap_dax_rw(), iomap_dax_fault() and
> iomap_dax_actor(). These are actually defined in fs/dax.c, though, so
> should be part of the "dax" namespace and not the "iomap" namespace.
> Rename them to dax_iomap_rw(), dax_iomap_fault() and dax_iomap_actor()
> respectively.
>
> Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
> Suggested-by: Dave Chinner <david@fromorbit.com>
> Reviewed-by: Christoph Hellwig <hch@lst.de>
> Reviewed-by: Jan Kara <jack@suse.cz>
> ---
>
> Dave, per Christoph's suggestion can you please send this to Linus for
> v4.9? This patch applies cleanly to both the current linux/master and the
> current xfs/for-next.
I'll plan to send this for 4.9-rc2 or 4.9-rc3 in the first batch of
fixes for the -rc1 merge.
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
WARNING: multiple messages have this Message-ID (diff)
From: Dave Chinner <david@fromorbit.com>
To: Ross Zwisler <ross.zwisler@linux.intel.com>
Cc: linux-kernel@vger.kernel.org,
Alexander Viro <viro@zeniv.linux.org.uk>,
Andrew Morton <akpm@linux-foundation.org>,
Christoph Hellwig <hch@lst.de>,
Dan Williams <dan.j.williams@intel.com>, Jan Kara <jack@suse.com>,
Matthew Wilcox <mawilcox@microsoft.com>,
linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org,
linux-nvdimm@ml01.01.org, linux-xfs@vger.kernel.org
Subject: Re: [PATCH] dax: correct dax iomap code namespace
Date: Tue, 11 Oct 2016 07:19:58 +1100 [thread overview]
Message-ID: <20161010201958.GD23194@dastard> (raw)
In-Reply-To: <1476126244-27673-1-git-send-email-ross.zwisler@linux.intel.com>
On Mon, Oct 10, 2016 at 01:04:04PM -0600, Ross Zwisler wrote:
> The recently added DAX functions that use the new struct iomap data
> structure were named iomap_dax_rw(), iomap_dax_fault() and
> iomap_dax_actor(). These are actually defined in fs/dax.c, though, so
> should be part of the "dax" namespace and not the "iomap" namespace.
> Rename them to dax_iomap_rw(), dax_iomap_fault() and dax_iomap_actor()
> respectively.
>
> Signed-off-by: Ross Zwisler <ross.zwisler@linux.intel.com>
> Suggested-by: Dave Chinner <david@fromorbit.com>
> Reviewed-by: Christoph Hellwig <hch@lst.de>
> Reviewed-by: Jan Kara <jack@suse.cz>
> ---
>
> Dave, per Christoph's suggestion can you please send this to Linus for
> v4.9? This patch applies cleanly to both the current linux/master and the
> current xfs/for-next.
I'll plan to send this for 4.9-rc2 or 4.9-rc3 in the first batch of
fixes for the -rc1 merge.
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
next prev parent reply other threads:[~2016-10-10 20:19 UTC|newest]
Thread overview: 182+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-07 21:08 [PATCH v5 00/17] re-enable DAX PMD support Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
[not found] ` <1475874544-24842-1-git-send-email-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-07 21:08 ` [PATCH v5 01/17] ext4: allow DAX writeback for hole punch Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` [PATCH v5 02/17] ext4: tell DAX the size of allocation holes Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` [PATCH v5 03/17] dax: remove buffer_size_valid() Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` [PATCH v5 05/17] ext2: return -EIO on ext2_iomap_end() failure Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-11 6:48 ` Jan Kara
2016-10-11 6:48 ` Jan Kara
2016-10-11 6:48 ` Jan Kara
2016-10-11 6:48 ` Jan Kara
2016-10-07 21:08 ` [PATCH v5 06/17] dax: make 'wait_table' global variable static Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` [PATCH v5 07/17] dax: remove the last BUG_ON() from fs/dax.c Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
[not found] ` <1475874544-24842-8-git-send-email-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-10 15:45 ` Christoph Hellwig
2016-10-10 15:45 ` Christoph Hellwig
2016-10-10 15:45 ` Christoph Hellwig
2016-10-10 15:45 ` Christoph Hellwig
2016-10-10 15:45 ` Christoph Hellwig
2016-10-11 6:50 ` Jan Kara
2016-10-11 6:50 ` Jan Kara
2016-10-11 6:50 ` Jan Kara
2016-10-11 6:50 ` Jan Kara
2016-10-11 6:50 ` Jan Kara
2016-10-07 21:08 ` [PATCH v5 08/17] dax: consistent variable naming for DAX entries Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` [PATCH v5 10/17] dax: remove dax_pmd_fault() Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` [PATCH v5 11/17] dax: correct dax iomap code namespace Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
[not found] ` <1475874544-24842-12-git-send-email-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-09 15:28 ` Christoph Hellwig
2016-10-09 15:28 ` Christoph Hellwig
2016-10-09 15:28 ` Christoph Hellwig
2016-10-09 15:28 ` Christoph Hellwig
[not found] ` <20161009152803.GA20111-jcswGhMUV9g@public.gmane.org>
2016-10-10 19:04 ` [PATCH] " Ross Zwisler
2016-10-10 19:04 ` Ross Zwisler
2016-10-10 19:04 ` Ross Zwisler
2016-10-10 19:04 ` Ross Zwisler
[not found] ` <1476126244-27673-1-git-send-email-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-10 20:19 ` Dave Chinner [this message]
2016-10-10 20:19 ` Dave Chinner
2016-10-10 20:19 ` Dave Chinner
2016-10-10 20:19 ` Dave Chinner
2016-10-07 21:08 ` [PATCH v5 12/17] dax: add dax_iomap_sector() helper function Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
[not found] ` <1475874544-24842-13-git-send-email-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-10 15:46 ` Christoph Hellwig
2016-10-10 15:46 ` Christoph Hellwig
2016-10-10 15:46 ` Christoph Hellwig
2016-10-10 15:46 ` Christoph Hellwig
2016-10-10 15:46 ` Christoph Hellwig
2016-10-11 7:06 ` Jan Kara
2016-10-11 7:06 ` Jan Kara
2016-10-11 7:06 ` Jan Kara
2016-10-11 7:06 ` Jan Kara
2016-10-07 21:09 ` [PATCH v5 15/17] dax: add struct iomap based DAX PMD support Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
[not found] ` <1475874544-24842-16-git-send-email-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-10 15:59 ` Christoph Hellwig
2016-10-10 15:59 ` Christoph Hellwig
2016-10-10 15:59 ` Christoph Hellwig
2016-10-10 15:59 ` Christoph Hellwig
[not found] ` <20161010155917.GA19978-jcswGhMUV9g@public.gmane.org>
2016-10-10 22:06 ` Ross Zwisler
2016-10-10 22:06 ` Ross Zwisler
2016-10-10 22:06 ` Ross Zwisler
2016-10-10 22:06 ` Ross Zwisler
2016-10-10 22:06 ` Ross Zwisler
2016-10-11 21:48 ` Ross Zwisler
2016-10-11 21:48 ` Ross Zwisler
2016-10-11 21:48 ` Ross Zwisler
2016-10-11 21:48 ` Ross Zwisler
2016-10-11 21:48 ` Ross Zwisler
2016-10-11 8:31 ` Jan Kara
2016-10-11 8:31 ` Jan Kara
2016-10-11 8:31 ` Jan Kara
2016-10-11 8:31 ` Jan Kara
2016-10-11 8:31 ` Jan Kara
[not found] ` <20161011083152.GG6952-4I4JzKEfoa/jFM9bn6wA6Q@public.gmane.org>
2016-10-11 22:51 ` Ross Zwisler
2016-10-11 22:51 ` Ross Zwisler
2016-10-11 22:51 ` Ross Zwisler
2016-10-11 22:51 ` Ross Zwisler
2016-10-11 22:51 ` Ross Zwisler
2016-10-12 7:45 ` Jan Kara
2016-10-12 7:45 ` Jan Kara
2016-10-12 7:45 ` Jan Kara
2016-10-07 21:09 ` [PATCH v5 16/17] xfs: use struct iomap based DAX PMD fault path Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-11 8:34 ` Jan Kara
2016-10-11 8:34 ` Jan Kara
2016-10-11 8:34 ` Jan Kara
2016-10-11 8:34 ` Jan Kara
2016-10-07 21:09 ` [PATCH v5 17/17] dax: remove "depends on BROKEN" from FS_DAX_PMD Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:08 ` [PATCH v5 04/17] ext2: remove support for DAX PMD faults Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` [PATCH v5 09/17] dax: coordinate locking for offsets in PMD range Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-07 21:08 ` Ross Zwisler
2016-10-10 15:46 ` Christoph Hellwig
2016-10-10 15:46 ` Christoph Hellwig
2016-10-10 15:46 ` Christoph Hellwig
2016-10-11 7:04 ` Jan Kara
2016-10-11 7:04 ` Jan Kara
2016-10-11 7:04 ` Jan Kara
2016-10-11 7:04 ` Jan Kara
2016-10-11 21:18 ` Ross Zwisler
2016-10-11 21:18 ` Ross Zwisler
2016-10-11 21:18 ` Ross Zwisler
2016-10-07 21:09 ` [PATCH v5 13/17] dax: dax_iomap_fault() needs to call iomap_end() Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-10 15:50 ` Christoph Hellwig
2016-10-10 15:50 ` Christoph Hellwig
2016-10-10 15:50 ` Christoph Hellwig
2016-10-10 15:50 ` Christoph Hellwig
[not found] ` <20161010155004.GD19343-jcswGhMUV9g@public.gmane.org>
2016-10-10 22:05 ` Ross Zwisler
2016-10-10 22:05 ` Ross Zwisler
2016-10-10 22:05 ` Ross Zwisler
2016-10-10 22:05 ` Ross Zwisler
[not found] ` <1475874544-24842-14-git-send-email-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-11 7:21 ` Jan Kara
2016-10-11 7:21 ` Jan Kara
2016-10-11 7:21 ` Jan Kara
2016-10-11 7:21 ` Jan Kara
2016-10-11 7:21 ` Jan Kara
2016-10-07 21:09 ` [PATCH v5 14/17] dax: move RADIX_DAX_* defines to dax.h Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
2016-10-07 21:09 ` Ross Zwisler
[not found] ` <1475874544-24842-15-git-send-email-ross.zwisler-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2016-10-10 15:50 ` Christoph Hellwig
2016-10-10 15:50 ` Christoph Hellwig
2016-10-10 15:50 ` Christoph Hellwig
2016-10-10 15:50 ` Christoph Hellwig
2016-10-10 15:50 ` Christoph Hellwig
2016-10-11 7:23 ` Jan Kara
2016-10-11 7:23 ` Jan Kara
2016-10-11 7:23 ` Jan Kara
2016-10-11 7:23 ` Jan Kara
2016-10-11 7:23 ` 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=20161010201958.GD23194@dastard \
--to=david-fqsqvqoi3ljby3ivrkzq2a@public.gmane.org \
--cc=akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
--cc=hch-jcswGhMUV9g@public.gmane.org \
--cc=jack-IBi9RG/b67k@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-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org \
--cc=linux-xfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mawilcox-0li6OtcxBFHby3iVrkZq2A@public.gmane.org \
--cc=ross.zwisler-VuQAYsv1563Yd54FQh9/CA@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.