Replace hardcoded HV_MAP_GPA_DEPOSIT_PAGES usage with hv_deposit_memory() which derives the deposit size from the hypercall status, and remove the now-unused constant.
The previous code always deposited a fixed 256 pages on insufficient memory, ignoring the actual demand reported by the hypervisor. hv_deposit_memory() handles different deposit statuses, aligning map-GPA retries with the rest of the codebase. This approach may require more allocation and deposit hypercall iterations, but avoids over-depositing large fixed chunks when fewer pages would suffice. Until any performance impact is measured, the more frugal and consistent behavior is preferred. Signed-off-by: Stanislav Kinsburskii <[email protected]> --- drivers/hv/mshv_root_hv_call.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/hv/mshv_root_hv_call.c b/drivers/hv/mshv_root_hv_call.c index 7f91096f95a8..317191462b63 100644 --- a/drivers/hv/mshv_root_hv_call.c +++ b/drivers/hv/mshv_root_hv_call.c @@ -16,7 +16,6 @@ /* Determined empirically */ #define HV_INIT_PARTITION_DEPOSIT_PAGES 208 -#define HV_MAP_GPA_DEPOSIT_PAGES 256 #define HV_UMAP_GPA_PAGES 512 #define HV_PAGE_COUNT_2M_ALIGNED(pg_count) (!((pg_count) & (0x200 - 1))) @@ -239,8 +238,7 @@ static int hv_do_map_gpa_hcall(u64 partition_id, u64 gfn, u64 page_struct_count, completed = hv_repcomp(status); if (hv_result_needs_memory(status)) { - ret = hv_call_deposit_pages(NUMA_NO_NODE, partition_id, - HV_MAP_GPA_DEPOSIT_PAGES); + ret = hv_deposit_memory(partition_id, status); if (ret) break;
