Re: [PATCH v3 3/3] mm: use numa_mem_id() in alloc_pages_node()

2015-07-30 Thread Johannes Weiner
On Thu, Jul 30, 2015 at 06:34:31PM +0200, Vlastimil Babka wrote: > numa_mem_id() is able to handle allocation from CPUs on memory-less nodes, > so it's a more robust fallback than the currently used numa_node_id(). Won't it fall through to the next closest memory node in the zonelist anyway? Is th

Re: [PATCH v3 2/3] mm: unify checks in alloc_pages_node() and __alloc_pages_node()

2015-07-30 Thread Johannes Weiner
xisting buggy callers, change the VM_BUG_ON > in __alloc_pages_node() to VM_WARN_ON. > > Signed-off-by: Vlastimil Babka > Acked-by: David Rientjes Acked-by: Johannes Weiner -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@

Re: [PATCH v3 1/3] mm: rename alloc_pages_exact_node to __alloc_pages_node

2015-07-30 Thread Johannes Weiner
istoph Lameter > Cc: Pekka Enberg > Cc: Joonsoo Kim > Cc: Naoya Horiguchi > Cc: Tony Luck > Cc: Fenghua Yu > Cc: Arnd Bergmann > Cc: Benjamin Herrenschmidt > Cc: Paul Mackerras > Acked-by: Michael Ellerman > Cc: Gleb Natapov > Cc: Paolo Bo

Re: commit 0bf1457f0cfca7b " mm: vmscan: do not swap anon pages just because free+file is low" causes heavy performance regression on paging

2014-04-24 Thread Johannes Weiner
Hi Rik, On Tue, Apr 22, 2014 at 10:40:17AM -0400, Rik van Riel wrote: > On 04/22/2014 07:57 AM, Christian Borntraeger wrote: > > On 22/04/14 12:55, Christian Borntraeger wrote: > >> While preparing/testing some KVM on s390 patches for the next merge window > >> (target is kvm/next which is based

Re: commit 0bf1457f0cfca7b " mm: vmscan: do not swap anon pages just because free+file is low" causes heavy performance regression on paging

2014-04-22 Thread Johannes Weiner
. > > According to /proc/vmstat the system is now in direct reclaim almost all the > time for every page fault (more than 10x more direct reclaims than kswap > reclaims) > With the patch being reverted everything is fine again. Ouch. Yes, I think we have to revert this for now. Ho