qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH COLO-Frame v6 30/31] COLO: Implement shutdown ch


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH COLO-Frame v6 30/31] COLO: Implement shutdown checkpoint
Date: Thu, 18 Jun 2015 16:55:11 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0


On 18/06/2015 10:58, zhanghailiang wrote:
> +void qemu_system_shutdown_request(void)
> +{
> +    trace_qemu_system_shutdown_request();
> +    /*
> +    * if in colo mode, we need do some significant work before respond to the
> +    * shutdown request.
> +    */
> +    if (loadvm_in_colo_state()) {
> +        return ; /* primary's responsibility */
> +    }
> +    if (migrate_in_colo_state()) {
> +        colo_shutdown_requested = 1;
> +        return;
> +    }

The patch looks sane---the only comment I have is that
loadvm_in_colo_state() and migrate_in_colo_state() do not sound like
great names.

Paolo

> +    qemu_system_shutdown_request_core();
> +}
> +



reply via email to

[Prev in Thread] Current Thread [Next in Thread]