From e62227977bbc161d2d0ae49951f9a4fbf02a354e Mon Sep 17 00:00:00 2001 From: José Bollo Date: Wed, 5 Apr 2017 18:08:27 +0200 Subject: Make daemon interface common MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit For the future, having only one piece of code to handle daemon interface to bindings is an improvement. Change-Id: I806c424eb1a82637d27bc84b921612789907d971 Signed-off-by: José Bollo --- bindings/samples/HelloWorld.c | 1 + src/CMakeLists.txt | 1 + src/afb-api-so-v1.c | 98 ++--------------------------------- src/afb-api-so-v2.c | 89 ++------------------------------ src/afb-ditf.c | 115 ++++++++++++++++++++++++++++++++++++++++++ src/afb-ditf.h | 47 +++++++++++++++++ 6 files changed, 174 insertions(+), 177 deletions(-) create mode 100644 src/afb-ditf.c create mode 100644 src/afb-ditf.h diff --git a/bindings/samples/HelloWorld.c b/bindings/samples/HelloWorld.c index 585154e5..3432d9f4 100644 --- a/bindings/samples/HelloWorld.c +++ b/bindings/samples/HelloWorld.c @@ -306,5 +306,6 @@ static const struct afb_binding plugin_desc = { const struct afb_binding *afbBindingV1Register (const struct afb_binding_interface *itf) { interface = itf; + NOTICE(interface, "hello plugin comes to live"); return &plugin_desc; } diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 83977e23..0566206e 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -65,6 +65,7 @@ ADD_LIBRARY(afb-lib STATIC afb-common.c afb-config.c afb-context.c + afb-ditf.c afb-evt.c afb-hook.c afb-hreq.c diff --git a/src/afb-api-so-v1.c b/src/afb-api-so-v1.c index 61b646a3..f9c1ef38 100644 --- a/src/afb-api-so-v1.c +++ b/src/afb-api-so-v1.c @@ -31,6 +31,7 @@ #include "afb-context.h" #include "afb-api-so.h" #include "afb-xreq.h" +#include "afb-ditf.h" #include "verbose.h" /* @@ -45,87 +46,11 @@ static const char afb_api_so_v1_service_event[] = "afbBindingV1ServiceEvent"; */ struct api_so_v1 { struct afb_binding *binding; /* descriptor */ - size_t apilength; /* length of the API name */ void *handle; /* context of dlopen */ struct afb_svc *service; /* handler for service started */ - struct afb_binding_interface interface; /* interface for the binding */ + struct afb_ditf ditf; /* daemon interface */ }; -static struct afb_event afb_api_so_event_make_cb(void *closure, const char *name); -static int afb_api_so_event_broadcast_cb(void *closure, const char *name, struct json_object *object); -static void afb_api_so_vverbose_cb(void *closure, int level, const char *file, int line, const char *fmt, va_list args); -static int afb_api_so_rootdir_get_fd(void *closure); -static int afb_api_so_rootdir_open_locale(void *closure, const char *filename, int flags, const char *locale); - -static const struct afb_daemon_itf daemon_itf = { - .event_broadcast = afb_api_so_event_broadcast_cb, - .get_event_loop = afb_common_get_event_loop, - .get_user_bus = afb_common_get_user_bus, - .get_system_bus = afb_common_get_system_bus, - .vverbose = afb_api_so_vverbose_cb, - .event_make = afb_api_so_event_make_cb, - .rootdir_get_fd = afb_api_so_rootdir_get_fd, - .rootdir_open_locale = afb_api_so_rootdir_open_locale -}; - -static struct afb_event afb_api_so_event_make_cb(void *closure, const char *name) -{ - size_t length; - char *event; - struct api_so_v1 *desc = closure; - - /* makes the event name */ - assert(desc->binding != NULL); - length = strlen(name); - event = alloca(length + 2 + desc->apilength); - memcpy(event, desc->binding->v1.prefix, desc->apilength); - event[desc->apilength] = '/'; - memcpy(event + desc->apilength + 1, name, length + 1); - - /* crate the event */ - return afb_evt_create_event(event); -} - -static int afb_api_so_event_broadcast_cb(void *closure, const char *name, struct json_object *object) -{ - size_t length; - char *event; - struct api_so_v1 *desc = closure; - - /* makes the event name */ - assert(desc->binding != NULL); - length = strlen(name); - event = alloca(length + 2 + desc->apilength); - memcpy(event, desc->binding->v1.prefix, desc->apilength); - event[desc->apilength] = '/'; - memcpy(event + desc->apilength + 1, name, length + 1); - - return afb_evt_broadcast(event, object); -} - -static void afb_api_so_vverbose_cb(void *closure, int level, const char *file, int line, const char *fmt, va_list args) -{ - char *p; - struct api_so_v1 *desc = closure; - - if (vasprintf(&p, fmt, args) < 0) - vverbose(level, file, line, fmt, args); - else { - verbose(level, file, line, "%s {binding %s}", p, desc->binding->v1.prefix); - free(p); - } -} - -static int afb_api_so_rootdir_get_fd(void *closure) -{ - return afb_common_rootdir_get_fd(); -} - -static int afb_api_so_rootdir_open_locale(void *closure, const char *filename, int flags, const char *locale) -{ - return afb_common_rootdir_open_locale(filename, flags, locale); -} - static const struct afb_verb_desc_v1 *search(struct api_so_v1 *desc, const char *name) { const struct afb_verb_desc_v1 *verb; @@ -194,8 +119,6 @@ int afb_api_so_v1_add(const char *path, void *handle) { struct api_so_v1 *desc; struct afb_binding *(*register_function) (const struct afb_binding_interface *interface); - struct afb_verb_desc_v1 fake_verb; - struct afb_binding fake_binding; /* retrieves the register function */ register_function = dlsym(handle, afb_api_so_v1_register); @@ -212,22 +135,11 @@ int afb_api_so_v1_add(const char *path, void *handle) desc->handle = handle; /* init the interface */ - desc->interface.verbosity = verbosity; - desc->interface.mode = AFB_MODE_LOCAL; - desc->interface.daemon.itf = &daemon_itf; - desc->interface.daemon.closure = desc; - - /* for log purpose, a fake binding is needed here */ - desc->binding = &fake_binding; - fake_binding.type = AFB_BINDING_VERSION_1; - fake_binding.v1.info = path; - fake_binding.v1.prefix = path; - fake_binding.v1.verbs = &fake_verb; - fake_verb.name = NULL; + afb_ditf_init(&desc->ditf, path); /* init the binding */ NOTICE("binding [%s] calling registering function %s", path, afb_api_so_v1_register); - desc->binding = register_function(&desc->interface); + desc->binding = register_function(&desc->ditf.interface); if (desc->binding == NULL) { ERROR("binding [%s] register function failed. continuing...", path); goto error2; @@ -256,7 +168,7 @@ int afb_api_so_v1_add(const char *path, void *handle) } /* records the binding */ - desc->apilength = strlen(desc->binding->v1.prefix); + afb_ditf_rename(&desc->ditf, desc->binding->v1.prefix); if (afb_apis_add(desc->binding->v1.prefix, (struct afb_api){ .closure = desc, .call = call_cb, diff --git a/src/afb-api-so-v2.c b/src/afb-api-so-v2.c index 4e34e03c..7df83dd8 100644 --- a/src/afb-api-so-v2.c +++ b/src/afb-api-so-v2.c @@ -26,6 +26,7 @@ #include "afb-apis.h" #include "afb-svc.h" +#include "afb-ditf.h" #include "afb-evt.h" #include "afb-common.h" #include "afb-context.h" @@ -43,87 +44,11 @@ static const char afb_api_so_v2_descriptor[] = "afbBindingV2"; */ struct api_so_v2 { struct afb_binding_v2 *binding; /* descriptor */ - size_t apilength; /* length of the API name */ void *handle; /* context of dlopen */ struct afb_svc *service; /* handler for service started */ - struct afb_binding_interface interface; /* interface for the binding */ + struct afb_ditf ditf; /* daemon interface */ }; -static struct afb_event afb_api_so_event_make_cb(void *closure, const char *name); -static int afb_api_so_event_broadcast_cb(void *closure, const char *name, struct json_object *object); -static void afb_api_so_vverbose_cb(void *closure, int level, const char *file, int line, const char *fmt, va_list args); -static int afb_api_so_rootdir_get_fd(void *closure); -static int afb_api_so_rootdir_open_locale(void *closure, const char *filename, int flags, const char *locale); - -static const struct afb_daemon_itf daemon_itf = { - .event_broadcast = afb_api_so_event_broadcast_cb, - .get_event_loop = afb_common_get_event_loop, - .get_user_bus = afb_common_get_user_bus, - .get_system_bus = afb_common_get_system_bus, - .vverbose = afb_api_so_vverbose_cb, - .event_make = afb_api_so_event_make_cb, - .rootdir_get_fd = afb_api_so_rootdir_get_fd, - .rootdir_open_locale = afb_api_so_rootdir_open_locale -}; - -static struct afb_event afb_api_so_event_make_cb(void *closure, const char *name) -{ - size_t length; - char *event; - struct api_so_v2 *desc = closure; - - /* makes the event name */ - assert(desc->binding != NULL); - length = strlen(name); - event = alloca(length + 2 + desc->apilength); - memcpy(event, desc->binding->api, desc->apilength); - event[desc->apilength] = '/'; - memcpy(event + desc->apilength + 1, name, length + 1); - - /* crate the event */ - return afb_evt_create_event(event); -} - -static int afb_api_so_event_broadcast_cb(void *closure, const char *name, struct json_object *object) -{ - size_t length; - char *event; - struct api_so_v2 *desc = closure; - - /* makes the event name */ - assert(desc->binding != NULL); - length = strlen(name); - event = alloca(length + 2 + desc->apilength); - memcpy(event, desc->binding->api, desc->apilength); - event[desc->apilength] = '/'; - memcpy(event + desc->apilength + 1, name, length + 1); - - return afb_evt_broadcast(event, object); -} - -static void afb_api_so_vverbose_cb(void *closure, int level, const char *file, int line, const char *fmt, va_list args) -{ - char *p; - struct api_so_v2 *desc = closure; - - if (vasprintf(&p, fmt, args) < 0) - vverbose(level, file, line, fmt, args); - else { - verbose(level, file, line, "%s {binding %s}", p, desc->binding->api); - free(p); - } -} - -static int afb_api_so_rootdir_get_fd(void *closure) -{ - return afb_common_rootdir_get_fd(); -} - -static int afb_api_so_rootdir_open_locale(void *closure, const char *filename, int flags, const char *locale) -{ - return afb_common_rootdir_open_locale(filename, flags, locale); -} - static const struct afb_verb_v2 *search(struct api_so_v2 *desc, const char *verb) { const struct afb_verb_v2 *result; @@ -178,7 +103,7 @@ static int service_start_cb(void *closure, int share_session, int onneed) /* get the event handler if any */ onevent = desc->binding->onevent; - desc->service = afb_svc_create_v2(share_session, onevent, start, &desc->interface); + desc->service = afb_svc_create_v2(share_session, onevent, start, &desc->ditf.interface); if (desc->service == NULL) { /* starting error */ ERROR("Starting service %s failed", desc->binding->api); @@ -229,17 +154,14 @@ int afb_api_so_v2_add(const char *path, void *handle) desc->handle = handle; /* init the interface */ - desc->interface.verbosity = verbosity; - desc->interface.mode = AFB_MODE_LOCAL; - desc->interface.daemon.itf = &daemon_itf; - desc->interface.daemon.closure = desc; + afb_ditf_init(&desc->ditf, binding->api); /* for log purpose, a fake binding is needed here */ /* init the binding */ if (binding->init) { NOTICE("binding %s [%s] calling init function", binding->api, path); - rc = binding->init(&desc->interface); + rc = binding->init(&desc->ditf.interface); if (rc < 0) { ERROR("binding %s [%s] initialisation function failed...", binding->api, path); goto error2; @@ -247,7 +169,6 @@ int afb_api_so_v2_add(const char *path, void *handle) } /* records the binding */ - desc->apilength = strlen(binding->api); if (afb_apis_add(binding->api, (struct afb_api){ .closure = desc, .call = call_cb, diff --git a/src/afb-ditf.c b/src/afb-ditf.c new file mode 100644 index 00000000..414824f5 --- /dev/null +++ b/src/afb-ditf.c @@ -0,0 +1,115 @@ +/* + * Copyright (C) 2016, 2017 "IoT.bzh" + * Author José Bollo + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#define _GNU_SOURCE +#define NO_BINDING_VERBOSE_MACRO + +#include +#include + +#include + +#include "afb-ditf.h" +#include "afb-evt.h" +#include "afb-common.h" +#include "verbose.h" + + +static struct afb_event event_make_cb(void *closure, const char *name); +static int event_broadcast_cb(void *closure, const char *name, struct json_object *object); +static void vverbose_cb(void *closure, int level, const char *file, int line, const char *fmt, va_list args); +static int rootdir_open_locale_cb(void *closure, const char *filename, int flags, const char *locale); + +static const struct afb_daemon_itf daemon_itf = { + .vverbose = vverbose_cb, + .event_make = event_make_cb, + .event_broadcast = event_broadcast_cb, + .get_event_loop = afb_common_get_event_loop, + .get_user_bus = afb_common_get_user_bus, + .get_system_bus = afb_common_get_system_bus, + .rootdir_get_fd = afb_common_rootdir_get_fd, + .rootdir_open_locale = rootdir_open_locale_cb +}; + +static void vverbose_cb(void *closure, int level, const char *file, int line, const char *fmt, va_list args) +{ + char *p; + struct afb_ditf *ditf = closure; + + if (vasprintf(&p, fmt, args) < 0) + vverbose(level, file, line, fmt, args); + else { + verbose(level, file, line, "%s {binding %s}", p, ditf->prefix); + free(p); + } +} + +static struct afb_event event_make_cb(void *closure, const char *name) +{ + size_t plen, nlen; + char *event; + struct afb_ditf *ditf = closure; + + /* makes the event name */ + plen = strlen(ditf->prefix); + nlen = strlen(name); + event = alloca(nlen + plen + 2); + memcpy(event, ditf->prefix, plen); + event[plen] = '/'; + memcpy(event + plen + 1, name, nlen + 1); + + /* create the event */ + return afb_evt_create_event(event); +} + +static int event_broadcast_cb(void *closure, const char *name, struct json_object *object) +{ + size_t plen, nlen; + char *event; + struct afb_ditf *ditf = closure; + + /* makes the event name */ + plen = strlen(ditf->prefix); + nlen = strlen(name); + event = alloca(nlen + plen + 2); + memcpy(event, ditf->prefix, plen); + event[plen] = '/'; + memcpy(event + plen + 1, name, nlen + 1); + + /* broadcast the event */ + return afb_evt_broadcast(event, object); +} + +static int rootdir_open_locale_cb(void *closure, const char *filename, int flags, const char *locale) +{ + return afb_common_rootdir_open_locale(filename, flags, locale); +} + +void afb_ditf_init(struct afb_ditf *ditf, const char *prefix) +{ + ditf->interface.verbosity = verbosity; + ditf->interface.mode = AFB_MODE_LOCAL; + ditf->interface.daemon.itf = &daemon_itf; + ditf->interface.daemon.closure = ditf; + afb_ditf_rename(ditf, prefix); +} + +void afb_ditf_rename(struct afb_ditf *ditf, const char *prefix) +{ + ditf->prefix = prefix; +} + diff --git a/src/afb-ditf.h b/src/afb-ditf.h new file mode 100644 index 00000000..7d56c604 --- /dev/null +++ b/src/afb-ditf.h @@ -0,0 +1,47 @@ +/* + * Copyright (C) 2016, 2017 "IoT.bzh" + * Author José Bollo + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#define _GNU_SOURCE +#define NO_BINDING_VERBOSE_MACRO + +#include +#include +#include + +#include + +#include "afb-apis.h" +#include "afb-svc.h" +#include "afb-evt.h" +#include "afb-common.h" +#include "afb-context.h" +#include "afb-api-so.h" +#include "afb-xreq.h" +#include "verbose.h" + + +struct afb_binding_interface; + +struct afb_ditf +{ + struct afb_binding_interface interface; + const char *prefix; +}; + +extern void afb_ditf_init(struct afb_ditf *ditf, const char *prefix); +extern void afb_ditf_rename(struct afb_ditf *ditf, const char *prefix); + -- cgit 1.2.3-korg