Change method returned value. Needed with send_request()
[apps/low-level-can-service.git] / src / diagnostic / diagnostic-manager.cpp
index 92ce33b..6867ec3 100644 (file)
 
 diagnostic_manager_t::diagnostic_manager_t()
        : request_list_entries_(MAX_REQUEST_ENTRIES), initialized_{false}
+{}
+
+bool diagnostic_manager_t::initialize(std::shared_ptr<can_bus_dev_t> cbd)
 {
+       // Mandatory to set the bus before intiliaze shims.
+       bus_ = cbd;
+
+       init_diagnostic_shims();
        reset();
+
+       initialized_ = true;
+       DEBUG(binder_interface, "initialize: Diagnostic Manager initialized");
+       return initialized_;
+}
+
+/**
+ * @brief initialize shims used by UDS lib and set initialized_ to true.
+ *  It is needed before used the diagnostic manager fully because shims are
+ *  required by most member functions.
+ */
+void diagnostic_manager_t::init_diagnostic_shims()
+{
+       shims_ = diagnostic_init_shims(shims_logger, shims_send, NULL);
+       DEBUG(binder_interface, "init_diagnostic_shims: Shims initialized");
 }
 
-diagnostic_manager_t::diagnostic_manager_t(can_bus_dev_t& bus)
-       : bus_(&bus), request_list_entries_(MAX_REQUEST_ENTRIES), initialized_{false}
+void diagnostic_manager_t::reset()
 {
-       reset();
+       if(initialized_)
+       {
+               DEBUG(binder_interface, "Clearing existing diagnostic requests");
+               cleanup_active_requests(true);
+       }
+
+       for(int i = 0; i < MAX_SIMULTANEOUS_DIAG_REQUESTS; i++)
+               free_request_entries_.push_back(request_list_entries_[i]);
 }
 
+
 void diagnostic_manager_t::find_and_erase(active_diagnostic_request_t* entry, std::vector<active_diagnostic_request_t*>& requests_list)
 {
        auto i = std::find(requests_list.begin(), requests_list.end(), entry);
@@ -109,36 +138,22 @@ void diagnostic_manager_t::cleanup_active_requests(bool force)
 
 /// @brief Note that this pops it off of whichver list it was on and returns it, so make
 /// sure to add it to some other list or it'll be lost.
-bool diagnostic_manager_t::lookup_recurring_request(const DiagnosticRequest* request)
+active_diagnostic_request_t* diagnostic_manager_t::find_recurring_request(const DiagnosticRequest* request)
 {
-       active_diagnostic_request_t existingEntry;
        for (auto& entry : recurring_requests_)
        {
                active_diagnostic_request_t* candidate = entry;
-               if(candidate->get_can_bus_dev()->get_device_name() == bus_->get_device_name() &&
-                       diagnostic_request_equals(&candidate->get_handle()->request, request))
+               if(diagnostic_request_equals(&candidate->get_handle()->request, request))
                {
                        find_and_erase(entry, recurring_requests_);
-                       return true;
+                       return entry;
                        break;
                }
        }
-       return false;
-}
-
-void diagnostic_manager_t::reset()
-{
-       if(initialized_)
-       {
-               DEBUG(binder_interface, "Clearing existing diagnostic requests");
-               cleanup_active_requests(true);
-       }
-
-       for(int i = 0; i < MAX_SIMULTANEOUS_DIAG_REQUESTS; i++)
-               free_request_entries_.push_back(request_list_entries_[i]);
+       return nullptr;
 }
 
-can_bus_dev_t* diagnostic_manager_t::get_can_bus_dev()
+std::shared_ptr<can_bus_dev_t> diagnostic_manager_t::get_can_bus_dev()
 {
        return bus_;
 }
@@ -208,7 +223,7 @@ bool diagnostic_manager_t::add_recurring_request(DiagnosticRequest* request, con
        cleanup_active_requests(false);
 
        bool added = true;
-       if(lookup_recurring_request(request))
+       if(find_recurring_request(request) == nullptr)
        {
                active_diagnostic_request_t* entry = get_free_entry();
 
@@ -244,9 +259,63 @@ bool diagnostic_manager_t::add_recurring_request(DiagnosticRequest* request, con
        return added;
 }
 
+bool diagnostic_manager_t::conflicting(active_diagnostic_request_t* request, active_diagnostic_request_t* candidate) const
+{
+       return (candidate->get_in_flight() && candidate != request &&
+                       candidate->get_can_bus_dev() == request->get_can_bus_dev() &&
+                       candidate->get_id() == request->get_id());
+}
+
+
+/// @brief Returns true if there are no other active requests to the same arbitration ID.
+bool diagnostic_manager_t::clear_to_send(active_diagnostic_request_t* request) const
+{
+       for ( auto entry : non_recurring_requests_)
+       {
+               if(conflicting(request, entry))
+                       return false;
+       }
+
+       for ( auto entry : recurring_requests_)
+       {
+               if(conflicting(request, entry))
+                       return false;
+       }
+       return true;
+}
+
+int diagnostic_manager_t::send_request(sd_event_source *s, uint64_t usec, void *userdata)
+{
+       diagnostic_manager_t& dm = configuration_t::instance().get_diagnostic_manager();
+       DiagnosticRequest* request = (DiagnosticRequest*)userdata;
+       active_diagnostic_request_t* adr = dm.find_recurring_request(request);
+
+       if(adr != nullptr && adr->get_can_bus_dev() == dm.get_can_bus_dev() && adr->should_send() &&
+               dm.clear_to_send(adr))
+       {
+               DEBUG(binder_interface, "Got active_diagnostic_request from recurring_requests_ queue.");
+               adr->get_frequency_clock().tick();
+               start_diagnostic_request(&dm.get_shims(), adr->get_handle());
+               if(adr->get_handle()->completed && !adr->get_handle()->success)
+               {
+                       DEBUG(binder_interface, "Fatal error sending diagnostic request");
+                       return 0;
+               }
+               adr->get_timeout_clock().tick();
+               adr->set_in_flight(true);
+               return 1;
+       }
+       return -1;
+}
+
+DiagnosticShims& diagnostic_manager_t::get_shims()
+{
+       return shims_;
+}
+
 bool diagnostic_manager_t::shims_send(const uint32_t arbitration_id, const uint8_t* data, const uint8_t size)
 {
-       can_bus_dev_t *can_bus_dev = configuration_t::instance().get_diagnostic_manager().get_can_bus_dev();
+       std::shared_ptr<can_bus_dev_t> can_bus_dev = configuration_t::instance().get_diagnostic_manager().get_can_bus_dev();
        return can_bus_dev->shims_send(arbitration_id, data, size);
 }
 
@@ -258,13 +327,3 @@ void diagnostic_manager_t::shims_logger(const char* m, ...)
 void diagnostic_manager_t::shims_timer()
 {}
 
-/**
- * @brief initialize shims used by UDS lib and set initialized_ to true.
- *  It is needed before used the diagnostic manager fully because shims are
- *  required by most member functions.
- */
-void diagnostic_manager_t::init_diagnostic_shims()
-{
-       shims_ = diagnostic_init_shims(shims_logger, shims_send, NULL);
-       initialized_ = true;
-}