search for: free_page_cmd_in

.... */ > wait_queue_head_t acked; > > @@ -93,6 +102,11 @@ struct virtio_balloon { > > /* To register callback in oom notifier call chain */ > struct notifier_block nb; > + > + /* Host to guest ctrlq cmd buf for free page report */ > + struct virtio_balloon_ctrlq_cmd free_page_cmd_in; > + /* Guest to Host ctrlq cmd buf for free page report */ > + struct virtio_balloon_ctrlq_cmd free_page_cmd_out; > }; > > static struct virtio_device_id id_table[] = { @@ -177,6 +191,26 @@ static void > send_balloon_page_sg(struct virtio_balloon *vb, > } > } > &g...

.... */ > wait_queue_head_t acked; > > @@ -93,6 +102,11 @@ struct virtio_balloon { > > /* To register callback in oom notifier call chain */ > struct notifier_block nb; > + > + /* Host to guest ctrlq cmd buf for free page report */ > + struct virtio_balloon_ctrlq_cmd free_page_cmd_in; > + /* Guest to Host ctrlq cmd buf for free page report */ > + struct virtio_balloon_ctrlq_cmd free_page_cmd_out; > }; > > static struct virtio_device_id id_table[] = { @@ -177,6 +191,26 @@ static void > send_balloon_page_sg(struct virtio_balloon *vb, > } > } > &g...

...iting for host to ack the pages we released. */ wait_queue_head_t acked; @@ -93,6 +102,11 @@ struct virtio_balloon { /* To register callback in oom notifier call chain */ struct notifier_block nb; + + /* Host to guest ctrlq cmd buf for free page report */ + struct virtio_balloon_ctrlq_cmd free_page_cmd_in; + /* Guest to Host ctrlq cmd buf for free page report */ + struct virtio_balloon_ctrlq_cmd free_page_cmd_out; }; static struct virtio_device_id id_table[] = { @@ -186,6 +200,24 @@ static int send_balloon_page_sg(struct virtio_balloon *vb, return err; } +static int send_free_page_sg(struct...

...*/ > wait_queue_head_t acked; > > @@ -93,6 +102,11 @@ struct virtio_balloon { > > /* To register callback in oom notifier call chain */ > struct notifier_block nb; > + > + /* Host to guest ctrlq cmd buf for free page report */ > + struct virtio_balloon_ctrlq_cmd free_page_cmd_in; > + /* Guest to Host ctrlq cmd buf for free page report */ > + struct virtio_balloon_ctrlq_cmd free_page_cmd_out; > }; > > static struct virtio_device_id id_table[] = { > @@ -186,6 +200,24 @@ static int send_balloon_page_sg(struct virtio_balloon *vb, > return err; >...

...*/ > wait_queue_head_t acked; > > @@ -93,6 +102,11 @@ struct virtio_balloon { > > /* To register callback in oom notifier call chain */ > struct notifier_block nb; > + > + /* Host to guest ctrlq cmd buf for free page report */ > + struct virtio_balloon_ctrlq_cmd free_page_cmd_in; > + /* Guest to Host ctrlq cmd buf for free page report */ > + struct virtio_balloon_ctrlq_cmd free_page_cmd_out; > }; > > static struct virtio_device_id id_table[] = { > @@ -186,6 +200,24 @@ static int send_balloon_page_sg(struct virtio_balloon *vb, > return err; >...

...works: there is only one stats buffer, which is used by the guest to report stats, and also used by the host to ask the guest for stats report. So the host can roll back one previous entry and what it gets will always be stat_vq_elem. Our case is a little more complex than that - we have both free_page_cmd_in (for host to guest command) and free_page_cmd_out (for guest to host command) buffer passed via ctrl_vq. When the host rolls back one entry, it may get the free_page_cmd_out buffer which can't be used as the host to guest buffer (i.e. free_page_elem held by the device). So a trick in the dr...

..._STOP) { > > + vb->report_free_page_stop = true; > > + } else if (cmd == VIRTIO_BALLOON_FREE_PAGE_F_START) { > > + vb->report_free_page_stop = false; > > + queue_work(vb->balloon_wq, &vb- > >report_free_page_work); > > + } > > + vb->free_page_cmd_in.class = > > + > VIRTIO_BALLOON_CTRLQ_CLASS_FREE_PAGE; > > + ctrlq_send_cmd(vb, &vb->free_page_cmd_in, true); > > + break; > > + default: > > + dev_warn(&vb->vdev->dev, "%s: cmd class not supported\n", > > + __func__); > >...

On Wed, Oct 11, 2017 at 02:03:20PM +0800, Wei Wang wrote: > On 10/10/2017 11:15 PM, Michael S. Tsirkin wrote: > > On Mon, Oct 02, 2017 at 04:38:01PM +0000, Wang, Wei W wrote: > > > On Sunday, October 1, 2017 11:19 AM, Michael S. Tsirkin wrote: > > > > On Sat, Sep 30, 2017 at 12:05:54PM +0800, Wei Wang wrote: > > > > > +static void ctrlq_send_cmd(struct

On Wed, Oct 11, 2017 at 02:03:20PM +0800, Wei Wang wrote: > On 10/10/2017 11:15 PM, Michael S. Tsirkin wrote: > > On Mon, Oct 02, 2017 at 04:38:01PM +0000, Wang, Wei W wrote: > > > On Sunday, October 1, 2017 11:19 AM, Michael S. Tsirkin wrote: > > > > On Sat, Sep 30, 2017 at 12:05:54PM +0800, Wei Wang wrote: > > > > > +static void ctrlq_send_cmd(struct

This patch series enhances the existing virtio-balloon with the following new features: 1) fast ballooning: transfer ballooned pages between the guest and host in chunks using sgs, instead of one array each time; and 2) free page block reporting: a new virtqueue to report guest free pages to the host. The second feature can be used to accelerate live migration of VMs. Here are some details: Live

This patch series enhances the existing virtio-balloon with the following new features: 1) fast ballooning: transfer ballooned pages between the guest and host in chunks using sgs, instead of one array each time; and 2) free page block reporting: a new virtqueue to report guest free pages to the host. The second feature can be used to accelerate live migration of VMs. Here are some details: Live

...report_free_page_stop = true; > > > + } else if (cmd == VIRTIO_BALLOON_FREE_PAGE_F_START) { > > > + vb->report_free_page_stop = false; > > > + queue_work(vb->balloon_wq, &vb- > > >report_free_page_work); > > > + } > > > + vb->free_page_cmd_in.class = > > > + > > VIRTIO_BALLOON_CTRLQ_CLASS_FREE_PAGE; > > > + ctrlq_send_cmd(vb, &vb->free_page_cmd_in, true); > > > + break; > > > + default: > > > + dev_warn(&vb->vdev->dev, "%s: cmd class not supported\n", >...

...report_free_page_stop = true; > > > + } else if (cmd == VIRTIO_BALLOON_FREE_PAGE_F_START) { > > > + vb->report_free_page_stop = false; > > > + queue_work(vb->balloon_wq, &vb- > > >report_free_page_work); > > > + } > > > + vb->free_page_cmd_in.class = > > > + > > VIRTIO_BALLOON_CTRLQ_CLASS_FREE_PAGE; > > > + ctrlq_send_cmd(vb, &vb->free_page_cmd_in, true); > > > + break; > > > + default: > > > + dev_warn(&vb->vdev->dev, "%s: cmd class not supported\n", >...