linux-kernel - Re: [PATCH] MIPS: ralink: select WEAK_REORDERING_BEYOND

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]

Date:   Mon, 15 Mar 2021 17:40:15 -0700
From:   Ilya Lipnitskiy <ilya.lipnitskiy@...il.com>
To:     Thomas Bogendoerfer <tsbogend@...ha.franken.de>
Cc:     John Crispin <john@...ozen.org>,
        linux-mediatek@...ts.infradead.org, linux-mips@...r.kernel.org,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] MIPS: ralink: select WEAK_REORDERING_BEYOND_LLSC on MT7621

Hi Thomas,

On Fri, Mar 12, 2021 at 2:34 AM Thomas Bogendoerfer
<tsbogend@...ha.franken.de> wrote:
>
> On Sun, Mar 07, 2021 at 11:08:06AM -0800, Ilya Lipnitskiy wrote:
> > Upstream a long-standing OpenWrt patch for RALINK MT7621 SoC. Selecting
> > WEAK_REORDERING_BEYOND_LLSC fixes random kernel hangs. This bug and fix
> > was reported by MediaTek WCN division [0].
> >
> > [0]: https://lists.infradead.org/pipermail/lede-commits/2017-August/004537.html
>
> I don't see the Mediatek WCN reporting there. Looking at the 1004K user
> manual I couldn't find a notice about such behaviour. So this looks
> more like a fix for papering over the real bug by just introducing
> a few more syncs. Could you please point me to where this is
> really documented ?
You may actually be right. After doing some testing we may not need
this change after all. Let me drop this patch for now. FYI, the
following commit may have fixed LL SC issues this change was covering
up: https://lore.kernel.org/lkml/20190613134933.048961704@infradead.org/

>
> Thomas.
>
> --
> Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
> good idea.                                                [ RFC1925, 2.3 ]
Ilya