On Mon, Jan 30, 2012 at 02:33:53PM +0100, Maxime Coquelin wrote:I catch your point.Any allocators might call the PASR Framework for DDR power savings. Currently,I did not review this series carefully and I know nothing about
only Linux Buddy allocator is patched, but HWMEM and PMEM physically
contiguous memory allocators will follow.
Linux Buddy allocator porting uses Buddy specificities to reduce the overhead
induced by the PASR Framework counter updates. Indeed, the PASR Framework is
called only when MAX_ORDER (4MB page blocs by default) buddies are
inserted/removed from the free lists.
To port PASR FW into a new allocator:
* Call pasr_put(phys_addr, size) each time a memory chunk becomes unused.
* Call pasr_get(phys_addr, size) each time a memory chunk becomes used.
Signed-off-by: Maxime Coquelin<maxime.coquelin@xxxxxxxxxxxxxx>
---
mm/page_alloc.c | 9 +++++++++
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index 03d8c48..c62fe11 100644
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -57,6 +57,7 @@
#include<linux/ftrace_event.h>
#include<linux/memcontrol.h>
#include<linux/prefetch.h>
+#include<linux/pasr.h>
#include<asm/tlbflush.h>
#include<asm/div64.h>
@@ -534,6 +535,7 @@ static inline void __free_one_page(struct page *page,
/* Our buddy is free, merge with it and move up one order. */
list_del(&buddy->lru);
zone->free_area[order].nr_free--;
+ pasr_kget(buddy, order);
rmv_page_order(buddy);
combined_idx = buddy_idx& page_idx;
page = page + (combined_idx - page_idx);
how you implemented PASR support but driver hooks like this in the
page allocator are heavily frowned upon. It is subject to abuse but
it adds overhead to the allocator although I note that you avoiding
putting hooks in the per-cpu page allocator.
I note that you hardcodeIn that RFC patch set, I assumed the PASR would be enabled in the config only in case it is used.
it so only PASR can use the hook but it looks like there is no way
of avoiding that overhead on platforms that do not have PASR if
it is enabled in the config.
At a glance, it appears to be doing aNote that we do that work only on MAX_ORDER pages, so it limits the overhead.
fair amount of work too - looking up maps, taking locks etc.
ThisOk.
potentially creates a new hot lock because in this paths, we have
per-zone locking but you are adding a PASR lock into the mix that
may be more coarse than zone->lock (I didn't check).
You may be able to use the existing arch_alloc_page() hook andYour proposal sounds good.
call PASR on architectures that support it if and only if PASR is
present and enabled by the administrator but even this is likely to be
unpopular as it'll have a measurable performance impact on platforms
with PASR (not to mention the PASR lock will be even heavier as it'll
now be also used for per-cpu page allocations). To get the hook you
want, you'd need to show significant benefit before they were happy with
the hook.
With this solution, we need in any case to add some hooks in the allocator to ensure the pages being allocated are refreshed.
What is more likely is that you will get pushed to doing something like
periodically scanning memory as part of a separate power management
module and calling into PASR if regions of memory that are found that
can be powered down in some ways.