linux-kernel - Re: [tip:x86/boot] x86/boot/e820: Basic cleanup of e820.c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:   Tue, 25 Apr 2017 16:57:50 -0700
From:   Yinghai Lu <yinghai@...nel.org>
To:     Tejun Heo <tj@...nel.org>, "Rafael J. Wysocki" <rjw@...k.pl>,
        Denys Vlasenko <dvlasenk@...hat.com>, athorlton@....com,
        jgross@...e.com, Dan Williams <dan.j.williams@...el.com>,
        Linus Torvalds <torvalds@...ux-foundation.org>,
        Yinghai Lu <yinghai@...nel.org>,
        Andy Lutomirski <luto@...nel.org>,
        Brian Gerst <brgerst@...il.com>, Paul Jackson <pj@....com>,
        Josh Poimboeuf <jpoimboe@...hat.com>,
        Wei Yang <richard.weiyang@...il.com>,
        Huang Ying <ying.huang@...el.com>,
        Ingo Molnar <mingo@...nel.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        "H. Peter Anvin" <hpa@...or.com>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Peter Zijlstra <peterz@...radead.org>,
        Borislav Petkov <bp@...en8.de>
Cc:     "linux-tip-commits@...r.kernel.org" 
        <linux-tip-commits@...r.kernel.org>,
        "kexec@...ts.infradead.org" <kexec@...ts.infradead.org>
Subject: Re: [tip:x86/boot] x86/boot/e820: Basic cleanup of e820.c

On Tue, Apr 11, 2017 at 12:37 AM, tip-bot for Ingo Molnar
<tipbot@...or.com> wrote:
> Commit-ID:  640e1b38b00550990cecd809021cd37716e45922
> Gitweb:     http://git.kernel.org/tip/640e1b38b00550990cecd809021cd37716e45922
> Author:     Ingo Molnar <mingo@...nel.org>
> AuthorDate: Sat, 28 Jan 2017 11:13:08 +0100
> Committer:  Ingo Molnar <mingo@...nel.org>
> CommitDate: Sat, 28 Jan 2017 14:42:27 +0100
>

> x86/boot/e820: Basic cleanup of e820.c
....
> @@ -951,49 +924,42 @@ void __init finish_e820_parsing(void)
>  static const char *__init e820_type_to_string(int e820_type)
>  {
>         switch (e820_type) {
> -       case E820_RESERVED_KERN:
> -       case E820_RAM:  return "System RAM";
> -       case E820_ACPI: return "ACPI Tables";
> -       case E820_NVS:  return "ACPI Non-volatile Storage";
> -       case E820_UNUSABLE:     return "Unusable memory";
> -       case E820_PRAM: return "Persistent Memory (legacy)";
> -       case E820_PMEM: return "Persistent Memory";
> -       default:        return "reserved";
> +       case E820_RESERVED_KERN: /* Fall-through: */
> +       case E820_RAM:           return "System RAM";
> +       case E820_ACPI:          return "ACPI Tables";
> +       case E820_NVS:           return "ACPI Non-volatile Storage";
> +       case E820_UNUSABLE:      return "Unusable memory";
> +       case E820_PRAM:          return "Persistent Memory (legacy)";
> +       case E820_PMEM:          return "Persistent Memory";
> +       default:                 return "Reserved";
>         }
>  }
>
...

Hi Ingo,

The reserved ==> Reserved change cause kexec warning.

Unknown type (Reserved) while parsing /sys/firmware/memmap/18/type.
Please report this as bug. Using RANGE_RESERVED now.
Unknown type (Reserved) while parsing /sys/firmware/memmap/16/type.
Please report this as bug. Using RANGE_RESERVED now.
Unknown type (Reserved) while parsing /sys/firmware/memmap/14/type.
Please report this as bug. Using RANGE_RESERVED now.
Unknown type (Reserved) while parsing /sys/firmware/memmap/22/type.
Please report this as bug. Using RANGE_RESERVED now.
Unknown type (Reserved) while parsing /sys/firmware/memmap/9/type.
Please report this as bug. Using RANGE_RESERVED now.
add_buffer: base:43fff6000 bufsz:80e0 memsz:a000
add_buffer: base:43fff1000 bufsz:44ce memsz:44ce
add_buffer: base:43c000000 bufsz:f4c5c0 memsz:3581000
add_buffer: base:439d0d000 bufsz:22f2060 memsz:22f2060
add_buffer: base:43fff0000 bufsz:70 memsz:70
add_buffer: base:43ffef000 bufsz:230 memsz:230
10:~/k # cat /sys/firmware/memmap/14/type
Reserved

also /proc/iomem have that changed too.

Yinghai