mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* kernel/cgroup/debug.c:1: warning: no structured comments found
@ 2023-11-21  2:32 kernel test robot
  2023-11-21  3:04 ` Waiman Long
  0 siblings, 1 reply; 4+ messages in thread
From: kernel test robot @ 2023-11-21  2:32 UTC (permalink / raw)
  To: Waiman Long; +Cc: oe-kbuild-all, linux-kernel, Tejun Heo

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   98b1cc82c4affc16f5598d4fa14b1858671b2263
commit: a28f8f5e995fe5964ae304444913536058f26e37 cgroup: Move debug cgroup to its own file
date:   6 years ago
config: i386-buildonly-randconfig-005-20231120 (https://download.01.org/0day-ci/archive/20231121/202311211005.Qwt1FHrO-lkp@intel.com/config)
compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231121/202311211005.Qwt1FHrO-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/202311211005.Qwt1FHrO-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> kernel/cgroup/debug.c:1: warning: no structured comments found


vim +1 kernel/cgroup/debug.c

   > 1	#include <linux/ctype.h>
     2	#include <linux/mm.h>
     3	#include <linux/slab.h>
     4	

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: kernel/cgroup/debug.c:1: warning: no structured comments found
  2023-11-21  2:32 kernel/cgroup/debug.c:1: warning: no structured comments found kernel test robot
@ 2023-11-21  3:04 ` Waiman Long
  2023-11-21  4:11   ` Randy Dunlap
  0 siblings, 1 reply; 4+ messages in thread
From: Waiman Long @ 2023-11-21  3:04 UTC (permalink / raw)
  To: kernel test robot; +Cc: oe-kbuild-all, linux-kernel, Tejun Heo


On 11/20/23 21:32, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   98b1cc82c4affc16f5598d4fa14b1858671b2263
> commit: a28f8f5e995fe5964ae304444913536058f26e37 cgroup: Move debug cgroup to its own file
> date:   6 years ago
> config: i386-buildonly-randconfig-005-20231120 (https://download.01.org/0day-ci/archive/20231121/202311211005.Qwt1FHrO-lkp@intel.com/config)
> compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231121/202311211005.Qwt1FHrO-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/202311211005.Qwt1FHrO-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
>>> kernel/cgroup/debug.c:1: warning: no structured comments found
>
> vim +1 kernel/cgroup/debug.c
>
>     > 1	#include <linux/ctype.h>
>       2	#include <linux/mm.h>
>       3	#include <linux/slab.h>
>       4	

Yes, there is no function in this file with structure comments. This 
debug controller is used for debugging only and is not supposed to be 
used on production systems anyway. Its interface is unstable and so I 
don't see a need to properly document its interface.

I will leave this file as it is for now.

Cheers,
Longman


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: kernel/cgroup/debug.c:1: warning: no structured comments found
  2023-11-21  3:04 ` Waiman Long
@ 2023-11-21  4:11   ` Randy Dunlap
  2023-11-22  2:08     ` Yujie Liu
  0 siblings, 1 reply; 4+ messages in thread
From: Randy Dunlap @ 2023-11-21  4:11 UTC (permalink / raw)
  To: Waiman Long, kernel test robot; +Cc: oe-kbuild-all, linux-kernel, Tejun Heo

Hi,

On 11/20/23 19:04, Waiman Long wrote:
> 
> On 11/20/23 21:32, kernel test robot wrote:
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>> head:   98b1cc82c4affc16f5598d4fa14b1858671b2263
>> commit: a28f8f5e995fe5964ae304444913536058f26e37 cgroup: Move debug cgroup to its own file
>> date:   6 years ago
>> config: i386-buildonly-randconfig-005-20231120 (https://download.01.org/0day-ci/archive/20231121/202311211005.Qwt1FHrO-lkp@intel.com/config)
>> compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
>> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231121/202311211005.Qwt1FHrO-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/202311211005.Qwt1FHrO-lkp@intel.com/
>>
>> All warnings (new ones prefixed by >>):
>>
>>>> kernel/cgroup/debug.c:1: warning: no structured comments found
>>
>> vim +1 kernel/cgroup/debug.c
>>
>>     > 1    #include <linux/ctype.h>
>>       2    #include <linux/mm.h>
>>       3    #include <linux/slab.h>
>>       4   
> 
> Yes, there is no function in this file with structure comments. This debug controller is used for debugging only and is not supposed to be used on production systems anyway. Its interface is unstable and so I don't see a need to properly document its interface.
> 
> I will leave this file as it is for now.

Yes :)


This warning usually happens when some file is referenced in the
Documentation/ tree while building 'make htmldocs', e.g.

But this source file is not used in Documentation/ at all,
so this report seems to have some kind of a problem.
Also, I could not reproduce the warning.

-- 
~Randy

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: kernel/cgroup/debug.c:1: warning: no structured comments found
  2023-11-21  4:11   ` Randy Dunlap
@ 2023-11-22  2:08     ` Yujie Liu
  0 siblings, 0 replies; 4+ messages in thread
From: Yujie Liu @ 2023-11-22  2:08 UTC (permalink / raw)
  To: Randy Dunlap
  Cc: Waiman Long, kernel test robot, oe-kbuild-all, linux-kernel, Tejun Heo

On Mon, Nov 20, 2023 at 08:11:07PM -0800, Randy Dunlap wrote:
> Hi,
> 
> On 11/20/23 19:04, Waiman Long wrote:
> > 
> > On 11/20/23 21:32, kernel test robot wrote:
> >> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> >> head:   98b1cc82c4affc16f5598d4fa14b1858671b2263
> >> commit: a28f8f5e995fe5964ae304444913536058f26e37 cgroup: Move debug cgroup to its own file
> >> date:   6 years ago
> >> config: i386-buildonly-randconfig-005-20231120 (https://download.01.org/0day-ci/archive/20231121/202311211005.Qwt1FHrO-lkp@intel.com/config)
> >> compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu2) 7.5.0
> >> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231121/202311211005.Qwt1FHrO-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/202311211005.Qwt1FHrO-lkp@intel.com/
> >>
> >> All warnings (new ones prefixed by >>):
> >>
> >>>> kernel/cgroup/debug.c:1: warning: no structured comments found
> >>
> >> vim +1 kernel/cgroup/debug.c
> >>
> >>     > 1    #include <linux/ctype.h>
> >>       2    #include <linux/mm.h>
> >>       3    #include <linux/slab.h>
> >>       4   
> > 
> > Yes, there is no function in this file with structure comments. This debug controller is used for debugging only and is not supposed to be used on production systems anyway. Its interface is unstable and so I don't see a need to properly document its interface.
> > 
> > I will leave this file as it is for now.
> 
> Yes :)
> 
> 
> This warning usually happens when some file is referenced in the
> Documentation/ tree while building 'make htmldocs', e.g.
> 
> But this source file is not used in Documentation/ at all,
> so this report seems to have some kind of a problem.
> Also, I could not reproduce the warning.

Thanks for the information. We've configured the bot to ignore this
warning. Sorry for the noise.

Best Regards,
Yujie

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-11-22  2:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-11-21  2:32 kernel/cgroup/debug.c:1: warning: no structured comments found kernel test robot
2023-11-21  3:04 ` Waiman Long
2023-11-21  4:11   ` Randy Dunlap
2023-11-22  2:08     ` Yujie Liu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®