All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: Andrew Morton <akpm@linux-foundation.org>,
	Dan Williams <dan.j.williams@intel.com>
Cc: "Felix Kuehling" <Felix.Kuehling@amd.com>,
	"Alex Deucher" <alexander.deucher@amd.com>,
	"Christian König" <christian.koenig@amd.com>,
	"Pan, Xinhui" <Xinhui.Pan@amd.com>,
	"Ben Skeggs" <bskeggs@redhat.com>,
	"Karol Herbst" <kherbst@redhat.com>,
	"Lyude Paul" <lyude@redhat.com>, "Jason Gunthorpe" <jgg@ziepe.ca>,
	"Alistair Popple" <apopple@nvidia.com>,
	"Logan Gunthorpe" <logang@deltatee.com>,
	"Ralph Campbell" <rcampbell@nvidia.com>,
	linux-kernel@vger.kernel.org, amd-gfx@lists.freedesktop.org,
	dri-devel@lists.freedesktop.org, nouveau@lists.freedesktop.org,
	nvdimm@lists.linux.dev, linux-mm@kvack.org
Subject: [PATCH 3/8] mm: remove pointless includes from <linux/hmm.h>
Date: Mon,  7 Feb 2022 07:32:44 +0100	[thread overview]
Message-ID: <20220207063249.1833066-4-hch@lst.de> (raw)
In-Reply-To: <20220207063249.1833066-1-hch@lst.de>

hmm.h pulls in the world for no good reason at all.  Remove the
includes and push a few ones into the users instead.

Signed-off-by: Christoph Hellwig <hch@lst.de>
---
 drivers/gpu/drm/amd/amdkfd/kfd_migrate.c | 1 +
 drivers/gpu/drm/nouveau/nouveau_dmem.c   | 1 +
 include/linux/hmm.h                      | 9 ++-------
 lib/test_hmm.c                           | 2 ++
 4 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
index ed5385137f4831..cb835f95a76e66 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
@@ -24,6 +24,7 @@
 #include <linux/hmm.h>
 #include <linux/dma-direction.h>
 #include <linux/dma-mapping.h>
+#include <linux/migrate.h>
 #include "amdgpu_sync.h"
 #include "amdgpu_object.h"
 #include "amdgpu_vm.h"
diff --git a/drivers/gpu/drm/nouveau/nouveau_dmem.c b/drivers/gpu/drm/nouveau/nouveau_dmem.c
index 3828aafd3ac46f..e886a3b9e08c7d 100644
--- a/drivers/gpu/drm/nouveau/nouveau_dmem.c
+++ b/drivers/gpu/drm/nouveau/nouveau_dmem.c
@@ -39,6 +39,7 @@
 
 #include <linux/sched/mm.h>
 #include <linux/hmm.h>
+#include <linux/migrate.h>
 
 /*
  * FIXME: this is ugly right now we are using TTM to allocate vram and we pin
diff --git a/include/linux/hmm.h b/include/linux/hmm.h
index 2fd2e91d5107c0..d5a6f101f843e6 100644
--- a/include/linux/hmm.h
+++ b/include/linux/hmm.h
@@ -9,14 +9,9 @@
 #ifndef LINUX_HMM_H
 #define LINUX_HMM_H
 
-#include <linux/kconfig.h>
-#include <linux/pgtable.h>
+#include <linux/mm.h>
 
-#include <linux/device.h>
-#include <linux/migrate.h>
-#include <linux/memremap.h>
-#include <linux/completion.h>
-#include <linux/mmu_notifier.h>
+struct mmu_interval_notifier;
 
 /*
  * On output:
diff --git a/lib/test_hmm.c b/lib/test_hmm.c
index 767538089a62e4..396beee6b061d4 100644
--- a/lib/test_hmm.c
+++ b/lib/test_hmm.c
@@ -26,6 +26,8 @@
 #include <linux/sched/mm.h>
 #include <linux/platform_device.h>
 #include <linux/rmap.h>
+#include <linux/mmu_notifier.h>
+#include <linux/migrate.h>
 
 #include "test_hmm_uapi.h"
 
-- 
2.30.2


WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@lst.de>
To: Andrew Morton <akpm@linux-foundation.org>,
	Dan Williams <dan.j.williams@intel.com>
Cc: nvdimm@lists.linux.dev, "Ralph Campbell" <rcampbell@nvidia.com>,
	"Alistair Popple" <apopple@nvidia.com>,
	dri-devel@lists.freedesktop.org, linux-mm@kvack.org,
	nouveau@lists.freedesktop.org,
	"Felix Kuehling" <Felix.Kuehling@amd.com>,
	"Pan, Xinhui" <Xinhui.Pan@amd.com>,
	linux-kernel@vger.kernel.org, amd-gfx@lists.freedesktop.org,
	"Jason Gunthorpe" <jgg@ziepe.ca>,
	"Ben Skeggs" <bskeggs@redhat.com>,
	"Alex Deucher" <alexander.deucher@amd.com>,
	"Logan Gunthorpe" <logang@deltatee.com>,
	"Christian König" <christian.koenig@amd.com>
Subject: [Nouveau] [PATCH 3/8] mm: remove pointless includes from <linux/hmm.h>
Date: Mon,  7 Feb 2022 07:32:44 +0100	[thread overview]
Message-ID: <20220207063249.1833066-4-hch@lst.de> (raw)
In-Reply-To: <20220207063249.1833066-1-hch@lst.de>

hmm.h pulls in the world for no good reason at all.  Remove the
includes and push a few ones into the users instead.

Signed-off-by: Christoph Hellwig <hch@lst.de>
---
 drivers/gpu/drm/amd/amdkfd/kfd_migrate.c | 1 +
 drivers/gpu/drm/nouveau/nouveau_dmem.c   | 1 +
 include/linux/hmm.h                      | 9 ++-------
 lib/test_hmm.c                           | 2 ++
 4 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
index ed5385137f4831..cb835f95a76e66 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
@@ -24,6 +24,7 @@
 #include <linux/hmm.h>
 #include <linux/dma-direction.h>
 #include <linux/dma-mapping.h>
+#include <linux/migrate.h>
 #include "amdgpu_sync.h"
 #include "amdgpu_object.h"
 #include "amdgpu_vm.h"
diff --git a/drivers/gpu/drm/nouveau/nouveau_dmem.c b/drivers/gpu/drm/nouveau/nouveau_dmem.c
index 3828aafd3ac46f..e886a3b9e08c7d 100644
--- a/drivers/gpu/drm/nouveau/nouveau_dmem.c
+++ b/drivers/gpu/drm/nouveau/nouveau_dmem.c
@@ -39,6 +39,7 @@
 
 #include <linux/sched/mm.h>
 #include <linux/hmm.h>
+#include <linux/migrate.h>
 
 /*
  * FIXME: this is ugly right now we are using TTM to allocate vram and we pin
diff --git a/include/linux/hmm.h b/include/linux/hmm.h
index 2fd2e91d5107c0..d5a6f101f843e6 100644
--- a/include/linux/hmm.h
+++ b/include/linux/hmm.h
@@ -9,14 +9,9 @@
 #ifndef LINUX_HMM_H
 #define LINUX_HMM_H
 
-#include <linux/kconfig.h>
-#include <linux/pgtable.h>
+#include <linux/mm.h>
 
-#include <linux/device.h>
-#include <linux/migrate.h>
-#include <linux/memremap.h>
-#include <linux/completion.h>
-#include <linux/mmu_notifier.h>
+struct mmu_interval_notifier;
 
 /*
  * On output:
diff --git a/lib/test_hmm.c b/lib/test_hmm.c
index 767538089a62e4..396beee6b061d4 100644
--- a/lib/test_hmm.c
+++ b/lib/test_hmm.c
@@ -26,6 +26,8 @@
 #include <linux/sched/mm.h>
 #include <linux/platform_device.h>
 #include <linux/rmap.h>
+#include <linux/mmu_notifier.h>
+#include <linux/migrate.h>
 
 #include "test_hmm_uapi.h"
 
-- 
2.30.2


WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@lst.de>
To: Andrew Morton <akpm@linux-foundation.org>,
	Dan Williams <dan.j.williams@intel.com>
Cc: nvdimm@lists.linux.dev, "Ralph Campbell" <rcampbell@nvidia.com>,
	"Alistair Popple" <apopple@nvidia.com>,
	dri-devel@lists.freedesktop.org,
	"Karol Herbst" <kherbst@redhat.com>,
	linux-mm@kvack.org, nouveau@lists.freedesktop.org,
	"Felix Kuehling" <Felix.Kuehling@amd.com>,
	"Pan, Xinhui" <Xinhui.Pan@amd.com>,
	linux-kernel@vger.kernel.org, amd-gfx@lists.freedesktop.org,
	"Jason Gunthorpe" <jgg@ziepe.ca>,
	"Ben Skeggs" <bskeggs@redhat.com>,
	"Alex Deucher" <alexander.deucher@amd.com>,
	"Logan Gunthorpe" <logang@deltatee.com>,
	"Christian König" <christian.koenig@amd.com>
Subject: [PATCH 3/8] mm: remove pointless includes from <linux/hmm.h>
Date: Mon,  7 Feb 2022 07:32:44 +0100	[thread overview]
Message-ID: <20220207063249.1833066-4-hch@lst.de> (raw)
In-Reply-To: <20220207063249.1833066-1-hch@lst.de>

hmm.h pulls in the world for no good reason at all.  Remove the
includes and push a few ones into the users instead.

Signed-off-by: Christoph Hellwig <hch@lst.de>
---
 drivers/gpu/drm/amd/amdkfd/kfd_migrate.c | 1 +
 drivers/gpu/drm/nouveau/nouveau_dmem.c   | 1 +
 include/linux/hmm.h                      | 9 ++-------
 lib/test_hmm.c                           | 2 ++
 4 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
index ed5385137f4831..cb835f95a76e66 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
@@ -24,6 +24,7 @@
 #include <linux/hmm.h>
 #include <linux/dma-direction.h>
 #include <linux/dma-mapping.h>
+#include <linux/migrate.h>
 #include "amdgpu_sync.h"
 #include "amdgpu_object.h"
 #include "amdgpu_vm.h"
diff --git a/drivers/gpu/drm/nouveau/nouveau_dmem.c b/drivers/gpu/drm/nouveau/nouveau_dmem.c
index 3828aafd3ac46f..e886a3b9e08c7d 100644
--- a/drivers/gpu/drm/nouveau/nouveau_dmem.c
+++ b/drivers/gpu/drm/nouveau/nouveau_dmem.c
@@ -39,6 +39,7 @@
 
 #include <linux/sched/mm.h>
 #include <linux/hmm.h>
+#include <linux/migrate.h>
 
 /*
  * FIXME: this is ugly right now we are using TTM to allocate vram and we pin
diff --git a/include/linux/hmm.h b/include/linux/hmm.h
index 2fd2e91d5107c0..d5a6f101f843e6 100644
--- a/include/linux/hmm.h
+++ b/include/linux/hmm.h
@@ -9,14 +9,9 @@
 #ifndef LINUX_HMM_H
 #define LINUX_HMM_H
 
-#include <linux/kconfig.h>
-#include <linux/pgtable.h>
+#include <linux/mm.h>
 
-#include <linux/device.h>
-#include <linux/migrate.h>
-#include <linux/memremap.h>
-#include <linux/completion.h>
-#include <linux/mmu_notifier.h>
+struct mmu_interval_notifier;
 
 /*
  * On output:
diff --git a/lib/test_hmm.c b/lib/test_hmm.c
index 767538089a62e4..396beee6b061d4 100644
--- a/lib/test_hmm.c
+++ b/lib/test_hmm.c
@@ -26,6 +26,8 @@
 #include <linux/sched/mm.h>
 #include <linux/platform_device.h>
 #include <linux/rmap.h>
+#include <linux/mmu_notifier.h>
+#include <linux/migrate.h>
 
 #include "test_hmm_uapi.h"
 
-- 
2.30.2


  parent reply	other threads:[~2022-02-07  6:33 UTC|newest]

Thread overview: 141+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-07  6:32 start sorting out the ZONE_DEVICE refcount mess Christoph Hellwig
2022-02-07  6:32 ` Christoph Hellwig
2022-02-07  6:32 ` [Nouveau] " Christoph Hellwig
2022-02-07  6:32 ` [PATCH 1/8] mm: remove a pointless CONFIG_ZONE_DEVICE check in memremap_pages Christoph Hellwig
2022-02-07  6:32   ` Christoph Hellwig
2022-02-07  6:32   ` [Nouveau] " Christoph Hellwig
2022-02-07 18:08   ` Dan Williams
2022-02-07 18:08     ` Dan Williams
2022-02-07 18:08     ` [Nouveau] " Dan Williams
2022-02-07 19:22   ` Jason Gunthorpe
2022-02-07 19:22     ` Jason Gunthorpe
2022-02-07 19:22     ` [Nouveau] " Jason Gunthorpe
2022-02-08  7:17   ` Chaitanya Kulkarni
2022-02-08  7:17     ` [Nouveau] " Chaitanya Kulkarni
2022-02-08  7:17     ` Chaitanya Kulkarni
2022-02-08  8:06   ` Muchun Song
2022-02-08  8:06     ` [Nouveau] " Muchun Song
2022-02-08  8:06     ` Muchun Song
2022-02-07  6:32 ` [PATCH 2/8] mm: remove the __KERNEL__ guard from <linux/mm.h> Christoph Hellwig
2022-02-07  6:32   ` Christoph Hellwig
2022-02-07  6:32   ` [Nouveau] " Christoph Hellwig
2022-02-07 18:08   ` Dan Williams
2022-02-07 18:08     ` Dan Williams
2022-02-07 18:08     ` [Nouveau] " Dan Williams
2022-02-07 19:26   ` Jason Gunthorpe
2022-02-07 19:26     ` [Nouveau] " Jason Gunthorpe
2022-02-07 19:26     ` Jason Gunthorpe
2022-02-08  7:21   ` Chaitanya Kulkarni
2022-02-08  7:21     ` [Nouveau] " Chaitanya Kulkarni
2022-02-08  7:21     ` Chaitanya Kulkarni
2022-02-08  8:07   ` Muchun Song
2022-02-08  8:07     ` [Nouveau] " Muchun Song
2022-02-08  8:07     ` Muchun Song
2022-02-07  6:32 ` Christoph Hellwig [this message]
2022-02-07  6:32   ` [PATCH 3/8] mm: remove pointless includes from <linux/hmm.h> Christoph Hellwig
2022-02-07  6:32   ` [Nouveau] " Christoph Hellwig
2022-02-07 14:01   ` Jason Gunthorpe
2022-02-07 14:01     ` Jason Gunthorpe
2022-02-07 14:01     ` [Nouveau] " Jason Gunthorpe
2022-02-08  7:27   ` Chaitanya Kulkarni
2022-02-08  7:27     ` [Nouveau] " Chaitanya Kulkarni
2022-02-08  7:27     ` Chaitanya Kulkarni
2022-02-07  6:32 ` [PATCH 4/8] mm: move free_devmap_managed_page to memremap.c Christoph Hellwig
2022-02-07  6:32   ` Christoph Hellwig
2022-02-07  6:32   ` [Nouveau] " Christoph Hellwig
2022-02-07 19:06   ` Dan Williams
2022-02-07 19:06     ` Dan Williams
2022-02-07 19:06     ` [Nouveau] " Dan Williams
2022-02-07 19:27   ` Jason Gunthorpe
2022-02-07 19:27     ` Jason Gunthorpe
2022-02-07 19:27     ` [Nouveau] " Jason Gunthorpe
2022-02-08  7:34   ` Chaitanya Kulkarni
2022-02-08  7:34     ` [Nouveau] " Chaitanya Kulkarni
2022-02-08  7:34     ` Chaitanya Kulkarni
2022-02-08  8:09   ` Muchun Song
2022-02-08  8:09     ` [Nouveau] " Muchun Song
2022-02-08  8:09     ` Muchun Song
2022-02-07  6:32 ` [PATCH 5/8] mm: simplify freeing of devmap managed pages Christoph Hellwig
2022-02-07  6:32   ` Christoph Hellwig
2022-02-07  6:32   ` [Nouveau] " Christoph Hellwig
2022-02-07 19:34   ` Jason Gunthorpe
2022-02-07 19:34     ` Jason Gunthorpe
2022-02-07 19:34     ` [Nouveau] " Jason Gunthorpe
2022-02-07 23:42   ` Dan Williams
2022-02-07 23:42     ` Dan Williams
2022-02-07 23:42     ` [Nouveau] " Dan Williams
2022-02-08  7:50   ` Chaitanya Kulkarni
2022-02-08  7:50     ` [Nouveau] " Chaitanya Kulkarni
2022-02-08  7:50     ` Chaitanya Kulkarni
2022-02-07  6:32 ` [PATCH 6/8] mm: don't include <linux/memremap.h> in <linux/mm.h> Christoph Hellwig
2022-02-07  6:32   ` Christoph Hellwig
2022-02-07  6:32   ` [Nouveau] " Christoph Hellwig
2022-02-07 17:38   ` Logan Gunthorpe
2022-02-07 17:38     ` Logan Gunthorpe
2022-02-07 17:38     ` [Nouveau] " Logan Gunthorpe
2022-02-07 19:35   ` Jason Gunthorpe
2022-02-07 19:35     ` Jason Gunthorpe
2022-02-07 19:35     ` [Nouveau] " Jason Gunthorpe
2022-02-07 21:19   ` Felix Kuehling
2022-02-07 21:19     ` Felix Kuehling
2022-02-07 21:19     ` [Nouveau] " Felix Kuehling
2022-02-08  6:46     ` Christoph Hellwig
2022-02-08  6:46       ` Christoph Hellwig
2022-02-08  6:46       ` [Nouveau] " Christoph Hellwig
2022-02-09 17:48     ` Christoph Hellwig
2022-02-09 17:48       ` Christoph Hellwig
2022-02-09 17:48       ` [Nouveau] " Christoph Hellwig
2022-02-10  2:10       ` Alistair Popple
2022-02-10  2:10         ` Alistair Popple
2022-02-10  2:10         ` [Nouveau] " Alistair Popple
2022-02-10  6:45         ` Christoph Hellwig
2022-02-10  6:45           ` Christoph Hellwig
2022-02-10  6:45           ` [Nouveau] " Christoph Hellwig
2022-02-10 21:00       ` Felix Kuehling
2022-02-10 21:00         ` Felix Kuehling
2022-02-10 21:00         ` [Nouveau] " Felix Kuehling
2022-02-07 23:49   ` Dan Williams
2022-02-07 23:49     ` Dan Williams
2022-02-07 23:49     ` [Nouveau] " Dan Williams
2022-02-08 23:53     ` Dan Williams
2022-02-08 23:53       ` Dan Williams
2022-02-08 23:53       ` [Nouveau] " Dan Williams
2022-02-09  6:22       ` Christoph Hellwig
2022-02-09  6:22         ` Christoph Hellwig
2022-02-09  6:22         ` [Nouveau] " Christoph Hellwig
2022-02-07  6:32 ` [PATCH 7/8] mm: remove the extra ZONE_DEVICE struct page refcount Christoph Hellwig
2022-02-07  6:32   ` Christoph Hellwig
2022-02-07  6:32   ` [Nouveau] " Christoph Hellwig
2022-02-07 19:21   ` Jason Gunthorpe
2022-02-07 19:21     ` Jason Gunthorpe
2022-02-07 19:21     ` [Nouveau] " Jason Gunthorpe
2022-02-08  2:25   ` Ralph Campbell
2022-02-08  2:25     ` Ralph Campbell
2022-02-08  2:25     ` [Nouveau] " Ralph Campbell
2022-02-09  3:30   ` Dan Williams
2022-02-09  3:30     ` Dan Williams
2022-02-09  3:30     ` [Nouveau] " Dan Williams
2022-02-09  6:23     ` Christoph Hellwig
2022-02-09  6:23       ` Christoph Hellwig
2022-02-09  6:23       ` [Nouveau] " Christoph Hellwig
2022-02-09 12:29       ` Jason Gunthorpe
2022-02-09 12:29         ` [Nouveau] " Jason Gunthorpe
2022-02-09 12:29         ` Jason Gunthorpe
2022-02-09 13:53         ` Christoph Hellwig
2022-02-09 13:53           ` Christoph Hellwig
2022-02-09 13:53           ` [Nouveau] " Christoph Hellwig
2022-02-09 14:14           ` Jason Gunthorpe
2022-02-09 14:14             ` Jason Gunthorpe
2022-02-09 14:14             ` [Nouveau] " Jason Gunthorpe
2022-02-07  6:32 ` [PATCH 8/8] fsdax: depend on ZONE_DEVICE || FS_DAX_LIMITED Christoph Hellwig
2022-02-07  6:32   ` Christoph Hellwig
2022-02-07  6:32   ` [Nouveau] " Christoph Hellwig
2022-02-07 19:36   ` Jason Gunthorpe
2022-02-07 19:36     ` Jason Gunthorpe
2022-02-07 19:36     ` [Nouveau] " Jason Gunthorpe
2022-02-07 23:51 ` start sorting out the ZONE_DEVICE refcount mess Logan Gunthorpe
2022-02-07 23:51   ` Logan Gunthorpe
2022-02-07 23:51   ` [Nouveau] " Logan Gunthorpe
2022-02-08  3:03 ` Miaohe Lin
2022-02-08  3:03   ` [Nouveau] " Miaohe Lin
2022-02-08  3:03   ` Miaohe Lin

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=20220207063249.1833066-4-hch@lst.de \
    --to=hch@lst.de \
    --cc=Felix.Kuehling@amd.com \
    --cc=Xinhui.Pan@amd.com \
    --cc=akpm@linux-foundation.org \
    --cc=alexander.deucher@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=apopple@nvidia.com \
    --cc=bskeggs@redhat.com \
    --cc=christian.koenig@amd.com \
    --cc=dan.j.williams@intel.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jgg@ziepe.ca \
    --cc=kherbst@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=logang@deltatee.com \
    --cc=lyude@redhat.com \
    --cc=nouveau@lists.freedesktop.org \
    --cc=nvdimm@lists.linux.dev \
    --cc=rcampbell@nvidia.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: 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.