linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
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)

  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