From: Christophe Leroy <christophe.leroy@csgroup.eu>
To: Andreas Schwab <schwab@linux-m68k.org>
Cc: Paul Mackerras <paulus@samba.org>,
linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] powerpc/32s: Setup the early hash table at all time.
Date: Fri, 30 Oct 2020 14:00:47 +0100
Message-ID: <20201030140047.Horde.TJJqKGzG9vSGbMRNIj-MPg7@messagerie.c-s.fr> (raw)
In-Reply-To: <87v9esaxlv.fsf@igel.home>
Andreas Schwab <schwab@linux-m68k.org> a écrit :
> On Okt 30 2020, Michael Ellerman wrote:
>
>> Andreas Schwab <schwab@linux-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
next prev parent reply index
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-01 15:35 Christophe Leroy
2020-10-09 6:03 ` Michael Ellerman
2020-10-29 21:07 ` Andreas Schwab
2020-10-30 5:19 ` Michael Ellerman
2020-10-30 9:09 ` Andreas Schwab
2020-10-30 13:00 ` Christophe Leroy [this message]
2020-10-30 13:11 ` Andreas Schwab
2020-11-03 7:41 ` Christophe Leroy
2020-11-03 8:43 ` Andreas Schwab
2020-11-03 8:56 ` Andreas Schwab
2020-11-03 18:58 ` Serge Belyshev
2020-11-04 6:44 ` Christophe Leroy
2020-11-04 7:57 ` Serge Belyshev
2020-11-04 9:28 ` Serge Belyshev
2020-11-04 14:53 ` Serge Belyshev
2020-11-07 9:12 ` Christophe Leroy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201030140047.Horde.TJJqKGzG9vSGbMRNIj-MPg7@messagerie.c-s.fr \
--to=christophe.leroy@csgroup.eu \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=paulus@samba.org \
--cc=schwab@linux-m68k.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
LinuxPPC-Dev Archive on lore.kernel.org
Archives are clonable:
git clone --mirror https://lore.kernel.org/linuxppc-dev/0 linuxppc-dev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 linuxppc-dev linuxppc-dev/ https://lore.kernel.org/linuxppc-dev \
linuxppc-dev@lists.ozlabs.org linuxppc-dev@ozlabs.org
public-inbox-index linuxppc-dev
Example config snippet for mirrors
Newsgroup available over NNTP:
nntp://nntp.lore.kernel.org/org.ozlabs.lists.linuxppc-dev
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git