Re: [PATCH] drm/i915: run kernel-doc on headers as part of HDRTEST

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 04, 2023 at 03:26:24PM +0300, Jani Nikula wrote:
> On Tue, 04 Apr 2023, kernel test robot <lkp@xxxxxxxxx> wrote:
> > Hi Jani,
> >
> > kernel test robot noticed the following build warnings:
> 
> Yeah, that's kind of the point of adding more checks. ;)

Indeed. I was in doubt if the include of the full dir was the right
way, but this reply shows it works as expected :)

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>

> 
> BR,
> Jani.
> 
> 
> >
> > [auto build test WARNING on drm-tip/drm-tip]
> >
> > url:    https://github.com/intel-lab-lkp/linux/commits/Jani-Nikula/drm-i915-run-kernel-doc-on-headers-as-part-of-HDRTEST/20230404-170637
> > base:   git://anongit.freedesktop.org/drm/drm-tip drm-tip
> > patch link:    https://lore.kernel.org/r/20230404090528.173075-1-jani.nikula%40intel.com
> > patch subject:  [PATCH] drm/i915: run kernel-doc on headers as part of HDRTEST
> > config: i386-defconfig (https://download.01.org/0day-ci/archive/20230404/202304042026.uZBs3cC2-lkp@xxxxxxxxx/config)
> > compiler: gcc-11 (Debian 11.3.0-8) 11.3.0
> > reproduce (this is a W=1 build):
> >         # https://github.com/intel-lab-lkp/linux/commit/336391cc647b2fbdf0ebd2eda8a10fb4509918b7
> >         git remote add linux-review https://github.com/intel-lab-lkp/linux
> >         git fetch --no-tags linux-review Jani-Nikula/drm-i915-run-kernel-doc-on-headers-as-part-of-HDRTEST/20230404-170637
> >         git checkout 336391cc647b2fbdf0ebd2eda8a10fb4509918b7
> >         # save the config file
> >         mkdir build_dir && cp config build_dir/.config
> >         make W=1 O=build_dir ARCH=i386 olddefconfig
> >         make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash drivers/gpu/
> >
> > If you fix the issue, kindly add following tag where applicable
> > | Reported-by: kernel test robot <lkp@xxxxxxxxx>
> > | Link: https://lore.kernel.org/oe-kbuild-all/202304042026.uZBs3cC2-lkp@xxxxxxxxx/
> >
> > All warnings (new ones prefixed by >>):
> >
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:27: warning: Function parameter or member 'size' not described in '__guc_capture_bufstate'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:27: warning: Function parameter or member 'data' not described in '__guc_capture_bufstate'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:27: warning: Function parameter or member 'rd' not described in '__guc_capture_bufstate'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:27: warning: Function parameter or member 'wr' not described in '__guc_capture_bufstate'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:59: warning: Function parameter or member 'link' not described in '__guc_capture_parsed_output'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:59: warning: Function parameter or member 'is_partial' not described in '__guc_capture_parsed_output'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:59: warning: Function parameter or member 'eng_class' not described in '__guc_capture_parsed_output'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:59: warning: Function parameter or member 'eng_inst' not described in '__guc_capture_parsed_output'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:59: warning: Function parameter or member 'guc_id' not described in '__guc_capture_parsed_output'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:59: warning: Function parameter or member 'lrca' not described in '__guc_capture_parsed_output'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:59: warning: Function parameter or member 'reginfo' not described in '__guc_capture_parsed_output'
> >>> drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:62: warning: wrong kernel-doc identifier on line:
> >     * struct guc_debug_capture_list_header / struct guc_debug_capture_list
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:80: warning: wrong kernel-doc identifier on line:
> >     * struct __guc_mmio_reg_descr / struct __guc_mmio_reg_descr_group
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:105: warning: wrong kernel-doc identifier on line:
> >     * struct guc_state_capture_header_t / struct guc_state_capture_t /
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:163: warning: Function parameter or member 'is_valid' not described in '__guc_capture_ads_cache'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:163: warning: Function parameter or member 'ptr' not described in '__guc_capture_ads_cache'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:163: warning: Function parameter or member 'size' not described in '__guc_capture_ads_cache'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:163: warning: Function parameter or member 'status' not described in '__guc_capture_ads_cache'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:216: warning: Function parameter or member 'ads_null_cache' not described in 'intel_guc_state_capture'
> >    drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h:216: warning: Function parameter or member 'max_mmio_per_node' not described in 'intel_guc_state_capture'
> > --
> >    drivers/gpu/drm/i915/i915_pmu.h:21: warning: cannot understand function prototype: 'enum i915_pmu_tracked_events '
> >    drivers/gpu/drm/i915/i915_pmu.h:32: warning: cannot understand function prototype: 'enum '
> >>> drivers/gpu/drm/i915/i915_pmu.h:41: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> >     * How many different events we track in the global PMU mask.
> > --
> >>> drivers/gpu/drm/i915/i915_request.h:176: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> >     * Request queue structure.
> >    drivers/gpu/drm/i915/i915_request.h:477: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> >     * Returns true if seq1 is later than seq2.
> > --
> >>> drivers/gpu/drm/i915/i915_vma.h:145: warning: expecting prototype for i915_vma_offset(). Prototype was for i915_vma_size() instead
> > --
> >>> drivers/gpu/drm/i915/intel_wakeref.h:118: warning: expecting prototype for intel_wakeref_get_if_in_use(). Prototype was for intel_wakeref_get_if_active() instead
> >
> >
> > vim +62 drivers/gpu/drm/i915/gt/uc/guc_capture_fwif.h
> >
> > 24492514ccbd57 Alan Previn 2022-03-21  14  
> > f5718a7265a6f5 Alan Previn 2022-03-21  15  /**
> > f5718a7265a6f5 Alan Previn 2022-03-21  16   * struct __guc_capture_bufstate
> > f5718a7265a6f5 Alan Previn 2022-03-21  17   *
> > f5718a7265a6f5 Alan Previn 2022-03-21  18   * Book-keeping structure used to track read and write pointers
> > f5718a7265a6f5 Alan Previn 2022-03-21  19   * as we extract error capture data from the GuC-log-buffer's
> > f5718a7265a6f5 Alan Previn 2022-03-21  20   * error-capture region as a stream of dwords.
> > f5718a7265a6f5 Alan Previn 2022-03-21  21   */
> > f5718a7265a6f5 Alan Previn 2022-03-21  22  struct __guc_capture_bufstate {
> > f5718a7265a6f5 Alan Previn 2022-03-21  23  	u32 size;
> > f5718a7265a6f5 Alan Previn 2022-03-21  24  	void *data;
> > f5718a7265a6f5 Alan Previn 2022-03-21  25  	u32 rd;
> > f5718a7265a6f5 Alan Previn 2022-03-21  26  	u32 wr;
> > f5718a7265a6f5 Alan Previn 2022-03-21 @27  };
> > f5718a7265a6f5 Alan Previn 2022-03-21  28  
> > f5718a7265a6f5 Alan Previn 2022-03-21  29  /**
> > f5718a7265a6f5 Alan Previn 2022-03-21  30   * struct __guc_capture_parsed_output - extracted error capture node
> > f5718a7265a6f5 Alan Previn 2022-03-21  31   *
> > f5718a7265a6f5 Alan Previn 2022-03-21  32   * A single unit of extracted error-capture output data grouped together
> > f5718a7265a6f5 Alan Previn 2022-03-21  33   * at an engine-instance level. We keep these nodes in a linked list.
> > 247f8071d58933 Alan Previn 2022-03-21  34   * See cachelist and outlist below.
> > f5718a7265a6f5 Alan Previn 2022-03-21  35   */
> > f5718a7265a6f5 Alan Previn 2022-03-21  36  struct __guc_capture_parsed_output {
> > f5718a7265a6f5 Alan Previn 2022-03-21  37  	/*
> > f5718a7265a6f5 Alan Previn 2022-03-21  38  	 * A single set of 3 capture lists: a global-list
> > f5718a7265a6f5 Alan Previn 2022-03-21  39  	 * an engine-class-list and an engine-instance list.
> > f5718a7265a6f5 Alan Previn 2022-03-21  40  	 * outlist in __guc_capture_parsed_output will keep
> > f5718a7265a6f5 Alan Previn 2022-03-21  41  	 * a linked list of these nodes that will eventually
> > f5718a7265a6f5 Alan Previn 2022-03-21  42  	 * be detached from outlist and attached into to
> > f5718a7265a6f5 Alan Previn 2022-03-21  43  	 * i915_gpu_codedump in response to a context reset
> > f5718a7265a6f5 Alan Previn 2022-03-21  44  	 */
> > f5718a7265a6f5 Alan Previn 2022-03-21  45  	struct list_head link;
> > f5718a7265a6f5 Alan Previn 2022-03-21  46  	bool is_partial;
> > f5718a7265a6f5 Alan Previn 2022-03-21  47  	u32 eng_class;
> > f5718a7265a6f5 Alan Previn 2022-03-21  48  	u32 eng_inst;
> > f5718a7265a6f5 Alan Previn 2022-03-21  49  	u32 guc_id;
> > f5718a7265a6f5 Alan Previn 2022-03-21  50  	u32 lrca;
> > f5718a7265a6f5 Alan Previn 2022-03-21  51  	struct gcap_reg_list_info {
> > f5718a7265a6f5 Alan Previn 2022-03-21  52  		u32 vfid;
> > f5718a7265a6f5 Alan Previn 2022-03-21  53  		u32 num_regs;
> > f5718a7265a6f5 Alan Previn 2022-03-21  54  		struct guc_mmio_reg *regs;
> > f5718a7265a6f5 Alan Previn 2022-03-21  55  	} reginfo[GUC_CAPTURE_LIST_TYPE_MAX];
> > f5718a7265a6f5 Alan Previn 2022-03-21  56  #define GCAP_PARSED_REGLIST_INDEX_GLOBAL   BIT(GUC_CAPTURE_LIST_TYPE_GLOBAL)
> > f5718a7265a6f5 Alan Previn 2022-03-21  57  #define GCAP_PARSED_REGLIST_INDEX_ENGCLASS BIT(GUC_CAPTURE_LIST_TYPE_ENGINE_CLASS)
> > f5718a7265a6f5 Alan Previn 2022-03-21  58  #define GCAP_PARSED_REGLIST_INDEX_ENGINST  BIT(GUC_CAPTURE_LIST_TYPE_ENGINE_INSTANCE)
> > f5718a7265a6f5 Alan Previn 2022-03-21  59  };
> > f5718a7265a6f5 Alan Previn 2022-03-21  60  
> > 24492514ccbd57 Alan Previn 2022-03-21  61  /**
> > 24492514ccbd57 Alan Previn 2022-03-21 @62   * struct guc_debug_capture_list_header / struct guc_debug_capture_list
> > 24492514ccbd57 Alan Previn 2022-03-21  63   *
> > 24492514ccbd57 Alan Previn 2022-03-21  64   * As part of ADS registration, these header structures (followed by
> > 24492514ccbd57 Alan Previn 2022-03-21  65   * an array of 'struct guc_mmio_reg' entries) are used to register with
> > 24492514ccbd57 Alan Previn 2022-03-21  66   * GuC microkernel the list of registers we want it to dump out prior
> > 24492514ccbd57 Alan Previn 2022-03-21  67   * to a engine reset.
> > 24492514ccbd57 Alan Previn 2022-03-21  68   */
> > 24492514ccbd57 Alan Previn 2022-03-21  69  struct guc_debug_capture_list_header {
> > 24492514ccbd57 Alan Previn 2022-03-21  70  	u32 info;
> > 24492514ccbd57 Alan Previn 2022-03-21  71  #define GUC_CAPTURELISTHDR_NUMDESCR GENMASK(15, 0)
> > 24492514ccbd57 Alan Previn 2022-03-21  72  } __packed;
> > 24492514ccbd57 Alan Previn 2022-03-21  73  
> 
> -- 
> Jani Nikula, Intel Open Source Graphics Center



[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux