From: "Kirill A. Shutemov" <kirill@shutemov.name>
To: Toshi Kani <toshi.kani@hpe.com>
Cc: jack@suse.cz, mawilcox@microsoft.com, hughd@google.com,
linux-nvdimm@lists.01.org, david@fromorbit.com,
linux-kernel@vger.kernel.org, linux-mm@kvack.org,
adilger.kernel@dilger.ca, kirill.shutemov@linux.intel.com,
linux-fsdevel@vger.kernel.org, tytso@mit.edu,
akpm@linux-foundation.org, mike.kravetz@oracle.com
Subject: Re: [PATCH v4 RESEND 0/2] Align mmap address for DAX pmd mappings
Date: Mon, 29 Aug 2016 23:48:42 +0300 [thread overview]
Message-ID: <20160829204842.GA27286@node.shutemov.name> (raw)
In-Reply-To: <1472497881-9323-1-git-send-email-toshi.kani@hpe.com>
On Mon, Aug 29, 2016 at 01:11:19PM -0600, Toshi Kani wrote:
> When CONFIG_FS_DAX_PMD is set, DAX supports mmap() using pmd page
> size. This feature relies on both mmap virtual address and FS
> block (i.e. physical address) to be aligned by the pmd page size.
> Users can use mkfs options to specify FS to align block allocations.
> However, aligning mmap address requires code changes to existing
> applications for providing a pmd-aligned address to mmap().
>
> For instance, fio with "ioengine=mmap" performs I/Os with mmap() [1].
> It calls mmap() with a NULL address, which needs to be changed to
> provide a pmd-aligned address for testing with DAX pmd mappings.
> Changing all applications that call mmap() with NULL is undesirable.
>
> This patch-set extends filesystems to align an mmap address for
> a DAX file so that unmodified applications can use DAX pmd mappings.
+Hugh
Can we get it used for shmem/tmpfs too?
I don't think we should duplicate essentially the same functionality in
multiple places.
--
Kirill A. Shutemov
_______________________________________________
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: "Kirill A. Shutemov" <kirill@shutemov.name>
To: Toshi Kani <toshi.kani@hpe.com>
Cc: akpm@linux-foundation.org, dan.j.williams@intel.com,
mawilcox@microsoft.com, ross.zwisler@linux.intel.com,
kirill.shutemov@linux.intel.com, david@fromorbit.com,
jack@suse.cz, tytso@mit.edu, adilger.kernel@dilger.ca,
mike.kravetz@oracle.com, linux-nvdimm@ml01.01.org,
linux-fsdevel@vger.kernel.org, linux-mm@kvack.org,
linux-kernel@vger.kernel.org, hughd@google.com
Subject: Re: [PATCH v4 RESEND 0/2] Align mmap address for DAX pmd mappings
Date: Mon, 29 Aug 2016 23:48:42 +0300 [thread overview]
Message-ID: <20160829204842.GA27286@node.shutemov.name> (raw)
In-Reply-To: <1472497881-9323-1-git-send-email-toshi.kani@hpe.com>
On Mon, Aug 29, 2016 at 01:11:19PM -0600, Toshi Kani wrote:
> When CONFIG_FS_DAX_PMD is set, DAX supports mmap() using pmd page
> size. This feature relies on both mmap virtual address and FS
> block (i.e. physical address) to be aligned by the pmd page size.
> Users can use mkfs options to specify FS to align block allocations.
> However, aligning mmap address requires code changes to existing
> applications for providing a pmd-aligned address to mmap().
>
> For instance, fio with "ioengine=mmap" performs I/Os with mmap() [1].
> It calls mmap() with a NULL address, which needs to be changed to
> provide a pmd-aligned address for testing with DAX pmd mappings.
> Changing all applications that call mmap() with NULL is undesirable.
>
> This patch-set extends filesystems to align an mmap address for
> a DAX file so that unmodified applications can use DAX pmd mappings.
+Hugh
Can we get it used for shmem/tmpfs too?
I don't think we should duplicate essentially the same functionality in
multiple places.
--
Kirill A. Shutemov
WARNING: multiple messages have this Message-ID (diff)
From: "Kirill A. Shutemov" <kirill@shutemov.name>
To: Toshi Kani <toshi.kani@hpe.com>
Cc: akpm@linux-foundation.org, dan.j.williams@intel.com,
mawilcox@microsoft.com, ross.zwisler@linux.intel.com,
kirill.shutemov@linux.intel.com, david@fromorbit.com,
jack@suse.cz, tytso@mit.edu, adilger.kernel@dilger.ca,
mike.kravetz@oracle.com, linux-nvdimm@lists.01.org,
linux-fsdevel@vger.kernel.org, linux-mm@kvack.org,
linux-kernel@vger.kernel.org, hughd@google.com
Subject: Re: [PATCH v4 RESEND 0/2] Align mmap address for DAX pmd mappings
Date: Mon, 29 Aug 2016 23:48:42 +0300 [thread overview]
Message-ID: <20160829204842.GA27286@node.shutemov.name> (raw)
In-Reply-To: <1472497881-9323-1-git-send-email-toshi.kani@hpe.com>
On Mon, Aug 29, 2016 at 01:11:19PM -0600, Toshi Kani wrote:
> When CONFIG_FS_DAX_PMD is set, DAX supports mmap() using pmd page
> size. This feature relies on both mmap virtual address and FS
> block (i.e. physical address) to be aligned by the pmd page size.
> Users can use mkfs options to specify FS to align block allocations.
> However, aligning mmap address requires code changes to existing
> applications for providing a pmd-aligned address to mmap().
>
> For instance, fio with "ioengine=mmap" performs I/Os with mmap() [1].
> It calls mmap() with a NULL address, which needs to be changed to
> provide a pmd-aligned address for testing with DAX pmd mappings.
> Changing all applications that call mmap() with NULL is undesirable.
>
> This patch-set extends filesystems to align an mmap address for
> a DAX file so that unmodified applications can use DAX pmd mappings.
+Hugh
Can we get it used for shmem/tmpfs too?
I don't think we should duplicate essentially the same functionality in
multiple places.
--
Kirill A. Shutemov
--
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>
next prev parent reply other threads:[~2016-08-29 20:48 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-29 19:11 [PATCH v4 RESEND 0/2] Align mmap address for DAX pmd mappings Toshi Kani
2016-08-29 19:11 ` Toshi Kani
2016-08-29 19:11 ` Toshi Kani
2016-08-29 19:11 ` [PATCH v4 RESEND 1/2] thp, dax: add thp_get_unmapped_area for " Toshi Kani
2016-08-29 19:11 ` Toshi Kani
2016-08-29 19:11 ` Toshi Kani
2016-08-29 19:34 ` Dan Williams
2016-08-29 19:34 ` Dan Williams
2016-08-29 19:34 ` Dan Williams
2016-08-29 20:44 ` Kani, Toshimitsu
2016-08-29 20:44 ` Kani, Toshimitsu
2016-08-29 20:44 ` Kani, Toshimitsu
2016-08-29 20:54 ` Dan Williams
2016-08-29 20:54 ` Dan Williams
2016-08-29 20:54 ` Dan Williams
2016-08-29 19:11 ` [PATCH v4 RESEND 2/2] ext2/4, xfs: call thp_get_unmapped_area() " Toshi Kani
2016-08-29 19:11 ` Toshi Kani
2016-08-29 19:11 ` Toshi Kani
2016-08-29 20:48 ` Kirill A. Shutemov [this message]
2016-08-29 20:48 ` [PATCH v4 RESEND 0/2] Align mmap address for DAX " Kirill A. Shutemov
2016-08-29 20:48 ` Kirill A. Shutemov
2016-08-29 21:32 ` Kani, Toshimitsu
2016-08-29 21:32 ` Kani, Toshimitsu
2016-08-29 21:32 ` Kani, Toshimitsu
2016-08-29 22:00 ` Kani, Toshimitsu
2016-08-29 22:00 ` Kani, Toshimitsu
2016-08-29 22:00 ` Kani, Toshimitsu
2016-09-08 10:57 ` Kirill A. Shutemov
2016-09-08 10:57 ` Kirill A. Shutemov
2016-09-08 10:57 ` Kirill A. Shutemov
2016-09-08 13:49 ` Kani, Toshimitsu
2016-09-08 13:49 ` Kani, Toshimitsu
2016-09-08 13:49 ` Kani, Toshimitsu
2016-09-08 23:21 ` Kani, Toshimitsu
2016-09-08 23:21 ` Kani, Toshimitsu
2016-09-08 23:21 ` Kani, Toshimitsu
2016-09-09 12:36 ` Kirill A. Shutemov
2016-09-09 12:36 ` Kirill A. Shutemov
2016-09-09 12:36 ` Kirill A. Shutemov
2016-09-09 17:21 ` Kani, Toshimitsu
2016-09-09 17:21 ` Kani, Toshimitsu
2016-09-09 17:21 ` Kani, Toshimitsu
2016-09-09 18:37 ` Kirill A. Shutemov
2016-09-09 18:37 ` Kirill A. Shutemov
2016-09-09 18:37 ` Kirill A. Shutemov
2016-09-09 18:37 ` Kirill A. Shutemov
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=20160829204842.GA27286@node.shutemov.name \
--to=kirill@shutemov.name \
--cc=adilger.kernel@dilger.ca \
--cc=akpm@linux-foundation.org \
--cc=david@fromorbit.com \
--cc=hughd@google.com \
--cc=jack@suse.cz \
--cc=kirill.shutemov@linux.intel.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linux-nvdimm@lists.01.org \
--cc=mawilcox@microsoft.com \
--cc=mike.kravetz@oracle.com \
--cc=toshi.kani@hpe.com \
--cc=tytso@mit.edu \
/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.