GMutex resources_lock;
+ gboolean is_release_marked[MM_RESOURCE_MANAGER_RES_TYPE_MAX];
+
mm_resource_manager_res_volume __max_resource_volumes
[MM_RESOURCE_MANAGER_RES_TYPE_MAX];
mm_resource_manager_res_volume __condition_volumes
mm_resource_manager_res_p res);
/* FALSE if resource is destroyed */
static gboolean __mark_resource_for_release(GPtrArray *resources, int index, mm_resource_manager_res_p resource);
-static void __send_release_cb_sync(mm_resource_manager_id id);
+static int __send_release_cb_sync(mm_resource_manager_id id);
static void __mm_resource_manager_release_callback(mm_resource_manager_s *handle,
mm_resource_manager_id id, mm_resource_manager_res_type_e type, mm_resource_manager_res_volume volume);
static void __mm_resource_manager_status_callback(mm_resource_manager_s *handle, mm_resource_manager_status_e status);
resource->type >= MM_RESOURCE_MANAGER_RES_TYPE_MAX,
MM_RESOURCE_MANAGER_ERROR_INVALID_OPERATION, "Type [%d] is out of range", resource->type);
MM_RM_RETVM_IF(resource->state < MM_RESOURCE_MANAGER_RES_STATE_FOR_ACQUIRE ||
- resource->state > MM_RESOURCE_MANAGER_RES_STATE_FOR_RELEASE,
- MM_RESOURCE_MANAGER_ERROR_INVALID_OPERATION, "State [%d] is out of range", resource->state);
+ resource->state >= MM_RESOURCE_MANAGER_RES_STATE_FOR_RELEASE,
+ MM_RESOURCE_MANAGER_ERROR_INVALID_OPERATION, "State [%d] is out of range or already released", resource->state);
res_name = _mm_resource_manager_get_res_str(resource->type);
return MM_RESOURCE_MANAGER_ERROR_INVALID_OPERATION;
}
+ handle->is_release_marked[resource->type] = TRUE;
MM_RM_INFO("(%s) is marked for release in RM #%"PRIu64, res_name, handle_id);
__mm_resources_lock(handle);
return MM_RESOURCE_MANAGER_ERROR_NONE;
}
+int _mm_resource_manager_release_other_resources(mm_resource_manager_s *handle)
+{
+ int idx;
+ int ret;
+ int len;
+
+ MM_RM_INFO("Enter %p", handle);
+
+ __mm_resource_handles_lock();
+ MM_RESOURCE_MANAGER_CHECK(handle);
+ __mm_resources_lock(handle);
+ handle->release_cb.is_invoked = TRUE;
+ MM_RM_INFO("other resource is released RM #%"PRIu64, _mm_rm_hash64(handle->id));
+ __mm_resource_handles_unlock();
+
+ len = handle->resources->len;
+
+ for (idx = 0; idx < len; idx++) {
+ if (!__mark_resource_for_release(handle->resources, idx, (mm_resource_manager_res_p) handle->resources->pdata[idx]))
+ MM_RM_INFO("[idx %d] resource length %d", idx, len);
+ }
+
+ MM_RM_INFO("[RELEASE] dbus_commit");
+ ret = __dbus_commit(handle);
+ if (ret == MM_RESOURCE_MANAGER_ERROR_NONE)
+ MM_RM_DEBUG("Changes in RM #%"PRIu64" have been committed successfully", _mm_rm_hash64(handle->id));
+ else
+ MM_RM_ERROR("Dbus commit request failed");
+
+ __mm_resources_unlock(handle);
+
+ MM_RM_INFO("All resources are marked for release in RM #%"PRIu64, _mm_rm_hash64(handle->id));
+
+ return ret;
+}
+
int _mm_resource_manager_mark_all_for_release(mm_resource_manager_h rm)
{
mm_resource_manager_s *handle = MM_RESOURCE_MANAGER(rm);
len = handle->resources->len;
for (idx = 0; idx < len; idx++) {
- if (!__mark_resource_for_release(handle->resources, idx, (mm_resource_manager_res_p) handle->resources->pdata[idx]))
- MM_RM_INFO("[idx %d] resource length %d", idx, len);
+ if (!__mark_resource_for_release(handle->resources, idx, (mm_resource_manager_res_p) handle->resources->pdata[idx])) {
+ MM_RM_WARNING("[idx %d] resource is already marked or released", idx);
+ __mm_resources_unlock(handle);
+ return MM_RESOURCE_MANAGER_ERROR_INVALID_OPERATION;
+ }
+ handle->is_release_marked[idx] = TRUE;
}
MM_RM_INFO("All resources are marked for release in RM #%"PRIu64, _mm_rm_hash64(handle->id));
__mm_resources_lock(handle);
__mm_resource_handles_unlock();
+ MM_RM_INFO("dbus_commit RM #%"PRIu64, _mm_rm_hash64(handle->id));
ret = __dbus_commit(handle);
if (ret == MM_RESOURCE_MANAGER_ERROR_NONE)
MM_RM_DEBUG("Changes in RM #%"PRIu64" have been committed successfully", _mm_rm_hash64(handle->id));
{
mm_resource_manager_res_s *resource;
mm_resource_manager_id handle_id;
+ mm_resource_manager_res_type_e t;
gboolean release_all = FALSE;
int idx;
int prev_len;
return;
}
+ for (t = MM_RESOURCE_MANAGER_RES_TYPE_VIDEO_DECODER; t < MM_RESOURCE_MANAGER_RES_TYPE_MAX; t++) {
+ if (handle->is_release_marked[t] && t == type) {
+ MM_RM_WARNING("mark_for_release() of %s is executed, which means that we can sync with rm dmn right now RM #%"PRIu64,
+ res_name, handle_id);
+ if (__send_release_cb_sync(handle->id) != MM_RESOURCE_MANAGER_ERROR_NONE)
+ MM_RM_ERROR("__send_release_cb_sync is failed");
+ __mm_resource_handles_unlock();
+ return;
+ }
+ }
+
__mm_resources_lock(handle);
__mm_resource_handles_unlock();
prev_len = handle->resources->len;
MM_RM_DEBUG("resource %p length %d", handle->resources, prev_len);
for (idx = 0; idx < prev_len; idx++) {
- resource = (mm_resource_manager_res_s*)handle->resources->pdata[idx];
+ resource = (mm_resource_manager_res_s *)handle->resources->pdata[idx];
if (resource->type == type && resource->volume == volume) {
/* FIXME : Set true in advance release callback invoking to prevent deadlock with resource marking */
handle->release_cb.is_invoked = TRUE;
release_all = ((mm_resource_manager_release_cb)handle->release_cb.cb)(handle, resource, handle->release_cb.user_data);
MM_RM_INFO("[%d] release_cb is completed", release_all);
- __send_release_cb_sync(handle->id);
+ if (__send_release_cb_sync(handle->id) != MM_RESOURCE_MANAGER_ERROR_NONE) {
+ MM_RM_ERROR("__send_release_cb_sync is failed");
+ handle->release_cb.is_invoked = FALSE;
+ break;
+ }
+
+ /* If there is only one resource, the release callback value must be reset considering return value in case the handle is reused. */
+ if (!release_all)
+ handle->release_cb.is_invoked = FALSE;
g_ptr_array_remove_index_fast(handle->resources, idx);
break;
}
}
-
- handle->release_cb.is_invoked = FALSE;
- MM_RM_WARNING("Reset release_cb release_cb.is_invoked RM #%"PRIu64, handle_id);
-
__mm_resources_unlock(handle);
+ MM_RM_DEBUG("[%d] RELEASE ALL", release_all);
+
if (release_all) {
- if (_mm_resource_manager_mark_all_for_release(handle) == MM_RESOURCE_MANAGER_ERROR_NONE &&
- _mm_resource_manager_commit(handle) == MM_RESOURCE_MANAGER_ERROR_NONE)
+ if (_mm_resource_manager_release_other_resources(handle) == MM_RESOURCE_MANAGER_ERROR_NONE) {
MM_RM_DEBUG("All resources are released after release cb");
- else
+ handle->release_cb.is_invoked = FALSE;
+ MM_RM_WARNING("Reset release_cb release_cb.is_invoked RM #%"PRIu64, handle_id);
+ } else {
MM_RM_WARNING("Resources cannot be released after release cb");
+ }
}
}
static void __mm_resources_lock(mm_resource_manager_s *h)
{
MM_RM_RETM_IF(!h, "handle is NULL");
+ LOGD(">> resource lock");
g_mutex_lock(&h->resources_lock);
}
{
MM_RM_RETM_IF(!h, "handle is NULL");
g_mutex_unlock(&h->resources_lock);
+ LOGD("<< resource unlock");
}
static int __check_resource(mm_resource_manager_s *rm, mm_resource_manager_res_type_e type, mm_resource_manager_res_volume volume)
MM_RM_DEBUG("Resource %p is successfully marked", resource);
break;
case MM_RESOURCE_MANAGER_RES_STATE_FOR_RELEASE:
- MM_RM_DEBUG("Resource %p is already marked", resource);
- break;
+ MM_RM_DEBUG("Resource %p is already marked or released", resource);
+ return FALSE;
}
return TRUE;
return FALSE;
}
-static void __send_release_cb_sync(mm_resource_manager_id id)
+static int __send_release_cb_sync(mm_resource_manager_id id)
{
+ int ret = MM_RESOURCE_MANAGER_ERROR_NONE;
int sync_fd;
MM_RM_DEBUG("Enter");
sync_fd = open(RELEASE_CB_SYNC_PATH, O_WRONLY | O_NONBLOCK);
MM_RM_DEBUG("[%d] opened %s", sync_fd, RELEASE_CB_SYNC_PATH);
- MM_RM_RETM_IF(sync_fd == -1, "Sync FIFO cannot be opened [errno %d]", errno);
+ MM_RM_RETVM_IF(sync_fd == -1, MM_RESOURCE_MANAGER_ERROR_INVALID_OPERATION,
+ "Sync FIFO cannot be opened [errno %d]", errno);
- if (write(sync_fd, &id, sizeof(id)) == sizeof(id))
+ if (write(sync_fd, &id, sizeof(id)) == sizeof(id)) {
MM_RM_INFO("[SYNC] message is sent successfully RM #%"PRIu64, _mm_rm_hash64(id));
- else
+ } else {
MM_RM_ERROR("[SYNC] message cannot be sent RM #%"PRIu64, _mm_rm_hash64(id));
+ ret = MM_RESOURCE_MANAGER_ERROR_INVALID_OPERATION;
+ }
- close(sync_fd);
+ MM_RM_RETVM_IF(close(sync_fd) == -1, MM_RESOURCE_MANAGER_ERROR_INVALID_OPERATION,
+ "[%d] close failed [errno %d]", sync_fd, errno);
MM_RM_DEBUG("[%d] closed", sync_fd);
+
+ return ret;
}
static void __init_lib()
break;
case MM_RESOURCE_MANAGER_RES_STATE_FOR_RELEASE:
g_ptr_array_remove_index_fast(handle->resources, i--);
+ handle->is_release_marked[resource->type] = FALSE;
break;
default:
break;
__mm_resource_handles_lock();
for (i = 0; i < handles->len; i++) {
- handle = (mm_resource_manager_s*)handles->pdata[i];
+ handle = (mm_resource_manager_s *)handles->pdata[i];
handle_id = handle->id;
MM_RM_HASH64(handle_id);
if (handle->dbus_proxy == object && handle_id == arg_id) {
+ if (handle->release_cb.is_invoked) {
+ MM_RM_WARNING("other resource release cb is already executed RM #%"PRIu64, handle_id);
+ if (__send_release_cb_sync(handle->id) != MM_RESOURCE_MANAGER_ERROR_NONE)
+ MM_RM_ERROR("__send_release_cb_sync is failed");
+ __mm_resource_handles_unlock();
+ return;
+ }
+ MM_RM_INFO("[release_callback] RM #%"PRIu64, handle_id);
__mm_resource_manager_release_callback(handle, arg_id, arg_resource_type, arg_volume);
return;
}
__mm_resource_handles_lock();
for (i = 0; i < handles->len; i++) {
- handle = (mm_resource_manager_s*)handles->pdata[i];
+ handle = (mm_resource_manager_s *)handles->pdata[i];
if (handle->dbus_proxy == object) {
__mm_resource_manager_status_callback(handle, arg_status);
return;