Commit 385d7e03 authored by Kinsbursky Stanislav's avatar Kinsbursky Stanislav Committed by Cyrill Gorcunov

parasite: some cleanup

1) Added few missed successfull status setup - this looks
   redundant though, but unified with other functions.
2) remove redundant argument in dump_pages_fini().
Signed-off-by: 's avatarStanislav Kinsbursky <skinsbursky@openvz.org>
Acked-by: 's avatarPavel Emelyanov <xemul@parallels.com>
Signed-off-by: 's avatarCyrill Gorcunov <gorcunov@openvz.org>
parent acf604c6
...@@ -106,6 +106,7 @@ static int dump_pages_init(parasite_status_t *st) ...@@ -106,6 +106,7 @@ static int dump_pages_init(parasite_status_t *st)
if (fd_pages[PG_SHARED] < 0) if (fd_pages[PG_SHARED] < 0)
goto err_s; goto err_s;
SET_PARASITE_STATUS(st, 0, 0);
return 0; return 0;
err_s: err_s:
...@@ -241,7 +242,7 @@ err: ...@@ -241,7 +242,7 @@ err:
return ret; return ret;
} }
static int dump_pages_fini(parasite_status_t *st) static int dump_pages_fini(void)
{ {
sys_close(fd_pages[PG_PRIV]); sys_close(fd_pages[PG_PRIV]);
sys_close(fd_pages[PG_SHARED]); sys_close(fd_pages[PG_SHARED]);
...@@ -395,6 +396,7 @@ static int init(struct parasite_init_args *args) ...@@ -395,6 +396,7 @@ static int init(struct parasite_init_args *args)
else else
reset_blocked = 1; reset_blocked = 1;
SET_PARASITE_STATUS(&args->status, ret, ret);
return ret; return ret;
} }
...@@ -433,7 +435,7 @@ static int __used parasite_service(unsigned long cmd, void *args, void *brk) ...@@ -433,7 +435,7 @@ static int __used parasite_service(unsigned long cmd, void *args, void *brk)
case PARASITE_CMD_DUMPPAGES_INIT: case PARASITE_CMD_DUMPPAGES_INIT:
return dump_pages_init((parasite_status_t *) args); return dump_pages_init((parasite_status_t *) args);
case PARASITE_CMD_DUMPPAGES_FINI: case PARASITE_CMD_DUMPPAGES_FINI:
return dump_pages_fini((parasite_status_t *) args); return dump_pages_fini();
case PARASITE_CMD_DUMPPAGES: case PARASITE_CMD_DUMPPAGES:
return dump_pages((struct parasite_dump_pages_args *)args); return dump_pages((struct parasite_dump_pages_args *)args);
case PARASITE_CMD_DUMP_SIGACTS: case PARASITE_CMD_DUMP_SIGACTS:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment