* [PATCH v2] memory-hotplug: remove redundant call of page_to_pfn
[not found] <1415764673-32152-1-git-send-email-zhenzhang.zhang@huawei.com>
@ 2014-11-12 3:56 ` Zhang Zhen
0 siblings, 0 replies; only message in thread
From: Zhang Zhen @ 2014-11-12 3:56 UTC (permalink / raw)
To: Andrew Morton
Cc: wangnan0, Yasuaki Ishimatsu, Linux MM, David Rientjes, Dave Hansen
This is just a small optimization. The start_pfn can be
obtained directly by phys_index << PFN_SECTION_SHIFT.
So the call of page_to_pfn() is redundant and remove it.
v2:
added the purpose of the patch to the description.
Signed-off-by: Zhang Zhen <zhenzhang.zhang@huawei.com>
Acked-by: Yasuaki Ishimatsu <isimatu.yasuaki@jp.fujitsu.com>
---
drivers/base/memory.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/base/memory.c b/drivers/base/memory.c
index 7c5d871..85be040 100644
--- a/drivers/base/memory.c
+++ b/drivers/base/memory.c
@@ -228,8 +228,8 @@ memory_block_action(unsigned long phys_index, unsigned long action, int online_t
struct page *first_page;
int ret;
- first_page = pfn_to_page(phys_index << PFN_SECTION_SHIFT);
- start_pfn = page_to_pfn(first_page);
+ start_pfn = phys_index << PFN_SECTION_SHIFT;
+ first_page = pfn_to_page(start_pfn);
switch (action) {
case MEM_ONLINE:
--
1.8.1.4
.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2014-11-12 3:56 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1415764673-32152-1-git-send-email-zhenzhang.zhang@huawei.com>
2014-11-12 3:56 ` [PATCH v2] memory-hotplug: remove redundant call of page_to_pfn Zhang Zhen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox