All of lore.kernel.org
 help / color / mirror / Atom feed
* DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)
@ 2016-09-08  4:32 ` Dan Williams
  0 siblings, 0 replies; 133+ messages in thread
From: Dan Williams @ 2016-09-08  4:32 UTC (permalink / raw)
  To: Xiao Guangrong
  Cc: Yumei Huang, Michal Hocko, KVM list, Linux MM, Gleb Natapov,
	mtosatti, linux-kernel@vger.kernel.org, Dave Hansen,
	Stefan Hajnoczi, linux-fsdevel, Paolo Bonzini, Andrew Morton,
	linux-nvdimm@lists.01.org

[ adding linux-fsdevel and linux-nvdimm ]

On Wed, Sep 7, 2016 at 8:36 PM, Xiao Guangrong
<guangrong.xiao@linux.intel.com> wrote:
[..]
> However, it is not easy to handle the case that the new VMA overlays with
> the old VMA
> already got by userspace. I think we have some choices:
> 1: One way is completely skipping the new VMA region as current kernel code
> does but i
>    do not think this is good as the later VMAs will be dropped.
>
> 2: show the un-overlayed portion of new VMA. In your case, we just show the
> region
>    (0x2000 -> 0x3000), however, it can not work well if the VMA is a new
> created
>    region with different attributions.
>
> 3: completely show the new VMA as this patch does.
>
> Which one do you prefer?
>

I don't have a preference, but perhaps this breakage and uncertainty
is a good opportunity to propose a more reliable interface for NVML to
get the information it needs?

My understanding is that it is looking for the VM_MIXEDMAP flag which
is already ambiguous for determining if DAX is enabled even if this
dynamic listing issue is fixed.  XFS has arranged for DAX to be a
per-inode capability and has an XFS-specific inode flag.  We can make
that a common inode flag, but it seems we should have a way to
interrogate the mapping itself in the case where the inode is unknown
or unavailable.  I'm thinking extensions to mincore to have flags for
DAX and possibly whether the page is part of a pte, pmd, or pud
mapping.  Just floating that idea before starting to look into the
implementation, comments or other ideas welcome...
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm

^ permalink raw reply	[flat|nested] 133+ messages in thread

end of thread, other threads:[~2016-12-21 21:33 UTC | newest]

Thread overview: 133+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-08  4:32 DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps) Dan Williams
2016-09-08  4:32 ` Dan Williams
2016-09-08  4:32 ` Dan Williams
2016-09-08 22:56 ` Ross Zwisler
2016-09-08 22:56   ` Ross Zwisler
2016-09-08 22:56   ` Ross Zwisler
2016-09-08 22:56   ` Ross Zwisler
2016-09-08 23:04   ` Dan Williams
2016-09-08 23:04     ` Dan Williams
2016-09-08 23:04     ` Dan Williams
2016-09-09  8:55     ` Xiao Guangrong
2016-09-09  8:55       ` Xiao Guangrong
2016-09-09  8:55       ` Xiao Guangrong
2016-09-09 15:40       ` Dan Williams
2016-09-09 15:40         ` Dan Williams
2016-09-09 15:40         ` Dan Williams
2016-09-12  6:00         ` Xiao Guangrong
2016-09-12  6:00           ` Xiao Guangrong
2016-09-12  6:00           ` Xiao Guangrong
2016-09-12  6:00           ` Xiao Guangrong
2016-09-12  3:44       ` Rudoff, Andy
2016-09-12  3:44         ` Rudoff, Andy
2016-09-12  3:44         ` Rudoff, Andy
2016-09-12  6:31         ` Xiao Guangrong
2016-09-12  6:31           ` Xiao Guangrong
2016-09-12  6:31           ` Xiao Guangrong
2016-09-12  1:40   ` Dave Chinner
2016-09-12  1:40     ` Dave Chinner
2016-09-12  1:40     ` Dave Chinner
2016-09-12  1:40     ` Dave Chinner
2016-09-15  5:55     ` Darrick J. Wong
2016-09-15  5:55       ` Darrick J. Wong
2016-09-15  5:55       ` Darrick J. Wong
2016-09-15  6:25       ` Dave Chinner
2016-09-15  6:25         ` Dave Chinner
2016-09-15  6:25         ` Dave Chinner
2016-09-12  5:27   ` Christoph Hellwig
2016-09-12  5:27     ` Christoph Hellwig
2016-09-12  5:27     ` Christoph Hellwig
2016-09-12  5:27     ` Christoph Hellwig
2016-09-12  7:25     ` Oliver O'Halloran
2016-09-12  7:25       ` Oliver O'Halloran
2016-09-12  7:25       ` Oliver O'Halloran
2016-09-12  7:51       ` Christoph Hellwig
2016-09-12  7:51         ` Christoph Hellwig
2016-09-12  8:05         ` Nicholas Piggin
2016-09-12  8:05           ` Nicholas Piggin
2016-09-12  8:05           ` Nicholas Piggin
2016-09-12 15:01           ` Christoph Hellwig
2016-09-12 15:01             ` Christoph Hellwig
2016-09-12 15:01             ` Christoph Hellwig
2016-09-12 15:01             ` Christoph Hellwig
2016-09-13  1:31             ` Nicholas Piggin
2016-09-13  1:31               ` Nicholas Piggin
2016-09-13  1:31               ` Nicholas Piggin
2016-09-13  1:31               ` Nicholas Piggin
2016-09-13  4:06               ` Dan Williams
2016-09-13  4:06                 ` Dan Williams
2016-09-13  4:06                 ` Dan Williams
2016-09-13  5:40                 ` Nicholas Piggin
2016-09-13  5:40                   ` Nicholas Piggin
2016-09-13  5:40                   ` Nicholas Piggin
2016-09-12 21:34           ` Dave Chinner
2016-09-12 21:34             ` Dave Chinner
2016-09-12 21:34             ` Dave Chinner
2016-09-13  1:53             ` Nicholas Piggin
2016-09-13  1:53               ` Nicholas Piggin
2016-09-13  1:53               ` Nicholas Piggin
2016-09-13  1:53               ` Nicholas Piggin
2016-09-13  7:17               ` Christoph Hellwig
2016-09-13  7:17                 ` Christoph Hellwig
2016-09-13  7:17                 ` Christoph Hellwig
2016-09-13  9:06                 ` Nicholas Piggin
2016-09-13  9:06                   ` Nicholas Piggin
2016-09-13  9:06                   ` Nicholas Piggin
2016-09-13  9:06                   ` Nicholas Piggin
2016-09-14  7:39               ` Dave Chinner
2016-09-14  7:39                 ` Dave Chinner
2016-09-14  7:39                 ` Dave Chinner
2016-09-14  7:39                 ` Dave Chinner
2016-09-14 10:19                 ` Nicholas Piggin
2016-09-14 10:19                   ` Nicholas Piggin
2016-09-14 10:19                   ` Nicholas Piggin
2016-09-14 10:19                   ` Nicholas Piggin
2016-09-15  2:31                   ` Dave Chinner
2016-09-15  2:31                     ` Dave Chinner
2016-09-15  2:31                     ` Dave Chinner
2016-09-15  2:31                     ` Dave Chinner
2016-09-15  3:49                     ` Nicholas Piggin
2016-09-15  3:49                       ` Nicholas Piggin
2016-09-15  3:49                       ` Nicholas Piggin
2016-09-15  3:49                       ` Nicholas Piggin
2016-09-15 10:32                       ` Dave Chinner
2016-09-15 10:32                         ` Dave Chinner
2016-09-15 10:32                         ` Dave Chinner
2016-09-15 10:32                         ` Dave Chinner
2016-09-15 11:42                         ` Nicholas Piggin
2016-09-15 11:42                           ` Nicholas Piggin
2016-09-15 11:42                           ` Nicholas Piggin
2016-09-15 11:42                           ` Nicholas Piggin
2016-09-15 22:33                           ` Dave Chinner
2016-09-15 22:33                             ` Dave Chinner
2016-09-15 22:33                             ` Dave Chinner
2016-09-15 22:33                             ` Dave Chinner
2016-09-16  5:54                             ` Nicholas Piggin
2016-09-16  5:54                               ` Nicholas Piggin
2016-09-16  5:54                               ` Nicholas Piggin
2016-09-16  5:54                               ` Nicholas Piggin
2016-12-19 21:11                               ` Ross Zwisler
2016-12-19 21:11                                 ` Ross Zwisler
2016-12-19 21:11                                 ` Ross Zwisler
2016-12-19 21:11                                 ` Ross Zwisler
2016-12-19 21:11                                 ` Ross Zwisler
2016-12-20  1:09                                 ` Darrick J. Wong
2016-12-20  1:09                                   ` Darrick J. Wong
2016-12-20  1:09                                   ` Darrick J. Wong
2016-12-20  1:18                                   ` Dan Williams
2016-12-20  1:18                                     ` Dan Williams
2016-12-21  0:40                                     ` Darrick J. Wong
2016-12-21  0:40                                       ` Darrick J. Wong
2016-12-21  0:40                                       ` Darrick J. Wong
2016-12-21  0:40                                       ` Darrick J. Wong
2016-12-21 16:53                                       ` Dan Williams
2016-12-21 16:53                                         ` Dan Williams
2016-12-21 16:53                                         ` Dan Williams
2016-12-21 16:53                                         ` Dan Williams
2016-12-21 21:24                                         ` Dave Chinner
2016-12-21 21:24                                           ` Dave Chinner
2016-12-21 21:24                                           ` Dave Chinner
2016-12-21 21:24                                           ` Dave Chinner
2016-12-21 21:33                                           ` Dan Williams
2016-12-21 21:33                                             ` Dan Williams
2016-12-21 21:33                                             ` Dan Williams

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.