On Tue, Jun 11, 2019 at 5:19 AM Nitesh Narayan Lal <nitesh@xxxxxxxxxx> wrote: > > > On 6/3/19 2:04 PM, Michael S. Tsirkin wrote: > > On Mon, Jun 03, 2019 at 01:03:04PM -0400, Nitesh Narayan Lal wrote: > >> This patch series proposes an efficient mechanism for communicating free memory > >> from a guest to its hypervisor. It especially enables guests with no page cache > >> (e.g., nvdimm, virtio-pmem) or with small page caches (e.g., ram > disk) to > >> rapidly hand back free memory to the hypervisor. > >> This approach has a minimal impact on the existing core-mm infrastructure. > > Could you help us compare with Alex's series? > > What are the main differences? > Sorry for the late reply, but I haven't been feeling too well during the > last week. > > The main differences are that this series uses a bitmap to track pages > that should be hinted to the hypervisor, while Alexander's series tracks > it directly in core-mm. Also in order to prevent duplicate hints > Alexander's series uses a newly defined page flag whereas I have added > another argument to __free_one_page. > For these reasons, Alexander's series is relatively more core-mm > invasive, while this series is lightweight (e.g., LOC). We'll have to > see if there are real performance differences. > > I'm planning on doing some further investigations/review/testing/... > once I'm back on track. BTW one thing I found is that I will likely need to add a new parameter like you did to __free_one_page as I need to defer setting the flag until after all of the merges have happened. Otherwise set the flag on a given page, and then after the merge that page may not be the one we ultimately add to the free list. I'll try to have an update with all of my changes ready before the end of this week. Thanks. - Alex
