[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V7 11/17] qapi: Add new command to query colo st
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH V7 11/17] qapi: Add new command to query colo status |
Date: |
Tue, 15 May 2018 16:26:08 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Zhang Chen <address@hidden> writes:
> Libvirt or other high level software can use this command query colo status.
> You can test this command like that:
> {'execute':'query-colo-status'}
>
> Signed-off-by: Zhang Chen <address@hidden>
> ---
> migration/colo.c | 34 ++++++++++++++++++++++++++++++++++
> qapi/migration.json | 33 +++++++++++++++++++++++++++++++++
> 2 files changed, 67 insertions(+)
>
> diff --git a/migration/colo.c b/migration/colo.c
> index 8ca6381..314344c 100644
> --- a/migration/colo.c
> +++ b/migration/colo.c
> @@ -29,6 +29,7 @@
> #include "net/colo.h"
> #include "block/block.h"
> #include "qapi/qapi-events-migration.h"
> +#include "qapi/qmp/qerror.h"
>
> static bool vmstate_loading;
> static Notifier packets_compare_notifier;
> @@ -237,6 +238,39 @@ void qmp_xen_colo_do_checkpoint(Error **errp)
> #endif
> }
>
> +COLOStatus *qmp_query_colo_status(Error **errp)
> +{
> + int state;
> + COLOStatus *s = g_new0(COLOStatus, 1);
> +
> + if (get_colo_mode() == COLO_MODE_UNKNOWN) {
> + error_setg(errp, QERR_FEATURE_DISABLED, "colo");
> + s->colo_running = false;
> + goto out;
> + } else if (get_colo_mode() == COLO_MODE_PRIMARY) {
> + state = migrate_get_current()->state;
> + } else {
> + state = migration_incoming_get_current()->state;
> + }
Indentation's off, as patchew pointed out. Perhaps the maintainer could
fix that for you.
> + s->colo_running = state == MIGRATION_STATUS_COLO;
> +
> +out:
> + s->mode = get_colo_mode();
I'm okay with this.
If you need to respin anyway, consider whether you like this better:
s->mode = get_colo_mode();
switch (s->mode) {
case COLO_MODE_UNKNOWN:
error_setg(errp, "COLO is disabled");
state = MIGRATION_STATUS_NONE;
break;
case COLO_MODE_PRIMARY:
state = migrate_get_current()->state;
break;
case COLO_MODE_SECONDARY:
state = migration_incoming_get_current()->state;
break;
default:
abort();
}
s->colo_running = state == MIGRATION_STATUS_COLO;
Aside: we should really finish off the QERR_ macros.
> +
> + switch (failover_get_state()) {
> + case FAILOVER_STATUS_NONE:
> + s->reason = COLO_EXIT_REASON_NONE;
> + break;
> + case FAILOVER_STATUS_REQUIRE:
> + s->reason = COLO_EXIT_REASON_REQUEST;
> + break;
> + default:
> + s->reason = COLO_EXIT_REASON_ERROR;
> + }
> +
> + return s;
> +}
> +
> static void colo_send_message(QEMUFile *f, COLOMessage msg,
> Error **errp)
> {
> diff --git a/qapi/migration.json b/qapi/migration.json
> index 55dae48..13589ba 100644
> --- a/qapi/migration.json
> +++ b/qapi/migration.json
> @@ -1220,3 +1220,36 @@
> # Since: 2.9
> ##
> { 'command': 'xen-colo-do-checkpoint' }
> +
> +##
> +# @COLOStatus:
> +#
> +# The result format for 'query-colo-status'.
> +#
> +# @mode: which COLO mode the VM was in when it exited.
Pardon my COLO-ignorance...
What does "when it exited" mean?
What's the value of @mode before "it exited"?
> +# @colo-running: true if COLO is running.
If COLO is running, has "it exited", yet?
> +#
> +# @reason: describes the reason for the COLO exit.
What's the value of @reason before a "COLO exit"?
Remember, while a COLO_EXIT event is tied to an exit, a
query-colo-status can be executed at any time, in particular before any
"COLO exit".
> +#
> +# Since: 2.13
> +##
> +{ 'struct': 'COLOStatus',
> + 'data': { 'mode': 'COLOMode', 'colo-running': 'bool', 'reason':
> 'COLOExitReason' } }
> +
> +##
> +# @query-colo-status:
> +#
> +# Query COLO status while the vm is running.
> +#
> +# Returns: A @COLOStatus object showing the status.
> +#
> +# Example:
> +#
> +# -> { "execute": "query-colo-status" }
> +# <- { "return": { "mode": "primary", "colo-running": true, "reason":
> "request" } }
> +#
> +# Since: 2.13
> +##
> +{ 'command': 'query-colo-status',
> + 'returns': 'COLOStatus' }
Almost ready :)
- [Qemu-devel] [PATCH V7 06/17] COLO: Remove colo_state migration struct, (continued)
- [Qemu-devel] [PATCH V7 06/17] COLO: Remove colo_state migration struct, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 05/17] COLO: Add block replication into colo process, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 13/17] COLO: flush host dirty ram from cache, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 01/17] filter-rewriter: fix memory leak for connection in connection_track_table, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 09/17] COLO: Flush memory data from ram cache, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 03/17] colo-compare: use notifier to notify packets comparing result, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 07/17] COLO: Load dirty pages into SVM's RAM cache firstly, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 02/17] colo-compare: implement the process of checkpoint, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 14/17] filter: Add handle_event method for NetFilterClass, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 11/17] qapi: Add new command to query colo status, Zhang Chen, 2018/05/14
- Re: [Qemu-devel] [PATCH V7 11/17] qapi: Add new command to query colo status,
Markus Armbruster <=
- [Qemu-devel] [PATCH V7 15/17] filter-rewriter: handle checkpoint and failover event, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 08/17] ram/COLO: Record the dirty pages that SVM received, Zhang Chen, 2018/05/14
- [Qemu-devel] [PATCH V7 10/17] qmp event: Add COLO_EXIT event to notify users while exited COLO, Zhang Chen, 2018/05/14
[Qemu-devel] [PATCH V7 12/17] savevm: split the process of different stages for loadvm/savevm, Zhang Chen, 2018/05/14
[Qemu-devel] [PATCH V7 17/17] COLO: quick failover process by kick COLO thread, Zhang Chen, 2018/05/14