X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?a=blobdiff_plain;f=src%2Fcan%2Fcan-bus.cpp;h=dfecea96d81a18f7721a311394c81c3e2dbd2193;hb=f0d7a6523955ee94a32ec4b62e2a207b23f62316;hp=a8a474cf7fa94abaeb8496cd98e3817fc5f05e42;hpb=7b131462d3b84f9fd5e87938c5f26d951d39a6cb;p=apps%2Flow-level-can-service.git diff --git a/src/can/can-bus.cpp b/src/can/can-bus.cpp index a8a474c..dfecea9 100644 --- a/src/can/can-bus.cpp +++ b/src/can/can-bus.cpp @@ -39,11 +39,6 @@ extern "C" #include } -/******************************************************************************** -* -* can_bus_t method implementation -* -*********************************************************************************/ /** * @brief Class constructor * @@ -56,6 +51,87 @@ can_bus_t::can_bus_t(int conf_file) } +int can_bus_t::process_can_signals(can_message_t& can_message) +{ + int processed_signals = 0; + std::vector signals; + openxc_DynamicField search_key, decoded_message; + openxc_VehicleMessage vehicle_message; + + /* First we have to found which can_signal_t it is */ + search_key = build_DynamicField((double)can_message.get_id()); + signals.clear(); + configuration_t::instance().find_can_signals(search_key, signals); + + /* Decoding the message ! Don't kill the messenger ! */ + for(auto& sig : signals) + { + std::lock_guard subscribed_signals_lock(get_subscribed_signals_mutex()); + std::map& s = get_subscribed_signals(); + + /* DEBUG message to make easier debugger STL containers... + DEBUG(binder_interface, "Operator[] key char: %s, event valid? %d", sig.generic_name, afb_event_is_valid(s[sig.generic_name])); + DEBUG(binder_interface, "Operator[] key string: %s, event valid? %d", sig.generic_name, afb_event_is_valid(s[std::string(sig.generic_name)])); + DEBUG(binder_interface, "Nb elt matched char: %d", (int)s.count(sig.generic_name)); + DEBUG(binder_interface, "Nb elt matched string: %d", (int)s.count(std::string(sig.generic_name)));*/ + if( s.find(sig->get_name()) != s.end() && afb_event_is_valid(s[sig->get_name()])) + { + decoded_message = decoder_t::translateSignal(*sig, can_message, configuration_t::instance().get_can_signals()); + + openxc_SimpleMessage s_message = build_SimpleMessage(sig->get_generic_name(), decoded_message); + vehicle_message = build_VehicleMessage(s_message); + + std::lock_guard decoded_can_message_lock(decoded_can_message_mutex_); + push_new_vehicle_message(vehicle_message); + new_decoded_can_message_.notify_one(); + processed_signals++; + } + } + + DEBUG(binder_interface, "process_can_signals: %d/%d CAN signals processed.", processed_signals, (int)signals.size()); + return processed_signals; +} + +int can_bus_t::process_diagnostic_signals(active_diagnostic_request_t* entry, const can_message_t& can_message) +{ + int processed_signals = 0; + openxc_VehicleMessage vehicle_message; + + diagnostic_manager_t& manager = configuration_t::instance().get_diagnostic_manager(); + + std::lock_guard subscribed_signals_lock(get_subscribed_signals_mutex()); + std::map& s = get_subscribed_signals(); + + if( s.find(entry->get_name()) != s.end() && afb_event_is_valid(s[entry->get_name()])) + { + if(manager.get_can_bus_dev() == entry->get_can_bus_dev() && entry->get_in_flight()) + { + DiagnosticResponse response = diagnostic_receive_can_frame( + // TODO: openXC todo task: eek, is bus address and array index this tightly coupled? + &manager.get_shims(), + entry->get_handle(), can_message.get_id(), can_message.get_data(), can_message.get_length()); + if(response.completed && entry->get_handle()->completed) + { + if(entry->get_handle()->success) + { + vehicle_message = manager.relay_diagnostic_response(entry, response); + std::lock_guard decoded_can_message_lock(decoded_can_message_mutex_); + push_new_vehicle_message(vehicle_message); + new_decoded_can_message_.notify_one(); + processed_signals++; + } + else + DEBUG(binder_interface, "Fatal error sending or receiving diagnostic request"); + } + else if(!response.completed && response.multi_frame) + // Reset the timeout clock while completing the multi-frame receive + entry->get_timeout_clock().tick(); + } + } + + return processed_signals; +} + /** * @brief thread to decoding raw CAN messages. * @@ -64,13 +140,15 @@ can_bus_t::can_bus_t(int conf_file) * subscription has been made. Can message will be decoded using translateSignal that will pass it to the * corresponding decoding function if there is one assigned for that signal. If not, it will be the default * noopDecoder function that will operate on it. +* +* Depending on the nature of message, if id match a diagnostic request corresponding id for a response +* then decoding a diagnostic message else use classic CAN signals decoding functions. +* +* TODO: make diagnostic messages parsing optionnal. */ void can_bus_t::can_decode_message() { can_message_t can_message; - std::vector signals; - openxc_VehicleMessage vehicle_message; - openxc_DynamicField search_key, decoded_message; while(is_decoding_) { @@ -78,34 +156,11 @@ void can_bus_t::can_decode_message() new_can_message_cv_.wait(can_message_lock); can_message = next_can_message(); - /* First we have to found which can_signal_t it is */ - search_key = build_DynamicField((double)can_message.get_id()); - signals.clear(); - configuration_t::instance().find_can_signals(search_key, signals); - - /* Decoding the message ! Don't kill the messenger ! */ - for(auto& sig : signals) - { - std::lock_guard subscribed_signals_lock(get_subscribed_signals_mutex()); - std::map& s = get_subscribed_signals(); - - /* DEBUG message to make easier debugger STL containers... - DEBUG(binder_interface, "Operator[] key char: %s, event valid? %d", sig.generic_name, afb_event_is_valid(s[sig.generic_name])); - DEBUG(binder_interface, "Operator[] key string: %s, event valid? %d", sig.generic_name, afb_event_is_valid(s[std::string(sig.generic_name)])); - DEBUG(binder_interface, "Nb elt matched char: %d", (int)s.count(sig.generic_name)); - DEBUG(binder_interface, "Nb elt matched string: %d", (int)s.count(std::string(sig.generic_name)));*/ - if( s.find(sig->get_generic_name()) != s.end() && afb_event_is_valid(s[sig->get_generic_name()])) - { - decoded_message = decoder_t::translateSignal(*sig, can_message, configuration_t::instance().get_can_signals()); - - openxc_SimpleMessage s_message = build_SimpleMessage(sig->get_generic_name(), decoded_message); - vehicle_message = build_VehicleMessage_with_SimpleMessage(openxc_DynamicField_Type::openxc_DynamicField_Type_NUM, s_message); - - std::lock_guard decoded_can_message_lock(decoded_can_message_mutex_); - push_new_vehicle_message(vehicle_message); - new_decoded_can_message_.notify_one(); - } - } + active_diagnostic_request_t* adr = configuration_t::instance().get_diagnostic_manager().is_diagnostic_response(can_message); + if(adr != nullptr) + process_diagnostic_signals(adr, can_message); + else + process_can_signals(can_message); } } @@ -186,16 +241,16 @@ int can_bus_t::init_can_dev() for(const auto& device : devices_name) { - can_devices_m_[device] = std::make_shared(device, i); - if (can_devices_m_[device]->open() == 0) + can_devices_.push_back(std::make_shared(device, i)); + if (can_devices_[i]->open() == 0) { - i++; DEBUG(binder_interface, "Start reading thread"); NOTICE(binder_interface, "%s device opened and reading", device.c_str()); - can_devices_m_[device]->start_reading(*this); + can_devices_[i]->start_reading(*this); } else ERROR(binder_interface, "Can't open device %s", device.c_str()); + i++; } NOTICE(binder_interface, "Initialized %d/%d can bus device(s)", i, t); @@ -232,15 +287,7 @@ std::vector can_bus_t::read_conf() jo = json_tokener_parse(fd_conf_content.c_str()); if (jo == NULL || !json_object_object_get_ex(jo, "canbus", &canbus)) - {/** -* @brief Telling if the pushing thread is running -* This is the boolean value on which the while loop -* take its condition. Set it to false will stop the -* according thread. -* -* @return true if pushing thread is running, false if not. -*/ - + { ERROR(binder_interface, "Can't find canbus node in the configuration file. Please review it."); ret.clear(); } @@ -349,8 +396,7 @@ void can_bus_t::push_new_vehicle_message(const openxc_VehicleMessage& v_msg) * * @return map can_bus_dev_m_ map */ -std::map> can_bus_t::get_can_devices() +const std::vector>& can_bus_t::get_can_devices() const { - return can_devices_m_; + return can_devices_; } -