From: Alexander Duyck <email@example.com> To: "Michael S. Tsirkin" <firstname.lastname@example.org> Cc: David Hildenbrand <email@example.com>, kvm list <firstname.lastname@example.org>, LKML <email@example.com>, Matthew Wilcox <firstname.lastname@example.org>, Michal Hocko <email@example.com>, linux-mm <firstname.lastname@example.org>, Andrew Morton <email@example.com>, Mel Gorman <firstname.lastname@example.org>, Vlastimil Babka <email@example.com>, Yang Zhang <firstname.lastname@example.org>, Nitesh Narayan Lal <email@example.com>, Konrad Rzeszutek Wilk <firstname.lastname@example.org>, Pankaj Gupta <email@example.com>, Rik van Riel <firstname.lastname@example.org>, email@example.com, Dave Hansen <firstname.lastname@example.org>, "Wang, Wei W" <email@example.com>, Andrea Arcangeli <firstname.lastname@example.org>, Paolo Bonzini <email@example.com>, Dan Williams <firstname.lastname@example.org>, Alexander Duyck <email@example.com>, Oscar Salvador <firstname.lastname@example.org> Subject: Re: [PATCH v14 6/6] virtio-balloon: Add support for providing unused page reports to host Date: Wed, 4 Dec 2019 09:53:53 -0800 [thread overview] Message-ID: <CAKgT0Uf3uhm5B6_aYphJEMLvWicMpnygfE=vYMDx7T4KSpMp-A@mail.gmail.com> (raw) In-Reply-To: <email@example.com> On Thu, Nov 28, 2019 at 9:00 AM Michael S. Tsirkin <firstname.lastname@example.org> wrote: > > On Thu, Nov 28, 2019 at 04:25:54PM +0100, David Hildenbrand wrote: > > On 19.11.19 22:46, Alexander Duyck wrote: > > > From: Alexander Duyck <email@example.com> > > > > > > Add support for the page reporting feature provided by virtio-balloon. > > > Reporting differs from the regular balloon functionality in that is is > > > much less durable than a standard memory balloon. Instead of creating a > > > list of pages that cannot be accessed the pages are only inaccessible > > > while they are being indicated to the virtio interface. Once the > > > interface has acknowledged them they are placed back into their respective > > > free lists and are once again accessible by the guest system. > > > > Maybe add something like "In contrast to ordinary balloon > > inflation/deflation, the guest can reuse all reported pages immediately > > after reporting has finished, without having to notify the hypervisor > > about it (e.g., VIRTIO_BALLOON_F_MUST_TELL_HOST does not apply)." > > Maybe we can make apply. The effect of reporting a page is effectively > putting it in a balloon then immediately taking it out. Maybe without > VIRTIO_BALLOON_F_MUST_TELL_HOST the pages can be reused before host > marked buffers used? > > We didn't teach existing page hinting to behave like this, but maybe we > should, and maybe it's not too late, not a long time passed > since it was merged, and the whole shrinker based thing > seems to have been broken ... The problem is the existing hinting implementation relies on pushing the memory to the point of OOM in order to avoid having to re-hint on pages. What it is looking for is a snapshot rather than a running tally. The page reporting bit approach would only work for the first migration. The problem is the bit is persistent and would leave unused pages flagged as reported if another migration starts so it wouldn't re-report those pages. > BTW generally UAPI patches will have to be sent to virtio-dev > mailing list before they are merged. Do you need just the QEMU patches submitted to virtio-dev or both the virtio kernel patches and the QEMU patches? One piece of feedback I got was that it was annoying that I was including virtio-dev since it requires a subscription to send to it. If you would like I could apply it on the QEMU patches which would make the changes more visible at least. Thanks. - Alex
next prev parent reply other threads:[~2019-12-04 17:54 UTC|newest] Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-11-19 21:46 [PATCH v14 0/6] mm / virtio: Provide support for unused page reporting Alexander Duyck 2019-11-19 21:46 ` [PATCH v14 1/6] mm: Adjust shuffle code to allow for future coalescing Alexander Duyck 2019-11-27 13:52 ` Mel Gorman 2019-11-19 21:46 ` [PATCH v14 2/6] mm: Use zone and order instead of free area in free_list manipulators Alexander Duyck 2019-11-27 13:54 ` Mel Gorman 2019-11-19 21:46 ` [PATCH v14 3/6] mm: Introduce Reported pages Alexander Duyck 2019-11-27 15:24 ` Mel Gorman 2019-11-27 17:22 ` Alexander Duyck 2019-11-27 18:35 ` Mel Gorman 2019-11-27 21:55 ` Alexander Duyck 2019-11-28 9:22 ` Mel Gorman 2019-11-29 19:25 ` Alexander Duyck 2019-11-19 21:46 ` [PATCH v14 4/6] mm: Add unused page reporting documentation Alexander Duyck 2019-11-19 21:46 ` [PATCH v14 5/6] virtio-balloon: Pull page poisoning config out of free page hinting Alexander Duyck 2019-11-19 21:46 ` [PATCH v14 6/6] virtio-balloon: Add support for providing unused page reports to host Alexander Duyck 2019-11-28 15:25 ` David Hildenbrand 2019-11-28 17:00 ` Michael S. Tsirkin 2019-12-04 17:48 ` Alexander Duyck 2019-12-04 17:53 ` Alexander Duyck [this message] 2019-11-29 21:13 ` Alexander Duyck 2019-12-01 11:46 ` Michael S. Tsirkin 2019-12-01 18:25 ` Alexander Duyck 2019-12-02 10:43 ` David Hildenbrand 2019-11-19 21:54 ` [PATCH v14 QEMU 1/3] virtio-ballon: Implement support for page poison tracking feature Alexander Duyck 2019-11-19 21:54 ` [PATCH v14 QEMU 2/3] virtio-balloon: Add bit to notify guest of unused page reporting Alexander Duyck 2019-11-19 21:54 ` [PATCH v14 QEMU 3/3] virtio-balloon: Provide a interface for " Alexander Duyck 2019-11-26 12:20 ` [PATCH v14 0/6] mm / virtio: Provide support " David Hildenbrand 2019-11-26 16:45 ` Alexander Duyck 2019-11-27 10:01 ` David Hildenbrand 2019-11-27 17:36 ` Alexander Duyck 2019-11-27 17:37 ` David Hildenbrand
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAKgT0Uf3uhm5B6_aYphJEMLvWicMpnygfE=vYMDx7T4KSpMp-A@mail.gmail.com' \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [PATCH v14 6/6] virtio-balloon: Add support for providing unused page reports to host' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).