On 13.11.18 20:53, Simon Glass wrote:
Hi Heinrich,
On 10 November 2018 at 14:30, Heinrich Schuchardt xypron.glpk@gmx.de wrote:
Do not use the sandbox's virtual address space for the internal structures of the memory map. This way we can eliminate a whole lot of unnecessary conversions.
The only conversion remaining is the one when adding known memory.
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de
lib/efi_loader/efi_memory.c | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-)
I much prefer this, but Alex was not too keen.
Uh, I thought you were the one who wanted to only expose fake addresses? I think the patch is great as is :). It definitely simplifies the code.
Also please can you add detailed comments to efi_mem_desc about what the addresses are for sandbox?
I'll wait for v2 then.
Alex