From: chenqiwu <chenqiwu@xxxxxxxxxx>
Remove duplicated pfn_to_section_nr() in pfn_valid() and pfn_present()
to increase executing efficiency of code.
Signed-off-by: chenqiwu <chenqiwu@xxxxxxxxxx>
---
include/linux/mmzone.h | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
index 462f687..c2b7868 100644
--- a/include/linux/mmzone.h
+++ b/include/linux/mmzone.h
@@ -1358,10 +1358,11 @@ static inline int pfn_section_valid(struct mem_section *ms, unsigned long pfn)
static inline int pfn_valid(unsigned long pfn)
{
struct mem_section *ms;
+ unsigned long sec_nr = pfn_to_section_nr(pfn);
- if (pfn_to_section_nr(pfn) >= NR_MEM_SECTIONS)
+ if (pfn >= NR_MEM_SECTIONS)
return 0;
- ms = __nr_to_section(pfn_to_section_nr(pfn));
+ ms = __nr_to_section(sec_nr);
if (!valid_section(ms))
return 0;
/*
@@ -1374,9 +1375,11 @@ static inline int pfn_valid(unsigned long pfn)
static inline int pfn_present(unsigned long pfn)
{
- if (pfn_to_section_nr(pfn) >= NR_MEM_SECTIONS)
+ unsigned long sec_nr = pfn_to_section_nr(pfn);
+
+ if (sec_nr >= NR_MEM_SECTIONS)
return 0;
- return present_section(__nr_to_section(pfn_to_section_nr(pfn)));
+ return present_section(__nr_to_section(sec_nr));
}
static inline unsigned long next_present_section_nr(unsigned long section_nr)
--
1.9.1
