> >
> > In particular, I think that you'll find that you will have to change
> > the heuristics in __alloc_pages_slowpath() where we currently do
> >
> > if ((did_some_progress && order <= PAGE_ALLOC_COSTLY_ORDER) || ..
> >
> > when the "did_some_progress" logic changes that radically.
> >
>
> Yes. But we can't simply do
>
> if (order <= PAGE_ALLOC_COSTLY_ORDER || ..
>
> because we won't be able to call out_of_memory(), can we?
>
Can you please try a simplified retry logic?
thanks
Hillf
--- a/mm/page_alloc.c Wed Oct 14 14:45:28 2015
+++ b/mm/page_alloc.c Wed Oct 14 15:43:31 2015
@@ -3154,8 +3154,7 @@ retry:
/* Keep reclaiming pages as long as there is reasonable progress */
pages_reclaimed += did_some_progress;
- if ((did_some_progress && order <= PAGE_ALLOC_COSTLY_ORDER) ||
- ((gfp_mask & __GFP_REPEAT) && pages_reclaimed < (1 << order))) {
+ if (did_some_progress) {
/* Wait for some write requests to complete then retry */
wait_iff_congested(ac->preferred_zone, BLK_RW_ASYNC, HZ/50);
goto retry;
--
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>