Re: [PATCH v4] powerpc: Avoid code patching freed init sections

2018-10-02 Thread Christophe LEROY
Le 03/10/2018 à 05:20, Michael Ellerman a écrit : Andreas Schwab writes: On Sep 14 2018, Michael Neuling wrote: This stops us from doing code patching in init sections after they've been freed. This breaks booting on PowerBook6,7, crashing very early. Crud, sorry. My CI setup tests

Re: [PATCH v4] powerpc: Avoid code patching freed init sections

2018-10-02 Thread Michael Ellerman
Andreas Schwab writes: > On Sep 14 2018, Michael Neuling wrote: > >> This stops us from doing code patching in init sections after they've >> been freed. > > This breaks booting on PowerBook6,7, crashing very early. Crud, sorry. My CI setup tests with the mac99 qemu model, but that boots happi

Re: [PATCH v4] powerpc: Avoid code patching freed init sections

2018-10-02 Thread Michael Neuling
On Tue, 2018-10-02 at 23:35 +0200, Andreas Schwab wrote: > On Sep 14 2018, Michael Neuling wrote: > > > This stops us from doing code patching in init sections after they've > > been freed. > > This breaks booting on PowerBook6,7, crashing very early. Sorry, Can you try? http://patchwork.ozlab

Re: [PATCH v4] powerpc: Avoid code patching freed init sections

2018-10-02 Thread Andreas Schwab
On Sep 14 2018, Michael Neuling wrote: > This stops us from doing code patching in init sections after they've > been freed. This breaks booting on PowerBook6,7, crashing very early. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510 2552 DF73 E7

Re: [v4] powerpc: Avoid code patching freed init sections

2018-10-01 Thread Michael Neuling
On Mon, 2018-10-01 at 13:25 +0200, Christophe LEROY wrote: > > Le 21/09/2018 à 13:59, Michael Ellerman a écrit : > > On Fri, 2018-09-14 at 01:14:11 UTC, Michael Neuling wrote: > > > This stops us from doing code patching in init sections after they've > > > been freed. > > > > > > In this chain:

Re: [v4] powerpc: Avoid code patching freed init sections

2018-10-01 Thread Christophe LEROY
Le 21/09/2018 à 13:59, Michael Ellerman a écrit : On Fri, 2018-09-14 at 01:14:11 UTC, Michael Neuling wrote: This stops us from doing code patching in init sections after they've been freed. In this chain: kvm_guest_init() -> kvm_use_magic_page() -> fault_in_pages_readable() -

Re: [v4] powerpc: Avoid code patching freed init sections

2018-09-21 Thread Michael Ellerman
On Fri, 2018-09-14 at 01:14:11 UTC, Michael Neuling wrote: > This stops us from doing code patching in init sections after they've > been freed. > > In this chain: > kvm_guest_init() -> > kvm_use_magic_page() -> > fault_in_pages_readable() -> >__get_user() -> > __get_u

Re: [PATCH v4] powerpc: Avoid code patching freed init sections

2018-09-18 Thread Michal Suchánek
On Tue, 18 Sep 2018 10:52:09 +0200 Christophe LEROY wrote: > > > Le 14/09/2018 à 06:22, Nicholas Piggin a écrit : > > On Fri, 14 Sep 2018 11:14:11 +1000 > > Michael Neuling wrote: > > > >> This stops us from doing code patching in init sections after > >> they've been freed. > >> > >> In this

Re: [PATCH v4] powerpc: Avoid code patching freed init sections

2018-09-18 Thread Christophe LEROY
Le 14/09/2018 à 06:22, Nicholas Piggin a écrit : On Fri, 14 Sep 2018 11:14:11 +1000 Michael Neuling wrote: This stops us from doing code patching in init sections after they've been freed. In this chain: kvm_guest_init() -> kvm_use_magic_page() -> fault_in_pages_readable() -

Re: [PATCH v4] powerpc: Avoid code patching freed init sections

2018-09-13 Thread Christophe LEROY
Le 14/09/2018 à 03:14, Michael Neuling a écrit : This stops us from doing code patching in init sections after they've been freed. In this chain: kvm_guest_init() -> kvm_use_magic_page() -> fault_in_pages_readable() -> __get_user() -> __get_user_nocheck() ->

Re: [PATCH v4] powerpc: Avoid code patching freed init sections

2018-09-13 Thread Nicholas Piggin
On Fri, 14 Sep 2018 11:14:11 +1000 Michael Neuling wrote: > This stops us from doing code patching in init sections after they've > been freed. > > In this chain: > kvm_guest_init() -> > kvm_use_magic_page() -> > fault_in_pages_readable() -> >__get_user() -> > __get_

[PATCH v4] powerpc: Avoid code patching freed init sections

2018-09-13 Thread Michael Neuling
This stops us from doing code patching in init sections after they've been freed. In this chain: kvm_guest_init() -> kvm_use_magic_page() -> fault_in_pages_readable() -> __get_user() -> __get_user_nocheck() -> barrier_nospec(); We have a code patching