[android-common:android16-6.12-desktop 1/1] drivers/gpu/drm/xe/xe_work_period.c:454: warning: Function parameter or struct member 'xe' not described in 'xe_work_period_suspend'

Hi Ryan,

FYI, the error/warning still remains.

tree:   https://android.googlesource.com/kernel/common android16-6.12-desktop
head:   24c37396da197463e1ff2bc69bb789e34dacad53
commit: c1b1ceb28458b92550c1e06b11f7d5b05bb36be8 [1/1] ANDROID: drm/xe: Add gpu_work_period tracepoint
config: i386-allmodconfig (https://download.01.org/0day-ci/archive/20260814/[email protected]/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260814/[email protected]/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 <[email protected]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

All warnings (new ones prefixed by >>):

>> drivers/gpu/drm/xe/xe_work_period.c:454: warning: Function parameter or struct member 'xe' not described in 'xe_work_period_suspend'
>> drivers/gpu/drm/xe/xe_work_period.c:474: warning: Function parameter or struct member 'xe' not described in 'xe_work_period_resume'
>> drivers/gpu/drm/xe/xe_work_period.c:500: warning: Function parameter or struct member 'xe' not described in 'xe_work_period_init'
>> drivers/gpu/drm/xe/xe_work_period.c:515: warning: Function parameter or struct member 'xe' not described in 'xe_work_period_fini'


vim +454 drivers/gpu/drm/xe/xe_work_period.c

   442	
   443	/**
   444	 * xe_work_period_suspend() - cancel all pending @xe_user workers
   445	 * @xe - the xe device
   446	 *
   447	 * Normally called during system suspend handlers or during @xe_device cleanup.
   448	 *
   449	 * Context: Process context. May take xe->work_period.lock if worker holds the
   450	 *	    last ref for its @xe_user.
   451	 * Return: void
   452	 */
   453	void xe_work_period_suspend(struct xe_device *xe)
 > 454	{
   455		struct xe_user *user = NULL;
   456		unsigned long i = 0;
   457	
   458		xa_for_each(&xe->work_period.users, i, user) {
   459			if (cancel_delayed_work_sync(&user->delay_work))
   460				xe_user_put(user);
   461		}
   462	}
   463	
   464	/**
   465	 * xe_work_period_resume() - schedule all previously suspended @xe_user workers
   466	 * @xe - the xe device
   467	 *
   468	 * Normally called during system resume handlers.
   469	 *
   470	 * Context: Process context.
   471	 * Return: void
   472	 */
   473	void xe_work_period_resume(struct xe_device *xe)
 > 474	{
   475		struct xe_user *user = NULL;
   476		unsigned long i = 0;
   477	
   478		/* no need to lock because schedule_next_work() atomically checks the
   479		 * validity of each user and acquires its own ref. If a user is dropped
   480		 * between reaching it in this iterator and attempting to get a ref, it
   481		 * can be safely excluded from rescheduling since there are no other
   482		 * remaining reference holders.
   483		 */
   484		xa_for_each(&xe->work_period.users, i, user) {
   485			user->last_timestamp_ns = ktime_get_raw_ns();
   486			schedule_next_work(user);
   487		}
   488	}
   489	
   490	/**
   491	 * xe_work_period_init() - setup work period reporting
   492	 * @xe - the xe device
   493	 *
   494	 * Normally called during @xe_device setup.
   495	 *
   496	 * Context: Any context.
   497	 * Return: void
   498	 */
   499	void xe_work_period_init(struct xe_device *xe)
 > 500	{
   501		mutex_init(&xe->work_period.lock);
   502		xa_init_flags(&xe->work_period.users, XA_FLAGS_ALLOC1);
   503	}
   504	
   505	/**
   506	 * xe_work_period_fini() - cleanup work period reporting
   507	 * @xe - the xe device
   508	 *
   509	 * Normally called during @xe_device cleanup.
   510	 *
   511	 * Context: Process context.
   512	 * Return: void
   513	 */
   514	void xe_work_period_fini(struct xe_device *xe)
 > 515	{

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki