> From: Kameron Carr <kameroncarr@xxxxxxxxxxxxxxxxxxx> > Sent: Thursday, June 22, 2023 1:07 PM > ... > Currently hv_free_hyperv_page() takes an unsigned long argument, which > is inconsistent with the void * return value from the corresponding > hv_alloc_hyperv_page() function and variants. This creates unnecessary > extra casting. > > Change the hv_free_hyperv_page() argument type to void *. > Also remove redundant casts from invocations of > hv_alloc_hyperv_page() and variants. > --- Hi Kameron, you forgot to add your Signed-off-by :-) Otherwise, it looks good to me. Reviewed-by: Dexuan Cui <decui@xxxxxxxxxxxxx>
