[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 03/20] migration: Tracepoint change in postcopy-run bottom ha
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH 03/20] migration: Tracepoint change in postcopy-run bottom half |
Date: |
Wed, 16 Feb 2022 19:00:12 +0000 |
User-agent: |
Mutt/2.1.5 (2021-12-30) |
* Peter Xu (peterx@redhat.com) wrote:
> Remove the old two tracepoints and they're even near each other:
>
> trace_loadvm_postcopy_handle_run_cpu_sync()
> trace_loadvm_postcopy_handle_run_vmstart()
>
> Add trace_loadvm_postcopy_handle_run_bh() with a finer granule trace.
>
> Signed-off-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> migration/savevm.c | 12 +++++++++---
> migration/trace-events | 3 +--
> 2 files changed, 10 insertions(+), 5 deletions(-)
>
> diff --git a/migration/savevm.c b/migration/savevm.c
> index 190cc5fc42..41e3238798 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -2006,13 +2006,19 @@ static void loadvm_postcopy_handle_run_bh(void
> *opaque)
> Error *local_err = NULL;
> MigrationIncomingState *mis = opaque;
>
> + trace_loadvm_postcopy_handle_run_bh("enter");
> +
> /* TODO we should move all of this lot into postcopy_ram.c or a shared
> code
> * in migration.c
> */
> cpu_synchronize_all_post_init();
>
> + trace_loadvm_postcopy_handle_run_bh("after cpu sync");
> +
> qemu_announce_self(&mis->announce_timer, migrate_announce_params());
>
> + trace_loadvm_postcopy_handle_run_bh("after announce");
> +
> /* Make sure all file formats flush their mutable metadata.
> * If we get an error here, just don't restart the VM yet. */
> bdrv_invalidate_cache_all(&local_err);
> @@ -2022,9 +2028,7 @@ static void loadvm_postcopy_handle_run_bh(void *opaque)
> autostart = false;
> }
>
> - trace_loadvm_postcopy_handle_run_cpu_sync();
> -
> - trace_loadvm_postcopy_handle_run_vmstart();
> + trace_loadvm_postcopy_handle_run_bh("after invalidate cache");
>
> dirty_bitmap_mig_before_vm_start();
>
> @@ -2037,6 +2041,8 @@ static void loadvm_postcopy_handle_run_bh(void *opaque)
> }
>
> qemu_bh_delete(mis->bh);
> +
> + trace_loadvm_postcopy_handle_run_bh("return");
> }
>
> /* After all discards we can start running and asking for pages */
> diff --git a/migration/trace-events b/migration/trace-events
> index 92596c00d8..1aec580e92 100644
> --- a/migration/trace-events
> +++ b/migration/trace-events
> @@ -16,8 +16,7 @@ loadvm_handle_recv_bitmap(char *s) "%s"
> loadvm_postcopy_handle_advise(void) ""
> loadvm_postcopy_handle_listen(const char *str) "%s"
> loadvm_postcopy_handle_run(void) ""
> -loadvm_postcopy_handle_run_cpu_sync(void) ""
> -loadvm_postcopy_handle_run_vmstart(void) ""
> +loadvm_postcopy_handle_run_bh(const char *str) "%s"
> loadvm_postcopy_handle_resume(void) ""
> loadvm_postcopy_ram_handle_discard(void) ""
> loadvm_postcopy_ram_handle_discard_end(void) ""
> --
> 2.32.0
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
- [PATCH 00/20] migration: Postcopy Preemption, Peter Xu, 2022/02/16
- [PATCH 01/20] migration: Dump sub-cmd name in loadvm_process_command tp, Peter Xu, 2022/02/16
- [PATCH 03/20] migration: Tracepoint change in postcopy-run bottom half, Peter Xu, 2022/02/16
- Re: [PATCH 03/20] migration: Tracepoint change in postcopy-run bottom half,
Dr. David Alan Gilbert <=
- [PATCH 04/20] migration: Introduce postcopy channels on dest node, Peter Xu, 2022/02/16
- [PATCH 02/20] migration: Finer grained tracepoints for POSTCOPY_LISTEN, Peter Xu, 2022/02/16
- [PATCH 05/20] migration: Dump ramblock and offset too when non-same-page detected, Peter Xu, 2022/02/16
- [PATCH 06/20] migration: Add postcopy_thread_create(), Peter Xu, 2022/02/16
- [PATCH 07/20] migration: Move static var in ram_block_from_stream() into global, Peter Xu, 2022/02/16
- [PATCH 08/20] migration: Add pss.postcopy_requested status, Peter Xu, 2022/02/16
- [PATCH 09/20] migration: Move migrate_allow_multifd and helpers into migration.c, Peter Xu, 2022/02/16