From: Andi Kleen <andi@firstfloor.org>
To: Dave Jones <davej@redhat.com>, Andi Kleen <andi@firstfloor.org>,
p.herz@profihost.ag, linux-kernel@vger.kernel.org
Subject: Re: Vanilla-Kernel 3 - page allocation failure
Date: Tue, 18 Oct 2011 20:59:13 +0200 [thread overview]
Message-ID: <20111018185913.GE15908@one.firstfloor.org> (raw)
In-Reply-To: <20111018170204.GA10551@redhat.com>
> We get reports like this fairly regularly, usually accompanied by
> "But I had lots of free memory and/or swap!"
I think the backtrace is also really bad. It just makes it look like a crash.
It just cries "please report me", even though there's usually no good
reason for it.
I understand it can be useful sometimes for debugging, but most of the
time it is unnecessary and just confusing. One good thing probably
would be some heuristic to see when to print the backtrace, and don't
print it in common situations.
>
> The order/mode stuff is completely opaque to end-users, who have no
> clue that there are different types of memory, and exhausting one type
> can happen even when plenty of other memory is free.
order should be probably replaced with a user readable size, agreed.
order:2 = "16 KB"
[note if anybody wants to reply now it should be "16 KiB", don't bother;
i'll ignore you]
>
> I've been toying with the idea of hacking up a patch to turn those mode
> flags into printing things like "mode:GFP_ATOMIC|GFP_NOIO" instead though, as I can
> never remember those flags off the top of my head.
> Still won't help end-users, but it would at least speed up diagnosing reports.
Better decode it: "from interrupt handler", "inside a file system"
Unfortunately there's no flag for GFP_ATOMIC but not in a interrupt handler,
but some code with broken locking abusing it. Perhaps there should be.
-Andi
--
ak@linux.intel.com -- Speaking for myself only.
next prev parent reply other threads:[~2011-10-18 18:59 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-18 10:25 Vanilla-Kernel 3 - page allocation failure Philipp Herz - Profihost AG
2011-10-18 11:32 ` Thadeu Lima de Souza Cascardo
2011-10-18 12:07 ` Philipp Herz - Profihost AG
2011-10-18 12:38 ` Thadeu Lima de Souza Cascardo
2011-10-18 13:24 ` Philipp Herz - Profihost AG
2011-10-18 14:35 ` Thadeu Lima de Souza Cascardo
2011-10-19 6:45 ` Philipp Herz - Profihost AG
2011-10-18 15:51 ` Andi Kleen
2011-10-18 17:02 ` Dave Jones
2011-10-18 18:59 ` Andi Kleen [this message]
2011-10-19 1:58 ` David Rientjes
2011-10-24 6:33 ` Philipp Herz - Profihost AG
2011-10-24 7:03 ` Eric Dumazet
2011-10-24 7:21 ` Philipp Herz - Profihost AG
2011-10-24 8:01 ` Eric Dumazet
2011-10-24 8:19 ` Philipp Herz - Profihost AG
2011-10-24 8:29 ` Eric Dumazet
2011-10-24 8:36 ` Stefan Priebe - Profihost AG
2011-10-26 20:26 ` David Rientjes
2011-10-27 7:13 ` Philipp Herz - Profihost AG
2011-10-27 20:08 ` David Rientjes
2011-10-20 21:11 ` Valdis.Kletnieks
2011-10-21 6:36 ` Philipp Herz - Profihost AG
-- strict thread matches above, loose matches on Subject: below --
2011-10-18 10:24 Philipp Herz - Profihost AG
2011-10-19 2:09 ` David Rientjes
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=20111018185913.GE15908@one.firstfloor.org \
--to=andi@firstfloor.org \
--cc=davej@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=p.herz@profihost.ag \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.