linux-kernel - Re: [PATCH v6 0/6] KASan for arm

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]

Date:   Fri, 17 Jan 2020 11:55:11 -0800
From:   Florian Fainelli <f.fainelli@...il.com>
To:     Linus Walleij <linus.walleij@...aro.org>,
        Florian Fainelli <f.fainelli@...il.com>
Cc:     Marco Felsch <m.felsch@...gutronix.de>,
        Mark Rutland <mark.rutland@....com>,
        Alexandre Belloni <alexandre.belloni@...tlin.com>,
        Michal Hocko <mhocko@...e.com>,
        Julien Thierry <julien.thierry@....com>,
        Catalin Marinas <catalin.marinas@....com>,
        Christoffer Dall <christoffer.dall@....com>,
        David Howells <dhowells@...hat.com>,
        Masahiro Yamada <yamada.masahiro@...ionext.com>,
        Andrey Ryabinin <ryabinin.a.a@...il.com>,
        Alexander Potapenko <glider@...gle.com>,
        kvmarm@...ts.cs.columbia.edu, Jonathan Corbet <corbet@....net>,
        Abbott Liu <liuwenliang@...wei.com>,
        Daniel Lezcano <daniel.lezcano@...aro.org>,
        Russell King <linux@...linux.org.uk>,
        kasan-dev <kasan-dev@...glegroups.com>,
        Geert Uytterhoeven <geert@...ux-m68k.org>,
        Dmitry Vyukov <dvyukov@...gle.com>,
        bcm-kernel-feedback-list <bcm-kernel-feedback-list@...adcom.com>,
        drjones@...hat.com, Vladimir Murzin <vladimir.murzin@....com>,
        Kees Cook <keescook@...omium.org>,
        Arnd Bergmann <arnd@...db.de>,
        Marc Zyngier <marc.zyngier@....com>,
        Andre Przywara <andre.przywara@....com>,
        Philippe Ombredanne <pombredanne@...b.com>,
        Jinbum Park <jinb.park7@...il.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        Sascha Hauer <kernel@...gutronix.de>,
        Linux ARM <linux-arm-kernel@...ts.infradead.org>,
        Nicolas Pitre <nico@...xnic.net>,
        Greg KH <gregkh@...uxfoundation.org>,
        Ard Biesheuvel <ard.biesheuvel@...aro.org>,
        Linux Doc Mailing List <linux-doc@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Rob Landley <rob@...dley.net>, philip@....systems,
        Andrew Morton <akpm@...ux-foundation.org>,
        Thomas Garnier <thgarnie@...gle.com>,
        "Kirill A . Shutemov" <kirill.shutemov@...ux.intel.com>
Subject: Re: [PATCH v6 0/6] KASan for arm

On 1/17/20 2:13 AM, Linus Walleij wrote:
> On Tue, Nov 19, 2019 at 1:14 AM Florian Fainelli <f.fainelli@...il.com> wrote:
>> On 11/15/19 3:44 AM, Marco Felsch wrote:
>>>
>>> With your v7 it is working on my imx6 but unfortunately I can't run my
>>> gstreamer testcase. My CPU load goes to 100% after starting gstreamer
>>> and nothing happens.. But the test_kasan module works =) So I decided to
>>> check a imx6quadplus but this target did not boot.. I used another
>>> toolchain for the imx6quadplus gcc-9 instead of gcc-8. So it seems that
>>> something went wrong during compilation. Because you didn't changed
>>> something within the logic.
>>>
>>> I wonder why we must not define the CONFIG_KASAN_SHADOW_OFFSET for arm.
>>
>> That is was oversight. I have pushed updates to the branch here:
>>
>> https://github.com/ffainelli/linux/pull/new/kasan-v7
> 
> I just git Kasan back on my radar because it needs to be fixed some day.
> 
> I took this branch for a ride on some QEMU and some real hardware.
> Here I use the test module and just hacked it into the kernel instead of
> as a module, it then crashes predictably but performs all the KASan
> tests first and it works file, as in provokes the right warnings from
> KASan.
> 
> Tested systems:
> 
> QEMU ARM RealView PBA8
> QEMU ARM RealView PBX A9
> QEMU ARM Versatile AB
> Hardware Integrator CP
> Hardware Versatile AB with IB2
> 
> Can we start to submit these patches to Russell's patch tracker?
> Any more testing I should be doing?

Let me submit and rebase v7 get the auto builders some days to see if it
exposes a new build issue and then we toss it to RMK's patch tracker and
fix bugs from there?
-- 
Florian