From: Ira Weiny <ira.weiny@intel.com> To: Christoph Hellwig <hch@lst.de> Cc: linux-nvdimm@lists.01.org, nouveau@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-mm@kvack.org, "Jérôme Glisse" <jglisse@redhat.com>, "Jason Gunthorpe" <jgg@mellanox.com>, "Ben Skeggs" <bskeggs@redhat.com>, linux-pci@vger.kernel.org Subject: Re: [PATCH 23/25] mm: sort out the DEVICE_PRIVATE Kconfig mess Date: Wed, 26 Jun 2019 14:36:21 -0700 [thread overview] Message-ID: <20190626213620.GA8399@iweiny-DESK2.sc.intel.com> (raw) In-Reply-To: <20190626122724.13313-24-hch@lst.de> On Wed, Jun 26, 2019 at 02:27:22PM +0200, Christoph Hellwig wrote: > The ZONE_DEVICE support doesn't depend on anything HMM related, just on > various bits of arch support as indicated by the architecture. Also > don't select the option from nouveau as it isn't present in many setups, > and depend on it instead. > > Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Ira Weiny <ira.weiny@intel.com> > --- > drivers/gpu/drm/nouveau/Kconfig | 2 +- > mm/Kconfig | 5 ++--- > 2 files changed, 3 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig > index dba2613f7180..6303d203ab1d 100644 > --- a/drivers/gpu/drm/nouveau/Kconfig > +++ b/drivers/gpu/drm/nouveau/Kconfig > @@ -85,10 +85,10 @@ config DRM_NOUVEAU_BACKLIGHT > config DRM_NOUVEAU_SVM > bool "(EXPERIMENTAL) Enable SVM (Shared Virtual Memory) support" > depends on ARCH_HAS_HMM > + depends on DEVICE_PRIVATE > depends on DRM_NOUVEAU > depends on STAGING > select HMM_MIRROR > - select DEVICE_PRIVATE > default n > help > Say Y here if you want to enable experimental support for > diff --git a/mm/Kconfig b/mm/Kconfig > index 6f35b85b3052..eecf037a54b3 100644 > --- a/mm/Kconfig > +++ b/mm/Kconfig > @@ -677,13 +677,13 @@ config ARCH_HAS_HMM_MIRROR > > config ARCH_HAS_HMM > bool > - default y > depends on (X86_64 || PPC64) > depends on ZONE_DEVICE > depends on MMU && 64BIT > depends on MEMORY_HOTPLUG > depends on MEMORY_HOTREMOVE > depends on SPARSEMEM_VMEMMAP > + default y > > config MIGRATE_VMA_HELPER > bool > @@ -709,8 +709,7 @@ config HMM_MIRROR > > config DEVICE_PRIVATE > bool "Unaddressable device memory (GPU memory, ...)" > - depends on ARCH_HAS_HMM > - select HMM > + depends on ZONE_DEVICE > select DEV_PAGEMAP_OPS > > help > -- > 2.20.1 > _______________________________________________ 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: Ira Weiny <ira.weiny@intel.com> To: Christoph Hellwig <hch@lst.de> Cc: "Dan Williams" <dan.j.williams@intel.com>, "Jérôme Glisse" <jglisse@redhat.com>, "Jason Gunthorpe" <jgg@mellanox.com>, "Ben Skeggs" <bskeggs@redhat.com>, linux-mm@kvack.org, nouveau@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-nvdimm@lists.01.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 23/25] mm: sort out the DEVICE_PRIVATE Kconfig mess Date: Wed, 26 Jun 2019 14:36:21 -0700 [thread overview] Message-ID: <20190626213620.GA8399@iweiny-DESK2.sc.intel.com> (raw) In-Reply-To: <20190626122724.13313-24-hch@lst.de> On Wed, Jun 26, 2019 at 02:27:22PM +0200, Christoph Hellwig wrote: > The ZONE_DEVICE support doesn't depend on anything HMM related, just on > various bits of arch support as indicated by the architecture. Also > don't select the option from nouveau as it isn't present in many setups, > and depend on it instead. > > Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: Ira Weiny <ira.weiny@intel.com> > --- > drivers/gpu/drm/nouveau/Kconfig | 2 +- > mm/Kconfig | 5 ++--- > 2 files changed, 3 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/Kconfig b/drivers/gpu/drm/nouveau/Kconfig > index dba2613f7180..6303d203ab1d 100644 > --- a/drivers/gpu/drm/nouveau/Kconfig > +++ b/drivers/gpu/drm/nouveau/Kconfig > @@ -85,10 +85,10 @@ config DRM_NOUVEAU_BACKLIGHT > config DRM_NOUVEAU_SVM > bool "(EXPERIMENTAL) Enable SVM (Shared Virtual Memory) support" > depends on ARCH_HAS_HMM > + depends on DEVICE_PRIVATE > depends on DRM_NOUVEAU > depends on STAGING > select HMM_MIRROR > - select DEVICE_PRIVATE > default n > help > Say Y here if you want to enable experimental support for > diff --git a/mm/Kconfig b/mm/Kconfig > index 6f35b85b3052..eecf037a54b3 100644 > --- a/mm/Kconfig > +++ b/mm/Kconfig > @@ -677,13 +677,13 @@ config ARCH_HAS_HMM_MIRROR > > config ARCH_HAS_HMM > bool > - default y > depends on (X86_64 || PPC64) > depends on ZONE_DEVICE > depends on MMU && 64BIT > depends on MEMORY_HOTPLUG > depends on MEMORY_HOTREMOVE > depends on SPARSEMEM_VMEMMAP > + default y > > config MIGRATE_VMA_HELPER > bool > @@ -709,8 +709,7 @@ config HMM_MIRROR > > config DEVICE_PRIVATE > bool "Unaddressable device memory (GPU memory, ...)" > - depends on ARCH_HAS_HMM > - select HMM > + depends on ZONE_DEVICE > select DEV_PAGEMAP_OPS > > help > -- > 2.20.1 >
next prev parent reply other threads:[~2019-06-26 21:36 UTC|newest] Thread overview: 142+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-06-26 12:26 dev_pagemap related cleanups v3 Christoph Hellwig 2019-06-26 12:26 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 01/25] mm: remove the unused ARCH_HAS_HMM_DEVICE Kconfig option Christoph Hellwig 2019-06-26 12:27 ` [PATCH 02/25] mm: remove the struct hmm_device infrastructure Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 03/25] mm: remove hmm_devmem_add_resource Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-27 16:18 ` Jason Gunthorpe 2019-06-27 16:54 ` Christoph Hellwig 2019-06-27 16:54 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 04/25] mm: remove MEMORY_DEVICE_PUBLIC support Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 16:00 ` Dan Williams 2019-06-26 16:00 ` Dan Williams 2019-06-26 17:14 ` Ira Weiny 2019-06-26 17:14 ` Ira Weiny 2019-06-26 17:14 ` Ira Weiny 2019-06-26 18:49 ` Ira Weiny 2019-06-26 18:49 ` Ira Weiny 2019-06-26 18:49 ` Ira Weiny 2019-06-26 12:27 ` [PATCH 05/25] mm: don't clear ->mapping in hmm_devmem_free Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 06/25] mm: export alloc_pages_vma Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:36 ` Michal Hocko 2019-06-26 12:36 ` Michal Hocko 2019-06-26 12:27 ` [PATCH 07/25] mm: factor out a devm_request_free_mem_region helper Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 08/25] memremap: validate the pagemap type passed to devm_memremap_pages Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 18:01 ` Ira Weiny 2019-06-26 18:01 ` Ira Weiny 2019-06-26 12:27 ` [PATCH 09/25] memremap: move dev_pagemap callbacks into a separate structure Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 10/25] memremap: pass a struct dev_pagemap to ->kill and ->cleanup Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 11/25] memremap: lift the devmap_enable manipulation into devm_memremap_pages Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 19:04 ` Ira Weiny 2019-06-26 19:04 ` Ira Weiny 2019-06-26 19:04 ` Ira Weiny 2019-06-27 8:50 ` Christoph Hellwig 2019-06-27 8:50 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 12/25] memremap: add a migrate_to_ram method to struct dev_pagemap_ops Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-27 16:29 ` Jason Gunthorpe 2019-06-27 16:29 ` Jason Gunthorpe 2019-06-27 16:53 ` Christoph Hellwig 2019-06-27 16:53 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 13/25] memremap: remove the data field in struct dev_pagemap Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 14/25] memremap: replace the altmap_valid field with a PGMAP_ALTMAP_VALID flag Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 21:13 ` Ira Weiny 2019-06-26 21:13 ` Ira Weiny 2019-06-26 12:27 ` [PATCH 15/25] memremap: provide an optional internal refcount in struct dev_pagemap Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 21:47 ` Ira Weiny 2019-06-26 21:47 ` Ira Weiny 2019-06-27 8:51 ` Christoph Hellwig 2019-06-27 8:51 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 16/25] device-dax: use the dev_pagemap internal refcount Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 21:48 ` Ira Weiny 2019-06-26 21:48 ` Ira Weiny 2019-06-26 21:48 ` Ira Weiny 2019-06-28 15:38 ` Jason Gunthorpe 2019-06-28 16:27 ` Dan Williams 2019-06-28 16:27 ` Dan Williams 2019-06-28 17:02 ` Jason Gunthorpe 2019-06-28 17:02 ` Jason Gunthorpe 2019-06-28 17:08 ` Dan Williams 2019-06-28 17:10 ` Dan Williams 2019-06-28 17:10 ` Dan Williams 2019-06-28 18:29 ` Jason Gunthorpe 2019-06-28 18:29 ` Jason Gunthorpe 2019-06-28 18:44 ` Dan Williams 2019-06-28 18:51 ` Christoph Hellwig 2019-06-28 18:51 ` Christoph Hellwig 2019-06-28 18:59 ` Dan Williams 2019-06-28 18:59 ` Dan Williams 2019-06-28 19:02 ` Christoph Hellwig 2019-06-28 19:02 ` Christoph Hellwig 2019-06-28 19:14 ` Dan Williams 2019-06-28 19:14 ` Dan Williams 2019-06-28 19:14 ` Dan Williams 2019-07-02 22:35 ` Andrew Morton 2019-07-02 22:35 ` Andrew Morton 2019-07-02 22:35 ` Andrew Morton 2019-06-26 12:27 ` [PATCH 17/25] PCI/P2PDMA: " Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 21:49 ` Ira Weiny 2019-06-26 21:49 ` Ira Weiny 2019-06-26 21:49 ` Ira Weiny 2019-06-27 18:49 ` Logan Gunthorpe 2019-06-26 12:27 ` [PATCH 18/25] nouveau: use alloc_page_vma directly Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 19/25] nouveau: use devm_memremap_pages directly Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 20/25] mm: remove hmm_vma_alloc_locked_page Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-27 16:26 ` Jason Gunthorpe 2019-06-26 12:27 ` [PATCH 21/25] mm: remove hmm_devmem_add Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 22/25] mm: simplify ZONE_DEVICE page private data Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` [PATCH 23/25] mm: sort out the DEVICE_PRIVATE Kconfig mess Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 21:36 ` Ira Weiny [this message] 2019-06-26 21:36 ` Ira Weiny 2019-06-26 12:27 ` [PATCH 24/25] mm: remove the HMM config option Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 21:38 ` Ira Weiny 2019-06-26 21:38 ` Ira Weiny 2019-06-26 21:38 ` Ira Weiny 2019-06-27 16:29 ` Jason Gunthorpe 2019-06-26 12:27 ` [PATCH 25/25] mm: don't select MIGRATE_VMA_HELPER from HMM_MIRROR Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig 2019-06-26 12:27 ` Christoph Hellwig -- strict thread matches above, loose matches on Subject: below -- 2019-06-17 12:27 dev_pagemap related cleanups v2 Christoph Hellwig 2019-06-17 12:27 ` [PATCH 23/25] mm: sort out the DEVICE_PRIVATE Kconfig mess Christoph Hellwig 2019-06-17 12:27 ` Christoph Hellwig 2019-06-17 12:27 ` 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=20190626213620.GA8399@iweiny-DESK2.sc.intel.com \ --to=ira.weiny@intel.com \ --cc=bskeggs@redhat.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=hch@lst.de \ --cc=jgg@mellanox.com \ --cc=jglisse@redhat.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=linux-nvdimm@lists.01.org \ --cc=linux-pci@vger.kernel.org \ --cc=nouveau@lists.freedesktop.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: 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.