[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <fc60cc71-80eb-4058-8d0a-dbd19a5c10ef@kernel.org>
Date: Thu, 5 Feb 2026 18:29:08 +0100
From: "David Hildenbrand (Arm)" <david@...nel.org>
To: Kiryl Shutsemau <kas@...nel.org>
Cc: "Pratik R. Sampat" <prsampat@....com>, linux-mm@...ck.org,
linux-coco@...ts.linux.dev, x86@...nel.org, linux-kernel@...r.kernel.org,
tglx@...utronix.de, mingo@...hat.com, bp@...en8.de,
dave.hansen@...ux.intel.com, ardb@...nel.org, akpm@...ux-foundation.org,
osalvador@...e.de, thomas.lendacky@....com, michael.roth@....com
Subject: Re: [PATCH v4 1/2] mm/memory_hotplug: Add support to accept memory
during hot-add
On 2/5/26 17:08, Kiryl Shutsemau wrote:
> On Thu, Feb 05, 2026 at 04:48:13PM +0100, David Hildenbrand (Arm) wrote:
>> On 2/5/26 11:48, Kiryl Shutsemau wrote:
>>>
>>> You are calling arch_accept_memory() on every memory allocation if the
>>> memory is not represented in the bitmap. Hard NAK.
>>
>> In which scenarios would we not have memory represented in the bitmap?
>> Guests with <4 GiB? (how does kexec work?) Anything else?
>
> We create the bitmap that covers all unaccepted memory.
Good! :)
>
> What memory is unaccepted is up to BIOS. Current implementation of edk2
> accepts the memory in the first 4G range of physical address space. It
> means we won't have bitmap for this range (unaccepted->phys_base >= 4G).
Ah, okay, this comes from the BIOS.
>
> If the whole VM is smaller than 4G we won't have the bitmap at all.
>
> We can allocate bitmap for all possible memory. Maybe upto max_possible_pfn?
> But we might not know the value in EFI stub. It costs 4k per 64GiB of
> physical address space.
That's what I would do. 4k per 64GiB sounds reasonable.
>
> Ideally, we want to know on boot:
>
> - what memory ranges are unaccepted - we have it;
> - what memory range can be removed or added after boot - we don't have it
The SRAT describes memory ranges where we can see hotplug memory. Is
that too late? We calculate max_possible_pfn based on that.
(don't ask me about special CXL windows and how they are advertised :) )
--
Cheers,
David
Powered by blists - more mailing lists