From: kernel test robot <lkp@intel.com>
To: Alan Previn <alan.previn.teres.alexis@intel.com>,
intel-gfx@lists.freedesktop.org
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org,
Alan Previn <alan.previn.teres.alexis@intel.com>
Subject: Re: [Intel-gfx] [PATCH v7 01/13] drm/i915/guc: Update GuC ADS size for error capture lists
Date: Mon, 28 Feb 2022 05:02:05 +0800 [thread overview]
Message-ID: <202202280456.Z5q4zizH-lkp@intel.com> (raw)
In-Reply-To: <20220226095541.1010534-2-alan.previn.teres.alexis@intel.com>
Hi Alan,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on drm-tip/drm-tip]
[cannot apply to drm-intel/for-linux-next drm-exynos/exynos-drm-next drm/drm-next tegra-drm/drm/tegra/for-next airlied/drm-next v5.17-rc5 next-20220225]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Alan-Previn/Add-GuC-Error-Capture-Support/20220226-175600
base: git://anongit.freedesktop.org/drm/drm-tip drm-tip
config: i386-allyesconfig (https://download.01.org/0day-ci/archive/20220228/202202280456.Z5q4zizH-lkp@intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project d271fc04d5b97b12e6b797c6067d3c96a8d7470e)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/9ea292ae1b189632494171f6dae9cbf7e4e5f30c
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Alan-Previn/Add-GuC-Error-Capture-Support/20220226-175600
git checkout 9ea292ae1b189632494171f6dae9cbf7e4e5f30c
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
>> drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c:223:1: error: no previous prototype for function 'intel_guc_capture_getlistsize' [-Werror,-Wmissing-prototypes]
intel_guc_capture_getlistsize(struct intel_guc *guc, u32 owner, u32 type, u32 classid,
^
drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c:222:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int
^
static
>> drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c:253:1: error: no previous prototype for function 'intel_guc_capture_getlist' [-Werror,-Wmissing-prototypes]
intel_guc_capture_getlist(struct intel_guc *guc, u32 owner, u32 type, u32 classid,
^
drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c:252:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int
^
static
drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c:332:6: error: no previous prototype for function 'intel_guc_capture_destroy' [-Werror,-Wmissing-prototypes]
void intel_guc_capture_destroy(struct intel_guc *guc)
^
drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c:332:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void intel_guc_capture_destroy(struct intel_guc *guc)
^
static
drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c:343:5: error: no previous prototype for function 'intel_guc_capture_init' [-Werror,-Wmissing-prototypes]
int intel_guc_capture_init(struct intel_guc *guc)
^
drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c:343:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int intel_guc_capture_init(struct intel_guc *guc)
^
static
4 errors generated.
vim +/intel_guc_capture_getlistsize +223 drivers/gpu/drm/i915/gt/uc/intel_guc_capture.c
221
222 int
> 223 intel_guc_capture_getlistsize(struct intel_guc *guc, u32 owner, u32 type, u32 classid,
224 size_t *size)
225 {
226 struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
227 struct __guc_state_capture_priv *gc = guc->capture.priv;
228 struct __guc_capture_ads_cache *cache = &gc->ads_cache[owner][type][classid];
229 int num_regs;
230
231 if (!gc->reglists)
232 return -ENODEV;
233
234 if (cache->is_valid) {
235 *size = cache->size;
236 return cache->status;
237 }
238
239 num_regs = guc_cap_list_num_regs(gc, owner, type, classid);
240 if (!num_regs) {
241 guc_capture_warn_with_list_info(i915, "Missing register list size",
242 owner, type, classid);
243 return -ENODATA;
244 }
245
246 *size = PAGE_ALIGN((sizeof(struct guc_debug_capture_list)) +
247 (num_regs * sizeof(struct guc_mmio_reg)));
248
249 return 0;
250 }
251
252 int
> 253 intel_guc_capture_getlist(struct intel_guc *guc, u32 owner, u32 type, u32 classid,
254 struct file **fileoutptr)
255 {
256 struct __guc_state_capture_priv *gc = guc->capture.priv;
257 struct __guc_capture_ads_cache *cache = &gc->ads_cache[owner][type][classid];
258 struct drm_i915_private *i915 = guc_to_gt(guc)->i915;
259 struct guc_debug_capture_list *listnode;
260 struct file *file;
261 u8 *caplist, *tmp;
262 size_t size = 0;
263 int ret, num_regs;
264
265 if (!gc->reglists)
266 return -ENODEV;
267
268 if (cache->is_valid) {
269 *fileoutptr = cache->file;
270 return cache->status;
271 }
272
273 ret = intel_guc_capture_getlistsize(guc, owner, type, classid, &size);
274 if (ret) {
275 cache->is_valid = true;
276 cache->file = NULL;
277 cache->size = 0;
278 cache->status = ret;
279 return ret;
280 }
281
282 caplist = kzalloc(size, GFP_KERNEL);
283 if (!caplist)
284 return -ENOMEM;
285
286 /* populate capture list header */
287 tmp = caplist;
288 num_regs = guc_cap_list_num_regs(guc->capture.priv, owner, type, classid);
289 listnode = (struct guc_debug_capture_list *)tmp;
290 listnode->header.info = FIELD_PREP(GUC_CAPTURELISTHDR_NUMDESCR, (u32)num_regs);
291
292 /* populate list of register descriptor */
293 tmp += sizeof(struct guc_debug_capture_list);
294 guc_capture_list_init(guc, owner, type, classid, (struct guc_mmio_reg *)tmp, num_regs);
295
296 /* ADS only takes file handles, so we keep that as our cache */
297 file = shmem_create_from_data("guc-err-cap", caplist, size);
298 kfree(caplist);
299 if (IS_ERR(file)) {
300 drm_warn(&i915->drm, "GuC-capture: cant create shmem for caplist = 0x%016lx", PTR_ERR(file));
301 return PTR_ERR(file);
302 }
303
304 /* cache this list */
305 cache->file = file;
306 cache->size = size;
307 cache->status = 0;
308 cache->is_valid = true;
309
310 *fileoutptr = file;
311
312 return 0;
313 }
314
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
next prev parent reply other threads:[~2022-02-27 21:02 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-26 9:55 [Intel-gfx] [PATCH v7 00/13] Add GuC Error Capture Support Alan Previn
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 01/13] drm/i915/guc: Update GuC ADS size for error capture lists Alan Previn
2022-02-26 20:22 ` Teres Alexis, Alan Previn
2022-02-27 21:02 ` kernel test robot [this message]
2022-03-10 5:30 ` Matthew Brost
2022-03-11 9:59 ` Teres Alexis, Alan Previn
2022-03-11 10:18 ` Teres Alexis, Alan Previn
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 02/13] drm/i915/guc: Add XE_LP static registers for GuC error capture Alan Previn
2022-03-10 0:05 ` Umesh Nerlige Ramappa
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 03/13] drm/i915/guc: Add XE_LP steered register lists support Alan Previn
2022-03-10 0:56 ` Umesh Nerlige Ramappa
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 04/13] drm/i915/guc: Add DG2 registers for GuC error state capture Alan Previn
2022-03-10 1:53 ` Umesh Nerlige Ramappa
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 05/13] drm/i915/guc: Add Gen9 " Alan Previn
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 06/13] drm/i915/guc: Add GuC's error state capture output structures Alan Previn
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 07/13] drm/i915/guc: Update GuC-log relay function names Alan Previn
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 08/13] drm/i915/guc: Add capture region into intel_guc_log Alan Previn
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 09/13] drm/i915/guc: Check sizing of guc_capture output Alan Previn
2022-02-28 22:32 ` kernel test robot
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 10/13] drm/i915/guc: Extract GuC error capture lists on G2H notification Alan Previn
2022-03-11 18:16 ` Umesh Nerlige Ramappa
2022-03-11 22:27 ` Teres Alexis, Alan Previn
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 11/13] drm/i915/guc: Pre-allocate output nodes for extraction Alan Previn
2022-03-11 19:40 ` Umesh Nerlige Ramappa
2022-03-11 19:46 ` Teres Alexis, Alan Previn
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 12/13] drm/i915/guc: Plumb GuC-capture into gpu_coredump Alan Previn
2022-02-26 9:55 ` [Intel-gfx] [PATCH v7 13/13] drm/i915/guc: Print the GuC error capture output register list Alan Previn
2022-03-11 5:26 ` Umesh Nerlige Ramappa
2022-02-26 10:11 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Add GuC Error Capture Support (rev7) Patchwork
2022-02-26 10:12 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2022-02-26 10:43 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
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=202202280456.Z5q4zizH-lkp@intel.com \
--to=lkp@intel.com \
--cc=alan.previn.teres.alexis@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=kbuild-all@lists.01.org \
--cc=llvm@lists.linux.dev \
/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