All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Roger Pau Monné" <roger.pau@citrix.com>
To: "Marek Marczykowski-Górecki" <marmarek@invisiblethingslab.com>
Cc: Jan Beulich <jbeulich@suse.com>,
	Patrick Plenefisch <simonpatp@gmail.com>,
	xen-devel@lists.xenproject.org, Juergen Gross <jgross@suse.com>
Subject: Re: E820 memory allocation issue on Threadripper platforms
Date: Wed, 17 Jan 2024 13:59:18 +0100	[thread overview]
Message-ID: <ZafPJvsIarRdy6BH@macbook> (raw)
In-Reply-To: <ZafC3apB4rjFUOXP@mail-itl>

On Wed, Jan 17, 2024 at 01:06:53PM +0100, Marek Marczykowski-Górecki wrote:
> On Tue, Jan 16, 2024 at 10:33:26AM +0100, Jan Beulich wrote:
> > ... as per
> > 
> > (XEN)  Dom0 kernel: 64-bit, PAE, lsb, paddr 0x1000000 -> 0x4a00000
> > 
> > there's an overlap with not exactly a hole, but with an
> > EfiACPIMemoryNVS region:
> > 
> > (XEN)  0000000100000-0000003159fff type=2 attr=000000000000000f
> > (XEN)  000000315a000-0000003ffffff type=7 attr=000000000000000f
> > (XEN)  0000004000000-0000004045fff type=10 attr=000000000000000f
> > (XEN)  0000004046000-0000009afefff type=7 attr=000000000000000f
> > 
> > (the 3rd of the 4 lines). Considering there's another region higher
> > up:
> > 
> > (XEN)  00000a747f000-00000a947efff type=10 attr=000000000000000f
> > 
> > I'm inclined to say it is poor firmware (or, far less likely, boot
> > loader) behavior to clobber a rather low and entirely arbitrary RAM
> > range, rather than consolidating all such regions near the top of
> > RAM below 4Gb.
> 
> FWIW, we have two more similar reports, with different motherboards and
> firmware versions, but the common factor is Threadripper CPU. It doesn't
> exclude firmware issue (it can be an issue in some common template, like
> edk2?), but makes it a bit less likely.
> 
> > There are further such odd regions, btw:
> > 
> > (XEN)  0000009aff000-0000009ffffff type=0 attr=000000000000000f
> > ...
> > (XEN)  000000b000000-000000b020fff type=0 attr=000000000000000f
> > 
> > If the kernel image was sufficiently much larger, these could become
> > a problem as well. Otoh if the kernel wasn't built with
> > CONFIG_PHYSICAL_START=0x1000000, i.e. to start at 16Mb, but at, say,
> > 2Mb, things should apparently work even with this unusual memory
> > layout (until the kernel would grow enough to again run into that
> > very region).
> 
> Shouldn't CONFIG_RELOCATABLE=y take care of this?

No, because PV doesn't use the native entry point.

> At least in the case
> of Qubes OS, it's enabled and the issue still happens.

I think for PV it should be possible to workaround this in Linux
itself, maybe by changing the pfn -> mfn relations of the kernel
area?

Those overlaps are not real, as the loaded kernel is scattered across
mfns, and those certainly belong to RAM regions in the memory map.

For PVH it's going to require some changes in Xen itself.

Roger.


  reply	other threads:[~2024-01-17 12:59 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-11  2:29 E820 memory allocation issue on Threadripper platforms Patrick Plenefisch
2024-01-11  8:37 ` Jan Beulich
2024-01-11  9:19   ` Xenia Ragiadakou
2024-01-11 10:13   ` Juergen Gross
2024-01-16  0:22     ` Patrick Plenefisch
2024-01-16  9:33       ` Jan Beulich
2024-01-17  6:12         ` Patrick Plenefisch
2024-01-17  8:46           ` Jan Beulich
2024-01-17 10:13             ` Roger Pau Monné
2024-01-17 10:40               ` Jan Beulich
2024-01-17 12:54                 ` Roger Pau Monné
2024-06-24 12:40                   ` Branden Sherrell
2024-06-24 12:54                     ` Jan Beulich
2024-06-24 13:07                       ` Branden Sherrell
2024-06-24 13:40                         ` Jan Beulich
2024-06-24 16:17                           ` Branden Sherrell
2024-01-18  6:23             ` Patrick Plenefisch
2024-01-18  8:27               ` Roger Pau Monné
2024-01-18  8:34                 ` Patrick Plenefisch
2024-01-18  9:46                   ` Roger Pau Monné
2024-01-18 12:41                     ` ACPI VFCT table too short on PVH dom0 (was: Re: E820 memory allocation issue on Threadripper platforms) Roger Pau Monné
2024-01-19  7:44                       ` Patrick Plenefisch
2024-01-19  9:54                         ` Huang Rui
2024-01-19 15:32                           ` Deucher, Alexander
2024-01-21  1:27                           ` Patrick Plenefisch
2024-01-19 11:06                         ` Roger Pau Monné
2024-01-21  1:33                           ` Patrick Plenefisch
2024-01-24  1:27                             ` Patrick Plenefisch
2024-01-24  8:23                               ` Roger Pau Monné
2024-01-25  0:20                                 ` Patrick Plenefisch
2024-01-18 16:03                     ` E820 memory allocation issue on Threadripper platforms Patrick Plenefisch
2024-01-18  9:01               ` Jan Beulich
2024-01-19 13:40               ` Marek Marczykowski-Górecki
2024-01-19 13:50                 ` Jan Beulich
2024-01-19 14:31                   ` Marek Marczykowski-Górecki
2024-03-10 14:06                 ` Marek Marczykowski-Górecki
2024-03-12 21:07                   ` Jason Andryuk
2024-03-12 22:00                     ` Marek Marczykowski-Górecki
2024-01-17  8:46           ` Roger Pau Monné
2024-01-17 12:06         ` Marek Marczykowski-Górecki
2024-01-17 12:59           ` Roger Pau Monné [this message]
2024-01-17 13:40             ` Jan Beulich

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=ZafPJvsIarRdy6BH@macbook \
    --to=roger.pau@citrix.com \
    --cc=jbeulich@suse.com \
    --cc=jgross@suse.com \
    --cc=marmarek@invisiblethingslab.com \
    --cc=simonpatp@gmail.com \
    --cc=xen-devel@lists.xenproject.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 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.