All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
To: Dan Williams <dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: Andrea Arcangeli
	<aarcange-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	Xiao Guangrong
	<guangrong.xiao-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>,
	Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>,
	linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Dave Hansen <dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org,
	linux-nvdimm-y27Ovi1pjclAfugRpC6u6w@public.gmane.org,
	Andrew Morton
	<akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>,
	"Kirill A. Shutemov"
	<kirill.shutemov-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Subject: Re: [RFC PATCH 1/2] mm, mincore2(): retrieve dax and tlb-size attributes of an address range
Date: Mon, 12 Sep 2016 23:44:05 -0700	[thread overview]
Message-ID: <20160913064405.GA21069@infradead.org> (raw)
In-Reply-To: <147361509579.17004.5258725187329709824.stgit-p8uTFz9XbKj2zm6wflaqv1nYeNYlB/vhral2JQCrhuEAvxtiuMwx3w@public.gmane.org>

On Sun, Sep 11, 2016 at 10:31:35AM -0700, Dan Williams wrote:
> As evidenced by this bug report [1], userspace libraries are interested
> in whether a mapping is DAX mapped, i.e. no intervening page cache.
> Rather than using the ambiguous VM_MIXEDMAP flag in smaps, provide an
> explicit "is dax" indication as a new flag in the page vector populated
> by mincore.

And how exactly does an implementation detail like DAX matter for an
application?  The only thing that might matter is the atomicy boundary,
but mincore is not the right interface for that.

WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@infradead.org>
To: Dan Williams <dan.j.williams@intel.com>
Cc: linux-mm@kvack.org, Andrea Arcangeli <aarcange@redhat.com>,
	Xiao Guangrong <guangrong.xiao@linux.intel.com>,
	Arnd Bergmann <arnd@arndb.de>,
	linux-nvdimm@ml01.01.org, linux-api@vger.kernel.org,
	Dave Hansen <dave.hansen@linux.intel.com>,
	linux-kernel@vger.kernel.org,
	Andrew Morton <akpm@linux-foundation.org>,
	"Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>
Subject: Re: [RFC PATCH 1/2] mm, mincore2(): retrieve dax and tlb-size attributes of an address range
Date: Mon, 12 Sep 2016 23:44:05 -0700	[thread overview]
Message-ID: <20160913064405.GA21069@infradead.org> (raw)
In-Reply-To: <147361509579.17004.5258725187329709824.stgit@dwillia2-desk3.amr.corp.intel.com>

On Sun, Sep 11, 2016 at 10:31:35AM -0700, Dan Williams wrote:
> As evidenced by this bug report [1], userspace libraries are interested
> in whether a mapping is DAX mapped, i.e. no intervening page cache.
> Rather than using the ambiguous VM_MIXEDMAP flag in smaps, provide an
> explicit "is dax" indication as a new flag in the page vector populated
> by mincore.

And how exactly does an implementation detail like DAX matter for an
application?  The only thing that might matter is the atomicy boundary,
but mincore is not the right interface for that.

--
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: Christoph Hellwig <hch@infradead.org>
To: Dan Williams <dan.j.williams@intel.com>
Cc: linux-mm@kvack.org, Andrea Arcangeli <aarcange@redhat.com>,
	Xiao Guangrong <guangrong.xiao@linux.intel.com>,
	Arnd Bergmann <arnd@arndb.de>,
	linux-nvdimm@ml01.01.org, linux-api@vger.kernel.org,
	Dave Hansen <dave.hansen@linux.intel.com>,
	linux-kernel@vger.kernel.org,
	Andrew Morton <akpm@linux-foundation.org>,
	"Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>
Subject: Re: [RFC PATCH 1/2] mm, mincore2(): retrieve dax and tlb-size attributes of an address range
Date: Mon, 12 Sep 2016 23:44:05 -0700	[thread overview]
Message-ID: <20160913064405.GA21069@infradead.org> (raw)
In-Reply-To: <147361509579.17004.5258725187329709824.stgit@dwillia2-desk3.amr.corp.intel.com>

On Sun, Sep 11, 2016 at 10:31:35AM -0700, Dan Williams wrote:
> As evidenced by this bug report [1], userspace libraries are interested
> in whether a mapping is DAX mapped, i.e. no intervening page cache.
> Rather than using the ambiguous VM_MIXEDMAP flag in smaps, provide an
> explicit "is dax" indication as a new flag in the page vector populated
> by mincore.

And how exactly does an implementation detail like DAX matter for an
application?  The only thing that might matter is the atomicy boundary,
but mincore is not the right interface for that.

  parent reply	other threads:[~2016-09-13  6:44 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-11 17:31 [RFC PATCH 1/2] mm, mincore2(): retrieve dax and tlb-size attributes of an address range Dan Williams
2016-09-11 17:31 ` Dan Williams
2016-09-11 17:31 ` Dan Williams
2016-09-11 17:31 ` [RFC PATCH 2/2] x86: wire up mincore2() Dan Williams
2016-09-11 17:31   ` Dan Williams
2016-09-11 17:31   ` Dan Williams
2016-09-11 17:31   ` Dan Williams
     [not found]   ` <147361510160.17004.6974628969361614698.stgit-p8uTFz9XbKj2zm6wflaqv1nYeNYlB/vhral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-09-13 18:44     ` Ingo Molnar
2016-09-13 18:44       ` Ingo Molnar
2016-09-13 18:44       ` Ingo Molnar
2016-09-13 18:44       ` Ingo Molnar
     [not found] ` <147361509579.17004.5258725187329709824.stgit-p8uTFz9XbKj2zm6wflaqv1nYeNYlB/vhral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-09-12  3:35   ` [RFC PATCH 1/2] mm, mincore2(): retrieve dax and tlb-size attributes of an address range Nicholas Piggin
2016-09-12  3:35     ` Nicholas Piggin
2016-09-12  3:35     ` Nicholas Piggin
2016-09-12  3:35     ` Nicholas Piggin
2016-09-12  3:35     ` Nicholas Piggin
2016-09-12 17:29     ` Dan Williams
2016-09-12 17:29       ` Dan Williams
2016-09-12 17:29       ` Dan Williams
2016-09-12 17:29       ` Dan Williams
     [not found]       ` <CAPcyv4hS7i1DApKPDB5PkfBNZVbk321FgP94kUDjmuyGXDidZg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-09-13  2:16         ` Nicholas Piggin
2016-09-13  2:16           ` Nicholas Piggin
2016-09-13  2:16           ` Nicholas Piggin
2016-09-13  2:16           ` Nicholas Piggin
2016-09-13  2:16           ` Nicholas Piggin
     [not found]           ` <20160913121645.652e6512-a5aMA/AkCkgK5Ils6ZIQy0EOCMrvLtNR@public.gmane.org>
2016-09-13  3:49             ` Dan Williams
2016-09-13  3:49               ` Dan Williams
2016-09-13  3:49               ` Dan Williams
2016-09-13  3:49               ` Dan Williams
2016-09-12  6:29   ` Oliver O'Halloran
2016-09-12  6:29     ` Oliver O'Halloran
2016-09-12  6:29     ` Oliver O'Halloran
     [not found]     ` <CAOSf1CHKY7LT0z+wpo7jUy3aYUDHCKDKwF0XoMwpKN4JwfYjeA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-09-12 17:25       ` Dan Williams
2016-09-12 17:25         ` Dan Williams
2016-09-12 17:25         ` Dan Williams
2016-09-12 17:25         ` Dan Williams
2016-09-12 10:09   ` Kirill A. Shutemov
2016-09-12 10:09     ` Kirill A. Shutemov
2016-09-12 10:09     ` Kirill A. Shutemov
2016-09-12 10:09     ` Kirill A. Shutemov
2016-09-12 17:15     ` Dan Williams
2016-09-12 17:15       ` Dan Williams
2016-09-12 17:15       ` Dan Williams
2016-09-13  6:44   ` Christoph Hellwig [this message]
2016-09-13  6:44     ` Christoph Hellwig
2016-09-13  6:44     ` Christoph Hellwig

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=20160913064405.GA21069@infradead.org \
    --to=hch-wegcikhe2lqwvfeawa7xhq@public.gmane.org \
    --cc=aarcange-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
    --cc=arnd-r2nGTMty4D4@public.gmane.org \
    --cc=dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=guangrong.xiao-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=kirill.shutemov-VuQAYsv1563Yd54FQh9/CA@public.gmane.org \
    --cc=linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org \
    --cc=linux-nvdimm-y27Ovi1pjclAfugRpC6u6w@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.