2017-04-05 19:24:34

by Mathias Krause

[permalink] [raw]
Subject: [PATCH v2] sparc: remove unused wp_works_ok macro

It's unused for ages, used to be required for ksyms.c back in the v1.1
times.

Signed-off-by: Mathias Krause <[email protected]>
Acked-by: David S. Miller <[email protected]>
---
Hi Dave,

same patch as v1 but as the tip folks took only the x86 parts, I think,
this one should go through the SPARC tree.

Cheers,
Mathias

arch/sparc/include/asm/processor_32.h | 6 ------
arch/sparc/include/asm/processor_64.h | 4 ----
2 files changed, 10 deletions(-)

diff --git a/arch/sparc/include/asm/processor_32.h b/arch/sparc/include/asm/processor_32.h
index 365d4cb267b4..dd27159819eb 100644
--- a/arch/sparc/include/asm/processor_32.h
+++ b/arch/sparc/include/asm/processor_32.h
@@ -18,12 +18,6 @@
#include <asm/signal.h>
#include <asm/page.h>

-/*
- * The sparc has no problems with write protection
- */
-#define wp_works_ok 1
-#define wp_works_ok__is_a_macro /* for versions in ksyms.c */
-
/* Whee, this is STACK_TOP + PAGE_SIZE and the lowest kernel address too...
* That one page is used to protect kernel from intruders, so that
* we can make our access_ok test faster
diff --git a/arch/sparc/include/asm/processor_64.h b/arch/sparc/include/asm/processor_64.h
index 6448cfc8292f..b58ee9018433 100644
--- a/arch/sparc/include/asm/processor_64.h
+++ b/arch/sparc/include/asm/processor_64.h
@@ -18,10 +18,6 @@
#include <asm/ptrace.h>
#include <asm/page.h>

-/* The sparc has no problems with write protection */
-#define wp_works_ok 1
-#define wp_works_ok__is_a_macro /* for versions in ksyms.c */
-
/*
* User lives in his very own context, and cannot reference us. Note
* that TASK_SIZE is a misnomer, it really gives maximum user virtual
--
1.7.10.4


2017-04-06 19:07:24

by David Miller

[permalink] [raw]
Subject: Re: [PATCH v2] sparc: remove unused wp_works_ok macro

From: Mathias Krause <[email protected]>
Date: Wed, 5 Apr 2017 21:24:20 +0200

> It's unused for ages, used to be required for ksyms.c back in the v1.1
> times.
>
> Signed-off-by: Mathias Krause <[email protected]>
> Acked-by: David S. Miller <[email protected]>
> ---
> Hi Dave,
>
> same patch as v1 but as the tip folks took only the x86 parts, I think,
> this one should go through the SPARC tree.

Ok, applied.