linux-kernel - Re: [OpenRISC] [PATCH 02/13] openrisc: define CPU_BIG

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20170901082803.GI2609@lianli.shorne-pla.net>
Date:   Fri, 1 Sep 2017 17:28:03 +0900
From:   Stafford Horne <shorne@...il.com>
To:     Geert Uytterhoeven <geert@...ux-m68k.org>
Cc:     LKML <linux-kernel@...r.kernel.org>,
        Jonas Bonn <jonas@...thpole.se>,
        Openrisc <openrisc@...ts.librecores.org>
Subject: Re: [OpenRISC] [PATCH 02/13] openrisc: define CPU_BIG_ENDIAN as true

On Fri, Sep 01, 2017 at 10:06:49AM +0200, Geert Uytterhoeven wrote:
> On Wed, Aug 30, 2017 at 11:58 PM, Stafford Horne <shorne@...il.com> wrote:
> > While working on SMP for OpenRISC I found this is needed for qrwlocks to
> > work correctly.  OpenRISC is big endian so this should have been here
> > already.
> >
> > Signed-off-by: Stafford Horne <shorne@...il.com>
> > ---
> >  arch/openrisc/Kconfig | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/arch/openrisc/Kconfig b/arch/openrisc/Kconfig
> > index ba8ae92ec5fb..8ddfc92ab813 100644
> > --- a/arch/openrisc/Kconfig
> > +++ b/arch/openrisc/Kconfig
> > @@ -29,6 +29,9 @@ config OPENRISC
> >         select CPU_NO_EFFICIENT_FFS if !OPENRISC_HAVE_INST_FF1
> >         select NO_BOOTMEM
> >
> > +config CPU_BIG_ENDIAN
> > +       def_bool y
> > +
> 
> This is already in Andrew's tree and linux-next, as part of commit
> 410ae809fee6c792 ("arch: define CPU_BIG_ENDIAN for all fixed big endian
> archs").

Thank you,  Yes I am cc'd on those patches.  I had added this patch myself
some time before 10ae809fee6c792 ("arch: define CPU_BIG_ENDIAN for all
fixed big endian archs") appeared.

Once that gets merged I will rebase and this will disappear.

-Stafford

> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds