* [PATCH 0/3] Drop unused MAX_PHYSADDR_BITS
@ 2020-07-23 23:15 ` Arvind Sankar
0 siblings, 0 replies; 21+ messages in thread
From: Arvind Sankar @ 2020-07-23 23:15 UTC (permalink / raw)
To: x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra, linux-sh,
Yoshinori Sato, Rich Felker, sparclinux, David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
This #define is not used anywhere, and has the wrong value on x86_64.
I tried digging into the history a bit, but it seems to have been unused
even in the initial merge of sparsemem in v2.6.13, when it was first
defined.
Arvind Sankar (3):
x86/mm: Drop unused MAX_PHYSADDR_BITS
sh/mm: Drop unused MAX_PHYSADDR_BITS
sparc: Drop unused MAX_PHYSADDR_BITS
arch/sh/include/asm/sparsemem.h | 4 +---
arch/sparc/include/asm/sparsemem.h | 1 -
arch/x86/include/asm/sparsemem.h | 6 +-----
3 files changed, 2 insertions(+), 9 deletions(-)
--
2.26.2
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 1/3] x86/mm: Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
@ 2020-07-23 23:15 ` Arvind Sankar
-1 siblings, 0 replies; 21+ messages in thread
From: Arvind Sankar @ 2020-07-23 23:15 UTC (permalink / raw)
To: x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra, linux-sh,
Yoshinori Sato, Rich Felker, sparclinux, David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
The macro is not used anywhere, and has an incorrect value (going by the
comment) on x86_64 since commit
c898faf91b3e ("x86: 46 bit physical address support on 64 bits")
To avoid confusion, just remove the definition.
Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
---
arch/x86/include/asm/sparsemem.h | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/arch/x86/include/asm/sparsemem.h b/arch/x86/include/asm/sparsemem.h
index 199218719a86..6bfc878f6771 100644
--- a/arch/x86/include/asm/sparsemem.h
+++ b/arch/x86/include/asm/sparsemem.h
@@ -10,24 +10,20 @@
* field of the struct page
*
* SECTION_SIZE_BITS 2^n: size of each section
- * MAX_PHYSADDR_BITS 2^n: max size of physical address space
- * MAX_PHYSMEM_BITS 2^n: how much memory we can have in that space
+ * MAX_PHYSMEM_BITS 2^n: max size of physical address space
*
*/
#ifdef CONFIG_X86_32
# ifdef CONFIG_X86_PAE
# define SECTION_SIZE_BITS 29
-# define MAX_PHYSADDR_BITS 36
# define MAX_PHYSMEM_BITS 36
# else
# define SECTION_SIZE_BITS 26
-# define MAX_PHYSADDR_BITS 32
# define MAX_PHYSMEM_BITS 32
# endif
#else /* CONFIG_X86_32 */
# define SECTION_SIZE_BITS 27 /* matt - 128 is convenient right now */
-# define MAX_PHYSADDR_BITS (pgtable_l5_enabled() ? 52 : 44)
# define MAX_PHYSMEM_BITS (pgtable_l5_enabled() ? 52 : 46)
#endif
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [PATCH 1/3] x86/mm: Drop unused MAX_PHYSADDR_BITS
@ 2020-07-23 23:15 ` Arvind Sankar
0 siblings, 0 replies; 21+ messages in thread
From: Arvind Sankar @ 2020-07-23 23:15 UTC (permalink / raw)
To: x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra, linux-sh,
Yoshinori Sato, Rich Felker, sparclinux, David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
The macro is not used anywhere, and has an incorrect value (going by the
comment) on x86_64 since commit
c898faf91b3e ("x86: 46 bit physical address support on 64 bits")
To avoid confusion, just remove the definition.
Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
---
arch/x86/include/asm/sparsemem.h | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/arch/x86/include/asm/sparsemem.h b/arch/x86/include/asm/sparsemem.h
index 199218719a86..6bfc878f6771 100644
--- a/arch/x86/include/asm/sparsemem.h
+++ b/arch/x86/include/asm/sparsemem.h
@@ -10,24 +10,20 @@
* field of the struct page
*
* SECTION_SIZE_BITS 2^n: size of each section
- * MAX_PHYSADDR_BITS 2^n: max size of physical address space
- * MAX_PHYSMEM_BITS 2^n: how much memory we can have in that space
+ * MAX_PHYSMEM_BITS 2^n: max size of physical address space
*
*/
#ifdef CONFIG_X86_32
# ifdef CONFIG_X86_PAE
# define SECTION_SIZE_BITS 29
-# define MAX_PHYSADDR_BITS 36
# define MAX_PHYSMEM_BITS 36
# else
# define SECTION_SIZE_BITS 26
-# define MAX_PHYSADDR_BITS 32
# define MAX_PHYSMEM_BITS 32
# endif
#else /* CONFIG_X86_32 */
# define SECTION_SIZE_BITS 27 /* matt - 128 is convenient right now */
-# define MAX_PHYSADDR_BITS (pgtable_l5_enabled() ? 52 : 44)
# define MAX_PHYSMEM_BITS (pgtable_l5_enabled() ? 52 : 46)
#endif
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread* [tip: x86/cleanups] x86/mm: Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
(?)
@ 2020-07-24 7:56 ` tip-bot2 for Arvind Sankar
-1 siblings, 0 replies; 21+ messages in thread
From: tip-bot2 for Arvind Sankar @ 2020-07-24 7:56 UTC (permalink / raw)
To: linux-tip-commits; +Cc: Arvind Sankar, Thomas Gleixner, x86, LKML
The following commit has been merged into the x86/cleanups branch of tip:
Commit-ID: 0a787b28b7a375ad9d5c77bc3922ae1a8305239e
Gitweb: https://git.kernel.org/tip/0a787b28b7a375ad9d5c77bc3922ae1a8305239e
Author: Arvind Sankar <nivedita@alum.mit.edu>
AuthorDate: Thu, 23 Jul 2020 19:15:42 -04:00
Committer: Thomas Gleixner <tglx@linutronix.de>
CommitterDate: Fri, 24 Jul 2020 09:53:06 +02:00
x86/mm: Drop unused MAX_PHYSADDR_BITS
The macro is not used anywhere, and has an incorrect value (going by the
comment) on x86_64 since commit c898faf91b3e ("x86: 46 bit physical address
support on 64 bits")
To avoid confusion, just remove the definition.
Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lkml.kernel.org/r/20200723231544.17274-2-nivedita@alum.mit.edu
---
arch/x86/include/asm/sparsemem.h | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/arch/x86/include/asm/sparsemem.h b/arch/x86/include/asm/sparsemem.h
index 1992187..6bfc878 100644
--- a/arch/x86/include/asm/sparsemem.h
+++ b/arch/x86/include/asm/sparsemem.h
@@ -10,24 +10,20 @@
* field of the struct page
*
* SECTION_SIZE_BITS 2^n: size of each section
- * MAX_PHYSADDR_BITS 2^n: max size of physical address space
- * MAX_PHYSMEM_BITS 2^n: how much memory we can have in that space
+ * MAX_PHYSMEM_BITS 2^n: max size of physical address space
*
*/
#ifdef CONFIG_X86_32
# ifdef CONFIG_X86_PAE
# define SECTION_SIZE_BITS 29
-# define MAX_PHYSADDR_BITS 36
# define MAX_PHYSMEM_BITS 36
# else
# define SECTION_SIZE_BITS 26
-# define MAX_PHYSADDR_BITS 32
# define MAX_PHYSMEM_BITS 32
# endif
#else /* CONFIG_X86_32 */
# define SECTION_SIZE_BITS 27 /* matt - 128 is convenient right now */
-# define MAX_PHYSADDR_BITS (pgtable_l5_enabled() ? 52 : 44)
# define MAX_PHYSMEM_BITS (pgtable_l5_enabled() ? 52 : 46)
#endif
^ permalink raw reply related [flat|nested] 21+ messages in thread* Re: [PATCH 1/3] x86/mm: Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
@ 2020-07-24 8:22 ` David Hildenbrand
-1 siblings, 0 replies; 21+ messages in thread
From: David Hildenbrand @ 2020-07-24 8:22 UTC (permalink / raw)
To: Arvind Sankar, x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra,
linux-sh, Yoshinori Sato, Rich Felker, sparclinux,
David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
On 24.07.20 01:15, Arvind Sankar wrote:
> The macro is not used anywhere, and has an incorrect value (going by the
> comment) on x86_64 since commit
> c898faf91b3e ("x86: 46 bit physical address support on 64 bits")
>
> To avoid confusion, just remove the definition.
>
> Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
> ---
> arch/x86/include/asm/sparsemem.h | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/arch/x86/include/asm/sparsemem.h b/arch/x86/include/asm/sparsemem.h
> index 199218719a86..6bfc878f6771 100644
> --- a/arch/x86/include/asm/sparsemem.h
> +++ b/arch/x86/include/asm/sparsemem.h
> @@ -10,24 +10,20 @@
> * field of the struct page
> *
> * SECTION_SIZE_BITS 2^n: size of each section
> - * MAX_PHYSADDR_BITS 2^n: max size of physical address space
> - * MAX_PHYSMEM_BITS 2^n: how much memory we can have in that space
> + * MAX_PHYSMEM_BITS 2^n: max size of physical address space
> *
> */
>
> #ifdef CONFIG_X86_32
> # ifdef CONFIG_X86_PAE
> # define SECTION_SIZE_BITS 29
> -# define MAX_PHYSADDR_BITS 36
> # define MAX_PHYSMEM_BITS 36
> # else
> # define SECTION_SIZE_BITS 26
> -# define MAX_PHYSADDR_BITS 32
> # define MAX_PHYSMEM_BITS 32
> # endif
> #else /* CONFIG_X86_32 */
> # define SECTION_SIZE_BITS 27 /* matt - 128 is convenient right now */
> -# define MAX_PHYSADDR_BITS (pgtable_l5_enabled() ? 52 : 44)
> # define MAX_PHYSMEM_BITS (pgtable_l5_enabled() ? 52 : 46)
> #endif
>
>
Reviewed-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 21+ messages in thread* Re: [PATCH 1/3] x86/mm: Drop unused MAX_PHYSADDR_BITS
@ 2020-07-24 8:22 ` David Hildenbrand
0 siblings, 0 replies; 21+ messages in thread
From: David Hildenbrand @ 2020-07-24 8:22 UTC (permalink / raw)
To: Arvind Sankar, x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra,
linux-sh, Yoshinori Sato, Rich Felker, sparclinux,
David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
On 24.07.20 01:15, Arvind Sankar wrote:
> The macro is not used anywhere, and has an incorrect value (going by the
> comment) on x86_64 since commit
> c898faf91b3e ("x86: 46 bit physical address support on 64 bits")
>
> To avoid confusion, just remove the definition.
>
> Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
> ---
> arch/x86/include/asm/sparsemem.h | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/arch/x86/include/asm/sparsemem.h b/arch/x86/include/asm/sparsemem.h
> index 199218719a86..6bfc878f6771 100644
> --- a/arch/x86/include/asm/sparsemem.h
> +++ b/arch/x86/include/asm/sparsemem.h
> @@ -10,24 +10,20 @@
> * field of the struct page
> *
> * SECTION_SIZE_BITS 2^n: size of each section
> - * MAX_PHYSADDR_BITS 2^n: max size of physical address space
> - * MAX_PHYSMEM_BITS 2^n: how much memory we can have in that space
> + * MAX_PHYSMEM_BITS 2^n: max size of physical address space
> *
> */
>
> #ifdef CONFIG_X86_32
> # ifdef CONFIG_X86_PAE
> # define SECTION_SIZE_BITS 29
> -# define MAX_PHYSADDR_BITS 36
> # define MAX_PHYSMEM_BITS 36
> # else
> # define SECTION_SIZE_BITS 26
> -# define MAX_PHYSADDR_BITS 32
> # define MAX_PHYSMEM_BITS 32
> # endif
> #else /* CONFIG_X86_32 */
> # define SECTION_SIZE_BITS 27 /* matt - 128 is convenient right now */
> -# define MAX_PHYSADDR_BITS (pgtable_l5_enabled() ? 52 : 44)
> # define MAX_PHYSMEM_BITS (pgtable_l5_enabled() ? 52 : 46)
> #endif
>
>
Reviewed-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 2/3] sh/mm: Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
@ 2020-07-23 23:15 ` Arvind Sankar
-1 siblings, 0 replies; 21+ messages in thread
From: Arvind Sankar @ 2020-07-23 23:15 UTC (permalink / raw)
To: x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra, linux-sh,
Yoshinori Sato, Rich Felker, sparclinux, David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
The macro is not used anywhere, so remove the definition.
Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
---
arch/sh/include/asm/sparsemem.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/arch/sh/include/asm/sparsemem.h b/arch/sh/include/asm/sparsemem.h
index 4eb899751e45..084706bb8cca 100644
--- a/arch/sh/include/asm/sparsemem.h
+++ b/arch/sh/include/asm/sparsemem.h
@@ -5,11 +5,9 @@
#ifdef __KERNEL__
/*
* SECTION_SIZE_BITS 2^N: how big each section will be
- * MAX_PHYSADDR_BITS 2^N: how much physical address space we have
- * MAX_PHYSMEM_BITS 2^N: how much memory we can have in that space
+ * MAX_PHYSMEM_BITS 2^N: how much physical address space we have
*/
#define SECTION_SIZE_BITS 26
-#define MAX_PHYSADDR_BITS 32
#define MAX_PHYSMEM_BITS 32
#endif
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 2/3] sh/mm: Drop unused MAX_PHYSADDR_BITS
@ 2020-07-23 23:15 ` Arvind Sankar
0 siblings, 0 replies; 21+ messages in thread
From: Arvind Sankar @ 2020-07-23 23:15 UTC (permalink / raw)
To: x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra, linux-sh,
Yoshinori Sato, Rich Felker, sparclinux, David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
The macro is not used anywhere, so remove the definition.
Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
---
arch/sh/include/asm/sparsemem.h | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/arch/sh/include/asm/sparsemem.h b/arch/sh/include/asm/sparsemem.h
index 4eb899751e45..084706bb8cca 100644
--- a/arch/sh/include/asm/sparsemem.h
+++ b/arch/sh/include/asm/sparsemem.h
@@ -5,11 +5,9 @@
#ifdef __KERNEL__
/*
* SECTION_SIZE_BITS 2^N: how big each section will be
- * MAX_PHYSADDR_BITS 2^N: how much physical address space we have
- * MAX_PHYSMEM_BITS 2^N: how much memory we can have in that space
+ * MAX_PHYSMEM_BITS 2^N: how much physical address space we have
*/
#define SECTION_SIZE_BITS 26
-#define MAX_PHYSADDR_BITS 32
#define MAX_PHYSMEM_BITS 32
#endif
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH 2/3] sh/mm: Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
@ 2020-07-24 8:22 ` David Hildenbrand
-1 siblings, 0 replies; 21+ messages in thread
From: David Hildenbrand @ 2020-07-24 8:22 UTC (permalink / raw)
To: Arvind Sankar, x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra,
linux-sh, Yoshinori Sato, Rich Felker, sparclinux,
David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
On 24.07.20 01:15, Arvind Sankar wrote:
> The macro is not used anywhere, so remove the definition.
>
> Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
> ---
> arch/sh/include/asm/sparsemem.h | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/arch/sh/include/asm/sparsemem.h b/arch/sh/include/asm/sparsemem.h
> index 4eb899751e45..084706bb8cca 100644
> --- a/arch/sh/include/asm/sparsemem.h
> +++ b/arch/sh/include/asm/sparsemem.h
> @@ -5,11 +5,9 @@
> #ifdef __KERNEL__
> /*
> * SECTION_SIZE_BITS 2^N: how big each section will be
> - * MAX_PHYSADDR_BITS 2^N: how much physical address space we have
> - * MAX_PHYSMEM_BITS 2^N: how much memory we can have in that space
> + * MAX_PHYSMEM_BITS 2^N: how much physical address space we have
> */
> #define SECTION_SIZE_BITS 26
> -#define MAX_PHYSADDR_BITS 32
> #define MAX_PHYSMEM_BITS 32
>
> #endif
>
Reviewed-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 2/3] sh/mm: Drop unused MAX_PHYSADDR_BITS
@ 2020-07-24 8:22 ` David Hildenbrand
0 siblings, 0 replies; 21+ messages in thread
From: David Hildenbrand @ 2020-07-24 8:22 UTC (permalink / raw)
To: Arvind Sankar, x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra,
linux-sh, Yoshinori Sato, Rich Felker, sparclinux,
David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
On 24.07.20 01:15, Arvind Sankar wrote:
> The macro is not used anywhere, so remove the definition.
>
> Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
> ---
> arch/sh/include/asm/sparsemem.h | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/arch/sh/include/asm/sparsemem.h b/arch/sh/include/asm/sparsemem.h
> index 4eb899751e45..084706bb8cca 100644
> --- a/arch/sh/include/asm/sparsemem.h
> +++ b/arch/sh/include/asm/sparsemem.h
> @@ -5,11 +5,9 @@
> #ifdef __KERNEL__
> /*
> * SECTION_SIZE_BITS 2^N: how big each section will be
> - * MAX_PHYSADDR_BITS 2^N: how much physical address space we have
> - * MAX_PHYSMEM_BITS 2^N: how much memory we can have in that space
> + * MAX_PHYSMEM_BITS 2^N: how much physical address space we have
> */
> #define SECTION_SIZE_BITS 26
> -#define MAX_PHYSADDR_BITS 32
> #define MAX_PHYSMEM_BITS 32
>
> #endif
>
Reviewed-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 3/3] sparc: Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
@ 2020-07-23 23:15 ` Arvind Sankar
-1 siblings, 0 replies; 21+ messages in thread
From: Arvind Sankar @ 2020-07-23 23:15 UTC (permalink / raw)
To: x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra, linux-sh,
Yoshinori Sato, Rich Felker, sparclinux, David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
The macro is not used anywhere, so remove the definition.
Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
---
arch/sparc/include/asm/sparsemem.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/sparc/include/asm/sparsemem.h b/arch/sparc/include/asm/sparsemem.h
index 1dd1b61432db..aa9a676bc341 100644
--- a/arch/sparc/include/asm/sparsemem.h
+++ b/arch/sparc/include/asm/sparsemem.h
@@ -7,7 +7,6 @@
#include <asm/page.h>
#define SECTION_SIZE_BITS 30
-#define MAX_PHYSADDR_BITS MAX_PHYS_ADDRESS_BITS
#define MAX_PHYSMEM_BITS MAX_PHYS_ADDRESS_BITS
#endif /* !(__KERNEL__) */
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH 3/3] sparc: Drop unused MAX_PHYSADDR_BITS
@ 2020-07-23 23:15 ` Arvind Sankar
0 siblings, 0 replies; 21+ messages in thread
From: Arvind Sankar @ 2020-07-23 23:15 UTC (permalink / raw)
To: x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra, linux-sh,
Yoshinori Sato, Rich Felker, sparclinux, David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
The macro is not used anywhere, so remove the definition.
Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
---
arch/sparc/include/asm/sparsemem.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/sparc/include/asm/sparsemem.h b/arch/sparc/include/asm/sparsemem.h
index 1dd1b61432db..aa9a676bc341 100644
--- a/arch/sparc/include/asm/sparsemem.h
+++ b/arch/sparc/include/asm/sparsemem.h
@@ -7,7 +7,6 @@
#include <asm/page.h>
#define SECTION_SIZE_BITS 30
-#define MAX_PHYSADDR_BITS MAX_PHYS_ADDRESS_BITS
#define MAX_PHYSMEM_BITS MAX_PHYS_ADDRESS_BITS
#endif /* !(__KERNEL__) */
--
2.26.2
^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH 3/3] sparc: Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
@ 2020-07-24 8:22 ` David Hildenbrand
-1 siblings, 0 replies; 21+ messages in thread
From: David Hildenbrand @ 2020-07-24 8:22 UTC (permalink / raw)
To: Arvind Sankar, x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra,
linux-sh, Yoshinori Sato, Rich Felker, sparclinux,
David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
On 24.07.20 01:15, Arvind Sankar wrote:
> The macro is not used anywhere, so remove the definition.
>
> Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
> ---
> arch/sparc/include/asm/sparsemem.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/sparc/include/asm/sparsemem.h b/arch/sparc/include/asm/sparsemem.h
> index 1dd1b61432db..aa9a676bc341 100644
> --- a/arch/sparc/include/asm/sparsemem.h
> +++ b/arch/sparc/include/asm/sparsemem.h
> @@ -7,7 +7,6 @@
> #include <asm/page.h>
>
> #define SECTION_SIZE_BITS 30
> -#define MAX_PHYSADDR_BITS MAX_PHYS_ADDRESS_BITS
> #define MAX_PHYSMEM_BITS MAX_PHYS_ADDRESS_BITS
>
> #endif /* !(__KERNEL__) */
>
Reviewed-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 3/3] sparc: Drop unused MAX_PHYSADDR_BITS
@ 2020-07-24 8:22 ` David Hildenbrand
0 siblings, 0 replies; 21+ messages in thread
From: David Hildenbrand @ 2020-07-24 8:22 UTC (permalink / raw)
To: Arvind Sankar, x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra,
linux-sh, Yoshinori Sato, Rich Felker, sparclinux,
David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
On 24.07.20 01:15, Arvind Sankar wrote:
> The macro is not used anywhere, so remove the definition.
>
> Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
> ---
> arch/sparc/include/asm/sparsemem.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/sparc/include/asm/sparsemem.h b/arch/sparc/include/asm/sparsemem.h
> index 1dd1b61432db..aa9a676bc341 100644
> --- a/arch/sparc/include/asm/sparsemem.h
> +++ b/arch/sparc/include/asm/sparsemem.h
> @@ -7,7 +7,6 @@
> #include <asm/page.h>
>
> #define SECTION_SIZE_BITS 30
> -#define MAX_PHYSADDR_BITS MAX_PHYS_ADDRESS_BITS
> #define MAX_PHYSMEM_BITS MAX_PHYS_ADDRESS_BITS
>
> #endif /* !(__KERNEL__) */
>
Reviewed-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 3/3] sparc: Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
@ 2020-07-24 23:37 ` David Miller
-1 siblings, 0 replies; 21+ messages in thread
From: David Miller @ 2020-07-24 23:37 UTC (permalink / raw)
To: nivedita
Cc: x86, dave.hansen, luto, peterz, linux-sh, ysato, dalias,
sparclinux, akpm, linux-mm, linux-kernel
From: Arvind Sankar <nivedita@alum.mit.edu>
Date: Thu, 23 Jul 2020 19:15:44 -0400
> The macro is not used anywhere, so remove the definition.
>
> Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
Acked-by: David S. Miller <davem@davemloft.net>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 3/3] sparc: Drop unused MAX_PHYSADDR_BITS
@ 2020-07-24 23:37 ` David Miller
0 siblings, 0 replies; 21+ messages in thread
From: David Miller @ 2020-07-24 23:37 UTC (permalink / raw)
To: nivedita
Cc: x86, dave.hansen, luto, peterz, linux-sh, ysato, dalias,
sparclinux, akpm, linux-mm, linux-kernel
From: Arvind Sankar <nivedita@alum.mit.edu>
Date: Thu, 23 Jul 2020 19:15:44 -0400
> The macro is not used anywhere, so remove the definition.
>
> Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
Acked-by: David S. Miller <davem@davemloft.net>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 0/3] Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
@ 2020-07-24 15:04 ` Dave Hansen
-1 siblings, 0 replies; 21+ messages in thread
From: Dave Hansen @ 2020-07-24 15:04 UTC (permalink / raw)
To: Arvind Sankar, x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra,
linux-sh, Yoshinori Sato, Rich Felker, sparclinux,
David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
On 7/23/20 4:15 PM, Arvind Sankar wrote:
> This #define is not used anywhere, and has the wrong value on x86_64.
Yeah, it certainly is unused.
> I tried digging into the history a bit, but it seems to have been unused
> even in the initial merge of sparsemem in v2.6.13, when it was first
> defined.
Yep, I don't even remember why we thought we needed it back then. Feel
free to add my ack on these, or at least the x86 one.
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 0/3] Drop unused MAX_PHYSADDR_BITS
@ 2020-07-24 15:04 ` Dave Hansen
0 siblings, 0 replies; 21+ messages in thread
From: Dave Hansen @ 2020-07-24 15:04 UTC (permalink / raw)
To: Arvind Sankar, x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra,
linux-sh, Yoshinori Sato, Rich Felker, sparclinux,
David S. Miller
Cc: Andrew Morton, linux-mm, linux-kernel
On 7/23/20 4:15 PM, Arvind Sankar wrote:
> This #define is not used anywhere, and has the wrong value on x86_64.
Yeah, it certainly is unused.
> I tried digging into the history a bit, but it seems to have been unused
> even in the initial merge of sparsemem in v2.6.13, when it was first
> defined.
Yep, I don't even remember why we thought we needed it back then. Feel
free to add my ack on these, or at least the x86 one.
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 0/3] Drop unused MAX_PHYSADDR_BITS
2020-07-23 23:15 ` Arvind Sankar
@ 2020-07-26 6:48 ` Mike Rapoport
-1 siblings, 0 replies; 21+ messages in thread
From: Mike Rapoport @ 2020-07-26 6:48 UTC (permalink / raw)
To: Arvind Sankar
Cc: x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra, linux-sh,
Yoshinori Sato, Rich Felker, sparclinux, David S. Miller,
Andrew Morton, linux-mm, linux-kernel
On Thu, Jul 23, 2020 at 07:15:41PM -0400, Arvind Sankar wrote:
> This #define is not used anywhere, and has the wrong value on x86_64.
>
> I tried digging into the history a bit, but it seems to have been unused
> even in the initial merge of sparsemem in v2.6.13, when it was first
> defined.
>
> Arvind Sankar (3):
> x86/mm: Drop unused MAX_PHYSADDR_BITS
> sh/mm: Drop unused MAX_PHYSADDR_BITS
> sparc: Drop unused MAX_PHYSADDR_BITS
For the series
Acked-by: Mike Rapoport <rppt@linux.ibm.com>
> arch/sh/include/asm/sparsemem.h | 4 +---
> arch/sparc/include/asm/sparsemem.h | 1 -
> arch/x86/include/asm/sparsemem.h | 6 +-----
> 3 files changed, 2 insertions(+), 9 deletions(-)
>
> --
> 2.26.2
>
>
--
Sincerely yours,
Mike.
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH 0/3] Drop unused MAX_PHYSADDR_BITS
@ 2020-07-26 6:48 ` Mike Rapoport
0 siblings, 0 replies; 21+ messages in thread
From: Mike Rapoport @ 2020-07-26 6:48 UTC (permalink / raw)
To: Arvind Sankar
Cc: x86, Dave Hansen, Andy Lutomirski, Peter Zijlstra, linux-sh,
Yoshinori Sato, Rich Felker, sparclinux, David S. Miller,
Andrew Morton, linux-mm, linux-kernel
On Thu, Jul 23, 2020 at 07:15:41PM -0400, Arvind Sankar wrote:
> This #define is not used anywhere, and has the wrong value on x86_64.
>
> I tried digging into the history a bit, but it seems to have been unused
> even in the initial merge of sparsemem in v2.6.13, when it was first
> defined.
>
> Arvind Sankar (3):
> x86/mm: Drop unused MAX_PHYSADDR_BITS
> sh/mm: Drop unused MAX_PHYSADDR_BITS
> sparc: Drop unused MAX_PHYSADDR_BITS
For the series
Acked-by: Mike Rapoport <rppt@linux.ibm.com>
> arch/sh/include/asm/sparsemem.h | 4 +---
> arch/sparc/include/asm/sparsemem.h | 1 -
> arch/x86/include/asm/sparsemem.h | 6 +-----
> 3 files changed, 2 insertions(+), 9 deletions(-)
>
> --
> 2.26.2
>
>
--
Sincerely yours,
Mike.
^ permalink raw reply [flat|nested] 21+ messages in thread