Re: [PATCH] x86/Hyper-V: Support for free page reporting

2020-11-24 Thread Matheus Castello
Hi Sunil, first thank you very much for your work here. I am using this patch in the latest mainline releases with WSL 2 and it is helping me a lot. Let me know if you're still interested in continuing to work on this patch. It would be great to have it included in the mainline. If you need an

RE: [EXTERNAL] Re: [PATCH] x86/Hyper-V: Support for free page reporting

2020-05-22 Thread Michael Kelley
From: Sunil Muthuswamy Sent: Friday, May 22, 2020 9:40 AM > > > > + if (hv_do_hypercall(HV_EXT_CALL_QUERY_CAPABILITIES, NULL, cap) == > > > + HV_STATUS_SUCCESS) > > > > You're using the input page as the output parameter. Ideally we should > > introduce hyperv_pcpu_output_arg page, but that

Re: [EXTERNAL] Re: [PATCH] x86/Hyper-V: Support for free page reporting

2020-05-22 Thread Alexander Duyck
On Fri, May 22, 2020 at 9:42 AM Sunil Muthuswamy wrote: > [...] > > > > > + order = get_order(sg->length); > > > + range = &hint->ranges[i]; > > > + range->address_space = 0; > > > > I guess this means all address spaces? > > 'address_space' is being used here just

RE: [EXTERNAL] Re: [PATCH] x86/Hyper-V: Support for free page reporting

2020-05-22 Thread Sunil Muthuswamy
> As the only usage of this function looks like > if (!(hv_query_ext_cap() & HV_EXT_CAPABILITY_MEMORY_COLD_DISCARD_HINT)) > > I would've change the interface to > > bool hv_query_ext_cap(u64 cap) > > so the usage would look like > > if (!(hv_query_ext_cap(HV_EXT_CAPABILITY_MEMORY_COLD_DISCARD_H

RE: [EXTERNAL] Re: [PATCH] x86/Hyper-V: Support for free page reporting

2020-05-22 Thread Sunil Muthuswamy
> > + if (hv_do_hypercall(HV_EXT_CALL_QUERY_CAPABILITIES, NULL, cap) == > > + HV_STATUS_SUCCESS) > > You're using the input page as the output parameter. Ideally we should > introduce hyperv_pcpu_output_arg page, but that would waste one page per > cpu just for this one call. > > So for n

Re: [PATCH] x86/Hyper-V: Support for free page reporting

2020-05-20 Thread kbuild test robot
Hi Sunil, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on next-20200519] [cannot apply to tip/auto-latest linus/master tip/x86/core asm-generic/master linux/master v5.7-rc6 v5.7-rc5 v5.7-rc4 v5.7-rc6] [if your patch is applied to the wrong git tree, please drop

Re: [PATCH] x86/Hyper-V: Support for free page reporting

2020-05-20 Thread Wei Liu
On Wed, May 20, 2020 at 10:59:22AM +0200, Vitaly Kuznetsov wrote: > Sunil Muthuswamy writes: [...] > > +EXPORT_SYMBOL_GPL(hv_query_ext_cap); > > diff --git a/arch/x86/kernel/cpu/mshyperv.c b/arch/x86/kernel/cpu/mshyperv.c > > index ebf34c7bc8bc..2de3f692c8bf 100644 > > --- a/arch/x86/kernel/cpu/ms

Re: [PATCH] x86/Hyper-V: Support for free page reporting

2020-05-20 Thread Wei Liu
On Tue, May 19, 2020 at 06:37:57PM +, Sunil Muthuswamy wrote: > Linux has support for free page reporting now (36e66c554b5c) for > virtualized environment. On Hyper-V when virtually backed VMs are > configured, Hyper-V will advertise cold memory discard capability, > when supported. This patch

Re: [PATCH] x86/Hyper-V: Support for free page reporting

2020-05-20 Thread Vitaly Kuznetsov
Sunil Muthuswamy writes: > Linux has support for free page reporting now (36e66c554b5c) for > virtualized environment. On Hyper-V when virtually backed VMs are > configured, Hyper-V will advertise cold memory discard capability, > when supported. This patch adds the support to hook into the free

[PATCH] x86/Hyper-V: Support for free page reporting

2020-05-19 Thread Sunil Muthuswamy
Linux has support for free page reporting now (36e66c554b5c) for virtualized environment. On Hyper-V when virtually backed VMs are configured, Hyper-V will advertise cold memory discard capability, when supported. This patch adds the support to hook into the free page reporting infrastructure and l