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. -- Tom
signature.asc
Description: PGP signature
