* [PATCH] mm: remove unused early_pfn_valid()
@ 2020-09-23 16:29 Mike Rapoport
2020-09-25 12:21 ` David Hildenbrand
2020-09-28 14:49 ` Mike Rapoport
0 siblings, 2 replies; 4+ messages in thread
From: Mike Rapoport @ 2020-09-23 16:29 UTC (permalink / raw)
To: Andrew Morton; +Cc: linux-mm, linux-kernel, Mike Rapoport, Mike Rapoport
From: Mike Rapoport <rppt@linux.ibm.com>
The early_pfn_valid() macro is defined by it is never used.
Remove it.
Signed-off-by: Mike Rapoport <rppt@linux.ibm.com>
---
include/linux/mmzone.h | 5 -----
1 file changed, 5 deletions(-)
diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
index 8379432f4f2f..38264363b0d4 100644
--- a/include/linux/mmzone.h
+++ b/include/linux/mmzone.h
@@ -1376,7 +1376,6 @@ static inline unsigned long next_present_section_nr(unsigned long section_nr)
#define pfn_to_nid(pfn) (0)
#endif
-#define early_pfn_valid(pfn) pfn_valid(pfn)
void sparse_init(void);
#else
#define sparse_init() do {} while (0)
@@ -1396,10 +1395,6 @@ struct mminit_pfnnid_cache {
int last_nid;
};
-#ifndef early_pfn_valid
-#define early_pfn_valid(pfn) (1)
-#endif
-
/*
* If it is possible to have holes within a MAX_ORDER_NR_PAGES, then we
* need to check pfn validity within that MAX_ORDER_NR_PAGES block.
--
2.28.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] mm: remove unused early_pfn_valid()
2020-09-23 16:29 [PATCH] mm: remove unused early_pfn_valid() Mike Rapoport
@ 2020-09-25 12:21 ` David Hildenbrand
2020-09-28 14:49 ` Mike Rapoport
1 sibling, 0 replies; 4+ messages in thread
From: David Hildenbrand @ 2020-09-25 12:21 UTC (permalink / raw)
To: Mike Rapoport, Andrew Morton; +Cc: linux-mm, linux-kernel, Mike Rapoport
On 23.09.20 18:29, Mike Rapoport wrote:
> From: Mike Rapoport <rppt@linux.ibm.com>
>
> The early_pfn_valid() macro is defined by it is never used.
> Remove it.
>
> Signed-off-by: Mike Rapoport <rppt@linux.ibm.com>
> ---
> include/linux/mmzone.h | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
> index 8379432f4f2f..38264363b0d4 100644
> --- a/include/linux/mmzone.h
> +++ b/include/linux/mmzone.h
> @@ -1376,7 +1376,6 @@ static inline unsigned long next_present_section_nr(unsigned long section_nr)
> #define pfn_to_nid(pfn) (0)
> #endif
>
> -#define early_pfn_valid(pfn) pfn_valid(pfn)
> void sparse_init(void);
> #else
> #define sparse_init() do {} while (0)
> @@ -1396,10 +1395,6 @@ struct mminit_pfnnid_cache {
> int last_nid;
> };
>
> -#ifndef early_pfn_valid
> -#define early_pfn_valid(pfn) (1)
> -#endif
> -
> /*
> * If it is possible to have holes within a MAX_ORDER_NR_PAGES, then we
> * need to check pfn validity within that MAX_ORDER_NR_PAGES block.
>
Think the last user was removed with
commit 73a6e474cb376921a311786652782155eac2fdf0
Author: Baoquan He <bhe@redhat.com>
Date: Wed Jun 3 15:57:55 2020 -0700
mm: memmap_init: iterate over memblock regions rather that check
each PFN
When called during boot the memmap_init_zone() function checks if
each PFN
is valid and actually belongs to the node being initialized using
early_pfn_valid() and early_pfn_in_nid().
Acked-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David / dhildenb
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mm: remove unused early_pfn_valid()
2020-09-23 16:29 [PATCH] mm: remove unused early_pfn_valid() Mike Rapoport
2020-09-25 12:21 ` David Hildenbrand
@ 2020-09-28 14:49 ` Mike Rapoport
2020-09-28 23:36 ` Andrew Morton
1 sibling, 1 reply; 4+ messages in thread
From: Mike Rapoport @ 2020-09-28 14:49 UTC (permalink / raw)
To: Andrew Morton; +Cc: linux-mm, linux-kernel, Mike Rapoport
ping?
On Wed, Sep 23, 2020 at 07:29:15PM +0300, Mike Rapoport wrote:
> From: Mike Rapoport <rppt@linux.ibm.com>
>
> The early_pfn_valid() macro is defined by it is never used.
> Remove it.
>
> Signed-off-by: Mike Rapoport <rppt@linux.ibm.com>
> ---
> include/linux/mmzone.h | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
> index 8379432f4f2f..38264363b0d4 100644
> --- a/include/linux/mmzone.h
> +++ b/include/linux/mmzone.h
> @@ -1376,7 +1376,6 @@ static inline unsigned long next_present_section_nr(unsigned long section_nr)
> #define pfn_to_nid(pfn) (0)
> #endif
>
> -#define early_pfn_valid(pfn) pfn_valid(pfn)
> void sparse_init(void);
> #else
> #define sparse_init() do {} while (0)
> @@ -1396,10 +1395,6 @@ struct mminit_pfnnid_cache {
> int last_nid;
> };
>
> -#ifndef early_pfn_valid
> -#define early_pfn_valid(pfn) (1)
> -#endif
> -
> /*
> * If it is possible to have holes within a MAX_ORDER_NR_PAGES, then we
> * need to check pfn validity within that MAX_ORDER_NR_PAGES block.
> --
> 2.28.0
>
--
Sincerely yours,
Mike.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mm: remove unused early_pfn_valid()
2020-09-28 14:49 ` Mike Rapoport
@ 2020-09-28 23:36 ` Andrew Morton
0 siblings, 0 replies; 4+ messages in thread
From: Andrew Morton @ 2020-09-28 23:36 UTC (permalink / raw)
To: Mike Rapoport; +Cc: linux-mm, linux-kernel, Mike Rapoport
On Mon, 28 Sep 2020 17:49:00 +0300 Mike Rapoport <rppt@kernel.org> wrote:
> ping?
>
Merged Sep 24.
https://www.ozlabs.org/~akpm/mmotm/broken-out/mm-remove-unused-early_pfn_valid.patch
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-09-28 23:36 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-23 16:29 [PATCH] mm: remove unused early_pfn_valid() Mike Rapoport
2020-09-25 12:21 ` David Hildenbrand
2020-09-28 14:49 ` Mike Rapoport
2020-09-28 23:36 ` Andrew Morton
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).