From: Kevin Hilman <khilman@linaro.org>
To: Johannes Weiner <hannes@cmpxchg.org>
Cc: Mel Gorman <mgorman@suse.de>,
Andrew Morton <akpm@linux-foundation.org>,
Rik van Riel <riel@surriel.com>,
Andrea Arcangeli <aarcange@redhat.com>,
Zlatko Calusic <zcalusic@bitsync.net>,
Minchan Kim <minchan@kernel.org>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org,
sfr@canb.auug.org.au,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
Olof Johansson <olof@lixom.net>,
Stephen Warren <swarren@wwwdotorg.org>
Subject: Re: [patch v2 3/3] mm: page_alloc: fair zone allocator policy
Date: Fri, 16 Aug 2013 10:17:01 -0700 [thread overview]
Message-ID: <87haepblo2.fsf@kernel.org> (raw)
In-Reply-To: <20130808041623.GL1845@cmpxchg.org> (Johannes Weiner's message of "Thu, 8 Aug 2013 00:16:23 -0400")
[resend, gmail sent the other in HTML, sorry]
Hi Johannes,
Johannes Weiner <hannes@cmpxchg.org> writes:
> On Wed, Aug 07, 2013 at 11:37:43AM -0400, Johannes Weiner wrote:
[...]
> Patch on top of mmotm:
>
> ---
> From: Johannes Weiner <hannes@cmpxchg.org>
> Subject: [patch] mm: page_alloc: use vmstats for fair zone allocation batching
>
> Avoid dirtying the same cache line with every single page allocation
> by making the fair per-zone allocation batch a vmstat item, which will
> turn it into batched percpu counters on SMP.
>
> Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>
I bisected several boot failures on various ARM platform in
next-20130816 down to this patch (commit 67131f9837 in linux-next.)
Simply reverting it got things booting again on top of -next. Example
boot crash below.
Kevin
[ 0.000000] Booting Linux on physical CPU 0x0
[ 0.000000] Linux version 3.11.0-rc5-next-20130816 (khilman@paris) (gcc version 4.7.2 (Ubuntu/Linaro 4.7.2-1ubuntu1) ) #30 SMP Fri Aug 16 09:47:32 PDT 2013
[ 0.000000] CPU: ARMv7 Processor [413fc082] revision 2 (ARMv7), cr=10c53c7d
[ 0.000000] CPU: PIPT / VIPT nonaliasing data cache, VIPT aliasing instruction cache
[ 0.000000] Machine: Generic AM33XX (Flattened Device Tree), model: TI AM335x BeagleBone
[ 0.000000] bootconsole [earlycon0] enabled
[ 0.000000] Memory policy: ECC disabled, Data cache writeback
[ 0.000000] On node 0 totalpages: 130816
[ 0.000000] free_area_init_node: node 0, pgdat c081d400, node_mem_map c12fc000
[ 0.000000] Normal zone: 1024 pages used for memmap
[ 0.000000] Normal zone: 0 pages reserved
[ 0.000000] Unable to handle kernel NULL pointer dereference at virtual address 00000026
[ 0.000000] pgd = c0004000
[ 0.000000] [00000026] *pgd=00000000
[ 0.000000] Internal error: Oops: 5 [#1] SMP ARM
[ 0.000000] Modules linked in:
[ 0.000000] CPU: 0 PID: 0 Comm: swapper Not tainted 3.11.0-rc5-next-20130816 #30
[ 0.000000] task: c0793c70 ti: c0788000 task.ti: c0788000
[ 0.000000] PC is at __mod_zone_page_state+0x2c/0xb4
[ 0.000000] LR is at mod_zone_page_state+0x2c/0x4c
[ 0.000000] pc : [<c00eb628>] lr : [<c00ebbf0>] psr: 60000193
[ 0.000000] sp : c0789e84 ip : 00000026 fp : c0789ef8
[ 0.000000] r10: c0789f04 r9 : c05149dc r8 : 00000000
[ 0.000000] r7 : 00000026 r6 : 00000000 r5 : c0791770 r4 : c0788000
[ 0.000000] r3 : 00000000 r2 : 0001fb00 r1 : 00000001 r0 : c081d400
[ 0.000000] Flags: nZCv IRQs off FIQs on Mode SVC_32 ISA ARM Segment kernel
[ 0.000000] Control: 10c5387d Table: 80004019 DAC: 00000017
[ 0>.000000] Process swapper (pid: 0, stack limit = 0xc0788240)
[ 0.000000] Stack: (0xc0789e84 to 0xc078a000)
[ 0.000000] 9e80: 60000193 0001fb00 00000001 c081d400 c0781570 c081d400 c081d400
[ 0.000000] 9ea0: 00000000 0001fb00 00020000 c0747dd4 c077c5f4 00000001 0009ff00 0001fb00
[ 0.000000] 9ec0: 00000400 00080000 00000000 bfffffff c07ebfb0 000a0000 c0789ef8 00080000
[ 0.000000] 9ee0: 000a0000 00020000 000001cf c07ec188 000000cf c072ef2c 00020000 00000000
[ 0.000000] 9f00: 00000000 00000100 00000000 00000000 00000000 00000000 c0821d0c dfefa000
[ 0.000000] 9f20: c07ebfb0 00000001 00000001 c073080c c0789fdc c076abd4 ffff1000 0009feff
[ 0.000000] 9f40: 00001000 00000007 c0734a14 c076abd4 c0821ca0 c0008000 c076c8f0 c07ec188
[ 0.000000] 9f60: 413fc082 c0789fdc c064b328 c072cc3c 00000000 10c53c7d c0d5e448 00000001
[ 0.000000] 9f80: 00000000 c076c8ec c079542c 80004059 413fc082 00000000 00000000 c04f6298
[ 0.000000] 9fa0: c064989c 00000001 00000000 c076c8ec c079542c 80004059 413fc082 00000000
[ 0.000000] 9fc0: 00000000 c07297ec 00000000 00000000 00000000 00000000 00000000 c076c8f0
[ 0.000000] 9fe0: 00000000 10c53c7d c07908e8 c076c8ec c079542c 80008074 00000000 00000000
[ 0.000000] [<c00eb628>] (__mod_zone_page_state+0x2c/0xb4) from [<c081d400>] (contig_page_data+0x0/0xd80)
[ 0.000000] Code: e7958103 e0867001 e2877025 e1a0c007 (e19cc0d8)
[ 0.000000] ---[ end trace 1b75b31a2719ed1c ]---
--
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/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2013-08-16 17:17 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-02 15:37 [patch v2 0/3] mm: improve page aging fairness between zones/nodes Johannes Weiner
2013-08-02 15:37 ` [patch v2 1/3] mm: vmscan: fix numa reclaim balance problem in kswapd Johannes Weiner
2013-08-07 14:15 ` Mel Gorman
2013-08-02 15:37 ` [patch v2 2/3] mm: page_alloc: rearrange watermark checking in get_page_from_freelist Johannes Weiner
2013-08-07 14:20 ` Mel Gorman
2013-08-02 15:37 ` [patch v2 3/3] mm: page_alloc: fair zone allocator policy Johannes Weiner
2013-08-02 17:51 ` Rik van Riel
2013-08-05 1:15 ` Minchan Kim
2013-08-05 3:43 ` Johannes Weiner
2013-08-05 4:48 ` Minchan Kim
2013-08-05 5:01 ` Johannes Weiner
2013-08-05 10:34 ` Wanpeng Li
2013-08-05 11:34 ` Andrea Arcangeli
2013-08-05 13:11 ` Wanpeng Li
2013-08-05 13:11 ` Wanpeng Li
2013-08-05 10:34 ` Wanpeng Li
2013-08-07 14:58 ` Mel Gorman
2013-08-07 15:37 ` Johannes Weiner
2013-08-08 4:16 ` Johannes Weiner
2013-08-08 9:21 ` Mel Gorman
2013-08-09 18:45 ` Rik van Riel
2013-08-16 17:07 ` Kevin Hilman
2013-08-16 17:17 ` Kevin Hilman [this message]
2013-08-16 20:18 ` Johannes Weiner
2013-08-16 21:24 ` Stephen Warren
2013-08-16 21:52 ` Kevin Hilman
2013-08-19 0:48 ` Stephen Rothwell
2014-04-02 14:26 ` Thomas Schwinge
2014-04-24 13:37 ` radeon: screen garbled after page allocator change, was: " Johannes Weiner
2014-04-25 21:47 ` Jerome Glisse
2014-04-25 21:50 ` Jerome Glisse
2014-04-25 23:03 ` Jerome Glisse
2014-04-28 8:03 ` Thomas Schwinge
2014-04-28 9:09 ` Thomas Schwinge
2014-04-27 3:31 ` Jerome Glisse
2014-04-27 19:55 ` Jerome Glisse
2014-04-28 7:30 ` Christian König
2014-04-28 12:51 ` Deucher, Alexander
2014-04-28 12:52 ` Deucher, Alexander
2014-04-28 8:09 ` Thomas Schwinge
2014-06-16 7:11 ` Thomas Schwinge
2013-08-02 19:59 ` [patch v2 0/3] mm: improve page aging fairness between zones/nodes Andrea Arcangeli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87haepblo2.fsf@kernel.org \
--to=khilman@linaro.org \
--cc=aarcange@redhat.com \
--cc=akpm@linux-foundation.org \
--cc=hannes@cmpxchg.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mgorman@suse.de \
--cc=minchan@kernel.org \
--cc=olof@lixom.net \
--cc=riel@surriel.com \
--cc=sfr@canb.auug.org.au \
--cc=swarren@wwwdotorg.org \
--cc=zcalusic@bitsync.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox