Re: [PATCH] dma-buf: move dma-buf symbols into the DMA_BUF module namespace

From: Daniel Vetter
Date: Tue Sep 28 2021 - 03:31:58 EST

Next message: David Hildenbrand: "Re: [RFC] arm64: mm: update max_pfn after memory hotplug"
Previous message: Ooi, Joyce: "RE: [PATCH] MAINTAINERS: Update Mun Yew Tham as Altera Triple Speed Ethernet Driver maintainer"
In reply to: Alex Deucher: "Re: [PATCH] dma-buf: move dma-buf symbols into the DMA_BUF module namespace"
Next in thread: Greg Kroah-Hartman: "Re: [PATCH] dma-buf: move dma-buf symbols into the DMA_BUF module namespace"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Sat, Sep 25, 2021 at 03:47:00PM +0200, Greg Kroah-Hartman wrote:
> In order to better track where in the kernel the dma-buf code is used,
> put the symbols in the namespace DMA_BUF and modify all users of the
> symbols to properly import the namespace to not break the build at the
> same time.
>
> Now the output of modinfo shows the use of these symbols, making it
> easier to watch for users over time:
>
> $ modinfo drivers/misc/fastrpc.ko | grep import
> import_ns: DMA_BUF
>
> Cc: Sumit Semwal <sumit.semwal@xxxxxxxxxx>
> Cc: "Christian König" <christian.koenig@xxxxxxx>
> Cc: Alex Deucher <alexander.deucher@xxxxxxx>
> Cc: "Pan, Xinhui" <Xinhui.Pan@xxxxxxx>
> Cc: David Airlie <airlied@xxxxxxxx>
> Cc: Daniel Vetter <daniel@xxxxxxxx>
> Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> Cc: Maxime Ripard <mripard@xxxxxxxxxx>
> Cc: Thomas Zimmermann <tzimmermann@xxxxxxx>
> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> Cc: Arnd Bergmann <arnd@xxxxxxxx>
> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> ---
>
> The topic of dma-buf came up in the Maintainer's summit yesterday, and
> one comment was to put the symbols in their own module namespace, to
> make it easier to notice and track who was using them. This patch does
> so, and finds some "interesting" users of the api already in the tree.

Yeah, the interesting ones is why I added the dma-buf wildcard match a
while ago. Since that landed I don't think anything escaped. Should we
perhaps also add

K: MODULE_IMPORT_NS(DMA_BUF);

to the dma-buf MAINATINERS entry? Entirely untested, also no idea whether
there's not a better way to match for module namespaces. Either way:

Acked-by: Daniel Vetter <daniel.vetter@xxxxxxxx>

>
> Only test-built on x86 allmodconfig, don't know what other arches will
> pick up, will let 0-day run on it for a bit...
>
> Comments?
>
>
>
> drivers/dma-buf/dma-buf.c | 34 +++++++++----------
> drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 3 ++
> drivers/gpu/drm/drm_gem_framebuffer_helper.c | 3 ++
> drivers/gpu/drm/drm_prime.c | 3 ++
> drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c | 3 ++
> drivers/gpu/drm/exynos/exynos_drm_gem.c | 3 ++
> drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c | 3 ++
> drivers/gpu/drm/tegra/gem.c | 3 ++
> drivers/gpu/drm/vmwgfx/ttm_object.c | 3 ++
> drivers/infiniband/core/umem_dmabuf.c | 3 ++
> .../media/common/videobuf2/videobuf2-core.c | 1 +
> .../common/videobuf2/videobuf2-dma-contig.c | 1 +
> .../media/common/videobuf2/videobuf2-dma-sg.c | 1 +
> .../common/videobuf2/videobuf2-vmalloc.c | 1 +
> drivers/misc/fastrpc.c | 1 +
> .../staging/media/tegra-vde/dmabuf-cache.c | 3 ++
> drivers/tee/tee_shm.c | 3 ++
> drivers/virtio/virtio_dma_buf.c | 1 +
> drivers/xen/gntdev-dmabuf.c | 3 ++
> samples/vfio-mdev/mbochs.c | 1 +
> 20 files changed, 60 insertions(+), 17 deletions(-)
>
> diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c
> index 63d32261b63f..6c2b5ea828a6 100644
> --- a/drivers/dma-buf/dma-buf.c
> +++ b/drivers/dma-buf/dma-buf.c
> @@ -610,7 +610,7 @@ struct dma_buf *dma_buf_export(const struct dma_buf_export_info *exp_info)
> module_put(exp_info->owner);
> return ERR_PTR(ret);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_export);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_export, DMA_BUF);
>
> /**
> * dma_buf_fd - returns a file descriptor for the given struct dma_buf
> @@ -634,7 +634,7 @@ int dma_buf_fd(struct dma_buf *dmabuf, int flags)
>
> return fd;
> }
> -EXPORT_SYMBOL_GPL(dma_buf_fd);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_fd, DMA_BUF);
>
> /**
> * dma_buf_get - returns the struct dma_buf related to an fd
> @@ -660,7 +660,7 @@ struct dma_buf *dma_buf_get(int fd)
>
> return file->private_data;
> }
> -EXPORT_SYMBOL_GPL(dma_buf_get);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_get, DMA_BUF);
>
> /**
> * dma_buf_put - decreases refcount of the buffer
> @@ -679,7 +679,7 @@ void dma_buf_put(struct dma_buf *dmabuf)
>
> fput(dmabuf->file);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_put);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_put, DMA_BUF);
>
> static void mangle_sg_table(struct sg_table *sg_table)
> {
> @@ -810,7 +810,7 @@ dma_buf_dynamic_attach(struct dma_buf *dmabuf, struct device *dev,
> dma_buf_detach(dmabuf, attach);
> return ERR_PTR(ret);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_dynamic_attach);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_dynamic_attach, DMA_BUF);
>
> /**
> * dma_buf_attach - Wrapper for dma_buf_dynamic_attach
> @@ -825,7 +825,7 @@ struct dma_buf_attachment *dma_buf_attach(struct dma_buf *dmabuf,
> {
> return dma_buf_dynamic_attach(dmabuf, dev, NULL, NULL);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_attach);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_attach, DMA_BUF);
>
> static void __unmap_dma_buf(struct dma_buf_attachment *attach,
> struct sg_table *sg_table,
> @@ -871,7 +871,7 @@ void dma_buf_detach(struct dma_buf *dmabuf, struct dma_buf_attachment *attach)
>
> kfree(attach);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_detach);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_detach, DMA_BUF);
>
> /**
> * dma_buf_pin - Lock down the DMA-buf
> @@ -901,7 +901,7 @@ int dma_buf_pin(struct dma_buf_attachment *attach)
>
> return ret;
> }
> -EXPORT_SYMBOL_GPL(dma_buf_pin);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_pin, DMA_BUF);
>
> /**
> * dma_buf_unpin - Unpin a DMA-buf
> @@ -922,7 +922,7 @@ void dma_buf_unpin(struct dma_buf_attachment *attach)
> if (dmabuf->ops->unpin)
> dmabuf->ops->unpin(attach);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_unpin);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_unpin, DMA_BUF);
>
> /**
> * dma_buf_map_attachment - Returns the scatterlist table of the attachment;
> @@ -1012,7 +1012,7 @@ struct sg_table *dma_buf_map_attachment(struct dma_buf_attachment *attach,
> #endif /* CONFIG_DMA_API_DEBUG */
> return sg_table;
> }
> -EXPORT_SYMBOL_GPL(dma_buf_map_attachment);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_map_attachment, DMA_BUF);
>
> /**
> * dma_buf_unmap_attachment - unmaps and decreases usecount of the buffer;might
> @@ -1048,7 +1048,7 @@ void dma_buf_unmap_attachment(struct dma_buf_attachment *attach,
> !IS_ENABLED(CONFIG_DMABUF_MOVE_NOTIFY))
> dma_buf_unpin(attach);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_unmap_attachment);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_unmap_attachment, DMA_BUF);
>
> /**
> * dma_buf_move_notify - notify attachments that DMA-buf is moving
> @@ -1068,7 +1068,7 @@ void dma_buf_move_notify(struct dma_buf *dmabuf)
> if (attach->importer_ops)
> attach->importer_ops->move_notify(attach);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_move_notify);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_move_notify, DMA_BUF);
>
> /**
> * DOC: cpu access
> @@ -1212,7 +1212,7 @@ int dma_buf_begin_cpu_access(struct dma_buf *dmabuf,
>
> return ret;
> }
> -EXPORT_SYMBOL_GPL(dma_buf_begin_cpu_access);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_begin_cpu_access, DMA_BUF);
>
> /**
> * dma_buf_end_cpu_access - Must be called after accessing a dma_buf from the
> @@ -1240,7 +1240,7 @@ int dma_buf_end_cpu_access(struct dma_buf *dmabuf,
>
> return ret;
> }
> -EXPORT_SYMBOL_GPL(dma_buf_end_cpu_access);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_end_cpu_access, DMA_BUF);
>
>
> /**
> @@ -1282,7 +1282,7 @@ int dma_buf_mmap(struct dma_buf *dmabuf, struct vm_area_struct *vma,
>
> return dmabuf->ops->mmap(dmabuf, vma);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_mmap);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_mmap, DMA_BUF);
>
> /**
> * dma_buf_vmap - Create virtual mapping for the buffer object into kernel
> @@ -1336,7 +1336,7 @@ int dma_buf_vmap(struct dma_buf *dmabuf, struct dma_buf_map *map)
> mutex_unlock(&dmabuf->lock);
> return ret;
> }
> -EXPORT_SYMBOL_GPL(dma_buf_vmap);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_vmap, DMA_BUF);
>
> /**
> * dma_buf_vunmap - Unmap a vmap obtained by dma_buf_vmap.
> @@ -1360,7 +1360,7 @@ void dma_buf_vunmap(struct dma_buf *dmabuf, struct dma_buf_map *map)
> }
> mutex_unlock(&dmabuf->lock);
> }
> -EXPORT_SYMBOL_GPL(dma_buf_vunmap);
> +EXPORT_SYMBOL_NS_GPL(dma_buf_vunmap, DMA_BUF);
>
> #ifdef CONFIG_DEBUG_FS
> static int dma_buf_debug_show(struct seq_file *s, void *unused)
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
> index 38dade421d46..38e144504649 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
> @@ -41,6 +41,7 @@
> #include <linux/swiotlb.h>
> #include <linux/dma-buf.h>
> #include <linux/sizes.h>
> +#include <linux/module.h>
>
> #include <drm/ttm/ttm_bo_api.h>
> #include <drm/ttm/ttm_bo_driver.h>
> @@ -59,6 +60,8 @@
> #include "amdgpu_res_cursor.h"
> #include "bif/bif_4_1_d.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> #define AMDGPU_TTM_VRAM_MAX_DW_READ (size_t)128
>
> static int amdgpu_ttm_backend_bind(struct ttm_device *bdev,
> diff --git a/drivers/gpu/drm/drm_gem_framebuffer_helper.c b/drivers/gpu/drm/drm_gem_framebuffer_helper.c
> index 3c75d79dbb65..746fd8c73845 100644
> --- a/drivers/gpu/drm/drm_gem_framebuffer_helper.c
> +++ b/drivers/gpu/drm/drm_gem_framebuffer_helper.c
> @@ -6,6 +6,7 @@
> */
>
> #include <linux/slab.h>
> +#include <linux/module.h>
>
> #include <drm/drm_damage_helper.h>
> #include <drm/drm_fb_helper.h>
> @@ -17,6 +18,8 @@
>
> #include "drm_internal.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> #define AFBC_HEADER_SIZE 16
> #define AFBC_TH_LAYOUT_ALIGNMENT 8
> #define AFBC_HDR_ALIGN 64
> diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c
> index deb23dbec8b5..d8ba95744410 100644
> --- a/drivers/gpu/drm/drm_prime.c
> +++ b/drivers/gpu/drm/drm_prime.c
> @@ -29,6 +29,7 @@
> #include <linux/export.h>
> #include <linux/dma-buf.h>
> #include <linux/rbtree.h>
> +#include <linux/module.h>
>
> #include <drm/drm.h>
> #include <drm/drm_drv.h>
> @@ -39,6 +40,8 @@
>
> #include "drm_internal.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> /**
> * DOC: overview and lifetime rules
> *
> diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c b/drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c
> index 6d8bed9c739d..6788ea8490d1 100644
> --- a/drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c
> +++ b/drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c
> @@ -5,10 +5,13 @@
>
> #include <drm/drm_prime.h>
> #include <linux/dma-buf.h>
> +#include <linux/module.h>
>
> #include "etnaviv_drv.h"
> #include "etnaviv_gem.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> static struct lock_class_key etnaviv_prime_lock_class;
>
> struct sg_table *etnaviv_gem_prime_get_sg_table(struct drm_gem_object *obj)
> diff --git a/drivers/gpu/drm/exynos/exynos_drm_gem.c b/drivers/gpu/drm/exynos/exynos_drm_gem.c
> index 4396224227d1..0a0c042a3155 100644
> --- a/drivers/gpu/drm/exynos/exynos_drm_gem.c
> +++ b/drivers/gpu/drm/exynos/exynos_drm_gem.c
> @@ -9,6 +9,7 @@
> #include <linux/dma-buf.h>
> #include <linux/pfn_t.h>
> #include <linux/shmem_fs.h>
> +#include <linux/module.h>
>
> #include <drm/drm_prime.h>
> #include <drm/drm_vma_manager.h>
> @@ -17,6 +18,8 @@
> #include "exynos_drm_drv.h"
> #include "exynos_drm_gem.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> static int exynos_drm_alloc_buf(struct exynos_drm_gem *exynos_gem, bool kvmap)
> {
> struct drm_device *dev = exynos_gem->base.dev;
> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
> index afa34111de02..abb854281347 100644
> --- a/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
> +++ b/drivers/gpu/drm/i915/gem/i915_gem_dmabuf.c
> @@ -7,11 +7,14 @@
> #include <linux/dma-buf.h>
> #include <linux/highmem.h>
> #include <linux/dma-resv.h>
> +#include <linux/module.h>
>
> #include "i915_drv.h"
> #include "i915_gem_object.h"
> #include "i915_scatterlist.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> I915_SELFTEST_DECLARE(static bool force_different_devices;)
>
> static struct drm_i915_gem_object *dma_buf_to_obj(struct dma_buf *buf)
> diff --git a/drivers/gpu/drm/tegra/gem.c b/drivers/gpu/drm/tegra/gem.c
> index 6ec598f5d5b3..d38fd7e12b57 100644
> --- a/drivers/gpu/drm/tegra/gem.c
> +++ b/drivers/gpu/drm/tegra/gem.c
> @@ -12,6 +12,7 @@
>
> #include <linux/dma-buf.h>
> #include <linux/iommu.h>
> +#include <linux/module.h>
>
> #include <drm/drm_drv.h>
> #include <drm/drm_prime.h>
> @@ -20,6 +21,8 @@
> #include "drm.h"
> #include "gem.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> static void tegra_bo_put(struct host1x_bo *bo)
> {
> struct tegra_bo *obj = host1x_to_tegra_bo(bo);
> diff --git a/drivers/gpu/drm/vmwgfx/ttm_object.c b/drivers/gpu/drm/vmwgfx/ttm_object.c
> index 04789b2bb2a2..899945f54dc7 100644
> --- a/drivers/gpu/drm/vmwgfx/ttm_object.c
> +++ b/drivers/gpu/drm/vmwgfx/ttm_object.c
> @@ -48,8 +48,11 @@
> #include <linux/spinlock.h>
> #include <linux/slab.h>
> #include <linux/atomic.h>
> +#include <linux/module.h>
> #include "ttm_object.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> /**
> * struct ttm_object_file
> *
> diff --git a/drivers/infiniband/core/umem_dmabuf.c b/drivers/infiniband/core/umem_dmabuf.c
> index e824baf4640d..2d14929543af 100644
> --- a/drivers/infiniband/core/umem_dmabuf.c
> +++ b/drivers/infiniband/core/umem_dmabuf.c
> @@ -6,9 +6,12 @@
> #include <linux/dma-buf.h>
> #include <linux/dma-resv.h>
> #include <linux/dma-mapping.h>
> +#include <linux/module.h>
>
> #include "uverbs.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> int ib_umem_dmabuf_map_pages(struct ib_umem_dmabuf *umem_dmabuf)
> {
> struct sg_table *sgt;
> diff --git a/drivers/media/common/videobuf2/videobuf2-core.c b/drivers/media/common/videobuf2/videobuf2-core.c
> index 508ac295eb06..773c68dcd158 100644
> --- a/drivers/media/common/videobuf2/videobuf2-core.c
> +++ b/drivers/media/common/videobuf2/videobuf2-core.c
> @@ -2978,3 +2978,4 @@ EXPORT_SYMBOL_GPL(vb2_thread_stop);
> MODULE_DESCRIPTION("Media buffer core framework");
> MODULE_AUTHOR("Pawel Osciak <pawel@xxxxxxxxxx>, Marek Szyprowski");
> MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(DMA_BUF);
> diff --git a/drivers/media/common/videobuf2/videobuf2-dma-contig.c b/drivers/media/common/videobuf2/videobuf2-dma-contig.c
> index a7f61ba85440..9a1a9baca2e4 100644
> --- a/drivers/media/common/videobuf2/videobuf2-dma-contig.c
> +++ b/drivers/media/common/videobuf2/videobuf2-dma-contig.c
> @@ -755,3 +755,4 @@ EXPORT_SYMBOL_GPL(vb2_dma_contig_set_max_seg_size);
> MODULE_DESCRIPTION("DMA-contig memory handling routines for videobuf2");
> MODULE_AUTHOR("Pawel Osciak <pawel@xxxxxxxxxx>");
> MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(DMA_BUF);
> diff --git a/drivers/media/common/videobuf2/videobuf2-dma-sg.c b/drivers/media/common/videobuf2/videobuf2-dma-sg.c
> index c5b06a509566..db90ebb8950f 100644
> --- a/drivers/media/common/videobuf2/videobuf2-dma-sg.c
> +++ b/drivers/media/common/videobuf2/videobuf2-dma-sg.c
> @@ -666,3 +666,4 @@ EXPORT_SYMBOL_GPL(vb2_dma_sg_memops);
> MODULE_DESCRIPTION("dma scatter/gather memory handling routines for videobuf2");
> MODULE_AUTHOR("Andrzej Pietrasiewicz");
> MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(DMA_BUF);
> diff --git a/drivers/media/common/videobuf2/videobuf2-vmalloc.c b/drivers/media/common/videobuf2/videobuf2-vmalloc.c
> index 83f95258ec8c..fa983897d0e9 100644
> --- a/drivers/media/common/videobuf2/videobuf2-vmalloc.c
> +++ b/drivers/media/common/videobuf2/videobuf2-vmalloc.c
> @@ -444,3 +444,4 @@ EXPORT_SYMBOL_GPL(vb2_vmalloc_memops);
> MODULE_DESCRIPTION("vmalloc memory handling routines for videobuf2");
> MODULE_AUTHOR("Pawel Osciak <pawel@xxxxxxxxxx>");
> MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(DMA_BUF);
> diff --git a/drivers/misc/fastrpc.c b/drivers/misc/fastrpc.c
> index beda610e6b30..fa5c067f1c1e 100644
> --- a/drivers/misc/fastrpc.c
> +++ b/drivers/misc/fastrpc.c
> @@ -1763,3 +1763,4 @@ static void fastrpc_exit(void)
> module_exit(fastrpc_exit);
>
> MODULE_LICENSE("GPL v2");
> +MODULE_IMPORT_NS(DMA_BUF);
> diff --git a/drivers/staging/media/tegra-vde/dmabuf-cache.c b/drivers/staging/media/tegra-vde/dmabuf-cache.c
> index a93b317885bf..a98d03419b8f 100644
> --- a/drivers/staging/media/tegra-vde/dmabuf-cache.c
> +++ b/drivers/staging/media/tegra-vde/dmabuf-cache.c
> @@ -12,9 +12,12 @@
> #include <linux/sched.h>
> #include <linux/slab.h>
> #include <linux/workqueue.h>
> +#include <linux/module.h>
>
> #include "vde.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> struct tegra_vde_cache_entry {
> enum dma_data_direction dma_dir;
> struct dma_buf_attachment *a;
> diff --git a/drivers/tee/tee_shm.c b/drivers/tee/tee_shm.c
> index 8a9384a64f3e..8a8deb95e918 100644
> --- a/drivers/tee/tee_shm.c
> +++ b/drivers/tee/tee_shm.c
> @@ -10,8 +10,11 @@
> #include <linux/slab.h>
> #include <linux/tee_drv.h>
> #include <linux/uio.h>
> +#include <linux/module.h>
> #include "tee_private.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> static void release_registered_pages(struct tee_shm *shm)
> {
> if (shm->pages) {
> diff --git a/drivers/virtio/virtio_dma_buf.c b/drivers/virtio/virtio_dma_buf.c
> index 5127a2f0c986..2521a75009c3 100644
> --- a/drivers/virtio/virtio_dma_buf.c
> +++ b/drivers/virtio/virtio_dma_buf.c
> @@ -86,3 +86,4 @@ int virtio_dma_buf_get_uuid(struct dma_buf *dma_buf,
> EXPORT_SYMBOL(virtio_dma_buf_get_uuid);
>
> MODULE_LICENSE("GPL");
> +MODULE_IMPORT_NS(DMA_BUF);
> diff --git a/drivers/xen/gntdev-dmabuf.c b/drivers/xen/gntdev-dmabuf.c
> index 4c13cbc99896..12e380db7f55 100644
> --- a/drivers/xen/gntdev-dmabuf.c
> +++ b/drivers/xen/gntdev-dmabuf.c
> @@ -14,6 +14,7 @@
> #include <linux/slab.h>
> #include <linux/types.h>
> #include <linux/uaccess.h>
> +#include <linux/module.h>
>
> #include <xen/xen.h>
> #include <xen/grant_table.h>
> @@ -21,6 +22,8 @@
> #include "gntdev-common.h"
> #include "gntdev-dmabuf.h"
>
> +MODULE_IMPORT_NS(DMA_BUF);
> +
> #ifndef GRANT_INVALID_REF
> /*
> * Note on usage of grant reference 0 as invalid grant reference:
> diff --git a/samples/vfio-mdev/mbochs.c b/samples/vfio-mdev/mbochs.c
> index c313ab4d1f4e..a83be6cd162f 100644
> --- a/samples/vfio-mdev/mbochs.c
> +++ b/samples/vfio-mdev/mbochs.c
> @@ -1493,5 +1493,6 @@ static void __exit mbochs_dev_exit(void)
> mbochs_class = NULL;
> }
>
> +MODULE_IMPORT_NS(DMA_BUF);
> module_init(mbochs_dev_init)
> module_exit(mbochs_dev_exit)
> --
> 2.33.0
>

--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch

Next message: David Hildenbrand: "Re: [RFC] arm64: mm: update max_pfn after memory hotplug"
Previous message: Ooi, Joyce: "RE: [PATCH] MAINTAINERS: Update Mun Yew Tham as Altera Triple Speed Ethernet Driver maintainer"
In reply to: Alex Deucher: "Re: [PATCH] dma-buf: move dma-buf symbols into the DMA_BUF module namespace"
Next in thread: Greg Kroah-Hartman: "Re: [PATCH] dma-buf: move dma-buf symbols into the DMA_BUF module namespace"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]