On Mon, Oct 20, 2025 at 02:14:00PM +0200, Gerd Hoffmann wrote:
On Fri, Oct 17, 2025 at 03:58:53PM +0200, Stefano Garzarella wrote:
On Wed, Oct 15, 2025 at 01:23:41PM +0200, Gerd Hoffmann wrote:
> Keep a linked list of the memory regions created by igvm.

Sorry, my knowledge of MemoryRegions is limited, but why do we need to do
this if we never access the list?
To keep references to regions?

Keep track of the things we have created, so we can clean them up
properly.  Which is indeed not needed in this patch, but we will
probably need that in the future for the bring-your-own-firmware
project (aka reboot guest with new IGVM image).

I see, so perhaps it makes sense to add this to the description, or postpone the patch until it is needed (not a strong opinion, but I'm not a fan of having code that isn't used).

Thanks,
Stefano


Reply via email to