[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/13] Add tracepoints for savevm section start/
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 07/13] Add tracepoints for savevm section start/end |
Date: |
Fri, 29 Jun 2012 20:32:49 +0000 |
On Fri, Jun 29, 2012 at 4:43 PM, Juan Quintela <address@hidden> wrote:
> This allows to know how long each section takes to save.
>
> An awk script like this tells us sections that takes more that 10ms
>
> $1 ~ /savevm_state_iterate_end/ {
> /* Print savevm_section_end line when > 10ms duration */
> if ($2 > 10000) {
> printf("%s times_missing=%u\n", $0, times_missing++);
> }
> }
>
> Signed-off-by: Juan Quintela <address@hidden>
--- missing
>
> fix ws tracepoints
>
> Signed-off-by: Juan Quintela <address@hidden>
duplicate
> ---
> savevm.c | 8 ++++++++
> trace-events | 5 +++++
> 2 files changed, 13 insertions(+)
>
> diff --git a/savevm.c b/savevm.c
> index d1d9020..987c6c0 100644
> --- a/savevm.c
> +++ b/savevm.c
> @@ -85,6 +85,7 @@
> #include "cpus.h"
> #include "memory.h"
> #include "qmp-commands.h"
> +#include "trace.h"
>
> #define SELF_ANNOUNCE_ROUNDS 5
>
> @@ -1625,11 +1626,14 @@ int qemu_savevm_state_iterate(QEMUFile *f)
> if (se->save_live_state == NULL)
> continue;
>
> + trace_savevm_section_start();
> /* Section type */
> qemu_put_byte(f, QEMU_VM_SECTION_PART);
> qemu_put_be32(f, se->section_id);
>
> ret = se->save_live_state(f, QEMU_VM_SECTION_PART, se->opaque);
> + trace_savevm_section_end(se->section_id);
> +
> if (ret <= 0) {
> /* Do not proceed to the next vmstate before this one reported
> completion of the current stage. This serializes the migration
> @@ -1659,11 +1663,13 @@ int qemu_savevm_state_complete(QEMUFile *f)
> if (se->save_live_state == NULL)
> continue;
>
> + trace_savevm_section_start();
> /* Section type */
> qemu_put_byte(f, QEMU_VM_SECTION_END);
> qemu_put_be32(f, se->section_id);
>
> ret = se->save_live_state(f, QEMU_VM_SECTION_END, se->opaque);
> + trace_savevm_section_end(se->section_id);
> if (ret < 0) {
> return ret;
> }
> @@ -1675,6 +1681,7 @@ int qemu_savevm_state_complete(QEMUFile *f)
> if (se->save_state == NULL && se->vmsd == NULL)
> continue;
>
> + trace_savevm_section_start();
> /* Section type */
> qemu_put_byte(f, QEMU_VM_SECTION_FULL);
> qemu_put_be32(f, se->section_id);
> @@ -1688,6 +1695,7 @@ int qemu_savevm_state_complete(QEMUFile *f)
> qemu_put_be32(f, se->version_id);
>
> vmstate_save(f, se);
> + trace_savevm_section_end(se->section_id);
> }
>
> qemu_put_byte(f, QEMU_VM_EOF);
> diff --git a/trace-events b/trace-events
> index c935ba2..d671118 100644
> --- a/trace-events
> +++ b/trace-events
> @@ -783,6 +783,11 @@ displaysurface_resize(void *display_state, void
> *display_surface, int width, int
> # vga.c
> ppm_save(const char *filename, void *display_surface) "%s surface=%p"
>
> +# savevm.c
> +
> +savevm_section_start(void) ""
> +savevm_section_end(unsigned int section_id) "section_id %u"
> +
> # hw/qxl.c
> disable qxl_interface_set_mm_time(int qid, uint32_t mm_time) "%d %d"
> disable qxl_io_write_vga(int qid, const char *mode, uint32_t addr, uint32_t
> val) "%d %s addr=%u val=%u"
> --
> 1.7.10.4
>
>
- [Qemu-devel] [PATCH v3 00/13] Migration tree, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 02/13] Add MigrationParams structure, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 01/13] Add missing check for host_from_stream_offset return value for RAM_SAVE_FLAG_PAGE, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 05/13] Add migration_end function, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 07/13] Add tracepoints for savevm section start/end, Juan Quintela, 2012/06/29
- Re: [Qemu-devel] [PATCH 07/13] Add tracepoints for savevm section start/end,
Blue Swirl <=
- [Qemu-devel] [PATCH 04/13] Add debugging infrastructure, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 11/13] Exit loop if we have been there too long, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 06/13] Add spent time for migration, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 03/13] Add save_block_hdr function, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 08/13] No need to iterate if we already are over the limit, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 12/13] dirty bitmap: abstract its use, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 10/13] Only calculate expected_time for stage 2, Juan Quintela, 2012/06/29
- [Qemu-devel] [PATCH 13/13] Maintain the number of dirty pages, Juan Quintela, 2012/06/29