[CRIU] [PATCH 04/10] unix: add ability to set callbacks for external sockets (v5)
Andrey Vagin
avagin at openvz.org
Wed Dec 18 03:36:33 PST 2013
We don't know a state behind an external socket. It depends on logic
of the program, which handles this socket.
This patch adds ability to load a library with callbacks for dumping
and restoring external sockets.
This patch introduces two callbacks cr_plugin_dump_unix_sk and
cr_plugin_restore_unix_sk. If a callback can not handle a socket, it
must return -ENOTSUP.
The main questions, what kind of information should be tranfered in
these callbacks. Pls, think a few minutes about that and send me
your opinion.
v2: Use uflags instread of adding a new field
v3: clean up
v4: Unsuitable callbacks return -ENOTSUP.
v5: set USK_CALLBACK, if a socket was dumped by callback.
Signed-off-by: Andrey Vagin <avagin at openvz.org>
---
include/criu-plugin.h | 5 ++++
include/image.h | 1 +
include/plugin.h | 4 +++
plugin.c | 62 ++++++++++++++++++++++++++++++++++++++++++
sk-unix.c | 75 ++++++++++++++++++++++++++++++++-------------------
5 files changed, 120 insertions(+), 27 deletions(-)
diff --git a/include/criu-plugin.h b/include/criu-plugin.h
index 44abb2a..b022a5f 100644
--- a/include/criu-plugin.h
+++ b/include/criu-plugin.h
@@ -20,7 +20,12 @@
#ifndef __CRIU_PLUGIN_H__
#define __CRIU_PLUGIN_H__
+#include <limits.h>
+
typedef int (cr_plugin_init_t)(void);
typedef void (cr_plugin_fini_t)(void);
+typedef int (cr_plugin_dump_unix_sk_t)(int fd, int id);
+typedef int (cr_plugin_restore_unix_sk_t)(int id);
+
#endif
diff --git a/include/image.h b/include/image.h
index 099f510..32ee0af 100644
--- a/include/image.h
+++ b/include/image.h
@@ -33,6 +33,7 @@
#define USK_EXTERN (1 << 0)
#define USK_SERVICE (1 << 1)
+#define USK_CALLBACK (1 << 2)
#define VMA_AREA_NONE (0 << 0)
#define VMA_AREA_REGULAR (1 << 0) /* Dumpable area */
diff --git a/include/plugin.h b/include/plugin.h
index 87a38ec..a70dde3 100644
--- a/include/plugin.h
+++ b/include/plugin.h
@@ -7,4 +7,8 @@
void cr_plugin_fini(void);
int cr_plugin_init(void);
+
+int cr_plugin_dump_unix_sk(int fd, int id);
+int cr_plugin_restore_unix_sk(int id);
+
#endif
diff --git a/plugin.c b/plugin.c
index 373629a..e65cd3f 100644
--- a/plugin.c
+++ b/plugin.c
@@ -14,6 +14,9 @@
struct cr_plugin_entry {
union {
cr_plugin_fini_t *cr_fini;
+
+ cr_plugin_dump_unix_sk_t *cr_plugin_dump_unix_sk;
+ cr_plugin_restore_unix_sk_t *cr_plugin_restore_unix_sk;
};
struct cr_plugin_entry *next;
@@ -21,10 +24,55 @@ struct cr_plugin_entry {
struct cr_plugins {
struct cr_plugin_entry *cr_fini;
+
+ struct cr_plugin_entry *cr_plugin_dump_unix_sk;
+ struct cr_plugin_entry *cr_plugin_restore_unix_sk;
};
struct cr_plugins cr_plugins;
+#define add_plugin_func(name) \
+ do { \
+ name ## _t *name; \
+ name = dlsym(h, #name); \
+ if (name) { \
+ struct cr_plugin_entry *__ce; \
+ __ce = xmalloc(sizeof(struct cr_plugin_entry)); \
+ if (__ce == NULL) \
+ return -1; \
+ __ce->name = name; \
+ __ce->next = cr_plugins.name; \
+ cr_plugins.name = __ce; \
+ } \
+ } while (0); \
+
+
+#define run_plugin_funcs(name, ...) ({ \
+ struct cr_plugin_entry *__ce = cr_plugins.name; \
+ int __ret = -ENOTSUP; \
+ \
+ while (__ce) { \
+ __ret = __ce->name(__VA_ARGS__); \
+ if (__ret == -ENOTSUP) { \
+ __ce = __ce->next; \
+ continue; \
+ } \
+ break; \
+ } \
+ \
+ __ret; \
+ }) \
+
+int cr_plugin_dump_unix_sk(int fd, int id)
+{
+ return run_plugin_funcs(cr_plugin_dump_unix_sk, fd, id);
+}
+
+int cr_plugin_restore_unix_sk(int id)
+{
+ return run_plugin_funcs(cr_plugin_restore_unix_sk, id);
+}
+
static int cr_lib_load(char *path)
{
struct cr_plugin_entry *ce;
@@ -38,6 +86,9 @@ static int cr_lib_load(char *path)
return -1;
}
+ add_plugin_func(cr_plugin_dump_unix_sk);
+ add_plugin_func(cr_plugin_restore_unix_sk);
+
ce = NULL;
f_fini = dlsym(h, "cr_plugin_fini");
if (f_fini) {
@@ -61,10 +112,21 @@ static int cr_lib_load(char *path)
return 0;
}
+#define cr_plugin_free(name) do { \
+ while (cr_plugins.name) { \
+ ce = cr_plugins.name; \
+ cr_plugins.name = cr_plugins.name->next; \
+ xfree(ce); \
+ } \
+} while (0) \
+
void cr_plugin_fini(void)
{
struct cr_plugin_entry *ce;
+ cr_plugin_free(cr_plugin_dump_unix_sk);
+ cr_plugin_free(cr_plugin_restore_unix_sk);
+
while (cr_plugins.cr_fini) {
ce = cr_plugins.cr_fini;
cr_plugins.cr_fini = cr_plugins.cr_fini->next;
diff --git a/sk-unix.c b/sk-unix.c
index f46546d..58ab805 100644
--- a/sk-unix.c
+++ b/sk-unix.c
@@ -7,6 +7,7 @@
#include <fcntl.h>
#include <sys/un.h>
#include <stdlib.h>
+#include <dlfcn.h>
#include "asm/types.h"
#include "libnetlink.h"
@@ -23,6 +24,7 @@
#include "sk-queue.h"
#include "mount.h"
#include "cr-service.h"
+#include "plugin.h"
#include "protobuf.h"
#include "protobuf/sk-unix.pb-c.h"
@@ -503,9 +505,31 @@ int unix_receive_one(struct nlmsghdr *h, void *arg)
static int dump_external_sockets(struct unix_sk_desc *peer)
{
struct unix_sk_desc *sk;
+ int ret;
list_for_each_entry(sk, &peer->peer_list, peer_node) {
- if (write_unix_entry(sk, peer, 0))
+ unsigned int uflags = 0;
+
+ ret = cr_plugin_dump_unix_sk(sk->fd, sk->sd.ino);
+ if (ret == -ENOTSUP) {
+ if (!opts.ext_unix_sk) {
+ show_one_unix("Runaway socket", peer);
+ pr_err("External socket is used. "
+ "Consider using --" USK_EXT_PARAM " option.\n");
+ return -1;
+ }
+
+ if (peer->type != SOCK_DGRAM) {
+ show_one_unix("Ext stream not supported", peer);
+ pr_err("Can't dump half of stream unix connection.\n");
+ return -1;
+ }
+ } else if (ret < 0)
+ return -1;
+ else
+ uflags |= USK_CALLBACK;
+
+ if (write_unix_entry(sk, peer, uflags))
return -1;
close_safe(&sk->fd);
}
@@ -528,19 +552,6 @@ int fix_external_unix_sockets(void)
BUG_ON(sk->sd.already_dumped);
- if (!opts.ext_unix_sk) {
- show_one_unix("Runaway socket", sk);
- pr_err("External socket is used. "
- "Consider using --" USK_EXT_PARAM " option.\n");
- goto err;
- }
-
- if (sk->type != SOCK_DGRAM) {
- show_one_unix("Ext stream not supported", sk);
- pr_err("Can't dump half of stream unix connection.\n");
- goto err;
- }
-
if (dump_external_sockets(sk))
goto err;
@@ -624,6 +635,9 @@ static int post_open_unix_sk(struct file_desc *d, int fd)
if (peer == NULL)
return 0;
+ if (ui->ue->uflags & USK_CALLBACK)
+ return 0;
+
pr_info("\tConnect %#x to %#x\n", ui->ue->ino, peer->ue->ino);
/* Skip external sockets */
@@ -861,6 +875,25 @@ static int open_unixsk_standalone(struct unix_sk_info *ui)
close(sks[1]);
sk = sks[0];
} else {
+ if (ui->ue->uflags & USK_CALLBACK) {
+ sk = cr_plugin_restore_unix_sk(ui->ue->ino);
+ if (sk >= 0)
+ goto out;
+ }
+
+ /*
+ * Connect to external sockets requires
+ * special option to be passed.
+ */
+ if (ui->peer && (ui->peer->ue->uflags & USK_EXTERN) &&
+ !(opts.ext_unix_sk)) {
+ pr_err("External socket found in image. "
+ "Consider using the --" USK_EXT_PARAM
+ "option to allow restoring it.\n");
+ return -1;
+ }
+
+
sk = socket(PF_UNIX, ui->ue->type, 0);
if (sk < 0) {
pr_perror("Can't make unix socket");
@@ -878,7 +911,7 @@ static int open_unixsk_standalone(struct unix_sk_info *ui)
return -1;
}
}
-
+out:
if (rst_file_params(sk, ui->ue->fown, ui->ue->flags))
return -1;
@@ -974,18 +1007,6 @@ int resolve_unix_peers(void)
return -1;
}
- /*
- * Connect to external sockets requires
- * special option to be passed.
- */
- if ((peer->ue->uflags & USK_EXTERN) &&
- !(opts.ext_unix_sk)) {
- pr_err("External socket found in image. "
- "Consider using the --" USK_EXT_PARAM " option "
- "to allow restoring it.\n");
- return -1;
- }
-
ui->peer = peer;
if (ui == peer)
/* socket connected to self %) */
--
1.8.3.1
More information about the CRIU
mailing list