linux-kernel - Re: Ping: [PATCH] LoongArch: Slightly clean up drdtime

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: <CAAhV-H4ELPjUgNOGQiU8o6zBAcE4=9JRLyoFcQprhGMrTBhd-A@mail.gmail.com>
Date:   Tue, 5 Dec 2023 16:07:52 +0800
From:   Huacai Chen <chenhuacai@...nel.org>
To:     Xi Ruoyao <xry111@...111.site>
Cc:     WANG Xuerui <kernel@...0n.name>, loongarch@...ts.linux.dev,
        linux-kernel@...r.kernel.org
Subject: Re: Ping: [PATCH] LoongArch: Slightly clean up drdtime

Queued, thanks.

Huacai

On Tue, Dec 5, 2023 at 4:01 PM Xi Ruoyao <xry111@...111.site> wrote:
>
> Ping.
>
> On Sun, 2023-11-26 at 20:17 +0800, Xi Ruoyao wrote:
> > As we are just discarding the stable clock ID, simply write it into
> > $zero instead of allocating a temporary register.
> >
> > Signed-off-by: Xi Ruoyao <xry111@...111.site>
> > ---
> >  arch/loongarch/include/asm/loongarch.h | 5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/loongarch/include/asm/loongarch.h b/arch/loongarch/include/asm/loongarch.h
> > index 9b4957cefa8a..46366e783c84 100644
> > --- a/arch/loongarch/include/asm/loongarch.h
> > +++ b/arch/loongarch/include/asm/loongarch.h
> > @@ -1098,12 +1098,11 @@
> >
> >  static __always_inline u64 drdtime(void)
> >  {
> > -     int rID = 0;
> >       u64 val = 0;
> >
> >       __asm__ __volatile__(
> > -             "rdtime.d %0, %1 \n\t"
> > -             : "=r"(val), "=r"(rID)
> > +             "rdtime.d %0, $zero\n\t"
> > +             : "=r"(val)
> >               :
> >               );
> >       return val;
>
> --
> Xi Ruoyao <xry111@...111.site>
> School of Aerospace Science and Technology, Xidian University