On Sun, 18 Jul 2021 at 14:54, Reinoud Zandijk <rein...@netbsd.org> wrote:
>
> A parameter max_size was added to the RAMBlockNotifier
> ram_block_added function. Use the max_size for pre allocation
> of hva space.
>
> Signed-off-by: Reinoud Zandijk <rein...@netbsd.org>
> ---
> target/i386/nvmm/nvmm-all.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/target/i386/nvmm/nvmm-all.c b/target/i386/nvmm/nvmm-all.c
> index dfa690d65d..fdcd37ed3e 100644
> --- a/target/i386/nvmm/nvmm-all.c
> +++ b/target/i386/nvmm/nvmm-all.c
> @@ -1134,13 +1134,14 @@ static MemoryListener nvmm_memory_listener = {
> };
>
> static void
> -nvmm_ram_block_added(RAMBlockNotifier *n, void *host, size_t size)
> +nvmm_ram_block_added(RAMBlockNotifier *n, void *host, size_t size,
> + size_t max_size)
> {
> struct nvmm_machine *mach = get_nvmm_mach();
> uintptr_t hva = (uintptr_t)host;
> int ret;
>
> - ret = nvmm_hva_map(mach, hva, size);
> + ret = nvmm_hva_map(mach, hva, max_size);
>
> if (ret == -1) {
> error_report("NVMM: Failed to map HVA, HostVA:%p "
This suggests that this code isn't being covered by our CI. Is
there something we can do to get it tested?
-- PMM
