From: Pankaj Gupta <pagupta@redhat.com> To: Zhang Yi <yi.z.zhang@linux.intel.com> Cc: jglisse@redhat.com, yu c zhang <yu.c.zhang@intel.com>, kvm@vger.kernel.org, linux-nvdimm@lists.01.org, jack@suse.cz, david@redhat.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, rkrcmar@redhat.com, pbonzini@redhat.com, hch@lst.de, yi z zhang <yi.z.zhang@intel.com> Subject: Re: [PATCH V4 3/4] mm: add a function to differentiate the pages is from DAX device memory Date: Tue, 18 Sep 2018 22:48:02 -0400 (EDT) [thread overview] Message-ID: <373427884.13987814.1537325282516.JavaMail.zimbra@redhat.com> (raw) In-Reply-To: <044309496afbb4121447dff6a453bd6b96d6068d.1534934405.git.yi.z.zhang@linux.intel.com> > > DAX driver hotplug the device memory and move it to memory zone, these > pages will be marked reserved flag, however, some other kernel componet > will misconceive these pages are reserved mmio (ex: we map these dev_dax > or fs_dax pages to kvm for DIMM/NVDIMM backend). Together with the type > MEMORY_DEVICE_FS_DAX, we can use is_dax_page() to differentiate the pages > is DAX device memory or not. > > Signed-off-by: Zhang Yi <yi.z.zhang@linux.intel.com> > Signed-off-by: Zhang Yu <yu.c.zhang@linux.intel.com> > Acked-by: Jan Kara <jack@suse.cz> > --- > include/linux/mm.h | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/include/linux/mm.h b/include/linux/mm.h > index 68a5121..de5cbc3 100644 > --- a/include/linux/mm.h > +++ b/include/linux/mm.h > @@ -889,6 +889,13 @@ static inline bool is_device_public_page(const struct > page *page) > page->pgmap->type == MEMORY_DEVICE_PUBLIC; > } > > +static inline bool is_dax_page(const struct page *page) > +{ > + return is_zone_device_page(page) && > + (page->pgmap->type == MEMORY_DEVICE_FS_DAX || > + page->pgmap->type == MEMORY_DEVICE_DEV_DAX); > +} > + > #else /* CONFIG_DEV_PAGEMAP_OPS */ > static inline void dev_pagemap_get_ops(void) > { > @@ -912,6 +919,11 @@ static inline bool is_device_public_page(const struct > page *page) > { > return false; > } > + > +static inline bool is_dax_page(const struct page *page) > +{ > + return false; > +} > #endif /* CONFIG_DEV_PAGEMAP_OPS */ > > static inline void get_page(struct page *page) > -- Reviewed-by: Pankaj Gupta <pagupta@redhat.com> > 2.7.4 > > _______________________________________________ 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: Pankaj Gupta <pagupta@redhat.com> To: Zhang Yi <yi.z.zhang@linux.intel.com> Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-nvdimm@lists.01.org, pbonzini@redhat.com, dan j williams <dan.j.williams@intel.com>, dave jiang <dave.jiang@intel.com>, yu c zhang <yu.c.zhang@intel.com>, david@redhat.com, jack@suse.cz, hch@lst.de, linux-mm@kvack.org, rkrcmar@redhat.com, jglisse@redhat.com, yi z zhang <yi.z.zhang@intel.com> Subject: Re: [PATCH V4 3/4] mm: add a function to differentiate the pages is from DAX device memory Date: Tue, 18 Sep 2018 22:48:02 -0400 (EDT) [thread overview] Message-ID: <373427884.13987814.1537325282516.JavaMail.zimbra@redhat.com> (raw) In-Reply-To: <044309496afbb4121447dff6a453bd6b96d6068d.1534934405.git.yi.z.zhang@linux.intel.com> > > DAX driver hotplug the device memory and move it to memory zone, these > pages will be marked reserved flag, however, some other kernel componet > will misconceive these pages are reserved mmio (ex: we map these dev_dax > or fs_dax pages to kvm for DIMM/NVDIMM backend). Together with the type > MEMORY_DEVICE_FS_DAX, we can use is_dax_page() to differentiate the pages > is DAX device memory or not. > > Signed-off-by: Zhang Yi <yi.z.zhang@linux.intel.com> > Signed-off-by: Zhang Yu <yu.c.zhang@linux.intel.com> > Acked-by: Jan Kara <jack@suse.cz> > --- > include/linux/mm.h | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/include/linux/mm.h b/include/linux/mm.h > index 68a5121..de5cbc3 100644 > --- a/include/linux/mm.h > +++ b/include/linux/mm.h > @@ -889,6 +889,13 @@ static inline bool is_device_public_page(const struct > page *page) > page->pgmap->type == MEMORY_DEVICE_PUBLIC; > } > > +static inline bool is_dax_page(const struct page *page) > +{ > + return is_zone_device_page(page) && > + (page->pgmap->type == MEMORY_DEVICE_FS_DAX || > + page->pgmap->type == MEMORY_DEVICE_DEV_DAX); > +} > + > #else /* CONFIG_DEV_PAGEMAP_OPS */ > static inline void dev_pagemap_get_ops(void) > { > @@ -912,6 +919,11 @@ static inline bool is_device_public_page(const struct > page *page) > { > return false; > } > + > +static inline bool is_dax_page(const struct page *page) > +{ > + return false; > +} > #endif /* CONFIG_DEV_PAGEMAP_OPS */ > > static inline void get_page(struct page *page) > -- Reviewed-by: Pankaj Gupta <pagupta@redhat.com> > 2.7.4 > >
next prev parent reply other threads:[~2018-09-19 2:48 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-08-22 10:55 [PATCH V4 0/4] Fix kvm misconceives NVDIMM pages as reserved mmio Zhang Yi 2018-08-22 10:55 ` Zhang Yi 2018-08-22 10:55 ` Zhang Yi 2018-08-22 10:55 ` [PATCH V4 1/4] kvm: remove redundant reserved page check Zhang Yi 2018-08-22 10:55 ` Zhang Yi 2018-08-22 10:55 ` Zhang Yi 2018-08-22 10:56 ` [PATCH V4 2/4] mm: introduce memory type MEMORY_DEVICE_DEV_DAX Zhang Yi 2018-08-22 10:56 ` Zhang Yi 2018-08-22 10:56 ` Zhang Yi 2018-09-19 2:43 ` Pankaj Gupta 2018-09-19 2:43 ` Pankaj Gupta 2018-09-19 2:43 ` Pankaj Gupta 2018-08-22 10:56 ` [PATCH V4 3/4] mm: add a function to differentiate the pages is from DAX device memory Zhang Yi 2018-08-22 10:56 ` Zhang Yi 2018-08-22 10:56 ` Zhang Yi 2018-09-19 2:48 ` Pankaj Gupta [this message] 2018-09-19 2:48 ` Pankaj Gupta 2018-08-22 10:58 ` [PATCH V4 4/4] kvm: add a check if pfn is from NVDIMM pmem Zhang Yi 2018-08-22 10:58 ` Zhang Yi 2018-08-22 10:58 ` Zhang Yi 2018-08-29 10:15 ` Pankaj Gupta 2018-08-29 10:15 ` Pankaj Gupta 2018-08-29 10:15 ` Pankaj Gupta 2018-08-30 19:23 ` Yi Zhang 2018-08-30 19:23 ` Yi Zhang 2018-08-30 19:23 ` Yi Zhang 2018-08-30 19:07 ` Dave Hansen 2018-08-30 19:07 ` Dave Hansen 2018-08-30 19:07 ` Dave Hansen 2018-08-31 16:39 ` Yi Zhang 2018-08-31 16:39 ` Yi Zhang 2018-08-31 16:39 ` Yi Zhang
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=373427884.13987814.1537325282516.JavaMail.zimbra@redhat.com \ --to=pagupta@redhat.com \ --cc=david@redhat.com \ --cc=hch@lst.de \ --cc=jack@suse.cz \ --cc=jglisse@redhat.com \ --cc=kvm@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=linux-nvdimm@lists.01.org \ --cc=pbonzini@redhat.com \ --cc=rkrcmar@redhat.com \ --cc=yi.z.zhang@intel.com \ --cc=yi.z.zhang@linux.intel.com \ --cc=yu.c.zhang@intel.com \ /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: linkBe 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.