Rename some of the classes removing can- prefix
[apps/agl-service-can-low-level.git] / low-can-binding / binding / low-can-cb.cpp
index 098f071..8edd1da 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2015, 2016 "IoT.bzh"
+ * Copyright (C) 2015, 2018 "IoT.bzh"
  * Author "Romain Forlot" <romain.forlot@iot.bzh>
  * Author "Loic Collignon" <loic.collignon@iot.bzh>
  *
@@ -31,8 +31,8 @@
 #include "application.hpp"
 #include "../can/can-encoder.hpp"
 #include "../can/can-bus.hpp"
-#include "../can/can-signals.hpp"
-#include "../can/can-message.hpp"
+#include "../can/signals.hpp"
+#include "../can/message/message.hpp"
 #include "../utils/signals.hpp"
 #include "../diagnostic/diagnostic-message.hpp"
 #include "../utils/openxc-utils.hpp"
@@ -70,12 +70,12 @@ void on_no_clients(std::shared_ptr<low_can_subscription_t> can_subscription, std
        s.erase(it);
 }
 
-static void push_n_notify(const can_message_t& cm)
+static void push_n_notify(std::shared_ptr<message_t> m)
 {
        can_bus_t& cbm = application_t::instance().get_can_bus_manager();
        {
                std::lock_guard<std::mutex> can_message_lock(cbm.get_can_message_mutex());
-               cbm.push_new_can_message(cm);
+               cbm.push_new_can_message(m);
        }
        cbm.get_new_can_message_cv().notify_one();
 }
@@ -83,23 +83,27 @@ static void push_n_notify(const can_message_t& cm)
 int read_message(sd_event_source *event_source, int fd, uint32_t revents, void *userdata)
 {
        low_can_subscription_t* can_subscription = (low_can_subscription_t*)userdata;
+
+
        if ((revents & EPOLLIN) != 0)
        {
-               can_message_t cm;
-               utils::socketcan_bcm_t& s = can_subscription->get_socket();
-               s >> cm;
+               std::shared_ptr<utils::socketcan_t> s = can_subscription->get_socket();
+               std::shared_ptr<message_t> message = s->read_message();
 
                // Sure we got a valid CAN message ?
-               if(! cm.get_id() == 0 && ! cm.get_length() == 0)
-                       {push_n_notify(cm);}
+               if (! message->get_id() == 0 && ! message->get_length() == 0)
+               {
+                       push_n_notify(message);
+               }
        }
 
        // check if error or hangup
        if ((revents & (EPOLLERR|EPOLLRDHUP|EPOLLHUP)) != 0)
        {
                sd_event_source_unref(event_source);
-               can_subscription->get_socket().close();
+               can_subscription->get_socket()->close();
        }
+
        return 0;
 }
 
@@ -109,35 +113,6 @@ int read_message(sd_event_source *event_source, int fd, uint32_t revents, void *
 ///
 ///*******************************************************************************/
 
-static int make_subscription_unsubscription(afb_req_t request,
-                                           std::shared_ptr<low_can_subscription_t>& can_subscription,
-                                           std::map<int, std::shared_ptr<low_can_subscription_t> >& s,
-                                           bool subscribe)
-{
-       /* Make the subscription or unsubscription to the event (if request is not null) */
-       if(request &&
-          ((subscribe ? afb_req_subscribe : afb_req_unsubscribe)(request, s[can_subscription->get_index()]->get_event())) < 0)
-       {
-               AFB_ERROR("Operation goes wrong for signal: %s", can_subscription->get_name().c_str());
-               return -1;
-       }
-       return 0;
-}
-
-static int create_event_handle(std::shared_ptr<low_can_subscription_t>& can_subscription,
-                              std::map<int, std::shared_ptr<low_can_subscription_t> >& s)
-{
-       int sub_index = can_subscription->get_index();
-       can_subscription->set_event(afb_daemon_make_event(can_subscription->get_name().c_str()));
-       s[sub_index] = can_subscription;
-       if (!afb_event_is_valid(s[sub_index]->get_event()))
-       {
-               AFB_ERROR("Can't create an event for %s, something goes wrong.", can_subscription->get_name().c_str());
-               return -1;
-       }
-       return 0;
-}
-
 /// @brief This will determine if an event handle needs to be created and checks if
 /// we got a valid afb_event to get subscribe or unsubscribe. After that launch the subscription or unsubscription
 /// against the application framework using that event handle.
@@ -146,30 +121,52 @@ static int subscribe_unsubscribe_signal(afb_req_t request,
                                        std::shared_ptr<low_can_subscription_t>& can_subscription,
                                        std::map<int, std::shared_ptr<low_can_subscription_t> >& s)
 {
-       int ret = -1;
+       int ret = 0;
        int sub_index = can_subscription->get_index();
+       bool subscription_exists = s.count(sub_index);
 
-       if (can_subscription && s.find(sub_index) != s.end())
+       // Susbcription part
+       if(subscribe)
        {
-               if (!afb_event_is_valid(s[sub_index]->get_event()) && !subscribe)
+               /* There is no valid request to subscribe so this must be an
+                * internal permanent diagnostic request. Skip the subscription
+                * part and don't register it into the current "low-can"
+                * subsciptions.
+                */
+               if(! request)
                {
-                       AFB_NOTICE("Event isn't valid, no need to unsubscribed.");
-                       ret = -1;
+                       return 0;
                }
-               ret = 0;
+
+               // Event doesn't exist , so let's create it
+               if (! subscription_exists &&
+                   (ret = can_subscription->subscribe(request)) < 0)
+                       return ret;
+
+               if(! subscription_exists)
+                               s[sub_index] = can_subscription;
+
+               return ret;
        }
-       else
+
+       // Unsubscrition part
+       if(! subscription_exists)
        {
-               /* Event doesn't exist , so let's create it */
-               s[sub_index] = can_subscription;
-               ret = create_event_handle(can_subscription, s);
+               AFB_NOTICE("There isn't any valid subscriptions for that request.");
+               return ret;
+       }
+       else if (subscription_exists &&
+                ! afb_event_is_valid(s[sub_index]->get_event()) )
+       {
+               AFB_NOTICE("Event isn't valid, no need to unsubscribed.");
+               return ret;
        }
 
-       // Checks if the event handler is correctly created, if it is, it
-       // performs the subscription or unsubscription operations.
-       if (ret < 0)
+       if( (ret = s[sub_index]->unsubscribe(request)) < 0)
                return ret;
-       return make_subscription_unsubscription(request, can_subscription, s, subscribe);
+       s.erase(sub_index);
+
+       return ret;
 }
 
 static int add_to_event_loop(std::shared_ptr<low_can_subscription_t>& can_subscription)
@@ -177,7 +174,7 @@ static int add_to_event_loop(std::shared_ptr<low_can_subscription_t>& can_subscr
                struct sd_event_source* event_source = nullptr;
                return ( sd_event_add_io(afb_daemon_get_event_loop(),
                        &event_source,
-                       can_subscription->get_socket().socket(),
+                       can_subscription->get_socket()->socket(),
                        EPOLLIN,
                        read_message,
                        can_subscription.get()));
@@ -244,21 +241,19 @@ static int subscribe_unsubscribe_diagnostic_messages(afb_req_t request,
        return rets;
 }
 
-static int subscribe_unsubscribe_can_signals(afb_req_t request,
+static int subscribe_unsubscribe_signals(afb_req_t request,
                                             bool subscribe,
-                                            std::vector<std::shared_ptr<can_signal_t> > can_signals,
+                                            std::vector<std::shared_ptr<signal_t> > signals,
                                             struct event_filter_t& event_filter,
                                             std::map<int, std::shared_ptr<low_can_subscription_t> >& s)
 {
        int rets = 0;
-       for(const auto& sig: can_signals)
+       for(const auto& sig: signals)
        {
                auto it =  std::find_if(s.begin(), s.end(), [&sig, &event_filter](std::pair<int, std::shared_ptr<low_can_subscription_t> > sub){ return sub.second->is_signal_subscription_corresponding(sig, event_filter) ; });
                std::shared_ptr<low_can_subscription_t> can_subscription;
                if(it != s.end())
-               {
-                        can_subscription = it->second;
-               }
+                       {can_subscription = it->second;}
                else
                {
                         can_subscription = std::make_shared<low_can_subscription_t>(low_can_subscription_t(event_filter));
@@ -272,7 +267,7 @@ static int subscribe_unsubscribe_can_signals(afb_req_t request,
                        {return -1;}
 
                rets++;
-               AFB_DEBUG("signal: %s subscribed", sig->get_name().c_str());
+               AFB_DEBUG("%s Signal: %s %ssubscribed", sig->get_message()->is_fd() ? "FD": "", sig->get_name().c_str(), subscribe ? "":"un");
        }
        return rets;
 }
@@ -282,7 +277,7 @@ static int subscribe_unsubscribe_can_signals(afb_req_t request,
 ///
 /// @param[in] afb_req request : contains original request use to subscribe or unsubscribe
 /// @param[in] subscribe boolean value, which chooses between a subscription operation or an unsubscription
-/// @param[in] signals -  struct containing vectors with can_signal_t and diagnostic_messages to subscribe
+/// @param[in] signals -  struct containing vectors with signal_t and diagnostic_messages to subscribe
 ///
 /// @return Number of correctly subscribed signal
 ///
@@ -298,7 +293,7 @@ static int subscribe_unsubscribe_signals(afb_req_t request,
        std::map<int, std::shared_ptr<low_can_subscription_t> >& s = sm.get_subscribed_signals();
 
        rets += subscribe_unsubscribe_diagnostic_messages(request, subscribe, signals.diagnostic_messages, event_filter, s, false);
-       rets += subscribe_unsubscribe_can_signals(request, subscribe, signals.can_signals, event_filter, s);
+       rets += subscribe_unsubscribe_signals(request, subscribe, signals.signals, event_filter, s);
 
        return rets;
 }
@@ -330,7 +325,7 @@ static int one_subscribe_unsubscribe(afb_req_t request,
        // subscribe or unsubscribe
        openxc_DynamicField search_key = build_DynamicField(tag);
        sf = utils::signals_manager_t::instance().find_signals(search_key);
-       if (sf.can_signals.empty() && sf.diagnostic_messages.empty())
+       if (sf.signals.empty() && sf.diagnostic_messages.empty())
        {
                AFB_NOTICE("No signal(s) found for %s.", tag.c_str());
                ret = -1;
@@ -413,12 +408,12 @@ static int send_frame(struct canfd_frame& cfd, const std::string& bus_name)
                return -1;
        }
 
-       std::map<std::string, std::shared_ptr<low_can_socket_t> >& cd = application_t::instance().get_can_devices();
+       std::map<std::string, std::shared_ptr<low_can_subscription_t> >& cd = application_t::instance().get_can_devices();
 
        if( cd.count(bus_name) == 0)
-               {cd[bus_name] = std::make_shared<low_can_socket_t>(low_can_socket_t());}
+               {cd[bus_name] = std::make_shared<low_can_subscription_t>(low_can_subscription_t());}
 
-       return cd[bus_name]->tx_send(cfd, bus_name);
+       return cd[bus_name]->tx_send(*cd[bus_name], cfd, bus_name);
 }
 
 static void write_raw_frame(afb_req_t request, const std::string& bus_name, json_object *json_value)
@@ -471,13 +466,13 @@ static void write_signal(afb_req_t request, const std::string& name, json_object
        sf = utils::signals_manager_t::instance().find_signals(search_key);
        openxc_DynamicField dynafield_value = build_DynamicField(json_value);
 
-       if (sf.can_signals.empty())
+       if (sf.signals.empty())
        {
                afb_req_fail_f(request, "No signal(s) found for %s. Message not sent.", name.c_str());
                return;
        }
 
-       std::shared_ptr<can_signal_t>& sig = sf.can_signals[0];
+       std::shared_ptr<signal_t>& sig = sf.signals[0];
        if(! sig->get_writable())
        {
                afb_req_fail_f(request, "%s isn't writable. Message not sent.", sig->get_name().c_str());
@@ -526,13 +521,13 @@ static struct json_object *get_signals_value(const std::string& name)
        openxc_DynamicField search_key = build_DynamicField(name);
        sf = utils::signals_manager_t::instance().find_signals(search_key);
 
-       if (sf.can_signals.empty())
+       if (sf.signals.empty())
        {
                AFB_WARNING("No signal(s) found for %s.", name.c_str());
                return NULL;
        }
        ans = json_object_new_array();
-       for(const auto& sig: sf.can_signals)
+       for(const auto& sig: sf.signals)
        {
                struct json_object *jobj = json_object_new_object();
                json_object_object_add(jobj, "event", json_object_new_string(sig->get_name().c_str()));
@@ -580,13 +575,13 @@ static struct json_object *list_can_message(const std::string& name)
        openxc_DynamicField search_key = build_DynamicField(name);
        sf = utils::signals_manager_t::instance().find_signals(search_key);
 
-       if (sf.can_signals.empty() && sf.diagnostic_messages.empty())
+       if (sf.signals.empty() && sf.diagnostic_messages.empty())
        {
                AFB_WARNING("No signal(s) found for %s.", name.c_str());
                return NULL;
        }
        ans = json_object_new_array();
-       for(const auto& sig: sf.can_signals)
+       for(const auto& sig: sf.signals)
        {
                json_object_array_add(ans,
                        json_object_new_string(sig->get_name().c_str()));
@@ -651,7 +646,7 @@ int init_binding(afb_api_t api)
        openxc_DynamicField search_key = build_DynamicField("diagnostic_messages.engine.speed");
        struct utils::signals_found sf = utils::signals_manager_t::instance().find_signals(search_key);
 
-       if(sf.can_signals.empty() && sf.diagnostic_messages.size() == 1)
+       if(sf.signals.empty() && sf.diagnostic_messages.size() == 1)
        {
                afb_req_t request = nullptr;