search for: _clear

...;s only needed in the balloon page case. > For the unused page case, we don't need it, since the free > page blocks are already chunks. > > > OK as far as it goes but you need much better isolation for it. > > Build a data structure with APIs such as _init, _cleanup, _add, _clear, > > _find_first, _find_next. > > Completely unrelated to pages, it just maintains bits. > > Then use it here. > > > > > > > static int oom_pages = OOM_VBALLOON_DEFAULT_PAGES; > > > module_param(oom_pages, int, S_IRUSR | S_IWUSR); > > >...

...;s only needed in the balloon page case. > For the unused page case, we don't need it, since the free > page blocks are already chunks. > > > OK as far as it goes but you need much better isolation for it. > > Build a data structure with APIs such as _init, _cleanup, _add, _clear, > > _find_first, _find_next. > > Completely unrelated to pages, it just maintains bits. > > Then use it here. > > > > > > > static int oom_pages = OOM_VBALLOON_DEFAULT_PAGES; > > > module_param(oom_pages, int, S_IRUSR | S_IWUSR); > > >...

...-by: Michael S. Tsirkin <mst at redhat.com> So we don't need the bitmap to talk to host, it is just a data structure we chose to maintain lists of pages, right? OK as far as it goes but you need much better isolation for it. Build a data structure with APIs such as _init, _cleanup, _add, _clear, _find_first, _find_next. Completely unrelated to pages, it just maintains bits. Then use it here. > --- > drivers/virtio/virtio_balloon.c | 384 +++++++++++++++++++++++++++++++++--- > include/uapi/linux/virtio_balloon.h | 13 ++ > 2 files changed, 374 insertions(+), 23 deletion...

...-by: Michael S. Tsirkin <mst at redhat.com> So we don't need the bitmap to talk to host, it is just a data structure we chose to maintain lists of pages, right? OK as far as it goes but you need much better isolation for it. Build a data structure with APIs such as _init, _cleanup, _add, _clear, _find_first, _find_next. Completely unrelated to pages, it just maintains bits. Then use it here. > --- > drivers/virtio/virtio_balloon.c | 384 +++++++++++++++++++++++++++++++++--- > include/uapi/linux/virtio_balloon.h | 13 ++ > 2 files changed, 374 insertions(+), 23 deletion...

...the balloon page case. >> For the unused page case, we don't need it, since the free >> page blocks are already chunks. >> >>> OK as far as it goes but you need much better isolation for it. >>> Build a data structure with APIs such as _init, _cleanup, _add, _clear, >>> _find_first, _find_next. >>> Completely unrelated to pages, it just maintains bits. >>> Then use it here. >>> >>> >>>> static int oom_pages = OOM_VBALLOON_DEFAULT_PAGES; >>>> module_param(oom_pages, int, S_IRUSR | S_IWU...

...he unused page case, we don't need it, since the free page > >> blocks are already chunks. > >> > >>> OK as far as it goes but you need much better isolation for it. > >>> Build a data structure with APIs such as _init, _cleanup, _add, > >>> _clear, _find_first, _find_next. > >>> Completely unrelated to pages, it just maintains bits. > >>> Then use it here. > >>> > >>> > >>>> static int oom_pages = OOM_VBALLOON_DEFAULT_PAGES; > >>>> module_param(oom_pages, int...

...he unused page case, we don't need it, since the free page > >> blocks are already chunks. > >> > >>> OK as far as it goes but you need much better isolation for it. > >>> Build a data structure with APIs such as _init, _cleanup, _add, > >>> _clear, _find_first, _find_next. > >>> Completely unrelated to pages, it just maintains bits. > >>> Then use it here. > >>> > >>> > >>>> static int oom_pages = OOM_VBALLOON_DEFAULT_PAGES; > >>>> module_param(oom_pages, int...

...04-14 07:09:13 UTC (rev 965) </span><span class="lines">@@ -85,7 +85,7 @@ </span><span class="cx"> </span><span class="cx"> Removes all items from the control. </span><span class="cx"> </span><del>-_Clear()_ also deletes the client data of the existing items if it is owned </del><ins>+_clear()_ also deletes the client data of the existing items if it is owned </ins><span class="cx"> by the control. </span><span class="cx"> </span><spa...

...er pages to chunk. It's only needed in the balloon page case. For the unused page case, we don't need it, since the free page blocks are already chunks. > OK as far as it goes but you need much better isolation for it. > Build a data structure with APIs such as _init, _cleanup, _add, _clear, > _find_first, _find_next. > Completely unrelated to pages, it just maintains bits. > Then use it here. > > >> static int oom_pages = OOM_VBALLOON_DEFAULT_PAGES; >> module_param(oom_pages, int, S_IRUSR | S_IWUSR); >> MODULE_PARM_DESC(oom_pages, "pages to...

This patch series implements two optimizations: 1) transfer pages in chuncks between the guest and host; 2) transfer the guest unused pages to the host so that they can be skipped to migrate in live migration. Changes: v8->v9: 1) Split the two new features, VIRTIO_BALLOON_F_BALLOON_CHUNKS and VIRTIO_BALLOON_F_MISC_VQ, which were mixed together in the previous implementation; 2) Simpler

This patch series implements two optimizations: 1) transfer pages in chuncks between the guest and host; 2) transfer the guest unused pages to the host so that they can be skipped to migrate in live migration. Changes: v8->v9: 1) Split the two new features, VIRTIO_BALLOON_F_BALLOON_CHUNKS and VIRTIO_BALLOON_F_MISC_VQ, which were mixed together in the previous implementation; 2) Simpler