[RFC PATCH V5 RESEND] mm readahead: Fix readahead fail for no localmemory and limit readahead pages

From: Raghavendra K T
Date: Mon Nov 25 2013 - 03:59:03 EST


Currently max_sane_readahead() returns zero on the cpu having no local memory node
which leads to readahead failure. Fix the readahead failure by returning
minimum of (requested pages, 4k). Users running application a on memory less cpu
which needs readahead such as streaming application see considerable boost in the
performance.

Result:
fadvise experiment with FADV_WILLNEED on a PPC machine having memoryless CPU
with 1GB testfile ( 12 iterations) yielded 46.66% improvement

kernel Avg Stddev
base_ppc 11.946833 1.34%
patched_ppc 6.3720833 1.80%

Below result proves that there is no impact on the normal NUMA cases w/ patch.

fadvise experiment with FADV_WILLNEED on a x240 machine with 1GB testfile
32GB* 4G RAM numa machine ( 12 iterations) yielded

Kernel Avg Stddev
base 7.2963 1.10 %
patched 7.2972 1.18 %

Reviewed-by: Jan Kara <jack@xxxxxxx>
Signed-off-by: Raghavendra K T <raghavendra.kt@xxxxxxxxxxxxxxxxxx>
---

Changes in V5:
- Updated the changelog with benefit seen (Andrew)
- Discard remote memroy term in comment since memoryless CPU
will have affinity to numa_mem_id() (David)
- Drop the 4k limit for normal readahead. (Jan Kara)

Changes in V4:
- Check for total node memory to decide whether we don't
have local memory (jan Kara)
- Add 4k page limit on readahead for normal and remote readahead (Linus)
(Linus suggestion was 16MB limit).

Changes in V3:
- Drop iterating over numa nodes that calculates total free pages (Linus)

Agree that we do not have control on allocation for readahead on a
particular numa node and hence for remote readahead we can not further
sanitize based on potential free pages of that node. and also we do
not want to itererate through all nodes to find total free pages.

Suggestions and comments welcome
mm/readahead.c | 21 +++++++++++++++++++--
1 file changed, 19 insertions(+), 2 deletions(-)

diff --git a/mm/readahead.c b/mm/readahead.c
index 0de2360..4c7343b 100644
--- a/mm/readahead.c
+++ b/mm/readahead.c
@@ -233,14 +233,31 @@ int force_page_cache_readahead(struct address_space *mapping, struct file *filp,
return 0;
}

+#define MAX_REMOTE_READAHEAD 4096UL
/*
* Given a desired number of PAGE_CACHE_SIZE readahead pages, return a
* sensible upper limit.
*/
unsigned long max_sane_readahead(unsigned long nr)
{
- return min(nr, (node_page_state(numa_node_id(), NR_INACTIVE_FILE)
- + node_page_state(numa_node_id(), NR_FREE_PAGES)) / 2);
+ unsigned long local_free_page;
+ int nid;
+
+ nid = numa_node_id();
+ if (node_present_pages(nid)) {
+ /*
+ * We sanitize readahead size depending on free memory in
+ * the local node.
+ */
+ local_free_page = node_page_state(nid, NR_INACTIVE_FILE)
+ + node_page_state(nid, NR_FREE_PAGES);
+ return min(nr, local_free_page / 2);
+ }
+ /*
+ * Ensure readahead success on a memoryless node cpu. But we limit
+ * the readahead to 4k pages to avoid trashing page cache.
+ */
+ return min(nr, MAX_REMOTE_READAHEAD);
}

/*
--
1.7.11.7

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/