* [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks
@ 2016-12-02 2:35 Ben Hutchings
2016-12-02 2:38 ` [PATCH 2/2] powerpc: Fix missing CRCs, add yet more asm-prototypes.h declarations Ben Hutchings
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Ben Hutchings @ 2016-12-02 2:35 UTC (permalink / raw)
To: linuxppc-dev; +Cc: Alexander Graf
[-- Attachment #1: Type: text/plain, Size: 1737 bytes --]
The symbols exported for use by MOL aren't getting CRCs and I was
about to fix that. But MOL is dead upstream, and the latest work on
it was to make it use KVM instead of its own kernel module. So remove
them instead.
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
---
arch/powerpc/kernel/head_32.S | 7 +------
arch/powerpc/mm/hash_low_32.S | 1 -
2 files changed, 1 insertion(+), 7 deletions(-)
diff --git a/arch/powerpc/kernel/head_32.S b/arch/powerpc/kernel/head_32.S
index 9d963547d243..87998430cd9b 100644
--- a/arch/powerpc/kernel/head_32.S
+++ b/arch/powerpc/kernel/head_32.S
@@ -735,11 +735,7 @@ END_MMU_FTR_SECTION_IFSET(MMU_FTR_NEED_DTLB_SW_LRU)
EXCEPTION(0x2c00, Trap_2c, unknown_exception, EXC_XFER_EE)
EXCEPTION(0x2d00, Trap_2d, unknown_exception, EXC_XFER_EE)
EXCEPTION(0x2e00, Trap_2e, unknown_exception, EXC_XFER_EE)
- EXCEPTION(0x2f00, MOLTrampoline, unknown_exception, EXC_XFER_EE_LITE)
-
- .globl mol_trampoline
- .set mol_trampoline, i0x2f00
- EXPORT_SYMBOL(mol_trampoline)
+ EXCEPTION(0x2f00, Trap_2f, unknown_exception, EXC_XFER_EE)
. = 0x3000
@@ -1289,7 +1285,6 @@ intercept_table:
.long 0, 0, 0, 0, 0, 0, 0, 0
.long 0, 0, 0, 0, 0, 0, 0, 0
.long 0, 0, 0, 0, 0, 0, 0, 0
-EXPORT_SYMBOL(intercept_table)
/* Room for two PTE pointers, usually the kernel and current user pointers
* to their respective root page table.
diff --git a/arch/powerpc/mm/hash_low_32.S b/arch/powerpc/mm/hash_low_32.S
index 09cc50c8dace..cddf14f60bf3 100644
--- a/arch/powerpc/mm/hash_low_32.S
+++ b/arch/powerpc/mm/hash_low_32.S
@@ -34,7 +34,6 @@
.globl mmu_hash_lock
mmu_hash_lock:
.space 4
-EXPORT_SYMBOL(mmu_hash_lock)
#endif /* CONFIG_SMP */
/*
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 811 bytes --]
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/2] powerpc: Fix missing CRCs, add yet more asm-prototypes.h declarations
2016-12-02 2:35 [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks Ben Hutchings
@ 2016-12-02 2:38 ` Ben Hutchings
2017-03-21 11:36 ` [2/2] " Michael Ellerman
2016-12-02 3:56 ` [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks Michael Ellerman
2017-03-21 11:36 ` [1/2] " Michael Ellerman
2 siblings, 1 reply; 6+ messages in thread
From: Ben Hutchings @ 2016-12-02 2:38 UTC (permalink / raw)
To: linuxppc-dev; +Cc: Daniel Axtens, Nicholas Piggin
[-- Attachment #1: Type: text/plain, Size: 1012 bytes --]
Add declarations for:
- __mfdcr, __mtdcr (if CONFIG_PPC_DCR_NATIVE=y; through <asm/dcr.h>)
- switch_mmu_context (if CONFIG_PPC_BOOK3S_64=n; through <asm/mmu_context.h>)
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
---
I checked all the symbols potentially exported from assembly under
arch/powerpc, and these seem to be the only ones not yet declared
through asm-prototypes.h (aside from those supporting Mac-on-Linux
which are removed by the previous patch).
Ben.
arch/powerpc/include/asm/asm-prototypes.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/powerpc/include/asm/asm-prototypes.h b/arch/powerpc/include/asm/asm-prototypes.h
index e0baba1535e6..ba0225ff38f8 100644
--- a/arch/powerpc/include/asm/asm-prototypes.h
+++ b/arch/powerpc/include/asm/asm-prototypes.h
@@ -18,6 +18,8 @@
#include <asm/checksum.h>
#include <asm/uaccess.h>
#include <asm/epapr_hcalls.h>
+#include <asm/dcr.h>
+#include <asm/mmu_context.h>
#include <uapi/asm/ucontext.h>
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 811 bytes --]
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks
2016-12-02 2:35 [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks Ben Hutchings
2016-12-02 2:38 ` [PATCH 2/2] powerpc: Fix missing CRCs, add yet more asm-prototypes.h declarations Ben Hutchings
@ 2016-12-02 3:56 ` Michael Ellerman
2016-12-02 5:05 ` Ben Hutchings
2017-03-21 11:36 ` [1/2] " Michael Ellerman
2 siblings, 1 reply; 6+ messages in thread
From: Michael Ellerman @ 2016-12-02 3:56 UTC (permalink / raw)
To: Ben Hutchings, linuxppc-dev
Cc: Alexander Graf, Benjamin Herrenschmidt, paulus
Ben Hutchings <ben@decadent.org.uk> writes:
> [ Unknown signature status ]
> The symbols exported for use by MOL aren't getting CRCs and I was
> about to fix that. But MOL is dead upstream, and the latest work on
> it was to make it use KVM instead of its own kernel module. So remove
> them instead.
Sounds good to me.
> diff --git a/arch/powerpc/kernel/head_32.S b/arch/powerpc/kernel/head_32.S
> index 9d963547d243..87998430cd9b 100644
> --- a/arch/powerpc/kernel/head_32.S
> +++ b/arch/powerpc/kernel/head_32.S
> @@ -735,11 +735,7 @@ END_MMU_FTR_SECTION_IFSET(MMU_FTR_NEED_DTLB_SW_LRU)
> EXCEPTION(0x2c00, Trap_2c, unknown_exception, EXC_XFER_EE)
> EXCEPTION(0x2d00, Trap_2d, unknown_exception, EXC_XFER_EE)
> EXCEPTION(0x2e00, Trap_2e, unknown_exception, EXC_XFER_EE)
> - EXCEPTION(0x2f00, MOLTrampoline, unknown_exception, EXC_XFER_EE_LITE)
> -
> - .globl mol_trampoline
> - .set mol_trampoline, i0x2f00
> - EXPORT_SYMBOL(mol_trampoline)
> + EXCEPTION(0x2f00, Trap_2f, unknown_exception, EXC_XFER_EE)
>
> . = 0x3000
>
> @@ -1289,7 +1285,6 @@ intercept_table:
> .long 0, 0, 0, 0, 0, 0, 0, 0
> .long 0, 0, 0, 0, 0, 0, 0, 0
> .long 0, 0, 0, 0, 0, 0, 0, 0
> -EXPORT_SYMBOL(intercept_table)
I don't see intercept_table used anywhere, so I think we can just remove
it entirely?
> diff --git a/arch/powerpc/mm/hash_low_32.S b/arch/powerpc/mm/hash_low_32.S
> index 09cc50c8dace..cddf14f60bf3 100644
> --- a/arch/powerpc/mm/hash_low_32.S
> +++ b/arch/powerpc/mm/hash_low_32.S
> @@ -34,7 +34,6 @@
> .globl mmu_hash_lock
> mmu_hash_lock:
> .space 4
> -EXPORT_SYMBOL(mmu_hash_lock)
> #endif /* CONFIG_SMP */
And now mmu_hash_lock is only used in this file, so doesn't need to be
global anymore.
I can fix both of those up here, will just wait a bit to see if anyone
else has comments.
cheers
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks
2016-12-02 3:56 ` [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks Michael Ellerman
@ 2016-12-02 5:05 ` Ben Hutchings
0 siblings, 0 replies; 6+ messages in thread
From: Ben Hutchings @ 2016-12-02 5:05 UTC (permalink / raw)
To: Michael Ellerman
Cc: linuxppc-dev, Alexander Graf, Benjamin Herrenschmidt, paulus
[-- Attachment #1: Type: text/plain, Size: 2353 bytes --]
On Fri, Dec 02, 2016 at 02:56:47PM +1100, Michael Ellerman wrote:
> Ben Hutchings <ben@decadent.org.uk> writes:
>
> > [ Unknown signature status ]
> > The symbols exported for use by MOL aren't getting CRCs and I was
> > about to fix that. But MOL is dead upstream, and the latest work on
> > it was to make it use KVM instead of its own kernel module. So remove
> > them instead.
>
> Sounds good to me.
>
> > diff --git a/arch/powerpc/kernel/head_32.S b/arch/powerpc/kernel/head_32.S
> > index 9d963547d243..87998430cd9b 100644
> > --- a/arch/powerpc/kernel/head_32.S
> > +++ b/arch/powerpc/kernel/head_32.S
> > @@ -735,11 +735,7 @@ END_MMU_FTR_SECTION_IFSET(MMU_FTR_NEED_DTLB_SW_LRU)
> > EXCEPTION(0x2c00, Trap_2c, unknown_exception, EXC_XFER_EE)
> > EXCEPTION(0x2d00, Trap_2d, unknown_exception, EXC_XFER_EE)
> > EXCEPTION(0x2e00, Trap_2e, unknown_exception, EXC_XFER_EE)
> > - EXCEPTION(0x2f00, MOLTrampoline, unknown_exception, EXC_XFER_EE_LITE)
> > -
> > - .globl mol_trampoline
> > - .set mol_trampoline, i0x2f00
> > - EXPORT_SYMBOL(mol_trampoline)
> > + EXCEPTION(0x2f00, Trap_2f, unknown_exception, EXC_XFER_EE)
> >
> > . = 0x3000
> >
> > @@ -1289,7 +1285,6 @@ intercept_table:
> > .long 0, 0, 0, 0, 0, 0, 0, 0
> > .long 0, 0, 0, 0, 0, 0, 0, 0
> > .long 0, 0, 0, 0, 0, 0, 0, 0
> > -EXPORT_SYMBOL(intercept_table)
>
> I don't see intercept_table used anywhere, so I think we can just remove
> it entirely?
That's strange, so it's actually constant data.
And... I've managed to confuse myself about which symbols are for
MOL. This one was actually used by rtlinux, which AFAICS is also
long dead.
> > diff --git a/arch/powerpc/mm/hash_low_32.S b/arch/powerpc/mm/hash_low_32.S
> > index 09cc50c8dace..cddf14f60bf3 100644
> > --- a/arch/powerpc/mm/hash_low_32.S
> > +++ b/arch/powerpc/mm/hash_low_32.S
> > @@ -34,7 +34,6 @@
> > .globl mmu_hash_lock
> > mmu_hash_lock:
> > .space 4
> > -EXPORT_SYMBOL(mmu_hash_lock)
> > #endif /* CONFIG_SMP */
>
> And now mmu_hash_lock is only used in this file, so doesn't need to be
> global anymore.
Right.
Ben.
> I can fix both of those up here, will just wait a bit to see if anyone
> else has comments.
>
> cheers
--
Ben Hutchings
It's easier to fight for one's principles than to live up to them.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 811 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [1/2] powerpc: Remove Mac-on-Linux hooks
2016-12-02 2:35 [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks Ben Hutchings
2016-12-02 2:38 ` [PATCH 2/2] powerpc: Fix missing CRCs, add yet more asm-prototypes.h declarations Ben Hutchings
2016-12-02 3:56 ` [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks Michael Ellerman
@ 2017-03-21 11:36 ` Michael Ellerman
2 siblings, 0 replies; 6+ messages in thread
From: Michael Ellerman @ 2017-03-21 11:36 UTC (permalink / raw)
To: Ben Hutchings, linuxppc-dev; +Cc: Alexander Graf
On Fri, 2016-12-02 at 02:35:52 UTC, Ben Hutchings wrote:
> The symbols exported for use by MOL aren't getting CRCs and I was
> about to fix that. But MOL is dead upstream, and the latest work on
> it was to make it use KVM instead of its own kernel module. So remove
> them instead.
>
> Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Applied to powerpc next, thanks.
https://git.kernel.org/powerpc/c/3072601375ffff92bddd0ce810dd64
cheers
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-03-21 11:36 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-02 2:35 [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks Ben Hutchings
2016-12-02 2:38 ` [PATCH 2/2] powerpc: Fix missing CRCs, add yet more asm-prototypes.h declarations Ben Hutchings
2017-03-21 11:36 ` [2/2] " Michael Ellerman
2016-12-02 3:56 ` [PATCH 1/2] powerpc: Remove Mac-on-Linux hooks Michael Ellerman
2016-12-02 5:05 ` Ben Hutchings
2017-03-21 11:36 ` [1/2] " Michael Ellerman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).