Re: [PATCH] doc: Always check kernel-doc

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

 



On Sat, Aug 19, 2023 at 2:14 AM Matthew Wilcox <willy@xxxxxxxxxxxxx> wrote:
>
> On Fri, Aug 18, 2023 at 10:49:37AM -0600, Jonathan Corbet wrote:
> > "Matthew Wilcox (Oracle)" <willy@xxxxxxxxxxxxx> writes:
> > > kernel-doc checks were initially enabled only for builds which had extra
> > > warnings enabled.  We have now eliminated enough kernel-doc warnings that
> > > we can enable kernel-doc checking by default.  This comes at a slight
> > > cost; for an allmodconfig build, make -j8 fs/ timings on my laptop
> > > increase by less than 5%:


Adding CONFIG_KDOC_CHECK or something
will allow people to avoid 5% build-time cost.
You can set "default y" or "default COMPILE_TEST".



> >
> > So I'm not opposed to this and can carry it in docs-next (after the
> > merge window, though, for something like this).  But, it seems to me, we
> > should copy Masahiro (added) on a build patch of this type.
>
> Thanks!  I've got a small collection of doc fixup patches redy to go;
> I'm going to spray them at maintainers and see what lands in this
> merge window.  I'm focusing on mm/ and fs/ since I know those areas
> better than others.  net/ is in good shape; only 25 lines of errors
> (21 of them in ceph).

Any single instance of warning may result in a rejection by Linus.
Anyway, we will see.




> I would suggest that we still have quite a lot of kernel-doc which is
> not incorporated into .rst files, which seems like a shame.  Does anyone
> have time to write a script that finds every file with kernel-doc in
> it, then finds which of those files do not have ".. kernel-doc::"
> lines in Documentation/ ?



-- 
Best Regards
Masahiro Yamada




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux