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
| ||
|
Date: Fri, 20 Mar 2020 17:00:35 +0100 From: Mauro Carvalho Chehab <mchehab+huawei@...nel.org> To: Thomas Gleixner <tglx@...utronix.de> Cc: Linux Doc Mailing List <linux-doc@...r.kernel.org>, linux-kernel@...r.kernel.org, Jonathan Corbet <corbet@....net>, Ingo Molnar <mingo@...hat.com>, Peter Zijlstra <peterz@...radead.org>, Darren Hart <dvhart@...radead.org> Subject: Re: [PATCH 04/17] kernel: futex.c: get rid of a docs build warning Em Fri, 20 Mar 2020 16:28:23 +0100 Thomas Gleixner <tglx@...utronix.de> escreveu: > Mauro Carvalho Chehab <mchehab+huawei@...nel.org> writes: > > The subject prefix for this is: 'futex:' Ok! > > > Adjust whitespaces and blank lines in order to get rid of this: > > > > ./kernel/futex.c:491: WARNING: Definition list ends without a blank line; unexpected unindent. > > > > Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@...nel.org> > > --- > > kernel/futex.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/kernel/futex.c b/kernel/futex.c > > index 67f004133061..dda6ddbc2e7d 100644 > > --- a/kernel/futex.c > > +++ b/kernel/futex.c > > @@ -486,7 +486,8 @@ static u64 get_inode_sequence_number(struct inode *inode) > > * The key words are stored in @key on success. > > * > > * For shared mappings (when @fshared), the key is: > > - * ( inode->i_sequence, page->index, offset_within_page ) > > + * ( inode->i_sequence, page->index, offset_within_page ) > > Sigh. Is there no better way to make this look sane both in the file and > in the docs? The enclosed patch would do the trick. I noticed another problem on it, however: "!@...ared" is not properly parsed by kernel-docs. It basically converts it to "!**fshared**", with is not what we want. IMHO, this specific case should be handled by kernel-doc script. I'll write a patch addressing this issue. Regards, Mauro --- [PATCH 04/17 v2] kernel: futex: get rid of a docs build warning Adjust whitespaces and blank lines in order to get rid of this: ./kernel/futex.c:491: WARNING: Definition list ends without a blank line; unexpected unindent. Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@...nel.org> diff --git a/kernel/futex.c b/kernel/futex.c index 67f004133061..81ad379cbadf 100644 --- a/kernel/futex.c +++ b/kernel/futex.c @@ -486,10 +486,13 @@ static u64 get_inode_sequence_number(struct inode *inode) * The key words are stored in @key on success. * * For shared mappings (when @fshared), the key is: + * * ( inode->i_sequence, page->index, offset_within_page ) + * * [ also see get_inode_sequence_number() ] * * For private mappings (or when !@...ared), the key is: + * * ( current->mm, address, 0 ) * * This allows (cross process, where applicable) identification of the futex
Powered by blists - more mailing lists