qiurui
/
Centos-kernel-stream-9
mirror of https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

x86/PCI: Tidy E820 removal messages 

Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2166398
Upstream Status: 00904bf64c2819d90a76407d79bdbc8918541320

commit 00904bf64c2819d90a76407d79bdbc8918541320
Author: Bjorn Helgaas <bhelgaas@google.com>
Date:   Thu Dec 8 13:03:40 2022 -0600

    x86/PCI: Tidy E820 removal messages

    These messages:

      clipped [mem size 0x00000000 64bit] to [mem size 0xfffffffffffa0000 64bit] for e820 entry [mem 0x0009f000-0x000fffff]

    aren't as useful as they could be because (a) the resource is often
    IORESOURCE_UNSET, so we print the size instead of the start/end and (b) we
    print the available resource even if it is empty after removing the E820
    entry.

    Print the available space by hand to avoid the IORESOURCE_UNSET problem and
    only if it's non-empty.  No functional change intended.

    Link: https://lore.kernel.org/r/20221208190341.1560157-4-helgaas@kernel.org
    Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
    Acked-by: Hans de Goede <hdegoede@redhat.com>

Signed-off-by: Myron Stowe <mstowe@redhat.com>
This commit is contained in:
Myron Stowe 2023-02-08 08:30:47 -07:00
parent c1f193f525
commit 5a1d2900b1
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
arch/x86/kernel/resource.c
View File

@ -42,8 +42,16 @@ static void remove_e820_regions(struct resource *avail)
resource_clip(avail, e820_start, e820_end);
if (orig.start != avail->start || orig.end != avail->end) {
pr_info("clipped %pR to %pR for e820 entry [mem %#010Lx-%#010Lx]\n",
&orig, avail, e820_start, e820_end);
pr_info("resource: avoiding allocation from e820 entry [mem %#010Lx-%#010Lx]\n",
e820_start, e820_end);
if (avail->end > avail->start)
/*
* Use %pa instead of %pR because "avail"
* is typically IORESOURCE_UNSET, so %pR
* shows the size instead of addresses.
*/
pr_info("resource: remaining [mem %pa-%pa] available\n",
&avail->start, &avail->end);
orig = *avail;
}
}