[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v10 6/7] migration/ram.c: add a function to disa
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v10 6/7] migration/ram.c: add a function to disable the bulk stage |
Date: |
Mon, 3 Dec 2018 13:31:18 +0800 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Mon, Dec 03, 2018 at 10:18:30AM +0800, Wei Wang wrote:
> This patch adds a function to enable a precopy notifier callback outside
> the migration subsystem to disable the bulk stage flag. This is needed by
> the free page optimization offered by virtio-balloon.
>
> Signed-off-by: Wei Wang <address@hidden>
> CC: Dr. David Alan Gilbert <address@hidden>
> CC: Juan Quintela <address@hidden>
> CC: Michael S. Tsirkin <address@hidden>
> CC: Peter Xu <address@hidden>
> ---
> include/migration/misc.h | 1 +
> migration/ram.c | 9 +++++++++
> 2 files changed, 10 insertions(+)
>
> diff --git a/include/migration/misc.h b/include/migration/misc.h
> index 15f8d00..47e7ff5 100644
> --- a/include/migration/misc.h
> +++ b/include/migration/misc.h
> @@ -37,6 +37,7 @@ void precopy_infrastructure_init(void);
> void precopy_add_notifier(NotifierWithReturn *n);
> void precopy_remove_notifier(NotifierWithReturn *n);
> int precopy_notify(PrecopyNotifyReason reason, Error **errp);
> +void precopy_disable_bulk_stage(void);
>
> void ram_mig_init(void);
> void qemu_guest_free_page_hint(void *addr, size_t len);
> diff --git a/migration/ram.c b/migration/ram.c
> index b90a3f2..739dc97 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -354,6 +354,15 @@ int precopy_notify(PrecopyNotifyReason reason, Error
> **errp)
> return notifier_with_return_list_notify(&precopy_notifier_list, &pnd);
> }
>
> +void precopy_disable_bulk_stage(void)
> +{
> + if (!ram_state) {
> + return;
> + }
> +
> + ram_state->ram_bulk_stage = false;
> +}
> +
This one is a bit tricky. E.g., I think it'll at least affect XBZRLE
and compression somehow. Since we will have the on-off switch for
balloon free page hinting so the user can at least decide what
features to use, with that I don't see much issue with it so far. But
I'd also like to see how other people see this change.
Regards,
--
Peter Xu
- [Qemu-devel] [PATCH v10 0/7] virtio-balloon: free page hint support, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 2/7] bitmap: bitmap_count_one_with_offset, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 1/7] bitmap: fix bitmap_count_one, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 3/7] migration: use bitmap_mutex in migration_bitmap_clear_dirty, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 4/7] migration: API to clear bits of guest free pages from the dirty bitmap, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 6/7] migration/ram.c: add a function to disable the bulk stage, Wei Wang, 2018/12/02
- Re: [Qemu-devel] [PATCH v10 6/7] migration/ram.c: add a function to disable the bulk stage,
Peter Xu <=
- [Qemu-devel] [PATCH v10 5/7] migration/ram.c: add a notifier chain for precopy, Wei Wang, 2018/12/02
- [Qemu-devel] [PATCH v10 7/7] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_HINT, Wei Wang, 2018/12/02