X-Git-Url: http://git.archive.openwrt.org/?p=project%2Fprocd.git;a=blobdiff_plain;f=service.c;h=da7ec9b1d4c1ae06fa6929a2be7a791a0ba5359f;hp=46f846dd40d47b383135b387498cb4d2bf868c61;hb=3688e3a3c27e71e4e20c47dc749a3fcb1fc9be20;hpb=84c2e44d4c1ae46418cc8f7891b31e9df4131d96

diff --git a/service.c b/service.c
index 46f846d..da7ec9b 100644
--- a/service.c
+++ b/service.c
@@ -19,7 +19,7 @@ service_instance_add(struct service *s, struct blob_attr *attr)
 	if (!in)
 		return;
 
-	instance_init(in, attr);
+	instance_init(in, s, attr);
 	vlist_add(&s->instances, &in->node, (void *) name);
 }
 
@@ -36,12 +36,15 @@ service_instance_update(struct vlist_tree *tree, struct vlist_node *node_new,
 		in_n = container_of(node_new, struct service_instance, node);
 
 	if (in_o && in_n) {
+		DPRINTF("Update instance %s::%s\n", in_o->srv->name, in_o->name);
 		instance_update(in_o, in_n);
 		instance_free(in_n);
 	} else if (in_o) {
+		DPRINTF("Free instance %s::%s\n", in_o->srv->name, in_o->name);
 		instance_stop(in_o, false);
 		instance_free(in_o);
 	} else if (in_n) {
+		DPRINTF("Create instance %s::%s\n", in_n->srv->name, in_n->name);
 		instance_start(in_n);
 	}
 }
@@ -50,50 +53,49 @@ static struct service *
 service_alloc(const char *name)
 {
 	struct service *s;
+	char *new_name;
+
+	s = calloc(1, sizeof(*s) + strlen(name) + 1);
+
+	new_name = (char *) s + 1;
+	strcpy(new_name, name);
 
-	s = calloc(1, sizeof(*s));
 	vlist_init(&s->instances, avl_strcmp, service_instance_update);
 	s->instances.keep_old = true;
+	s->name = new_name;
+	s->avl.key = s->name;
 
 	return s;
 }
 
 enum {
-	SERVICE_ATTR_NAME,
-	SERVICE_ATTR_SCRIPT,
-	SERVICE_ATTR_INSTANCES,
-	__SERVICE_ATTR_MAX
+	SERVICE_SET_NAME,
+	SERVICE_SET_SCRIPT,
+	SERVICE_SET_INSTANCES,
+	__SERVICE_SET_MAX
 };
 
-static const struct blobmsg_policy service_attrs[__SERVICE_ATTR_MAX] = {
-	[SERVICE_ATTR_NAME] = { "name", BLOBMSG_TYPE_STRING },
-	[SERVICE_ATTR_SCRIPT] = { "script", BLOBMSG_TYPE_STRING },
-	[SERVICE_ATTR_INSTANCES] = { "instances", BLOBMSG_TYPE_TABLE },
+static const struct blobmsg_policy service_set_attrs[__SERVICE_SET_MAX] = {
+	[SERVICE_SET_NAME] = { "name", BLOBMSG_TYPE_STRING },
+	[SERVICE_SET_SCRIPT] = { "script", BLOBMSG_TYPE_STRING },
+	[SERVICE_SET_INSTANCES] = { "instances", BLOBMSG_TYPE_TABLE },
 };
 
 
 static int
 service_update(struct service *s, struct blob_attr *config, struct blob_attr **tb)
 {
-	struct blob_attr *old_config = s->config;
 	struct blob_attr *cur;
 	int rem;
 
-	/* only the pointer changes, the content stays the same,
-	 * no avl update necessary */
-	s->name = s->avl.key = blobmsg_data(tb[SERVICE_ATTR_NAME]);
-	s->config = config;
-
-	if (tb[SERVICE_ATTR_INSTANCES]) {
+	if (tb[SERVICE_SET_INSTANCES]) {
 		vlist_update(&s->instances);
-		blobmsg_for_each_attr(cur, tb[SERVICE_ATTR_INSTANCES], rem) {
+		blobmsg_for_each_attr(cur, tb[SERVICE_SET_INSTANCES], rem) {
 			service_instance_add(s, cur);
 		}
 		vlist_flush(&s->instances);
 	}
 
-	free(old_config);
-
 	return 0;
 }
 
@@ -106,21 +108,27 @@ service_delete(struct service *s)
 	free(s);
 }
 
+enum {
+	SERVICE_ATTR_NAME,
+	__SERVICE_ATTR_MAX,
+};
+
+static const struct blobmsg_policy service_attrs[__SERVICE_ATTR_MAX] = {
+	[SERVICE_ATTR_NAME] = { "name", BLOBMSG_TYPE_STRING },
+};
+
+
 static int
 service_handle_set(struct ubus_context *ctx, struct ubus_object *obj,
 		   struct ubus_request_data *req, const char *method,
 		   struct blob_attr *msg)
 {
-	struct blob_attr *tb[__SERVICE_ATTR_MAX], *cur;
+	struct blob_attr *tb[__SERVICE_SET_MAX], *cur;
 	struct service *s = NULL;
 	const char *name;
 	int ret = UBUS_STATUS_INVALID_ARGUMENT;
 
-	msg = blob_memdup(msg);
-	if (!msg)
-		return UBUS_STATUS_UNKNOWN_ERROR;
-
-	blobmsg_parse(service_attrs, __SERVICE_ATTR_MAX, tb, blob_data(msg), blob_len(msg));
+	blobmsg_parse(service_set_attrs, __SERVICE_SET_MAX, tb, blob_data(msg), blob_len(msg));
 	cur = tb[SERVICE_ATTR_NAME];
 	if (!cur)
 		goto free;
@@ -128,9 +136,12 @@ service_handle_set(struct ubus_context *ctx, struct ubus_object *obj,
 	name = blobmsg_data(cur);
 
 	s = avl_find_element(&services, name, s, avl);
-	if (s)
+	if (s) {
+		DPRINTF("Update service %s\n", name);
 		return service_update(s, msg, tb);
+	}
 
+	DPRINTF("Create service %s\n", name);
 	s = service_alloc(name);
 	if (!s)
 		return UBUS_STATUS_UNKNOWN_ERROR;
@@ -148,6 +159,20 @@ free:
 	return ret;
 }
 
+static void
+service_dump(struct service *s)
+{
+	struct service_instance *in;
+	void *c, *i;
+
+	c = blobmsg_open_table(&b, s->name);
+	i = blobmsg_open_table(&b, "instances");
+	vlist_for_each_element(&s->instances, in, node)
+		instance_dump(&b, in);
+	blobmsg_close_table(&b, i);
+	blobmsg_close_table(&b, c);
+}
+
 static int
 service_handle_list(struct ubus_context *ctx, struct ubus_object *obj,
 		    struct ubus_request_data *req, const char *method,
@@ -156,37 +181,23 @@ service_handle_list(struct ubus_context *ctx, struct ubus_object *obj,
 	struct service *s;
 
 	blob_buf_init(&b, 0);
-	avl_for_each_element(&services, s, avl) {
-		void *c;
-
-		c = blobmsg_open_table(&b, s->name);
-		blobmsg_close_table(&b, c);
-	}
+	avl_for_each_element(&services, s, avl)
+		service_dump(s);
 
 	ubus_send_reply(ctx, req, b.head);
 
 	return 0;
 }
 
-enum {
-	SERVICE_DEL_NAME,
-	__SERVICE_DEL_MAX,
-};
-
-static const struct blobmsg_policy service_del_attrs[__SERVICE_DEL_MAX] = {
-	[SERVICE_DEL_NAME] = { "name", BLOBMSG_TYPE_STRING },
-};
-
-
 static int
 service_handle_delete(struct ubus_context *ctx, struct ubus_object *obj,
 		    struct ubus_request_data *req, const char *method,
 		    struct blob_attr *msg)
 {
-	struct blob_attr *tb[__SERVICE_DEL_MAX], *cur;
+	struct blob_attr *tb[__SERVICE_ATTR_MAX], *cur;
 	struct service *s, *tmp;
 
-	blobmsg_parse(service_del_attrs, __SERVICE_DEL_MAX, tb, blob_data(msg), blob_len(msg));
+	blobmsg_parse(service_attrs, __SERVICE_ATTR_MAX, tb, blob_data(msg), blob_len(msg));
 
 	cur = tb[SERVICE_ATTR_NAME];
 	if (!cur) {
@@ -203,10 +214,38 @@ service_handle_delete(struct ubus_context *ctx, struct ubus_object *obj,
 	return 0;
 }
 
+static int
+service_handle_update(struct ubus_context *ctx, struct ubus_object *obj,
+		      struct ubus_request_data *req, const char *method,
+		      struct blob_attr *msg)
+{
+	struct blob_attr *tb[__SERVICE_ATTR_MAX], *cur;
+	struct service *s;
+
+	blobmsg_parse(service_attrs, __SERVICE_ATTR_MAX, tb, blob_data(msg), blob_len(msg));
+
+	cur = tb[SERVICE_ATTR_NAME];
+	if (!cur)
+		return UBUS_STATUS_INVALID_ARGUMENT;
+
+	s = avl_find_element(&services, blobmsg_data(cur), s, avl);
+	if (!s)
+		return UBUS_STATUS_NOT_FOUND;
+
+	if (!strcmp(method, "update_start"))
+		vlist_update(&s->instances);
+	else
+		vlist_flush(&s->instances);
+
+	return 0;
+}
+
 static struct ubus_method main_object_methods[] = {
-	{ .name = "list", .handler = service_handle_list },
-	{ .name = "set", .handler = service_handle_set },
-	{ .name = "delete", .handler = service_handle_delete },
+	UBUS_METHOD("set", service_handle_set, service_set_attrs),
+	UBUS_METHOD("list", service_handle_list, service_attrs),
+	UBUS_METHOD("delete", service_handle_delete, service_attrs),
+	UBUS_METHOD("update_start", service_handle_update, service_attrs),
+	UBUS_METHOD("update_complete", service_handle_update, service_attrs),
 };
 
 static struct ubus_object_type main_object_type =