On Fri, Jan 24, 2025 at 01:39:45PM +0200, Ilias Apalodimas wrote:
Heinrich, Tobias
There's a slight problem that I forgot when commenting v2.
Heinrich's idea of plugging this into blkmap is eventually the right thing to do.
However, when I started coding this I only added the pmem memory as 'reserved' in the DT hoping that would work. Unfortunately, this depends on a kernel config option. I've managed to track down the problem here[0], but I haven't found time to test it properly and send it upstream. So for this feature to work reliably we *need* to remove the memory map we hand over to the OS.
Since using EFI memmap function into the blkmap code makes no sense, can we perhaps merge v2 (or a variant of it), which only targets EFI, with an explanation of *why* while I try to sort out the kernel issue?
[0] https://git.linaro.org/people/ilias.apalodimas/net-next.git/commit/?h=main
I think this is a reasonable path forward, FWIW.