lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20171212101819.oq3mtmifqbovytgh@gmail.com> Date: Tue, 12 Dec 2017 11:18:19 +0100 From: Ingo Molnar <mingo@...nel.org> To: Jonathan Corbet <corbet@....net> Cc: Randy Dunlap <rdunlap@...radead.org>, Ingo Molnar <mingo@...hat.com>, Peter Zijlstra <peterz@...radead.org>, LKML <linux-kernel@...r.kernel.org>, Al Viro <viro@...IV.linux.org.uk> Subject: Re: sched: Update kerneldoc for sched_rr_get_interval() * Jonathan Corbet <corbet@....net> wrote: > On Mon, 11 Dec 2017 15:14:21 -0800 > Randy Dunlap <rdunlap@...radead.org> wrote: > > > On 12/11/2017 02:56 PM, Jonathan Corbet wrote: > > > Commit abca5fc535a3e ("sched_rr_get_interval(): move compat to native, get > > > rid of set_fs()") changed the prototype of that function but left the > > > kerneldoc comments unchanged, leading to these docs-build warnings: > > > > > > ./kernel/sched/core.c:5113: warning: No description found for parameter 't' > > > ./kernel/sched/core.c:5113: warning: Excess function parameter 'interval' > > > description in 'sched_rr_get_interval' > > > > > > Update the documentation (noting that it's not a user-space address > > > anymore) and make the docs build a little quieter. > > > > > > Signed-off-by: Jonathan Corbet <corbet@....net> > > > > I sent a patch for this on DEC-03 to Al, Ingo, PeterZ, and LKML. > > https://marc.info/?l=linux-kernel&m=151233595424349&w=2 > > > > or I could ack your patch. :) > > OK, looking more closely, your patch is more correct than mine and is the > one that should be applied. Sorry I didn't see it before. Yeah, I have applied Randy's patch to the sched/urgent tree yesterday: 2064a5ab0470: sched/core: Fix kernel-doc warnings after code movement It's still not pushed out though, because I'm chasing an (unrelated) regression. Thanks, Ingo
Powered by blists - more mailing lists