<div dir="ltr">IMO "return-pages" sounds the best out of those and stays relatively consistent with the kernel and qemu terminology for this feature. <div><br></div><div>I personally don't see a huge problem with the current name, but I've also been staring at the words "free page reporting" for too long.   </div><div><br></div><div>Cheers!</div><div>-- Nico </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Oct 14, 2020 at 1:56 AM Michal Privoznik <<a href="mailto:mprivozn@redhat.com">mprivozn@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 10/13/20 7:57 PM, Peter Krempa wrote:<br>
> On Tue, Oct 13, 2020 at 18:47:39 +0200, Michal Privoznik wrote:<br>
>> On 10/13/20 5:10 PM, Michal Privoznik wrote:<br>
>>> On 10/13/20 1:35 AM, Nico Pache wrote:<br>
>>>> gitlab issue: <a href="https://gitlab.com/libvirt/libvirt/-/issues/79" rel="noreferrer" target="_blank">https://gitlab.com/libvirt/libvirt/-/issues/79</a><br>
>>>><br>
>>>> The virtio-balloon device now has the ability to report free pages<br>
>>>> back to the hypervisor for reuse by other programs.<br>
>><br>
>> Is this something that we might want to report? I mean, we have 'virsh<br>
>> dommemstat $dom' which under the hood calls:<br>
>><br>
>> {"execute":"qom-get","arguments":{"path":"/machine/peripheral/balloon0","property":"guest-stats"},"id":"libvirt-400"}<br>
> <br>
> As I've pointed out in earlier review, I think that the feature name is<br>
> a bit misleading. It sounds like a statistic, not something that<br>
> actually returns memory to the host. The docs are now better but still<br>
> leave a lot of room for imagination.<br>
> <br>
> IMO, if the feature is mainly for returning memory to the host, the<br>
> 'reporting' word should not have been used.<br>
> <br>
<br>
Oh sorry I missed that. I a penance I will post a cleanup patch. How <br>
does "free-pages" or "return-pages" or even "discard-pages" sound?<br>
<br>
Michal<br>
<br>
</blockquote></div>