Message ID | 1377264653-31031-2-git-send-email-lilei@linux.vnet.ibm.com |
---|---|
State | New |
Headers | show |
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);
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>
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);
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(-)