linux-kernel - Re: [PATCH] s390/delay: Fix comment typo

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <YtfJgyVQzgkzSouS@osiris>
Date:   Wed, 20 Jul 2022 11:23:15 +0200
From:   Heiko Carstens <hca@...ux.ibm.com>
To:     Alexander Gordeev <agordeev@...ux.ibm.com>
Cc:     Jason Wang <wangborong@...rlc.com>, svens@...ux.ibm.com,
        gor@...ux.ibm.com, borntraeger@...ux.ibm.com,
        linux-s390@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] s390/delay: Fix comment typo

On Tue, Jul 19, 2022 at 08:22:15PM +0200, Alexander Gordeev wrote:
> On Fri, Jul 15, 2022 at 12:36:10PM +0800, Jason Wang wrote:
> > The double `that' is duplicated in line 19, remove one.
> > 
> > Signed-off-by: Jason Wang <wangborong@...rlc.com>
> > ---
> >  arch/s390/lib/delay.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/arch/s390/lib/delay.c b/arch/s390/lib/delay.c
> > index f7f5adea8940..c32bc8f7c1fd 100644
> > --- a/arch/s390/lib/delay.c
> > +++ b/arch/s390/lib/delay.c
> > @@ -16,7 +16,7 @@ void __delay(unsigned long loops)
> >          /*
> >           * To end the bloody studid and useless discussion about the
> >           * BogoMips number I took the liberty to define the __delay
> > -         * function in a way that that resulting BogoMips number will
> > +         * function in a way that resulting BogoMips number will
> >           * yield the megahertz number of the cpu. The important function
> >           * is udelay and that is done using the tod clock. -- martin.
> >           */
> 
> Applied, thanks!

Ahem :) That should have been actually "fixed" to "that the", instead
of removing the double "that". And by the way, just two lines above is
another obvious typo...

But seriously, the whole comment is entirely outdated and simply
wrong.  We don't use __delay() to calculate the BogoMips number, nor
does the function return a value that correlates with the megahertz
number of the cpu nowadays. It just loops a random time (where the cpu
is free to detect the pointless loop and optimize it away anyway).

So the right thing to do would be to remove the complete comment, and
maybe add a new comment - but please don't apply this patch.