From: kernel test robot <lkp@intel.com>
To: Danilo Krummrich <dakr@redhat.com>,
airlied@gmail.com, daniel@ffwll.ch, matthew.brost@intel.com,
thomas.hellstrom@linux.intel.com, sarah.walker@imgtec.com,
donald.robson@imgtec.com, boris.brezillon@collabora.com,
christian.koenig@amd.com, faith.ekstrand@collabora.com
Cc: oe-kbuild-all@lists.linux.dev, nouveau@lists.freedesktop.org,
Danilo Krummrich <dakr@redhat.com>,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH drm-misc-next v2 2/7] drm/gpuvm: rename struct drm_gpuva_manager to struct drm_gpuvm
Date: Thu, 7 Sep 2023 16:51:37 +0800 [thread overview]
Message-ID: <202309071613.s6ztMEyU-lkp@intel.com> (raw)
In-Reply-To: <20230906214723.4393-3-dakr@redhat.com>
Hi Danilo,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 6bd3d8da51ca1ec97c724016466606aec7739b9f]
url: https://github.com/intel-lab-lkp/linux/commits/Danilo-Krummrich/drm-gpuva_mgr-allow-building-as-module/20230907-054931
base: 6bd3d8da51ca1ec97c724016466606aec7739b9f
patch link: https://lore.kernel.org/r/20230906214723.4393-3-dakr%40redhat.com
patch subject: [PATCH drm-misc-next v2 2/7] drm/gpuvm: rename struct drm_gpuva_manager to struct drm_gpuvm
reproduce: (https://download.01.org/0day-ci/archive/20230907/202309071613.s6ztMEyU-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202309071613.s6ztMEyU-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> ./include/drm/drm_gpuva_mgr.h:138: warning: Function parameter or member 'vm' not described in 'drm_gpuva'
vim +138 ./include/drm/drm_gpuva_mgr.h
e6303f323b1ad9 Danilo Krummrich 2023-07-20 60
e6303f323b1ad9 Danilo Krummrich 2023-07-20 61 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 62 * struct drm_gpuva - structure to track a GPU VA mapping
e6303f323b1ad9 Danilo Krummrich 2023-07-20 63 *
e6303f323b1ad9 Danilo Krummrich 2023-07-20 64 * This structure represents a GPU VA mapping and is associated with a
3142f8b7e68331 Danilo Krummrich 2023-09-06 65 * &drm_gpuvm.
e6303f323b1ad9 Danilo Krummrich 2023-07-20 66 *
e6303f323b1ad9 Danilo Krummrich 2023-07-20 67 * Typically, this structure is embedded in bigger driver structures.
e6303f323b1ad9 Danilo Krummrich 2023-07-20 68 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 69 struct drm_gpuva {
e6303f323b1ad9 Danilo Krummrich 2023-07-20 70 /**
3142f8b7e68331 Danilo Krummrich 2023-09-06 71 * @gpuvm: the &drm_gpuvm this object is associated with
e6303f323b1ad9 Danilo Krummrich 2023-07-20 72 */
3142f8b7e68331 Danilo Krummrich 2023-09-06 73 struct drm_gpuvm *vm;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 74
e6303f323b1ad9 Danilo Krummrich 2023-07-20 75 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 76 * @flags: the &drm_gpuva_flags for this mapping
e6303f323b1ad9 Danilo Krummrich 2023-07-20 77 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 78 enum drm_gpuva_flags flags;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 79
e6303f323b1ad9 Danilo Krummrich 2023-07-20 80 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 81 * @va: structure containing the address and range of the &drm_gpuva
e6303f323b1ad9 Danilo Krummrich 2023-07-20 82 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 83 struct {
e6303f323b1ad9 Danilo Krummrich 2023-07-20 84 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 85 * @addr: the start address
e6303f323b1ad9 Danilo Krummrich 2023-07-20 86 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 87 u64 addr;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 88
e6303f323b1ad9 Danilo Krummrich 2023-07-20 89 /*
e6303f323b1ad9 Danilo Krummrich 2023-07-20 90 * @range: the range
e6303f323b1ad9 Danilo Krummrich 2023-07-20 91 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 92 u64 range;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 93 } va;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 94
e6303f323b1ad9 Danilo Krummrich 2023-07-20 95 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 96 * @gem: structure containing the &drm_gem_object and it's offset
e6303f323b1ad9 Danilo Krummrich 2023-07-20 97 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 98 struct {
e6303f323b1ad9 Danilo Krummrich 2023-07-20 99 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 100 * @offset: the offset within the &drm_gem_object
e6303f323b1ad9 Danilo Krummrich 2023-07-20 101 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 102 u64 offset;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 103
e6303f323b1ad9 Danilo Krummrich 2023-07-20 104 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 105 * @obj: the mapped &drm_gem_object
e6303f323b1ad9 Danilo Krummrich 2023-07-20 106 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 107 struct drm_gem_object *obj;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 108
e6303f323b1ad9 Danilo Krummrich 2023-07-20 109 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 110 * @entry: the &list_head to attach this object to a &drm_gem_object
e6303f323b1ad9 Danilo Krummrich 2023-07-20 111 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 112 struct list_head entry;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 113 } gem;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 114
e6303f323b1ad9 Danilo Krummrich 2023-07-20 115 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 116 * @rb: structure containing data to store &drm_gpuvas in a rb-tree
e6303f323b1ad9 Danilo Krummrich 2023-07-20 117 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 118 struct {
e6303f323b1ad9 Danilo Krummrich 2023-07-20 119 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 120 * @rb: the rb-tree node
e6303f323b1ad9 Danilo Krummrich 2023-07-20 121 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 122 struct rb_node node;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 123
e6303f323b1ad9 Danilo Krummrich 2023-07-20 124 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 125 * @entry: The &list_head to additionally connect &drm_gpuvas
e6303f323b1ad9 Danilo Krummrich 2023-07-20 126 * in the same order they appear in the interval tree. This is
e6303f323b1ad9 Danilo Krummrich 2023-07-20 127 * useful to keep iterating &drm_gpuvas from a start node found
e6303f323b1ad9 Danilo Krummrich 2023-07-20 128 * through the rb-tree while doing modifications on the rb-tree
e6303f323b1ad9 Danilo Krummrich 2023-07-20 129 * itself.
e6303f323b1ad9 Danilo Krummrich 2023-07-20 130 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 131 struct list_head entry;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 132
e6303f323b1ad9 Danilo Krummrich 2023-07-20 133 /**
e6303f323b1ad9 Danilo Krummrich 2023-07-20 134 * @__subtree_last: needed by the interval tree, holding last-in-subtree
e6303f323b1ad9 Danilo Krummrich 2023-07-20 135 */
e6303f323b1ad9 Danilo Krummrich 2023-07-20 136 u64 __subtree_last;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 137 } rb;
e6303f323b1ad9 Danilo Krummrich 2023-07-20 @138 };
e6303f323b1ad9 Danilo Krummrich 2023-07-20 139
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2023-09-07 16:11 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-06 21:47 [PATCH drm-misc-next v2 0/7] [RFC] DRM GPUVA Manager GPU-VM features Danilo Krummrich
2023-09-06 21:47 ` [PATCH drm-misc-next v2 1/7] drm: gpuva_mgr: allow building as module Danilo Krummrich
2023-09-06 21:47 ` [PATCH drm-misc-next v2 2/7] drm/gpuvm: rename struct drm_gpuva_manager to struct drm_gpuvm Danilo Krummrich
2023-09-07 7:56 ` Boris Brezillon
2023-09-07 8:54 ` Danilo Krummrich
2023-09-07 8:51 ` kernel test robot [this message]
2023-09-06 21:47 ` [PATCH drm-misc-next v2 3/7] drm/nouveau: uvmm: rename 'umgr' to 'base' Danilo Krummrich
2023-09-06 21:47 ` [PATCH drm-misc-next v2 4/7] drm/gpuvm: common dma-resv per struct drm_gpuvm Danilo Krummrich
2023-09-06 21:47 ` [PATCH drm-misc-next v2 5/7] drm/gpuvm: add an abstraction for a VM / BO combination Danilo Krummrich
2023-09-07 8:16 ` Boris Brezillon
2023-09-07 9:11 ` Danilo Krummrich
2023-09-07 8:42 ` Boris Brezillon
2023-09-07 9:41 ` Danilo Krummrich
2023-09-07 11:05 ` Boris Brezillon
2023-09-06 21:47 ` [PATCH drm-misc-next v2 6/7] drm/gpuvm: generalize dma_resv/extobj handling and GEM validation Danilo Krummrich
2023-09-07 0:12 ` kernel test robot
2023-09-06 21:47 ` [PATCH drm-misc-next v2 7/7] drm/nouveau: GPUVM dma-resv/extobj handling, " Danilo Krummrich
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=202309071613.s6ztMEyU-lkp@intel.com \
--to=lkp@intel.com \
--cc=airlied@gmail.com \
--cc=boris.brezillon@collabora.com \
--cc=christian.koenig@amd.com \
--cc=dakr@redhat.com \
--cc=daniel@ffwll.ch \
--cc=donald.robson@imgtec.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=faith.ekstrand@collabora.com \
--cc=linux-kernel@vger.kernel.org \
--cc=matthew.brost@intel.com \
--cc=nouveau@lists.freedesktop.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=sarah.walker@imgtec.com \
--cc=thomas.hellstrom@linux.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox