From e60c874f5d4a6dbeb6d23aa5d3b2913f0d09dc29 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Mon, 30 May 2016 19:58:51 +0900 Subject: [PATCH 01/16] sensord: fix to not loading sensor which is already loaded - a sensor has to be loaded just once. Change-Id: I9cbd1897e6a582a85ed90cc8f36812503c2d08c3 Signed-off-by: kibak.yoon --- src/server/sensor_loader.cpp | 7 +++++-- src/server/sensor_loader.h | 1 + 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/server/sensor_loader.cpp b/src/server/sensor_loader.cpp index 5e0e0f9..a873d71 100644 --- a/src/server/sensor_loader.cpp +++ b/src/server/sensor_loader.cpp @@ -63,8 +63,8 @@ sensor_loader::~sensor_loader() sensor_device_map_t::iterator it_device; std::vector::iterator it_handle; - for (it_device = m_devices.begin(); it_device != m_devices.end();) - it_device = m_devices.erase(it_device); + for (it_device = m_active_devices.begin(); it_device != m_active_devices.end();) + it_device = m_active_devices.erase(it_device); for (it_handle = m_handles.begin(); it_handle != m_handles.end(); ++it_handle) dlclose(*it_handle); @@ -220,6 +220,9 @@ void sensor_loader::create_physical_sensors(sensor_type_t type) std::shared_ptr sensor_ptr(sensor); m_sensors.insert(std::make_pair(_type, sensor_ptr)); + m_active_devices[it->first] = it->second; + m_devices.erase(it->first); + _I("created [%s] sensor", sensor->get_name()); } } diff --git a/src/server/sensor_loader.h b/src/server/sensor_loader.h index c10c4bb..4d9f179 100644 --- a/src/server/sensor_loader.h +++ b/src/server/sensor_loader.h @@ -56,6 +56,7 @@ private: sensor_map_t m_sensors; sensor_device_map_t m_devices; + sensor_device_map_t m_active_devices; std::vector m_handles; public: static sensor_loader& get_instance(void); -- 2.7.4 From 6a64accc185cf655b20b6d5b5055133c997f25bb Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Wed, 1 Jun 2016 00:13:28 +0900 Subject: [PATCH 02/16] sensord: rename return variables from err to ret - change the condition about state is not equal to PSMODE_NORMAL Change-Id: I7d8de4b7434a5c857760ae81789b8805f9c38607 Signed-off-by: kibak.yoon --- src/client/client.cpp | 10 +++++----- src/server/sensor_loader.cpp | 6 +++--- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/client/client.cpp b/src/client/client.cpp index e77b1b3..861aa43 100644 --- a/src/client/client.cpp +++ b/src/client/client.cpp @@ -119,18 +119,18 @@ void clean_up(void) static int get_power_save_state(void) { - int err; + int ret; int state = 0; int pm_state, ps_state; - err = vconf_get_int(VCONFKEY_PM_STATE, &pm_state); + ret = vconf_get_int(VCONFKEY_PM_STATE, &pm_state); - if (!err && pm_state == VCONFKEY_PM_STATE_LCDOFF) + if (!ret && pm_state == VCONFKEY_PM_STATE_LCDOFF) state |= SENSOR_OPTION_ON_IN_SCREEN_OFF; - err = vconf_get_int(VCONFKEY_SETAPPL_PSMODE, &ps_state); + ret = vconf_get_int(VCONFKEY_SETAPPL_PSMODE, &ps_state); - if (!err && ps_state == SETTING_PSMODE_NORMAL) + if (!ret && ps_state != SETTING_PSMODE_NORMAL) state |= SENSOR_OPTION_ON_IN_POWERSAVE_MODE; return state; diff --git a/src/server/sensor_loader.cpp b/src/server/sensor_loader.cpp index a873d71..0d12915 100644 --- a/src/server/sensor_loader.cpp +++ b/src/server/sensor_loader.cpp @@ -326,7 +326,7 @@ bool sensor_loader::get_paths_from_dir(const string &dir_path, vector &h struct dirent dir_entry; struct dirent *result; string name; - int error; + int ret; dir = opendir(dir_path.c_str()); @@ -336,9 +336,9 @@ bool sensor_loader::get_paths_from_dir(const string &dir_path, vector &h } while (true) { - error = readdir_r(dir, &dir_entry, &result); + ret = readdir_r(dir, &dir_entry, &result); - if (error != 0) + if (ret != 0) continue; if (result == NULL) -- 2.7.4 From 31ea2cd85c02df30cb51e301c59d1fe37e829c71 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Wed, 1 Jun 2016 00:15:20 +0900 Subject: [PATCH 03/16] sensord: fix comments properly about getting sensor data logic - remove wrong tab before assignment Change-Id: I49f18ea005c3f6f69209a30ebe4d03f0029353d9 Signed-off-by: kibak.yoon --- src/server/command_worker.cpp | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/src/server/command_worker.cpp b/src/server/command_worker.cpp index b2a61aa..13c8260 100644 --- a/src/server/command_worker.cpp +++ b/src/server/command_worker.cpp @@ -706,30 +706,28 @@ bool command_worker::cmd_get_data(void *payload) state = m_module->get_cache(&data); - // In case of not getting sensor data, wait short time and retry again - // 1. changing interval to be less than 10ms - // 2. In case of first time, wait for INIT_WAIT_TIME - // 3. at another time, wait for WAIT_TIME - // 4. retrying to get data - // 5. repeat 2 ~ 4 operations RETRY_CNT times - // 6. reverting back to original interval + /* if there is no cached data, wait short time and retry to get data again */ if (state == -ENODATA) { - const int RETRY_CNT = 10; + const int RETRY_CNT = 10; int retry = 0; unsigned int interval = m_module->get_interval(m_client_id, false); + /* 1. change interval to 10ms. */ if (interval > GET_DATA_MIN_INTERVAL) { m_module->add_interval(m_client_id, GET_DATA_MIN_INTERVAL, false); adjusted = true; } + /* 2. try to get sensor data increasing the waited time(20ms, 40ms, 80ms, 160ms, 160ms...) */ + /* 3. if data cannot be found in 10 times, stop it. */ while ((state == -ENODATA) && (retry++ < RETRY_CNT)) { _I("Wait sensor[%#llx] data updated for client [%d] #%d", m_sensor_id, m_client_id, retry); usleep(WAIT_TIME(retry)); state = m_module->get_cache(&data); } + /* 4. revert to original interval */ if (adjusted) m_module->add_interval(m_client_id, interval, false); } -- 2.7.4 From 6c74491e0abe56efac3f217ddcd34323cc135272 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Wed, 8 Jun 2016 12:45:22 +0900 Subject: [PATCH 04/16] sensord: store unknown rotation event when auto_rotation sensor starts - when sensor starts, it would be better to store unknown rotation event for getting auto_rotation sensor data. Change-Id: I68e25ec9460b7959b200185ef674adc2293a3b55 Signed-off-by: kibak.yoon --- src/sensor/auto_rotation/auto_rotation_sensor.cpp | 3 +++ src/server/sensor_base.cpp | 4 ++-- src/server/sensor_base.h | 3 ++- 3 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/sensor/auto_rotation/auto_rotation_sensor.cpp b/src/sensor/auto_rotation/auto_rotation_sensor.cpp index 0cebedd..470d5d8 100644 --- a/src/sensor/auto_rotation/auto_rotation_sensor.cpp +++ b/src/sensor/auto_rotation/auto_rotation_sensor.cpp @@ -196,8 +196,11 @@ bool auto_rotation_sensor::set_wakeup(int wakeup) bool auto_rotation_sensor::on_start(void) { + int length; m_rotation = AUTO_ROTATION_DEGREE_UNKNOWN; + get_data(&m_last_data, &length); + m_alg->start(); m_accel_sensor->add_interval((intptr_t)this , m_interval, true); diff --git a/src/server/sensor_base.cpp b/src/server/sensor_base.cpp index dd78a2a..5ff2566 100644 --- a/src/server/sensor_base.cpp +++ b/src/server/sensor_base.cpp @@ -31,11 +31,11 @@ using std::make_pair; using std::vector; sensor_base::sensor_base() -: m_id(SENSOR_ID_INVALID) +: m_last_data(NULL) +, m_id(SENSOR_ID_INVALID) , m_permission(SENSOR_PERMISSION_STANDARD) , m_started(false) , m_client(0) -, m_last_data(NULL) { } diff --git a/src/server/sensor_base.h b/src/server/sensor_base.h index 22f0e66..04883d8 100644 --- a/src/server/sensor_base.h +++ b/src/server/sensor_base.h @@ -77,6 +77,8 @@ public: int get_permission(void); protected: + sensor_data_t *m_last_data; + void set_permission(int permission); unsigned long long get_timestamp(void); @@ -93,7 +95,6 @@ private: unsigned int m_client; cmutex m_client_mutex; - sensor_data_t *m_last_data; cmutex m_data_cache_mutex; virtual int set_attribute(int32_t attribute, int32_t value); -- 2.7.4 From 4e3a78c1fd7e44c714ad54edebc8f23723907a4f Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Tue, 7 Jun 2016 21:34:47 +0900 Subject: [PATCH 05/16] sensord: support attribute interface & clean up option - SENSORD_PAUSE_POLICY( = sensor option) - SENSORD_AXIS_ORIENTATION - SENSORD_ATTRIBUTE_* Change-Id: Ida4039b55fc72a5cff4cbe021ef79c33429c3948 Signed-off-by: kibak.yoon --- src/client/client.cpp | 74 +++++++++++++++++++++++++++++++++------------- src/shared/sensor_common.h | 18 +++++++++++ 2 files changed, 71 insertions(+), 21 deletions(-) diff --git a/src/client/client.cpp b/src/client/client.cpp index 861aa43..99b96d0 100644 --- a/src/client/client.cpp +++ b/src/client/client.cpp @@ -44,9 +44,13 @@ using std::vector; #define DEFAULT_INTERVAL POLL_10HZ_MS +#define CONVERT_OPTION_PAUSE_POLICY(option) \ + (option == SENSOR_OPTION_DEFAULT || option == SENSOR_OPTION_ALWAYS_ON) ? \ + (option ^ 0b11) : option + static cmutex lock; -static int g_power_save_state = 0; +static int g_power_save_state = SENSORD_PAUSE_NONE; static int get_power_save_state(void); static void power_save_state_cb(keynode_t *node, void *data); @@ -126,12 +130,12 @@ static int get_power_save_state(void) ret = vconf_get_int(VCONFKEY_PM_STATE, &pm_state); if (!ret && pm_state == VCONFKEY_PM_STATE_LCDOFF) - state |= SENSOR_OPTION_ON_IN_SCREEN_OFF; + state |= SENSORD_PAUSE_ON_DISPLAY_OFF; ret = vconf_get_int(VCONFKEY_SETAPPL_PSMODE, &ps_state); if (!ret && ps_state != SETTING_PSMODE_NORMAL) - state |= SENSOR_OPTION_ON_IN_POWERSAVE_MODE; + state |= SENSORD_PAUSE_ON_POWERSAVE_MODE; return state; } @@ -633,7 +637,7 @@ API int sensord_connect(sensor_t sensor) _I("%s[%d] connects with %s[%d]", get_client_name(), client_id, get_sensor_name(sensor_id), handle); - sensor_client_info::get_instance().set_sensor_params(handle, SENSOR_STATE_STOPPED, SENSOR_OPTION_DEFAULT); + sensor_client_info::get_instance().set_sensor_params(handle, SENSOR_STATE_STOPPED, SENSORD_PAUSE_ALL); if (!sensor_registered) { if (!cmd_channel->cmd_hello(sensor_id)) { @@ -829,6 +833,7 @@ API bool sensord_start(int handle, int option) sensor_rep prev_rep, cur_rep; bool ret; int prev_state, prev_option; + int pause; AUTOLOCK(lock); @@ -843,14 +848,16 @@ API bool sensord_start(int handle, int option) _I("%s starts %s[%d], with option: %d, power save state: %d", get_client_name(), get_sensor_name(sensor_id), handle, option, g_power_save_state); - if (g_power_save_state && !(g_power_save_state & option)) { - sensor_client_info::get_instance().set_sensor_params(handle, SENSOR_STATE_PAUSED, option); + pause = CONVERT_OPTION_PAUSE_POLICY(option); + + if (g_power_save_state && (g_power_save_state & pause)) { + sensor_client_info::get_instance().set_sensor_params(handle, SENSOR_STATE_PAUSED, pause); return true; } sensor_client_info::get_instance().get_sensor_rep(sensor_id, prev_rep); sensor_client_info::get_instance().get_sensor_params(handle, prev_state, prev_option); - sensor_client_info::get_instance().set_sensor_params(handle, SENSOR_STATE_STARTED, option); + sensor_client_info::get_instance().set_sensor_params(handle, SENSOR_STATE_STARTED, pause); sensor_client_info::get_instance().get_sensor_rep(sensor_id, cur_rep); ret = change_sensor_rep(sensor_id, prev_rep, cur_rep); @@ -968,46 +975,52 @@ API bool sensord_change_event_max_batch_latency(int handle, unsigned int event_t return change_event_batch(handle, event_type, prev_interval, max_batch_latency); } -API bool sensord_set_option(int handle, int option) +static int change_pause_policy(int handle, int pause) { sensor_id_t sensor_id; sensor_rep prev_rep, cur_rep; int sensor_state; bool ret; - int prev_state, prev_option; + int prev_state, prev_pause; AUTOLOCK(lock); + retvm_if((pause < 0) || (pause >= SENSORD_PAUSE_END), -EINVAL, + "Invalid pause value : %d, handle: %d, %s, %s", + pause, handle, get_sensor_name(sensor_id), get_client_name()); + if (!sensor_client_info::get_instance().get_sensor_state(handle, sensor_state)|| !sensor_client_info::get_instance().get_sensor_id(handle, sensor_id)) { _E("client %s failed to get handle information", get_client_name()); - return false; + return -EPERM; } - retvm_if((option < 0) || (option >= SENSOR_OPTION_END), false, "Invalid option value : %d, handle: %d, %s, %s", - option, handle, get_sensor_name(sensor_id), get_client_name()); - sensor_client_info::get_instance().get_sensor_rep(sensor_id, prev_rep); - sensor_client_info::get_instance().get_sensor_params(handle, prev_state, prev_option); + sensor_client_info::get_instance().get_sensor_params(handle, prev_state, prev_pause); if (g_power_save_state) { - if ((option & g_power_save_state) && (sensor_state == SENSOR_STATE_PAUSED)) - sensor_client_info::get_instance().set_sensor_state(handle, SENSOR_STATE_STARTED); - else if (!(option & g_power_save_state) && (sensor_state == SENSOR_STATE_STARTED)) + if ((pause & g_power_save_state) && (sensor_state == SENSOR_STATE_STARTED)) sensor_client_info::get_instance().set_sensor_state(handle, SENSOR_STATE_PAUSED); + else if (!(pause & g_power_save_state) && (sensor_state == SENSOR_STATE_PAUSED)) + sensor_client_info::get_instance().set_sensor_state(handle, SENSOR_STATE_STARTED); } - sensor_client_info::get_instance().set_sensor_option(handle, option); + sensor_client_info::get_instance().set_sensor_option(handle, pause); sensor_client_info::get_instance().get_sensor_rep(sensor_id, cur_rep); ret = change_sensor_rep(sensor_id, prev_rep, cur_rep); if (!ret) - sensor_client_info::get_instance().set_sensor_option(handle, prev_option); + sensor_client_info::get_instance().set_sensor_option(handle, prev_pause); - return ret; + return (ret ? OP_SUCCESS : OP_ERROR); } -API int sensord_set_attribute_int(int handle, int attribute, int value) +static int change_axis_orientation(int handle, int axis_orientation) +{ + return OP_SUCCESS; +} + +static int change_attribute_int(int handle, int attribute, int value) { sensor_id_t sensor_id; command_channel *cmd_channel; @@ -1039,6 +1052,25 @@ API int sensord_set_attribute_int(int handle, int attribute, int value) return OP_SUCCESS; } +API bool sensord_set_option(int handle, int option) +{ + return (change_pause_policy(handle, CONVERT_OPTION_PAUSE_POLICY(option)) == OP_SUCCESS); +} + +API int sensord_set_attribute_int(int handle, int attribute, int value) +{ + switch (attribute) { + case SENSORD_ATTRIBUTE_PAUSE_POLICY: + return change_pause_policy(handle, value); + case SENSORD_ATTRIBUTE_AXIS_ORIENTATION: + return change_axis_orientation(handle, value); + default: + return change_attribute_int(handle, attribute, value); + } + + return OP_SUCCESS; +} + API int sensord_set_attribute_str(int handle, int attribute, const char *value, int value_len) { sensor_id_t sensor_id; diff --git a/src/shared/sensor_common.h b/src/shared/sensor_common.h index a1c0541..d308549 100644 --- a/src/shared/sensor_common.h +++ b/src/shared/sensor_common.h @@ -75,6 +75,24 @@ enum sensor_option_t { typedef enum sensor_option_t sensor_option_e; #endif +enum sensord_attribute_e { + SENSORD_ATTRIBUTE_AXIS_ORIENTATION = 1, + SENSORD_ATTRIBUTE_PAUSE_POLICY, +}; + +enum sensord_axis_e { + SENSORD_AXIS_DEVICE_ORIENTED = 1, + SENSORD_AXIS_DISPLAY_ORIENTED, +}; + +enum sensord_pause_e { + SENSORD_PAUSE_NONE = 0, + SENSORD_PAUSE_ON_DISPLAY_OFF = 1, + SENSORD_PAUSE_ON_POWERSAVE_MODE = 2, + SENSORD_PAUSE_ALL = 3, + SENSORD_PAUSE_END, +}; + enum poll_interval_t { POLL_100HZ_MS = 10, POLL_50HZ_MS = 20, -- 2.7.4 From 2d29dbd9dffa335735afcbc0098388fbb76e1150 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Wed, 8 Jun 2016 13:42:44 +0900 Subject: [PATCH 06/16] sensord: rename option to pause policy Change-Id: I2bcb62312af65f693aa83a1071bac1eee3d3a4a5 Signed-off-by: kibak.yoon --- src/client/client.cpp | 24 ++++++++++++------------ src/client/client_common.cpp | 4 ++-- src/client/command_channel.cpp | 24 ++++++++++++------------ src/client/command_channel.h | 2 +- src/client/sensor_client_info.cpp | 28 ++++++++++++++-------------- src/client/sensor_client_info.h | 10 +++++----- src/client/sensor_event_listener.cpp | 4 ++-- src/client/sensor_handle_info.cpp | 2 +- src/client/sensor_handle_info.h | 2 +- src/server/client_info_manager.cpp | 4 ++-- src/server/client_info_manager.h | 2 +- src/server/client_sensor_record.cpp | 6 +++--- src/server/client_sensor_record.h | 2 +- src/server/command_worker.cpp | 18 +++++++++--------- src/server/command_worker.h | 2 +- src/server/sensor_usage.cpp | 2 +- src/server/sensor_usage.h | 2 +- src/shared/command_common.h | 6 +++--- 18 files changed, 72 insertions(+), 72 deletions(-) diff --git a/src/client/client.cpp b/src/client/client.cpp index 99b96d0..7effcda 100644 --- a/src/client/client.cpp +++ b/src/client/client.cpp @@ -224,7 +224,7 @@ void restore_session(void) sensor_rep prev_rep, cur_rep; prev_rep.active = false; - prev_rep.option = SENSOR_OPTION_DEFAULT; + prev_rep.pause_policy = SENSORD_PAUSE_ALL; prev_rep.interval = 0; sensor_client_info::get_instance().get_sensor_rep(*it_sensor, cur_rep); @@ -273,9 +273,9 @@ static bool change_sensor_rep(sensor_id_t sensor_id, sensor_rep &prev_rep, senso get_events_diff(prev_rep.event_types, cur_rep.event_types, add_event_types, del_event_types); if (cur_rep.active) { - if (prev_rep.option != cur_rep.option) { - if (!cmd_channel->cmd_set_option(cur_rep.option)) { - _E("Sending cmd_set_option(%d, %s, %d) failed for %s", client_id, get_sensor_name(sensor_id), cur_rep.option, get_client_name()); + if (prev_rep.pause_policy != cur_rep.pause_policy) { + if (!cmd_channel->cmd_set_pause_policy(cur_rep.pause_policy)) { + _E("Sending cmd_set_pause_policy(%d, %s, %d) failed for %s", client_id, get_sensor_name(sensor_id), cur_rep.pause_policy, get_client_name()); return false; } } @@ -832,7 +832,7 @@ API bool sensord_start(int handle, int option) sensor_id_t sensor_id; sensor_rep prev_rep, cur_rep; bool ret; - int prev_state, prev_option; + int prev_state, prev_pause; int pause; AUTOLOCK(lock); @@ -856,14 +856,14 @@ API bool sensord_start(int handle, int option) } sensor_client_info::get_instance().get_sensor_rep(sensor_id, prev_rep); - sensor_client_info::get_instance().get_sensor_params(handle, prev_state, prev_option); + sensor_client_info::get_instance().get_sensor_params(handle, prev_state, prev_pause); sensor_client_info::get_instance().set_sensor_params(handle, SENSOR_STATE_STARTED, pause); sensor_client_info::get_instance().get_sensor_rep(sensor_id, cur_rep); ret = change_sensor_rep(sensor_id, prev_rep, cur_rep); if (!ret) - sensor_client_info::get_instance().set_sensor_params(handle, prev_state, prev_option); + sensor_client_info::get_instance().set_sensor_params(handle, prev_state, prev_pause); return ret; } @@ -873,7 +873,7 @@ API bool sensord_stop(int handle) sensor_id_t sensor_id; int sensor_state; bool ret; - int prev_state, prev_option; + int prev_state, prev_pause; sensor_rep prev_rep, cur_rep; @@ -891,14 +891,14 @@ API bool sensord_stop(int handle) _I("%s stops sensor %s[%d]", get_client_name(), get_sensor_name(sensor_id), handle); sensor_client_info::get_instance().get_sensor_rep(sensor_id, prev_rep); - sensor_client_info::get_instance().get_sensor_params(handle, prev_state, prev_option); + sensor_client_info::get_instance().get_sensor_params(handle, prev_state, prev_pause); sensor_client_info::get_instance().set_sensor_state(handle, SENSOR_STATE_STOPPED); sensor_client_info::get_instance().get_sensor_rep(sensor_id, cur_rep); ret = change_sensor_rep(sensor_id, prev_rep, cur_rep); if (!ret) - sensor_client_info::get_instance().set_sensor_params(handle, prev_state, prev_option); + sensor_client_info::get_instance().set_sensor_params(handle, prev_state, prev_pause); return ret; } @@ -1004,13 +1004,13 @@ static int change_pause_policy(int handle, int pause) else if (!(pause & g_power_save_state) && (sensor_state == SENSOR_STATE_PAUSED)) sensor_client_info::get_instance().set_sensor_state(handle, SENSOR_STATE_STARTED); } - sensor_client_info::get_instance().set_sensor_option(handle, pause); + sensor_client_info::get_instance().set_sensor_pause_policy(handle, pause); sensor_client_info::get_instance().get_sensor_rep(sensor_id, cur_rep); ret = change_sensor_rep(sensor_id, prev_rep, cur_rep); if (!ret) - sensor_client_info::get_instance().set_sensor_option(handle, prev_pause); + sensor_client_info::get_instance().set_sensor_pause_policy(handle, prev_pause); return (ret ? OP_SUCCESS : OP_ERROR); } diff --git a/src/client/client_common.cpp b/src/client/client_common.cpp index a91dbb0..ec17a1f 100644 --- a/src/client/client_common.cpp +++ b/src/client/client_common.cpp @@ -95,9 +95,9 @@ void print_event_occurrence_log(sensor_handle_info &info) if ((count != 1) && (count % log_per_count != 0)) return; - _D("%s receives %s[%d][state: %d, option: %d, count: %d]", get_client_name(), + _D("%s receives %s[%d][state: %d, pause policy: %d, count: %d]", get_client_name(), get_sensor_name(info.m_sensor_id), info.m_handle, - info.m_sensor_state, info.m_sensor_option, count); + info.m_sensor_state, info.m_pause_policy, count); } /* diff --git a/src/client/command_channel.cpp b/src/client/command_channel.cpp index ff49a54..39c3d4d 100644 --- a/src/client/command_channel.cpp +++ b/src/client/command_channel.cpp @@ -339,33 +339,33 @@ bool command_channel::cmd_stop(void) return true; } -bool command_channel::cmd_set_option(int option) +bool command_channel::cmd_set_pause_policy(int pause_policy) { cpacket *packet; - cmd_set_option_t *cmd_set_option; + cmd_set_pause_policy_t *cmd_set_pause_policy; cmd_done_t *cmd_done; - packet = new(std::nothrow) cpacket(sizeof(cmd_set_option_t)); + packet = new(std::nothrow) cpacket(sizeof(cmd_set_pause_policy_t)); retvm_if(!packet, false, "Failed to allocate memory"); - packet->set_cmd(CMD_SET_OPTION); + packet->set_cmd(CMD_SET_PAUSE_POLICY); - cmd_set_option = (cmd_set_option_t*)packet->data(); - cmd_set_option->option = option; + cmd_set_pause_policy = (cmd_set_pause_policy_t*)packet->data(); + cmd_set_pause_policy->pause_policy = pause_policy; - _I("%s send cmd_set_option(client_id=%d, %s, option=%d)", - get_client_name(), m_client_id, get_sensor_name(m_sensor_id), option); + _I("%s send cmd_set_pause_policy(client_id=%d, %s, pause_policy=%d)", + get_client_name(), m_client_id, get_sensor_name(m_sensor_id), pause_policy); if (!command_handler(packet, (void **)&cmd_done)) { - _E("Client %s failed to send/receive command for sensor[%s] with client_id [%d], option[%d]", - get_client_name(), get_sensor_name(m_sensor_id), m_client_id, option); + _E("Client %s failed to send/receive command for sensor[%s] with client_id [%d], pause_policy[%d]", + get_client_name(), get_sensor_name(m_sensor_id), m_client_id, pause_policy); delete packet; return false; } if (cmd_done->value < 0) { - _E("Client %s got error[%d] from server for sensor[%s] with client_id [%d], option[%d]", - get_client_name(), cmd_done->value, get_sensor_name(m_sensor_id), m_client_id, option); + _E("Client %s got error[%d] from server for sensor[%s] with client_id [%d], pause_policy[%d]", + get_client_name(), cmd_done->value, get_sensor_name(m_sensor_id), m_client_id, pause_policy); delete[] (char *)cmd_done; delete packet; diff --git a/src/client/command_channel.h b/src/client/command_channel.h index 15b579e..2dd3589 100644 --- a/src/client/command_channel.h +++ b/src/client/command_channel.h @@ -39,7 +39,7 @@ public: bool cmd_byebye(void); bool cmd_start(void); bool cmd_stop(void); - bool cmd_set_option(int option); + bool cmd_set_pause_policy(int pause_policy); bool cmd_register_event(unsigned int event_type); bool cmd_register_events(event_type_vector &event_vec); bool cmd_unregister_event(unsigned int event_type); diff --git a/src/client/sensor_client_info.cpp b/src/client/sensor_client_info.cpp index ecb453e..6ac154c 100644 --- a/src/client/sensor_client_info.cpp +++ b/src/client/sensor_client_info.cpp @@ -59,7 +59,7 @@ int sensor_client_info::create_handle(sensor_id_t sensor) handle_info.m_sensor_id = sensor; handle_info.m_sensor_state = SENSOR_STATE_STOPPED; - handle_info.m_sensor_option = SENSOR_OPTION_DEFAULT; + handle_info.m_pause_policy = SENSORD_PAUSE_ALL; handle_info.m_handle = handle; handle_info.m_accuracy = -1; handle_info.m_accuracy_cb = NULL; @@ -163,7 +163,7 @@ bool sensor_client_info::unregister_accuracy_cb(int handle) return true; } -bool sensor_client_info::set_sensor_params(int handle, int sensor_state, int sensor_option) +bool sensor_client_info::set_sensor_params(int handle, int sensor_state, int pause_policy) { AUTOLOCK(m_handle_info_lock); @@ -175,12 +175,12 @@ bool sensor_client_info::set_sensor_params(int handle, int sensor_state, int sen } it_handle->second.m_sensor_state = sensor_state; - it_handle->second.m_sensor_option = sensor_option; + it_handle->second.m_pause_policy = pause_policy; return true; } -bool sensor_client_info::get_sensor_params(int handle, int &sensor_state, int &sensor_option) +bool sensor_client_info::get_sensor_params(int handle, int &sensor_state, int &pause_policy) { AUTOLOCK(m_handle_info_lock); @@ -192,7 +192,7 @@ bool sensor_client_info::get_sensor_params(int handle, int &sensor_state, int &s } sensor_state = it_handle->second.m_sensor_state; - sensor_option = it_handle->second.m_sensor_option; + pause_policy = it_handle->second.m_pause_policy; return true; } @@ -213,7 +213,7 @@ bool sensor_client_info::set_sensor_state(int handle, int sensor_state) return true; } -bool sensor_client_info::set_sensor_option(int handle, int sensor_option) +bool sensor_client_info::set_sensor_pause_policy(int handle, int pause_policy) { AUTOLOCK(m_handle_info_lock); @@ -224,7 +224,7 @@ bool sensor_client_info::set_sensor_option(int handle, int sensor_option) return false; } - it_handle->second.m_sensor_option = sensor_option; + it_handle->second.m_pause_policy = pause_policy; return true; } @@ -323,7 +323,7 @@ void sensor_client_info::get_sensor_rep(sensor_id_t sensor, sensor_rep& rep) { const unsigned int INVALID_BATCH_VALUE = std::numeric_limits::max(); rep.active = is_sensor_active(sensor); - rep.option = get_active_option(sensor); + rep.pause_policy = get_active_pause_policy(sensor); if (!get_active_batch(sensor, rep.interval, rep.latency)) { rep.interval = INVALID_BATCH_VALUE; rep.latency = INVALID_BATCH_VALUE; @@ -441,11 +441,11 @@ bool sensor_client_info::get_active_batch(sensor_id_t sensor, unsigned int &inte return true; } -unsigned int sensor_client_info::get_active_option(sensor_id_t sensor) +unsigned int sensor_client_info::get_active_pause_policy(sensor_id_t sensor) { - int active_option = SENSOR_OPTION_DEFAULT; + int active_pause = SENSORD_PAUSE_ALL; bool active_sensor_found = false; - int option; + int pause; AUTOLOCK(m_handle_info_lock); @@ -455,8 +455,8 @@ unsigned int sensor_client_info::get_active_option(sensor_id_t sensor) if ((it_handle->second.m_sensor_id == sensor) && (it_handle->second.m_sensor_state == SENSOR_STATE_STARTED)) { active_sensor_found = true; - option = it_handle->second.m_sensor_option; - active_option = (option > active_option) ? option : active_option; + pause = it_handle->second.m_pause_policy; + active_pause = (pause < active_pause) ? pause: active_pause; } ++it_handle; @@ -465,7 +465,7 @@ unsigned int sensor_client_info::get_active_option(sensor_id_t sensor) if (!active_sensor_found) _D("Active sensor[%#llx] is not found for client %s", sensor, get_client_name()); - return active_option; + return active_pause; } bool sensor_client_info::get_sensor_id(int handle, sensor_id_t &sensor) diff --git a/src/client/sensor_client_info.h b/src/client/sensor_client_info.h index 275e674..ac3a56f 100644 --- a/src/client/sensor_client_info.h +++ b/src/client/sensor_client_info.h @@ -53,7 +53,7 @@ typedef unordered_map sensor_command_channel_map typedef struct sensor_rep { bool active; - int option; + int pause_policy; unsigned int interval; unsigned int latency; event_type_vector event_types; @@ -71,10 +71,10 @@ public: bool register_accuracy_cb(int handle, sensor_accuracy_changed_cb_t cb, void* user_data); bool unregister_accuracy_cb(int handle); - bool set_sensor_params(int handle, int sensor_state, int sensor_option); - bool get_sensor_params(int handle, int &sensor_state, int &sensor_option); + bool set_sensor_params(int handle, int sensor_state, int sensor_pause_policy); + bool get_sensor_params(int handle, int &sensor_state, int &sensor_pause_policy); bool set_sensor_state(int handle, int sensor_state); - bool set_sensor_option(int handle, int sensor_option); + bool set_sensor_pause_policy(int handle, int pause_policy); bool set_event_batch(int handle, unsigned int event_type, unsigned int interval, unsigned int latency); bool set_accuracy(int handle, int accuracy); bool set_bad_accuracy(int handle, int bad_accuracy); @@ -83,7 +83,7 @@ public: void get_sensor_rep(sensor_id_t sensor, sensor_rep& rep); bool get_active_batch(sensor_id_t sensor, unsigned int &interval, unsigned int &latency); - unsigned int get_active_option(sensor_id_t sensor_id); + unsigned int get_active_pause_policy(sensor_id_t sensor_id); void get_active_event_types(sensor_id_t sensor_id, event_type_vector &active_event_types); bool get_sensor_id(int handle, sensor_id_t &sensor_id); diff --git a/src/client/sensor_event_listener.cpp b/src/client/sensor_event_listener.cpp index 3182b78..1b9cd07 100644 --- a/src/client/sensor_event_listener.cpp +++ b/src/client/sensor_event_listener.cpp @@ -82,12 +82,12 @@ void sensor_event_listener::operate_sensor(sensor_id_t sensor, int power_save_st if ((it_handle->second.m_sensor_state == SENSOR_STATE_STARTED) && power_save_state && - !(it_handle->second.m_sensor_option & power_save_state)) { + (it_handle->second.m_pause_policy & power_save_state)) { m_client_info.set_sensor_state(it_handle->first, SENSOR_STATE_PAUSED); _I("%s's %s[%d] is paused", get_client_name(), get_sensor_name(sensor), it_handle->first); } else if ((it_handle->second.m_sensor_state == SENSOR_STATE_PAUSED) && - (!power_save_state || (it_handle->second.m_sensor_option & power_save_state))) { + (!power_save_state || !(it_handle->second.m_pause_policy & power_save_state))) { m_client_info.set_sensor_state(it_handle->first, SENSOR_STATE_STARTED); _I("%s's %s[%d] is resumed", get_client_name(), get_sensor_name(sensor), it_handle->first); } diff --git a/src/client/sensor_handle_info.cpp b/src/client/sensor_handle_info.cpp index 6dc1984..175c567 100644 --- a/src/client/sensor_handle_info.cpp +++ b/src/client/sensor_handle_info.cpp @@ -29,7 +29,7 @@ sensor_handle_info::sensor_handle_info() : m_handle(0) , m_sensor_id(UNKNOWN_SENSOR) , m_sensor_state(SENSOR_STATE_UNKNOWN) -, m_sensor_option(SENSOR_OPTION_DEFAULT) +, m_pause_policy(SENSORD_PAUSE_ALL) , m_bad_accuracy(false) , m_accuracy(-1) , m_accuracy_cb(NULL) diff --git a/src/client/sensor_handle_info.h b/src/client/sensor_handle_info.h index 00eba28..f64b2a6 100644 --- a/src/client/sensor_handle_info.h +++ b/src/client/sensor_handle_info.h @@ -33,7 +33,7 @@ public: int m_handle; sensor_id_t m_sensor_id; int m_sensor_state; - int m_sensor_option; + int m_pause_policy; int m_bad_accuracy; int m_accuracy; sensor_accuracy_changed_cb_t m_accuracy_cb; diff --git a/src/server/client_info_manager.cpp b/src/server/client_info_manager.cpp index efdaec0..4a72308 100644 --- a/src/server/client_info_manager.cpp +++ b/src/server/client_info_manager.cpp @@ -131,7 +131,7 @@ bool client_info_manager::get_batch(int client_id, sensor_id_t sensor_id, unsign return it_record->second.get_batch(sensor_id, interval, latency); } -bool client_info_manager::set_option(int client_id, sensor_id_t sensor_id, int option) +bool client_info_manager::set_pause_policy(int client_id, sensor_id_t sensor_id, int pause_policy) { AUTOLOCK(m_mutex); @@ -144,7 +144,7 @@ bool client_info_manager::set_option(int client_id, sensor_id_t sensor_id, int o return false; } - if (!it_record->second.set_option(sensor_id, option)) + if (!it_record->second.set_pause_policy(sensor_id, pause_policy)) return false; return true; diff --git a/src/server/client_info_manager.h b/src/server/client_info_manager.h index b65c4bf..ad7add8 100644 --- a/src/server/client_info_manager.h +++ b/src/server/client_info_manager.h @@ -52,7 +52,7 @@ public: bool set_batch(int client_id, sensor_id_t sensor_id, unsigned int interval, unsigned int latency); bool get_batch(int client_id, sensor_id_t sensor_id, unsigned int &interval, unsigned int &latency); - bool set_option(int client_id, sensor_id_t sensor_id, int option); + bool set_pause_policy(int client_id, sensor_id_t sensor_id, int pause_policy); bool set_start(int client_id, sensor_id_t sensor_id, bool start); bool is_started(int client_id, sensor_id_t sensor_id); diff --git a/src/server/client_sensor_record.cpp b/src/server/client_sensor_record.cpp index e2e647e..8eeafca 100644 --- a/src/server/client_sensor_record.cpp +++ b/src/server/client_sensor_record.cpp @@ -72,16 +72,16 @@ bool client_sensor_record::unregister_event(sensor_id_t sensor_id, unsigned int return true; } -bool client_sensor_record::set_option(sensor_id_t sensor_id, int option) +bool client_sensor_record::set_pause_policy(sensor_id_t sensor_id, int pause_policy) { auto it_usage = m_sensor_usages.find(sensor_id); if (it_usage == m_sensor_usages.end()) { sensor_usage usage; - usage.m_option = option; + usage.m_pause_policy = pause_policy; m_sensor_usages.insert(pair(sensor_id, usage)); } else { - it_usage->second.m_option = option; + it_usage->second.m_pause_policy = pause_policy; } return true; diff --git a/src/server/client_sensor_record.h b/src/server/client_sensor_record.h index defc3d9..ad4835c 100644 --- a/src/server/client_sensor_record.h +++ b/src/server/client_sensor_record.h @@ -47,7 +47,7 @@ public: bool set_batch(sensor_id_t sensor_id, unsigned int interval, unsigned int latency); bool get_batch(sensor_id_t sensor_id, unsigned int &interval, unsigned int &latency); - bool set_option(sensor_id_t sensor_id, int option); + bool set_pause_policy(sensor_id_t sensor_id, int pause_policy); bool set_start(sensor_id_t sensor_id, bool start); bool is_started(sensor_id_t sensor_id); diff --git a/src/server/command_worker.cpp b/src/server/command_worker.cpp index 13c8260..f8a6075 100644 --- a/src/server/command_worker.cpp +++ b/src/server/command_worker.cpp @@ -82,7 +82,7 @@ void command_worker::init_cmd_handlers(void) m_cmd_handlers[CMD_STOP] = &command_worker::cmd_stop; m_cmd_handlers[CMD_REG] = &command_worker::cmd_register_event; m_cmd_handlers[CMD_UNREG] = &command_worker::cmd_unregister_event; - m_cmd_handlers[CMD_SET_OPTION] = &command_worker::cmd_set_option; + m_cmd_handlers[CMD_SET_PAUSE_POLICY] = &command_worker::cmd_set_pause_policy; m_cmd_handlers[CMD_SET_BATCH] = &command_worker::cmd_set_batch; m_cmd_handlers[CMD_UNSET_BATCH] = &command_worker::cmd_unset_batch; m_cmd_handlers[CMD_GET_DATA] = &command_worker::cmd_get_data; @@ -658,23 +658,23 @@ out: return true; } -bool command_worker::cmd_set_option(void *payload) +bool command_worker::cmd_set_pause_policy(void *payload) { - cmd_set_option_t *cmd; + cmd_set_pause_policy_t *cmd; long ret_value = OP_ERROR; - cmd = (cmd_set_option_t*)payload; + cmd = (cmd_set_pause_policy_t*)payload; if (!is_permission_allowed()) { - _E("Permission denied to set interval for client [%d], for sensor [%#llx] with option [%d] to client info manager", - m_client_id, m_sensor_id, cmd->option); + _E("Permission denied to set interval for client [%d], for sensor [%#llx] with pause_policy [%d] to client info manager", + m_client_id, m_sensor_id, cmd->pause_policy); ret_value = OP_ERROR; goto out; } - if (!get_client_info_manager().set_option(m_client_id, m_sensor_id, cmd->option)) { - _E("Failed to set option for client [%d], for sensor [%#llx] with option [%d] to client info manager", - m_client_id, m_sensor_id, cmd->option); + if (!get_client_info_manager().set_pause_policy(m_client_id, m_sensor_id, cmd->pause_policy)) { + _E("Failed to set pause_policy for client [%d], for sensor [%#llx] with pause_policy [%d] to client info manager", + m_client_id, m_sensor_id, cmd->pause_policy); ret_value = OP_ERROR; goto out; } diff --git a/src/server/command_worker.h b/src/server/command_worker.h index 78661ad..3e1b62c 100644 --- a/src/server/command_worker.h +++ b/src/server/command_worker.h @@ -69,7 +69,7 @@ private: bool cmd_unregister_event(void *payload); bool cmd_set_batch(void *payload); bool cmd_unset_batch(void *payload); - bool cmd_set_option(void *payload); + bool cmd_set_pause_policy(void *payload); bool cmd_get_data(void *payload); bool cmd_set_attribute_int(void *payload); bool cmd_set_attribute_str(void *payload); diff --git a/src/server/sensor_usage.cpp b/src/server/sensor_usage.cpp index 780bdff..e4f0ffc 100644 --- a/src/server/sensor_usage.cpp +++ b/src/server/sensor_usage.cpp @@ -25,7 +25,7 @@ sensor_usage::sensor_usage() : m_interval(POLL_1HZ_MS) , m_latency(0) -, m_option(SENSOR_OPTION_DEFAULT) +, m_pause_policy(SENSORD_PAUSE_ALL) , m_start(false) { } diff --git a/src/server/sensor_usage.h b/src/server/sensor_usage.h index 91c4571..e3fd247 100644 --- a/src/server/sensor_usage.h +++ b/src/server/sensor_usage.h @@ -29,7 +29,7 @@ class sensor_usage { public: unsigned int m_interval; unsigned int m_latency; - int m_option; + int m_pause_policy; reg_event_vector m_reg_events; bool m_start; diff --git a/src/shared/command_common.h b/src/shared/command_common.h index 3973a92..2afb3bd 100644 --- a/src/shared/command_common.h +++ b/src/shared/command_common.h @@ -42,7 +42,7 @@ enum packet_type_t { CMD_STOP, CMD_REG, CMD_UNREG, - CMD_SET_OPTION, + CMD_SET_PAUSE_POLICY, CMD_SET_BATCH, CMD_UNSET_BATCH, CMD_GET_DATA, @@ -122,8 +122,8 @@ typedef struct { } cmd_unset_batch_t; typedef struct { - int option; -} cmd_set_option_t; + int pause_policy; +} cmd_set_pause_policy_t; typedef struct { int attribute; -- 2.7.4 From 48a6c5a2953fcb3156d839200ce81471c73eab17 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Thu, 2 Jun 2016 13:49:19 +0900 Subject: [PATCH 07/16] sensord: move operate_sensor function to sensor_client_info - it would be better to move operate_sensor() to sensor_client_info class, because it is only related with that class. - because "operate_sensor" name is ambiguous, rename it to set_pause_policy. Change-Id: I593f1380015e7d38ed413e5cda8141718942d23a Signed-off-by: kibak.yoon --- src/client/client.cpp | 2 +- src/client/sensor_client_info.cpp | 24 ++++++++++++++++++++++++ src/client/sensor_client_info.h | 2 ++ src/client/sensor_event_listener.cpp | 34 ---------------------------------- src/client/sensor_event_listener.h | 3 --- 5 files changed, 27 insertions(+), 38 deletions(-) diff --git a/src/client/client.cpp b/src/client/client.cpp index 7effcda..e818bd8 100644 --- a/src/client/client.cpp +++ b/src/client/client.cpp @@ -164,7 +164,7 @@ static void power_save_state_cb(keynode_t *node, void *data) while (it_sensor != sensors.end()) { sensor_client_info::get_instance().get_sensor_rep(*it_sensor, prev_rep); - sensor_event_listener::get_instance().operate_sensor(*it_sensor, cur_power_save_state); + sensor_client_info::get_instance().set_pause_policy(*it_sensor, cur_power_save_state); sensor_client_info::get_instance().get_sensor_rep(*it_sensor, cur_rep); change_sensor_rep(*it_sensor, prev_rep, cur_rep); diff --git a/src/client/sensor_client_info.cpp b/src/client/sensor_client_info.cpp index 6ac154c..fcd4e80 100644 --- a/src/client/sensor_client_info.cpp +++ b/src/client/sensor_client_info.cpp @@ -616,6 +616,30 @@ bool sensor_client_info::is_event_active(int handle, unsigned int event_type, un return true; } +void sensor_client_info::set_pause_policy(sensor_id_t sensor, int pause_policy) +{ + sensor_handle_info_map handles_info; + + get_sensor_handle_info(sensor, handles_info); + + for (auto it_handle = handles_info.begin(); it_handle != handles_info.end(); ++it_handle) { + if (it_handle->second.m_sensor_id != sensor) + continue; + + if (pause_policy && (it_handle->second.m_pause_policy & pause_policy)) { + if (it_handle->second.m_sensor_state == SENSOR_STATE_STARTED) { + set_sensor_state(it_handle->first, SENSOR_STATE_PAUSED); + _I("%s's %s[%d] is paused", get_client_name(), get_sensor_name(sensor), it_handle->first); + } + } else { + if (it_handle->second.m_sensor_state == SENSOR_STATE_PAUSED) { + set_sensor_state(it_handle->first, SENSOR_STATE_STARTED); + _I("%s's %s[%d] is resumed", get_client_name(), get_sensor_name(sensor), it_handle->first); + } + } + } +} + void sensor_client_info::clear(void) { close_command_channel(); diff --git a/src/client/sensor_client_info.h b/src/client/sensor_client_info.h index ac3a56f..1d2ba9b 100644 --- a/src/client/sensor_client_info.h +++ b/src/client/sensor_client_info.h @@ -107,6 +107,8 @@ public: void get_sensor_handle_info(sensor_id_t sensor, sensor_handle_info_map &handles_info); void get_all_handle_info(sensor_handle_info_map &handles_info); + void set_pause_policy(sensor_id_t sensor, int power_save_state); + void clear(void); sensor_client_info(); diff --git a/src/client/sensor_event_listener.cpp b/src/client/sensor_event_listener.cpp index 1b9cd07..ec8da17 100644 --- a/src/client/sensor_event_listener.cpp +++ b/src/client/sensor_event_listener.cpp @@ -60,40 +60,6 @@ sensor_event_listener& sensor_event_listener::get_instance(void) return inst; } -bool sensor_event_listener::start_handle(int handle) -{ - return m_client_info.set_sensor_state(handle, SENSOR_STATE_STARTED); -} - -bool sensor_event_listener::stop_handle(int handle) -{ - return m_client_info.set_sensor_state(handle, SENSOR_STATE_STOPPED); -} - -void sensor_event_listener::operate_sensor(sensor_id_t sensor, int power_save_state) -{ - sensor_handle_info_map handles_info; - - m_client_info.get_sensor_handle_info(sensor, handles_info); - - for (auto it_handle = handles_info.begin(); it_handle != handles_info.end(); ++it_handle) { - if (it_handle->second.m_sensor_id != sensor) - continue; - - if ((it_handle->second.m_sensor_state == SENSOR_STATE_STARTED) && - power_save_state && - (it_handle->second.m_pause_policy & power_save_state)) { - m_client_info.set_sensor_state(it_handle->first, SENSOR_STATE_PAUSED); - _I("%s's %s[%d] is paused", get_client_name(), get_sensor_name(sensor), it_handle->first); - - } else if ((it_handle->second.m_sensor_state == SENSOR_STATE_PAUSED) && - (!power_save_state || !(it_handle->second.m_pause_policy & power_save_state))) { - m_client_info.set_sensor_state(it_handle->first, SENSOR_STATE_STARTED); - _I("%s's %s[%d] is resumed", get_client_name(), get_sensor_name(sensor), it_handle->first); - } - } -} - client_callback_info* sensor_event_listener::handle_calibration_cb(sensor_handle_info &handle_info, unsigned event_type, unsigned long long time, int accuracy) { unsigned int cal_event_type = get_calibration_event_type(event_type); diff --git a/src/client/sensor_event_listener.h b/src/client/sensor_event_listener.h index 5bb92ab..417d0ad 100644 --- a/src/client/sensor_event_listener.h +++ b/src/client/sensor_event_listener.h @@ -66,10 +66,7 @@ typedef void (*hup_observer_t)(void); class sensor_event_listener { public: static sensor_event_listener& get_instance(void); - bool start_handle(int handle); - bool stop_handle(int handle); - void operate_sensor(sensor_id_t sensor, int power_save_state); void get_listening_sensors(sensor_id_vector &sensors); bool start_event_listener(void); -- 2.7.4 From cba95e8c6a0981d952980ea07f2fb4354a9df709 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Thu, 2 Jun 2016 14:07:47 +0900 Subject: [PATCH 08/16] sensord: change to use is_started() instead of using member variable directly - it would be better to use member function Change-Id: I2475768459215731d36bdd3ef00df263688c4431 Signed-off-by: kibak.yoon --- src/client/sensor_client_info.cpp | 28 ++++++++++++---------------- src/client/sensor_event_listener.cpp | 4 ++-- src/client/sensor_handle_info.cpp | 5 +++++ src/client/sensor_handle_info.h | 3 +++ 4 files changed, 22 insertions(+), 18 deletions(-) diff --git a/src/client/sensor_client_info.cpp b/src/client/sensor_client_info.cpp index fcd4e80..62de536 100644 --- a/src/client/sensor_client_info.cpp +++ b/src/client/sensor_client_info.cpp @@ -410,7 +410,6 @@ bool sensor_client_info::get_active_batch(sensor_id_t sensor, unsigned int &inte unsigned int min_interval = POLL_MAX_HZ_MS; unsigned int min_latency = std::numeric_limits::max(); - bool active_sensor_found = false; unsigned int _interval; unsigned int _latency; @@ -420,17 +419,16 @@ bool sensor_client_info::get_active_batch(sensor_id_t sensor, unsigned int &inte while (it_handle != m_sensor_handle_infos.end()) { if ((it_handle->second.m_sensor_id == sensor) && - (it_handle->second.m_sensor_state == SENSOR_STATE_STARTED)) { - active_sensor_found = true; - it_handle->second.get_batch(_interval, _latency); - min_interval = (_interval < min_interval) ? _interval : min_interval; - min_latency = (_latency < min_latency) ? _latency : min_latency; + it_handle->second.is_started()) { + it_handle->second.get_batch(_interval, _latency); + min_interval = (_interval < min_interval) ? _interval : min_interval; + min_latency = (_latency < min_latency) ? _latency : min_latency; } ++it_handle; } - if (!active_sensor_found) { + if (!is_sensor_active(sensor)) { _D("Active sensor[%#llx] is not found for client %s", sensor, get_client_name()); return false; } @@ -444,7 +442,6 @@ bool sensor_client_info::get_active_batch(sensor_id_t sensor, unsigned int &inte unsigned int sensor_client_info::get_active_pause_policy(sensor_id_t sensor) { int active_pause = SENSORD_PAUSE_ALL; - bool active_sensor_found = false; int pause; AUTOLOCK(m_handle_info_lock); @@ -453,16 +450,15 @@ unsigned int sensor_client_info::get_active_pause_policy(sensor_id_t sensor) while (it_handle != m_sensor_handle_infos.end()) { if ((it_handle->second.m_sensor_id == sensor) && - (it_handle->second.m_sensor_state == SENSOR_STATE_STARTED)) { - active_sensor_found = true; - pause = it_handle->second.m_pause_policy; - active_pause = (pause < active_pause) ? pause: active_pause; + it_handle->second.is_started()) { + pause = it_handle->second.m_pause_policy; + active_pause = (pause < active_pause) ? pause: active_pause; } ++it_handle; } - if (!active_sensor_found) + if (!is_sensor_active(sensor)) _D("Active sensor[%#llx] is not found for client %s", sensor, get_client_name()); return active_pause; @@ -510,8 +506,8 @@ void sensor_client_info::get_active_event_types(sensor_id_t sensor, event_type_v while (it_handle != m_sensor_handle_infos.end()) { if ((it_handle->second.m_sensor_id == sensor) && - (it_handle->second.m_sensor_state == SENSOR_STATE_STARTED)) - it_handle->second.get_reg_event_types(event_types); + it_handle->second.is_started()) + it_handle->second.get_reg_event_types(event_types); ++it_handle; } @@ -586,7 +582,7 @@ bool sensor_client_info::is_sensor_active(sensor_id_t sensor) while (it_handle != m_sensor_handle_infos.end()) { if ((it_handle->second.m_sensor_id == sensor) && - (it_handle->second.m_sensor_state == SENSOR_STATE_STARTED)) + it_handle->second.is_started()) return true; ++it_handle; diff --git a/src/client/sensor_event_listener.cpp b/src/client/sensor_event_listener.cpp index ec8da17..80c280b 100644 --- a/src/client/sensor_event_listener.cpp +++ b/src/client/sensor_event_listener.cpp @@ -128,8 +128,8 @@ void sensor_event_listener::handle_events(void* event) event_info = sensor_handle_info.get_reg_event_info(event_type); if ((sensor_handle_info.m_sensor_id != sensor_id) || - (sensor_handle_info.m_sensor_state != SENSOR_STATE_STARTED) || - !event_info) + !sensor_handle_info.is_started() || + !event_info) continue; if (event_info->m_fired) diff --git a/src/client/sensor_handle_info.cpp b/src/client/sensor_handle_info.cpp index 175c567..b394990 100644 --- a/src/client/sensor_handle_info.cpp +++ b/src/client/sensor_handle_info.cpp @@ -160,3 +160,8 @@ unsigned int sensor_handle_info::get_reg_event_count(void) { return m_reg_event_infos.size(); } + +bool sensor_handle_info::is_started(void) +{ + return (m_sensor_state == SENSOR_STATE_STARTED); +} diff --git a/src/client/sensor_handle_info.h b/src/client/sensor_handle_info.h index f64b2a6..28e5a01 100644 --- a/src/client/sensor_handle_info.h +++ b/src/client/sensor_handle_info.h @@ -54,6 +54,9 @@ public: void clear_all_events(void); static unsigned long long renew_event_id(void); + + bool is_started(void); + private: event_info_map m_reg_event_infos; static unsigned long long m_event_id; -- 2.7.4 From a305da78e60760135f49e9783b8d6f39cb04d791 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Thu, 2 Jun 2016 14:11:04 +0900 Subject: [PATCH 09/16] sensord: support the passive mode which listens sensor event only if a client wants to be listening sensor event which other clients start sensor, use sensord_set_passive_mode() before sensord_register_event(). Change-Id: Idbe59ed663c8a0bc268c406afcdc8cac13286b98 Signed-off-by: kibak.yoon --- src/client/client.cpp | 34 ++++++++++++++++++++++++++++++++++ src/client/sensor_client_info.cpp | 30 ++++++++++++++++++++++++++++++ src/client/sensor_client_info.h | 4 ++++ src/client/sensor_handle_info.cpp | 13 ++++++++++++- src/client/sensor_handle_info.h | 21 ++++++++++++--------- src/client/sensor_internal.h | 1 + 6 files changed, 93 insertions(+), 10 deletions(-) diff --git a/src/client/client.cpp b/src/client/client.cpp index e818bd8..769aa9d 100644 --- a/src/client/client.cpp +++ b/src/client/client.cpp @@ -682,6 +682,28 @@ API bool sensord_disconnect(int handle) _I("%s disconnects with %s[%d]", get_client_name(), get_sensor_name(sensor_id), handle); + if (sensor_client_info::get_instance().get_passive_mode(handle)) { + _W("%s[%d] for %s is on passive mode while disconnecting.", + get_sensor_name(sensor_id), handle, get_client_name()); + + command_channel *cmd_channel; + event_type_vector event_types; + sensor_client_info::get_instance().get_active_event_types(sensor_id, event_types); + + for (auto it = event_types.begin(); it != event_types.end(); ++it) + sensord_unregister_event(handle, *it); + + if (!sensor_client_info::get_instance().get_command_channel(sensor_id, &cmd_channel)) { + _E("client %s failed to get command channel for %s", get_client_name(), get_sensor_name(sensor_id)); + return false; + } + + if (!cmd_channel->cmd_unset_batch()) { + _E("Sending cmd_unset_interval(%d, %s) failed for %s", client_id, get_sensor_name(sensor_id), get_client_name()); + return false; + } + } + if (sensor_state != SENSOR_STATE_STOPPED) { _W("%s[%d] for %s is not stopped before disconnecting.", get_sensor_name(sensor_id), handle, get_client_name()); @@ -781,6 +803,9 @@ API bool sensord_unregister_event(int handle, unsigned int event_type) sensor_client_info::get_instance().get_sensor_rep(sensor_id, cur_rep); ret = change_sensor_rep(sensor_id, prev_rep, cur_rep); + if (sensor_client_info::get_instance().get_passive_mode(handle)) + sensor_client_info::get_instance().set_passive_mode(handle, false); + if (!ret) sensor_client_info::get_instance().register_event(handle, event_type, prev_interval, prev_latency, prev_cb, prev_user_data); @@ -1197,3 +1222,12 @@ API bool sensord_register_hub_event(int handle, unsigned int event_type, unsigne return false; } +API bool sensord_set_passive_mode(int handle, bool passive) +{ + if (!sensor_client_info::get_instance().set_passive_mode(handle, passive)) { + _E("Failed to set passive mode %d", passive); + return false; + } + + return true; +} diff --git a/src/client/sensor_client_info.cpp b/src/client/sensor_client_info.cpp index 62de536..08b5a9d 100644 --- a/src/client/sensor_client_info.cpp +++ b/src/client/sensor_client_info.cpp @@ -213,6 +213,36 @@ bool sensor_client_info::set_sensor_state(int handle, int sensor_state) return true; } +bool sensor_client_info::get_passive_mode(int handle) +{ + AUTOLOCK(m_handle_info_lock); + + auto it_handle = m_sensor_handle_infos.find(handle); + + if (it_handle == m_sensor_handle_infos.end()) { + _E("Handle[%d] is not found for client %s", handle, get_client_name()); + return false; + } + + return it_handle->second.get_passive_mode(); +} + +bool sensor_client_info::set_passive_mode(int handle, bool passive) +{ + AUTOLOCK(m_handle_info_lock); + + auto it_handle = m_sensor_handle_infos.find(handle); + + if (it_handle == m_sensor_handle_infos.end()) { + _E("Handle[%d] is not found for client %s", handle, get_client_name()); + return false; + } + + it_handle->second.set_passive_mode(passive); + + return true; +} + bool sensor_client_info::set_sensor_pause_policy(int handle, int pause_policy) { AUTOLOCK(m_handle_info_lock); diff --git a/src/client/sensor_client_info.h b/src/client/sensor_client_info.h index 1d2ba9b..7c46323 100644 --- a/src/client/sensor_client_info.h +++ b/src/client/sensor_client_info.h @@ -74,6 +74,10 @@ public: bool set_sensor_params(int handle, int sensor_state, int sensor_pause_policy); bool get_sensor_params(int handle, int &sensor_state, int &sensor_pause_policy); bool set_sensor_state(int handle, int sensor_state); + + bool get_passive_mode(int handle); + bool set_passive_mode(int handle, bool passive); + bool set_sensor_pause_policy(int handle, int pause_policy); bool set_event_batch(int handle, unsigned int event_type, unsigned int interval, unsigned int latency); bool set_accuracy(int handle, int accuracy); diff --git a/src/client/sensor_handle_info.cpp b/src/client/sensor_handle_info.cpp index b394990..f1e9c65 100644 --- a/src/client/sensor_handle_info.cpp +++ b/src/client/sensor_handle_info.cpp @@ -34,6 +34,7 @@ sensor_handle_info::sensor_handle_info() , m_accuracy(-1) , m_accuracy_cb(NULL) , m_accuracy_user_data(NULL) +, m_passive(false) { } @@ -161,7 +162,17 @@ unsigned int sensor_handle_info::get_reg_event_count(void) return m_reg_event_infos.size(); } +bool sensor_handle_info::get_passive_mode(void) +{ + return m_passive; +} + +void sensor_handle_info::set_passive_mode(bool passive) +{ + m_passive = passive; +} + bool sensor_handle_info::is_started(void) { - return (m_sensor_state == SENSOR_STATE_STARTED); + return (m_sensor_state == SENSOR_STATE_STARTED) || m_passive; } diff --git a/src/client/sensor_handle_info.h b/src/client/sensor_handle_info.h index 28e5a01..c74c3a5 100644 --- a/src/client/sensor_handle_info.h +++ b/src/client/sensor_handle_info.h @@ -30,15 +30,6 @@ typedef std::unordered_map event_info_map; class sensor_handle_info { public: - int m_handle; - sensor_id_t m_sensor_id; - int m_sensor_state; - int m_pause_policy; - int m_bad_accuracy; - int m_accuracy; - sensor_accuracy_changed_cb_t m_accuracy_cb; - void *m_accuracy_user_data; - sensor_handle_info(); ~sensor_handle_info(); @@ -55,8 +46,20 @@ public: void clear_all_events(void); static unsigned long long renew_event_id(void); + bool get_passive_mode(void); + void set_passive_mode(bool passive); bool is_started(void); + int m_handle; + sensor_id_t m_sensor_id; + int m_sensor_state; + int m_pause_policy; + int m_bad_accuracy; + int m_accuracy; + sensor_accuracy_changed_cb_t m_accuracy_cb; + void *m_accuracy_user_data; + bool m_passive; + private: event_info_map m_reg_event_infos; static unsigned long long m_event_id; diff --git a/src/client/sensor_internal.h b/src/client/sensor_internal.h index c4c3fce..3cc6641 100644 --- a/src/client/sensor_internal.h +++ b/src/client/sensor_internal.h @@ -386,6 +386,7 @@ int sensord_external_connect(const char *key, sensor_external_command_cb_t cb, v bool sensord_external_disconnect(int handle); bool sensord_external_post(int handle, unsigned long long timestamp, const float* data, int data_cnt); +bool sensord_set_passive_mode(int handle, bool passive); /** * @} */ -- 2.7.4 From da9123180996923a1449f47d5a4965ca4780ffc2 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Wed, 8 Jun 2016 20:18:58 +0900 Subject: [PATCH 10/16] sensord: restore attributes after sensor daemon is restarted sensor attributes also has to restored when daemon is restarted, like listener, interval, batch latency. Change-Id: Id9bfb37fc9ab9b71caca08639f2106710a65741f Signed-off-by: kibak.yoon --- src/client/client.cpp | 42 +++++++++++++++++++++++++++++++++++++++ src/client/sensor_client_info.cpp | 32 +++++++++++++++++++++++++++++ src/client/sensor_client_info.h | 24 +++++++--------------- src/client/sensor_handle_info.h | 6 ++++++ 4 files changed, 87 insertions(+), 17 deletions(-) diff --git a/src/client/client.cpp b/src/client/client.cpp index 769aa9d..e26ba85 100644 --- a/src/client/client.cpp +++ b/src/client/client.cpp @@ -172,6 +172,40 @@ static void power_save_state_cb(keynode_t *node, void *data) } } +bool restore_attributes(int client_id, sensor_id_t sensor, command_channel *cmd_channel) +{ + sensor_handle_info_map handle_infos; + + sensor_client_info::get_instance().get_sensor_handle_info(sensor, handle_infos); + + for (auto it_handles = handle_infos.begin(); it_handles != handle_infos.end(); ++it_handles) { + sensor_handle_info info = it_handles->second; + + for (auto it = info.attributes_int.begin(); it != info.attributes_int.end(); ++it) { + int attribute = it->first; + int value = it->second; + if (!cmd_channel->cmd_set_attribute_int(attribute, value)) { + _E("Failed to send cmd_set_attribute_int(%d, %d) for %s", + client_id, value, get_client_name()); + return false; + } + } + + for (auto it = info.attributes_str.begin(); it != info.attributes_str.end(); ++it) { + int attribute = it->first; + const char *value = it->second.c_str(); + int value_len = it->second.size(); + if (!cmd_channel->cmd_set_attribute_str(attribute, value, value_len)) { + _E("Failed to send cmd_set_attribute_str(%d, %d, %s) for %s", + client_id, value_len, value, get_client_name()); + return false; + } + } + } + + return true; +} + void restore_session(void) { AUTOLOCK(lock); @@ -233,6 +267,10 @@ void restore_session(void) goto FAILED; } + if (!restore_attributes(client_id, *it_sensor, cmd_channel)) { + _E("Failed to restore attributes(%s) for %s", get_sensor_name(*it_sensor), get_client_name()); + goto FAILED; + } ++it_sensor; } @@ -1074,6 +1112,8 @@ static int change_attribute_int(int handle, int attribute, int value) return -EPERM; } + sensor_client_info::get_instance().set_attribute(handle, attribute, value); + return OP_SUCCESS; } @@ -1130,6 +1170,8 @@ API int sensord_set_attribute_str(int handle, int attribute, const char *value, return -EPERM; } + sensor_client_info::get_instance().set_attribute(handle, attribute, value); + return OP_SUCCESS; } diff --git a/src/client/sensor_client_info.cpp b/src/client/sensor_client_info.cpp index 08b5a9d..54a6d7e 100644 --- a/src/client/sensor_client_info.cpp +++ b/src/client/sensor_client_info.cpp @@ -666,6 +666,38 @@ void sensor_client_info::set_pause_policy(sensor_id_t sensor, int pause_policy) } } +bool sensor_client_info::set_attribute(int handle, int attribute, int value) +{ + AUTOLOCK(m_handle_info_lock); + + auto it_handle = m_sensor_handle_infos.find(handle); + + if (it_handle == m_sensor_handle_infos.end()) { + _E("Handle[%d] is not found for client %s", handle, get_client_name()); + return false; + } + + it_handle->second.attributes_int[attribute] = value; + + return true; +} + +bool sensor_client_info::set_attribute(int handle, int attribute, std::string value) +{ + AUTOLOCK(m_handle_info_lock); + + auto it_handle = m_sensor_handle_infos.find(handle); + + if (it_handle == m_sensor_handle_infos.end()) { + _E("Handle[%d] is not found for client %s", handle, get_client_name()); + return false; + } + + it_handle->second.attributes_str[attribute] = value; + + return true; +} + void sensor_client_info::clear(void) { close_command_channel(); diff --git a/src/client/sensor_client_info.h b/src/client/sensor_client_info.h index 7c46323..ab3a384 100644 --- a/src/client/sensor_client_info.h +++ b/src/client/sensor_client_info.h @@ -29,27 +29,14 @@ #include #include #include -#include -#include -#include -#include #include #include #include -using std::unordered_map; -using std::vector; -using std::string; -using std::queue; -using std::mutex; -using std::lock_guard; -using std::unique_lock; -using std::condition_variable; - -typedef vector handle_vector; -typedef vector sensor_id_vector; -typedef unordered_map sensor_handle_info_map; -typedef unordered_map sensor_command_channel_map; +typedef std::vector handle_vector; +typedef std::vector sensor_id_vector; +typedef std::unordered_map sensor_handle_info_map; +typedef std::unordered_map sensor_command_channel_map; typedef struct sensor_rep { bool active; @@ -78,6 +65,9 @@ public: bool get_passive_mode(int handle); bool set_passive_mode(int handle, bool passive); + bool set_attribute(int handle, int attribute, int value); + bool set_attribute(int handle, int attribute, std::string value); + bool set_sensor_pause_policy(int handle, int pause_policy); bool set_event_batch(int handle, unsigned int event_type, unsigned int interval, unsigned int latency); bool set_accuracy(int handle, int accuracy); diff --git a/src/client/sensor_handle_info.h b/src/client/sensor_handle_info.h index c74c3a5..b62f646 100644 --- a/src/client/sensor_handle_info.h +++ b/src/client/sensor_handle_info.h @@ -25,8 +25,12 @@ #include #include #include +#include +#include typedef std::unordered_map event_info_map; +typedef std::map sensor_attribute_int_map; +typedef std::map sensor_attribute_str_map; class sensor_handle_info { public: @@ -59,6 +63,8 @@ public: sensor_accuracy_changed_cb_t m_accuracy_cb; void *m_accuracy_user_data; bool m_passive; + sensor_attribute_int_map attributes_int; + sensor_attribute_str_map attributes_str; private: event_info_map m_reg_event_infos; -- 2.7.4 From 8fe04eb190fda9eb38b93150f9140837d865915c Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Tue, 7 Jun 2016 14:02:41 +0900 Subject: [PATCH 11/16] sensord: set to UINT_MAX batch count when there is no batch latency for sensor - if there is no client which uses batch latency, batch count should be set the default value, not 0 but max batch count. if sensord sets the UINT_MAX as a default value, HAL will set the proper value comparing between this sensor's max batch count and UINT_MAX. Change-Id: I3aff8d24c00ca6b232d33c803b1783cb60325abb Signed-off-by: kibak.yoon --- src/server/sensor_base.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/server/sensor_base.cpp b/src/server/sensor_base.cpp index 5ff2566..a5a40bb 100644 --- a/src/server/sensor_base.cpp +++ b/src/server/sensor_base.cpp @@ -18,6 +18,7 @@ */ #include +#include #include #include #include @@ -264,10 +265,10 @@ bool sensor_base::delete_batch(int client_id) cur_max = m_sensor_info_list.get_max_batch(); if (!cur_max) { - _I("No latency for sensor[%#llx] by client[%d] deleting latency, so set to default 0 ms", + _I("No latency for sensor[%#llx] by client[%d] deleting latency, so set to default count", get_id(), client_id); - set_batch_latency(0); + set_batch_latency(UINT_MAX); } else if (cur_max != prev_max) { _I("Max latency for sensor[%#llx] is changed from %dms to %dms by client[%d] deleting latency", get_id(), prev_max, cur_max, client_id); -- 2.7.4 From 7df328c69c213aa972f0277fd9390ee332b88be6 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Mon, 20 Jun 2016 12:46:07 +0900 Subject: [PATCH 12/16] sensord: change sensor axis policy by using attribute setting via C-API - SENSORD_AXIS_DEVICE_ORIENTED - SENSORD_AXIS_DISPLAY_ORIENTED Change-Id: Iab201d26c1d957d58ee48a00d4aa1403aa034f9e Signed-off-by: kibak.yoon --- src/client/client.cpp | 1 + src/client/sensor_event_listener.cpp | 14 ++++++++++++-- src/client/sensor_event_listener.h | 4 ++++ 3 files changed, 17 insertions(+), 2 deletions(-) diff --git a/src/client/client.cpp b/src/client/client.cpp index e26ba85..b900db4 100644 --- a/src/client/client.cpp +++ b/src/client/client.cpp @@ -1080,6 +1080,7 @@ static int change_pause_policy(int handle, int pause) static int change_axis_orientation(int handle, int axis_orientation) { + sensor_event_listener::get_instance().set_sensor_axis(axis_orientation); return OP_SUCCESS; } diff --git a/src/client/sensor_event_listener.cpp b/src/client/sensor_event_listener.cpp index 80c280b..fdc622a 100644 --- a/src/client/sensor_event_listener.cpp +++ b/src/client/sensor_event_listener.cpp @@ -45,6 +45,7 @@ sensor_event_listener::sensor_event_listener() , m_thread_state(THREAD_STATE_TERMINATE) , m_hup_observer(NULL) , m_client_info(sensor_client_info::get_instance()) +, m_axis(SENSORD_AXIS_DEVICE_ORIENTED) , m_display_rotation(AUTO_ROTATION_DEGREE_UNKNOWN) { } @@ -204,11 +205,19 @@ bool sensor_event_listener::is_valid_callback(client_callback_info *cb_info) return m_client_info.is_event_active(cb_info->handle, cb_info->event_type, cb_info->event_id); } +void sensor_event_listener::set_sensor_axis(int axis) +{ + m_axis = axis; +} + void sensor_event_listener::align_sensor_axis(sensor_t sensor, sensor_data_t *data) { sensor_type_t type = sensor_to_sensor_info(sensor)->get_type(); - if (type != ACCELEROMETER_SENSOR && type != GYROSCOPE_SENSOR && type != GRAVITY_SENSOR) + if (m_axis != SENSORD_AXIS_DISPLAY_ORIENTED) + return; + + if (type != ACCELEROMETER_SENSOR && type != GYROSCOPE_SENSOR && type != GRAVITY_SENSOR && type != LINEAR_ACCEL_SENSOR) return; float x, y; @@ -250,7 +259,8 @@ gboolean sensor_event_listener::callback_dispatcher(gpointer data) if (cb_info->accuracy_cb) cb_info->accuracy_cb(cb_info->sensor, cb_info->timestamp, cb_info->accuracy, cb_info->accuracy_user_data); - ((sensor_cb_t) cb_info->cb)(cb_info->sensor, cb_info->event_type, (sensor_data_t *) cb_info->sensor_data.get(), cb_info->user_data); + sensor_event_listener::get_instance().align_sensor_axis(cb_info->sensor, (sensor_data_t *)cb_info->sensor_data.get()); + ((sensor_cb_t) cb_info->cb)(cb_info->sensor, cb_info->event_type, (sensor_data_t *)cb_info->sensor_data.get(), cb_info->user_data); delete cb_info; diff --git a/src/client/sensor_event_listener.h b/src/client/sensor_event_listener.h index 417d0ad..9631a63 100644 --- a/src/client/sensor_event_listener.h +++ b/src/client/sensor_event_listener.h @@ -74,6 +74,8 @@ public: void clear(void); void set_hup_observer(hup_observer_t observer); + + void set_sensor_axis(int axis); void set_display_rotation(int rt); private: @@ -97,6 +99,8 @@ private: sensor_client_info &m_client_info; /* WC1's rotation control */ + /* SENSORD_AXIS_DEVICE_ORIENTED, SENSORD_AXIS_DISPLAY_ORIENTED */ + int m_axis; int m_display_rotation; sensor_event_listener(); -- 2.7.4 From bcc7f159bdef472d1310bc55ce3b829b94456678 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Thu, 2 Jun 2016 17:48:10 +0900 Subject: [PATCH 13/16] sensord: add new internal sensor types Change-Id: I9c7bb390184877f41d379fa1bb0115dd7edecc3b Signed-off-by: kibak.yoon --- src/hal/sensor_hal_types.h | 59 +++++++++++++++++++++++++---------------- src/shared/sensor_types.h | 66 ++++++++++++++++++++++++++++++++-------------- 2 files changed, 82 insertions(+), 43 deletions(-) diff --git a/src/hal/sensor_hal_types.h b/src/hal/sensor_hal_types.h index 91630c4..9b7ccde 100644 --- a/src/hal/sensor_hal_types.h +++ b/src/hal/sensor_hal_types.h @@ -71,12 +71,47 @@ typedef enum { SENSOR_DEVICE_HUMAN_PEDOMETER = 0x300, SENSOR_DEVICE_HUMAN_SLEEP_MONITOR, + SENSOR_DEVICE_HUMAN_SLEEP_DETECTOR, + SENSOR_DEVICE_HUMAN_STRESS_MONITOR, + + SENSOR_DEVICE_EXERCISE_WALKING = 0x400, + SENSOR_DEVICE_EXERCISE_RUNNING, + SENSOR_DEVICE_EXERCISE_HIKING, + SENSOR_DEVICE_EXERCISE_CYCLING, + SENSOR_DEVICE_EXERCISE_STAIR_CLIMBING, + SENSOR_DEVICE_EXERCISE_ELLIPTICAL, + SENSOR_DEVICE_EXERCISE_INDOOR_CYCLING, + SENSOR_DEVICE_EXERCISE_ROWING, + SENSOR_DEVICE_EXERCISE_STEPPER, SENSOR_DEVICE_FUSION = 0x900, SENSOR_DEVICE_AUTO_ROTATION, SENSOR_DEVICE_AUTO_BRIGHTNESS, - SENSOR_DEVICE_CONTEXT = 0x1000, + SENSOR_DEVICE_GESTURE_MOVEMENT = 0x1200, + SENSOR_DEVICE_GESTURE_WRIST_UP, + SENSOR_DEVICE_GESTURE_WRIST_DOWN, + SENSOR_DEVICE_GESTURE_MOVEMENT_STATE, + + SENSOR_DEVICE_GPS_BATCH = 0x1A00, + SENSOR_DEVICE_ACTIVITY_TRACKER, + + SENSOR_DEVICE_HRM_CTRL = 0x1A80, + SENSOR_DEVICE_EXERCISE_COACH, + SENSOR_DEVICE_ACTIVITY_LEVEL_MONITOR, + + SENSOR_DEVICE_WEAR_STATUS = 0x2000, + SENSOR_DEVICE_WEAR_ON_MONITOR, + SENSOR_DEVICE_NO_MOVE_DETECTOR, + SENSOR_DEVICE_RESTING_HR, + SENSOR_DEVICE_STEP_LEVEL_MONITOR, + SENSOR_DEVICE_EXERCISE, + SENSOR_DEVICE_EXERCISE_HR, + SENSOR_DEVICE_WORKOUT, + SENSOR_DEVICE_CYCLE_MONITOR, + SENSOR_DEVICE_STAIR_TRACKER, + + SENSOR_DEVICE_CONTEXT = 0x7000, SENSOR_DEVICE_MOTION, SENSOR_DEVICE_PIR, SENSOR_DEVICE_PIR_LONG, @@ -87,32 +122,10 @@ typedef enum { SENSOR_DEVICE_HRM_RAW, SENSOR_DEVICE_TILT, SENSOR_DEVICE_ROTATION_VECTOR_RAW, - SENSOR_DEVICE_EXERCISE, SENSOR_DEVICE_GSR, SENSOR_DEVICE_SIMSENSE, SENSOR_DEVICE_PPG, - SENSOR_DEVICE_GESTURE_MOVEMENT = 0x1200, - SENSOR_DEVICE_GESTURE_WRIST_UP, - SENSOR_DEVICE_GESTURE_WRIST_DOWN, - SENSOR_DEVICE_GESTURE_MOVEMENT_STATE, - - SENSOR_DEVICE_WEAR_STATUS = 0x1A00, - SENSOR_DEVICE_WEAR_ON_MONITOR, - SENSOR_DEVICE_GPS_BATCH, - SENSOR_DEVICE_ACTIVITY_TRACKER, - SENSOR_DEVICE_SLEEP_DETECTOR, - SENSOR_DEVICE_NO_MOVE_DETECTOR = 0x1A80, - SENSOR_DEVICE_HRM_CTRL, - SENSOR_DEVICE_EXERCISE_COACH, - SENSOR_DEVICE_EXERCISE_HR, - SENSOR_DEVICE_RESTING_HR, - SENSOR_DEVICE_STEP_LEVEL_MONITOR, - SENSOR_DEVICE_ACTIVITY_LEVEL_MONITOR, - SENSOR_DEVICE_CYCLE_MONITOR, - SENSOR_DEVICE_STRESS_MONITOR, - SENSOR_DEVICE_AUTOSESSION_EXERCISE, - SENSOR_DEVICE_STAIR_TRACKER, } sensor_device_type; /* diff --git a/src/shared/sensor_types.h b/src/shared/sensor_types.h index cb8e65e..5c890b9 100644 --- a/src/shared/sensor_types.h +++ b/src/shared/sensor_types.h @@ -54,12 +54,47 @@ extern "C" \ DEF_SENSOR_VALUE(HUMAN_PEDOMETER_SENSOR, 0x300) \ DEF_SENSOR(HUMAN_SLEEP_MONITOR_SENSOR) \ + DEF_SENSOR(HUMAN_SLEEP_DETECTOR_SENSOR) \ + DEF_SENSOR(HUMAN_STRESS_MONITOR_SENSOR) \ + \ + DEF_SENSOR_VALUE(EXERCISE_WALKING_SENSOR, 0x400) \ + DEF_SENSOR(EXERCISE_RUNNING_SENSOR) \ + DEF_SENSOR(EXERCISE_HIKING_SENSOR) \ + DEF_SENSOR(EXERCISE_CYCLING_SENSOR) \ + DEF_SENSOR(EXERCISE_STAIR_CLIMBING_SENSOR) \ + DEF_SENSOR(EXERCISE_ELLIPTICAL_SENSOR) \ + DEF_SENSOR(EXERCISE_INDOOR_CYCLING_SENSOR) \ + DEF_SENSOR(EXERCISE_ROWING_SENSOR) \ + DEF_SENSOR(EXERCISE_STEPPER_SENSOR) \ \ DEF_SENSOR_VALUE(FUSION_SENSOR, 0x900) \ DEF_SENSOR(AUTO_ROTATION_SENSOR) \ DEF_SENSOR(AUTO_BRIGHTNESS_SENSOR) \ \ - DEF_SENSOR_VALUE(CONTEXT_SENSOR, 0x1000) \ + DEF_SENSOR_VALUE(GESTURE_MOVEMENT_SENSOR, 0x1200) \ + DEF_SENSOR(GESTURE_WRIST_UP_SENSOR) \ + DEF_SENSOR(GESTURE_WRIST_DOWN_SENSOR) \ + DEF_SENSOR(GESTURE_MOVEMENT_STATE_SENSOR) \ + \ + DEF_SENSOR_VALUE(GPS_BATCH_SENSOR, 0x1A00) \ + DEF_SENSOR(ACTIVITY_TRACKER_SENSOR) \ + \ + DEF_SENSOR_VALUE(HRM_CTRL_SENSOR, 0x1A80) \ + DEF_SENSOR(EXERCISE_COACH_SENSOR) \ + DEF_SENSOR(ACTIVITY_LEVEL_MONITOR_SENSOR) \ + \ + DEF_SENSOR_VALUE(WEAR_STATUS_SENSOR, 0x2000) \ + DEF_SENSOR(WEAR_ON_MONITOR_SENSOR) \ + DEF_SENSOR(NO_MOVE_DETECTOR_SENSOR) \ + DEF_SENSOR(RESTING_HR_SENSOR) \ + DEF_SENSOR(STEP_LEVEL_MONITOR_SENSOR) \ + DEF_SENSOR(EXERCISE_SENSOR) \ + DEF_SENSOR(EXERCISE_HR_SENSOR) \ + DEF_SENSOR(WORKOUT_SENSOR) \ + DEF_SENSOR(CYCLE_MONITOR_SENSOR) \ + DEF_SENSOR(STAIR_TRACKER_SENSOR) \ + \ + DEF_SENSOR_VALUE(CONTEXT_SENSOR, 0x7000) \ DEF_SENSOR(MOTION_SENSOR) \ DEF_SENSOR(PIR_SENSOR) \ DEF_SENSOR(PIR_LONG_SENSOR) \ @@ -70,30 +105,21 @@ extern "C" DEF_SENSOR(HRM_RAW_SENSOR) \ DEF_SENSOR(TILT_SENSOR) \ DEF_SENSOR(RV_RAW_SENSOR) \ - DEF_SENSOR(EXERCISE_SENSOR) \ - \ - DEF_SENSOR_VALUE(GESTURE_MOVEMENT_SENSOR, 0x1200) \ - DEF_SENSOR(GESTURE_WRIST_UP_SENSOR) \ - DEF_SENSOR(GESTURE_WRIST_DOWN_SENSOR) \ - DEF_SENSOR(GESTURE_MOVEMENT_STATE_SENSOR) \ - \ - DEF_SENSOR_VALUE(WEAR_STATUS_SENSOR, 0x1A00) \ - DEF_SENSOR(WEAR_ON_MONITOR_SENSOR) \ - DEF_SENSOR(GPS_BATCH_SENSOR) \ - DEF_SENSOR(ACTIVITY_TRACKER_SENSOR) \ - DEF_SENSOR(SLEEP_DETECTOR_SENSOR) \ -#define BIO_HRM_SENSOR HRM_SENSOR -#define BIO_LED_GREEN_SENSOR HRM_LED_GREEN_SENSOR -#define BIO_LED_IR_SENSOR HRM_LED_IR_SENSOR -#define BIO_LED_RED_SENSOR HRM_LED_RED_SENSOR -#define BIO_SENSOR HRM_RAW_SENSOR +#define BIO_HRM_SENSOR HRM_SENSOR +#define BIO_LED_GREEN_SENSOR HRM_LED_GREEN_SENSOR +#define BIO_LED_IR_SENSOR HRM_LED_IR_SENSOR +#define BIO_LED_RED_SENSOR HRM_LED_RED_SENSOR +#define BIO_SENSOR HRM_RAW_SENSOR +#define SLEEP_DETECTOR_SENSOR HUMAN_SLEEP_DETECTOR_SENSOR +#define STRESS_MONITOR_SENSOR HUMAN_STRESS_MONITOR_SENSOR +#define AUTOSESSION_EXERCISE_SENSOR WORKOUT_SENSOR DECLARE_SENSOR_ENUM(sensor_type_t, SENSOR_TYPE) enum proxi_change_state { - PROXIMITY_STATE_NEAR = 0, - PROXIMITY_STATE_FAR = 1, + PROXIMITY_STATE_NEAR = 0, + PROXIMITY_STATE_FAR = 1, }; enum auto_rotation_state { -- 2.7.4 From 7562c18dbaad2cd013843373635055456120b487 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Tue, 21 Jun 2016 12:45:25 +0900 Subject: [PATCH 14/16] sensord: rename and reorder sensor types - EXERCISE_COACH -> EXERCISE_STANDALONE - EXERCISE -> EXERCISE_COMPANION - EXERCISE_STAIR_CLIMBING -> X Change-Id: I43290e81ad9175bc2c63b46db7d6c98dcea62046 Signed-off-by: kibak.yoon --- src/hal/sensor_hal_types.h | 8 ++++---- src/shared/sensor_types.h | 7 ++++--- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/src/hal/sensor_hal_types.h b/src/hal/sensor_hal_types.h index 9b7ccde..ffa96f1 100644 --- a/src/hal/sensor_hal_types.h +++ b/src/hal/sensor_hal_types.h @@ -78,7 +78,6 @@ typedef enum { SENSOR_DEVICE_EXERCISE_RUNNING, SENSOR_DEVICE_EXERCISE_HIKING, SENSOR_DEVICE_EXERCISE_CYCLING, - SENSOR_DEVICE_EXERCISE_STAIR_CLIMBING, SENSOR_DEVICE_EXERCISE_ELLIPTICAL, SENSOR_DEVICE_EXERCISE_INDOOR_CYCLING, SENSOR_DEVICE_EXERCISE_ROWING, @@ -97,7 +96,6 @@ typedef enum { SENSOR_DEVICE_ACTIVITY_TRACKER, SENSOR_DEVICE_HRM_CTRL = 0x1A80, - SENSOR_DEVICE_EXERCISE_COACH, SENSOR_DEVICE_ACTIVITY_LEVEL_MONITOR, SENSOR_DEVICE_WEAR_STATUS = 0x2000, @@ -105,7 +103,7 @@ typedef enum { SENSOR_DEVICE_NO_MOVE_DETECTOR, SENSOR_DEVICE_RESTING_HR, SENSOR_DEVICE_STEP_LEVEL_MONITOR, - SENSOR_DEVICE_EXERCISE, + SENSOR_DEVICE_EXERCISE_STANDALONE, SENSOR_DEVICE_EXERCISE_HR, SENSOR_DEVICE_WORKOUT, SENSOR_DEVICE_CYCLE_MONITOR, @@ -219,8 +217,10 @@ typedef struct { } diffs[SENSOR_PEDOMETER_DATA_DIFFS_SIZE]; } sensor_pedometer_data_t; +#define CONVERT_TYPE_ATTR(type, index) ((type) << 8 | 0x80 | (index)) + enum sensor_attribute { - SENSOR_ATTR_ACTIVITY = 0x100, + SENSOR_ATTR_ACTIVITY = CONVERT_TYPE_ATTR(SENSOR_DEVICE_ACTIVITY_TRACKER, 0x1), }; enum sensor_activity { diff --git a/src/shared/sensor_types.h b/src/shared/sensor_types.h index 5c890b9..0612f72 100644 --- a/src/shared/sensor_types.h +++ b/src/shared/sensor_types.h @@ -61,7 +61,6 @@ extern "C" DEF_SENSOR(EXERCISE_RUNNING_SENSOR) \ DEF_SENSOR(EXERCISE_HIKING_SENSOR) \ DEF_SENSOR(EXERCISE_CYCLING_SENSOR) \ - DEF_SENSOR(EXERCISE_STAIR_CLIMBING_SENSOR) \ DEF_SENSOR(EXERCISE_ELLIPTICAL_SENSOR) \ DEF_SENSOR(EXERCISE_INDOOR_CYCLING_SENSOR) \ DEF_SENSOR(EXERCISE_ROWING_SENSOR) \ @@ -78,9 +77,9 @@ extern "C" \ DEF_SENSOR_VALUE(GPS_BATCH_SENSOR, 0x1A00) \ DEF_SENSOR(ACTIVITY_TRACKER_SENSOR) \ + DEF_SENSOR(EXERCISE_COMPANION_SENSOR) \ \ DEF_SENSOR_VALUE(HRM_CTRL_SENSOR, 0x1A80) \ - DEF_SENSOR(EXERCISE_COACH_SENSOR) \ DEF_SENSOR(ACTIVITY_LEVEL_MONITOR_SENSOR) \ \ DEF_SENSOR_VALUE(WEAR_STATUS_SENSOR, 0x2000) \ @@ -88,7 +87,7 @@ extern "C" DEF_SENSOR(NO_MOVE_DETECTOR_SENSOR) \ DEF_SENSOR(RESTING_HR_SENSOR) \ DEF_SENSOR(STEP_LEVEL_MONITOR_SENSOR) \ - DEF_SENSOR(EXERCISE_SENSOR) \ + DEF_SENSOR(EXERCISE_STANDALONE_SENSOR) \ DEF_SENSOR(EXERCISE_HR_SENSOR) \ DEF_SENSOR(WORKOUT_SENSOR) \ DEF_SENSOR(CYCLE_MONITOR_SENSOR) \ @@ -114,6 +113,8 @@ extern "C" #define SLEEP_DETECTOR_SENSOR HUMAN_SLEEP_DETECTOR_SENSOR #define STRESS_MONITOR_SENSOR HUMAN_STRESS_MONITOR_SENSOR #define AUTOSESSION_EXERCISE_SENSOR WORKOUT_SENSOR +#define EXERCISE_COACH_SENSOR EXERCISE_STANDALONE_SENSOR +#define EXERCISE_SENSOR EXERCISE_COMPANION_SENSOR DECLARE_SENSOR_ENUM(sensor_type_t, SENSOR_TYPE) -- 2.7.4 From fda09ec118138917b72afad8ab21f12988ce33e9 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Wed, 22 Jun 2016 19:07:50 +0900 Subject: [PATCH 15/16] sensord: fix the bug which converts wrong pause policy Change-Id: I655ccde45a65637672fb9c23141e66b32d34c2c7 Signed-off-by: kibak.yoon --- src/client/client.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/client/client.cpp b/src/client/client.cpp index b900db4..129958c 100644 --- a/src/client/client.cpp +++ b/src/client/client.cpp @@ -44,9 +44,7 @@ using std::vector; #define DEFAULT_INTERVAL POLL_10HZ_MS -#define CONVERT_OPTION_PAUSE_POLICY(option) \ - (option == SENSOR_OPTION_DEFAULT || option == SENSOR_OPTION_ALWAYS_ON) ? \ - (option ^ 0b11) : option +#define CONVERT_OPTION_PAUSE_POLICY(option) ((option) ^ 0b11) static cmutex lock; -- 2.7.4 From a0b71120db332e7ae23da7ebb3dd1f2c8db2fa91 Mon Sep 17 00:00:00 2001 From: "kibak.yoon" Date: Mon, 27 Jun 2016 13:27:25 +0900 Subject: [PATCH 16/16] sensord: clean up switch-case statement if attribute is not global. - if it is a default attribute in switch-case statement, break and call set_attribute_int() function. Change-Id: I1bf0403ad268a8465190439acd9c1ebe320cda2c Signed-off-by: kibak.yoon --- src/client/client.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/client/client.cpp b/src/client/client.cpp index 129958c..171cb88 100644 --- a/src/client/client.cpp +++ b/src/client/client.cpp @@ -1129,10 +1129,10 @@ API int sensord_set_attribute_int(int handle, int attribute, int value) case SENSORD_ATTRIBUTE_AXIS_ORIENTATION: return change_axis_orientation(handle, value); default: - return change_attribute_int(handle, attribute, value); + break; } - return OP_SUCCESS; + return change_attribute_int(handle, attribute, value); } API int sensord_set_attribute_str(int handle, int attribute, const char *value, int value_len) -- 2.7.4