On Thu, Feb 19, 2026 at 10:09:32PM +0000, Stanislav Kinsburskii wrote: > 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; > > >
Reviewed-by: Anirudh Rayabharam (Microsoft) <[email protected]>
