From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Andrew Morton <akpm@linux-foundation.org>,
Thomas Gleixner <tglx@linutronix.de>,
Matthew Wilcox <willy@infradead.org>,
Ingo Molnar <mingo@redhat.com>, Jens Axboe <axboe@kernel.dk>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
David Miller <davem@davemloft.net>,
Al Viro <viro@zeniv.linux.org.uk>,
Dave Airlie <airlied@gmail.com>, Tejun Heo <tj@kernel.org>,
Ted Ts'o <tytso@google.com>, Mike Snitzer <snitzer@redhat.com>,
linux-mm <linux-mm@kvack.org>, Daniel Vacek <neelx@redhat.com>,
Pavel Tatashin <pasha.tatashin@oracle.com>,
Mel Gorman <mgorman@techsingularity.net>
Subject: Re: Instability in current -git tree
Date: Tue, 17 Jul 2018 10:59:03 +0800 [thread overview]
Message-ID: <CAKv+Gu-qQOr5rLzdb-gTUD-QSOXQ=L7njO2Aqk4-jg5FD1kTgQ@mail.gmail.com> (raw)
In-Reply-To: <CA+55aFwD0cXbD6wW_2gs0kXRk-VsF05oE+4M6J=OoVj-wOOGSg@mail.gmail.com>
On 14 July 2018 at 08:20, Linus Torvalds <torvalds@linux-foundation.org> wrote:
> On Fri, Jul 13, 2018 at 4:51 PM Linus Torvalds
> <torvalds@linux-foundation.org> wrote:
>>
>> I'm building a "replace VM_BUG_ON() with proper printk's instead" right now.
>
> Ok, the machine now stays up, and I get messages like
>
> Removed VM_BUG_ON()!
> pfn c2400 - c25ff
> zone DMA32 DMA
> zone pfn 1000 1
>
> Removed VM_BUG_ON()!
> pfn c0a00 - c0bff
> zone DMA32 DMA
> zone pfn 1000 1
>
> Removed VM_BUG_ON()!
> pfn c2200 - c23ff
> zone DMA DMA32
> zone pfn 1 1000
>
> instead.
>
> That's from
>
> + printk("Removed VM_BUG_ON()!\n");
> + printk(" pfn %lx - %lx\n", page_to_pfn(start_page),
> page_to_pfn(end_page));
> + printk(" zone %s %s\n", page_zone(start_page)->name,
> page_zone(end_page)->name);
> + printk(" zone pfn %lx %lx\n",
> page_zone(start_page)->zone_start_pfn,
> page_zone(end_page)->zone_start_pfn);
>
> inside an if() statement that replaced that VM_BUG_ON().
>
> WTF? That's just odd.
>
> But everything seems to work fine, and now it doesn't crash.
>
> But there's something really odd going on wrt page_zone() and/or page_to_pfn().
>
> page_to_pfn() implies this is just regular memory in the 3GB area. It
> is likely related to this:
>
> BIOS-e820: [mem 0x00000000c0b33000-0x00000000c226cfff] reserved
> BIOS-e820: [mem 0x00000000c226d000-0x00000000c227efff] ACPI data
> BIOS-e820: [mem 0x00000000c227f000-0x00000000c2439fff] usable
> BIOS-e820: [mem 0x00000000c243a000-0x00000000c2a61fff] ACPI NVS
> BIOS-e820: [mem 0x00000000c2a62000-0x00000000c32fefff] reserved
> BIOS-e820: [mem 0x00000000c32ff000-0x00000000c32fffff] usable
> BIOS-e820: [mem 0x00000000c3300000-0x00000000c7ffffff] reserved
>
> I dunno. It's a bit odd. I'm not sure I understand that VM_BUG_ON().
> Adding Ard (who worked on the memblock_next_valid_pfn() thing not that
> long ago) and must have hit this same BUG_ON() because he modified it
> not that long ago.
>
> Ard, I triggered the VM_BUG_ON() in mm/page_alloc.c:2016, with a call trace opf
>
> RIP: move_pfreepages_block()
> Call Trace:
> steal_suitable_fallback
> get_page_from_freelist
> ...
>
> just for some context.
>
Pavel's fix for this issue in commit e181ae0c5db9 is causing boot
problems on i686 for me.
Is anyone else seeing the same?
I get no output whatsoever when booting a i386_defconfig kernel under
qemu/kvm (without EFI)
next prev parent reply other threads:[~2018-07-17 2:59 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CA+55aFyARQV302+mXNYznrOOjzW+yxbcv+=OkD43dG6G1ktoMQ@mail.gmail.com>
[not found] ` <alpine.DEB.2.21.1807140031440.2644@nanos.tec.linutronix.de>
[not found] ` <CA+55aFzBx1haeM2QSFvhaW2t_HVK78Y=bKvsiJmOZztwkZ-y7Q@mail.gmail.com>
[not found] ` <CA+55aFzVGa57apuzDMBLgWQQRcm3BNBs1UEg-G_2o7YW1i=o2Q@mail.gmail.com>
[not found] ` <CA+55aFy9NJZeqT7h_rAgbKUZLjzfxvDPwneFQracBjVhY53aQQ@mail.gmail.com>
2018-07-13 23:48 ` Andrew Morton
2018-07-13 23:51 ` Linus Torvalds
2018-07-13 23:58 ` Andrew Morton
2018-07-14 0:19 ` Pavel Tatashin
2018-07-14 0:28 ` Linus Torvalds
2018-07-14 0:46 ` Pavel Tatashin
2018-07-14 2:40 ` Linus Torvalds
2018-07-14 3:03 ` Pavel Tatashin
2018-07-14 3:25 ` Linus Torvalds
2018-07-14 3:28 ` Pavel Tatashin
2018-07-14 13:39 ` Pavel Tatashin
2018-07-14 17:11 ` Linus Torvalds
2018-07-14 17:29 ` Linus Torvalds
2018-07-16 12:06 ` Michal Hocko
2018-07-16 12:09 ` Pavel Tatashin
2018-07-16 12:29 ` Michal Hocko
2018-07-16 13:26 ` Pavel Tatashin
2018-07-16 14:12 ` Michal Hocko
2018-07-16 13:39 ` Oscar Salvador
2018-07-14 3:04 ` Linus Torvalds
2018-07-14 0:20 ` Linus Torvalds
2018-07-14 9:28 ` Ard Biesheuvel
2018-07-17 2:59 ` Ard Biesheuvel [this message]
2018-07-17 3:14 ` Ard Biesheuvel
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='CAKv+Gu-qQOr5rLzdb-gTUD-QSOXQ=L7njO2Aqk4-jg5FD1kTgQ@mail.gmail.com' \
--to=ard.biesheuvel@linaro.org \
--cc=airlied@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=axboe@kernel.dk \
--cc=davem@davemloft.net \
--cc=gregkh@linuxfoundation.org \
--cc=linux-mm@kvack.org \
--cc=mgorman@techsingularity.net \
--cc=mingo@redhat.com \
--cc=neelx@redhat.com \
--cc=pasha.tatashin@oracle.com \
--cc=snitzer@redhat.com \
--cc=tglx@linutronix.de \
--cc=tj@kernel.org \
--cc=torvalds@linux-foundation.org \
--cc=tytso@google.com \
--cc=viro@zeniv.linux.org.uk \
--cc=willy@infradead.org \
/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