* [PATCH] percpu: fix per_cpu_ptr_to_phys() handling of non-page-aligned addresses.
@ 2011-12-15 19:25 Eugene Surovegin
2011-12-15 19:43 ` Tejun Heo
0 siblings, 1 reply; 2+ messages in thread
From: Eugene Surovegin @ 2011-12-15 19:25 UTC (permalink / raw)
To: tj; +Cc: ptesarik, xiyou.wangcong, linux-mm, linux-kernel, vgoyal
per_cpu_ptr_to_phys() incorrectly rounds up its result for non-kmalloc case
to the page boundary, which is bogus for any non-page-aligned address.
This fixes the only in-tree user of this function - sysfs handler for
per-cpu 'crash_notes' physical address. The manifestation of this bug is
missing 'CORE' ELF notes in kdump.
Signed-off-by: Eugene Surovegin <ebs@ebshome.net>
---
mm/percpu.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/mm/percpu.c b/mm/percpu.c
index 3bb810a..716eb4a 100644
--- a/mm/percpu.c
+++ b/mm/percpu.c
@@ -1023,9 +1023,11 @@ phys_addr_t per_cpu_ptr_to_phys(void *addr)
if (!is_vmalloc_addr(addr))
return __pa(addr);
else
- return page_to_phys(vmalloc_to_page(addr));
+ return page_to_phys(vmalloc_to_page(addr)) +
+ offset_in_page(addr);
} else
- return page_to_phys(pcpu_addr_to_page(addr));
+ return page_to_phys(pcpu_addr_to_page(addr)) +
+ offset_in_page(addr);
}
/**
--
1.7.5.1
--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH] percpu: fix per_cpu_ptr_to_phys() handling of non-page-aligned addresses
2011-12-15 19:25 [PATCH] percpu: fix per_cpu_ptr_to_phys() handling of non-page-aligned addresses Eugene Surovegin
@ 2011-12-15 19:43 ` Tejun Heo
0 siblings, 0 replies; 2+ messages in thread
From: Tejun Heo @ 2011-12-15 19:43 UTC (permalink / raw)
To: Eugene Surovegin; +Cc: ptesarik, xiyou.wangcong, linux-mm, linux-kernel, vgoyal
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-12-15 19:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-15 19:25 [PATCH] percpu: fix per_cpu_ptr_to_phys() handling of non-page-aligned addresses Eugene Surovegin
2011-12-15 19:43 ` Tejun Heo
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).