Re: [Qemu-devel] [PATCH resend 2/3] savevm: fix wrong error set by ram_control_load_hook()

["Michael R. Hines" <mrhines@linux.vnet.ibm.com>]

On 08/23/2013 09:30 AM, Lei Li wrote:
> It should set negative error value if there has been an error.
>
> Signed-off-by: Lei Li <lilei@linux.vnet.ibm.com>
> ---
>   savevm.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/savevm.c b/savevm.c
> index 95a11f9..a0be109 100644
> --- a/savevm.c
> +++ b/savevm.c
> @@ -649,7 +649,7 @@ void ram_control_after_iterate(QEMUFile *f, uint64_t flags)
>
>   void ram_control_load_hook(QEMUFile *f, uint64_t flags)
>   {
> -    int ret = 0;
> +    int ret = -EINVAL;
>
>       if (f->ops->hook_ram_load) {
>           ret = f->ops->hook_ram_load(f, f->opaque, flags);

Reviewed-By: Michael R. Hines <mrhines@us.ibm.com>