linux-kernel - Re: [PATCH] powerpc/32s: Setup the early hash table at all time.

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: <20201030140047.Horde.TJJqKGzG9vSGbMRNIj-MPg7@messagerie.c-s.fr>
Date:   Fri, 30 Oct 2020 14:00:47 +0100
From:   Christophe Leroy <christophe.leroy@...roup.eu>
To:     Andreas Schwab <schwab@...ux-m68k.org>
Cc:     linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org,
        Paul Mackerras <paulus@...ba.org>,
        Benjamin Herrenschmidt <benh@...nel.crashing.org>,
        Michael Ellerman <mpe@...erman.id.au>
Subject: Re: [PATCH] powerpc/32s: Setup the early hash table at all time.

Andreas Schwab <schwab@...ux-m68k.org> a écrit :

> On Okt 30 2020, Michael Ellerman wrote:
>
>> Andreas Schwab <schwab@...ux-m68k.org> writes:
>>> On Okt 01 2020, Christophe Leroy wrote:
>>>
>>>> At the time being, an early hash table is set up when
>>>> CONFIG_KASAN is selected.
>>>>
>>>> There is nothing wrong with setting such an early hash table
>>>> all the time, even if it is not used. This is a statically
>>>> allocated 256 kB table which lies in the init data section.
>>>>
>>>> This makes the code simpler and may in the future allow to
>>>> setup early IO mappings with fixmap instead of hard coding BATs.
>>>>
>>>> Put create_hpte() and flush_hash_pages() in the .ref.text section
>>>> in order to avoid warning for the reference to early_hash[]. This
>>>> reference is removed by MMU_init_hw_patch() before init memory is
>>>> freed.
>>>
>>> This breaks booting on the iBook G4.
>>
>> Do you get an oops or anything?
>
> Nope, nothing at all.
>

Under qemu it was working.

Can you share your .config, or are you using one of the defconfig ?

Christophe