Merge "st-hal: fix session back to active due to playback event"
diff --git a/st_session.c b/st_session.c
index 0ccdeaf..9b6d5c6 100644
--- a/st_session.c
+++ b/st_session.c
@@ -2646,7 +2646,12 @@
st_ses->hw_session_started = true;
} else {
ALOGE("%s:[%d] failed to restart", __func__, st_ses->sm_handle);
- st_ses->hw_session_started = false;
+ /*
+ * lower layers like gcs/lsm need to handle double stop calls properly
+ * to avoid possible crash, as some of the clean ups are already issued
+ * during fptrs->restart() when it's failed.
+ */
+ stop_hw_session(st_ses, hw_ses, true);
}
return status;
@@ -2784,7 +2789,7 @@
return 0;
}
-static inline int prepapre_second_stage_for_client(st_session_t *stc_ses)
+static inline int prepare_second_stage_for_client(st_session_t *stc_ses)
{
struct listnode *node = NULL;
st_arm_second_stage_t *st_sec_stage = NULL;
@@ -4591,6 +4596,11 @@
STATE_TRANSITION(st_ses, loaded_state_fn);
break;
+ case ST_SES_EV_RESUME:
+ if (stc_ses->paused == true)
+ stc_ses->paused = false;
+ break;
+
case ST_SES_EV_STOP:
status = stop_session(st_ses, hw_ses, false);
if (status)
@@ -5260,6 +5270,7 @@
hw_ses->fptrs->stop_buffering(hw_ses);
if (hw_ses->sthw_cfg_updated || ev->ev_id == ST_SES_EV_START) {
status = stop_session(st_ses, hw_ses, false);
+ STATE_TRANSITION(st_ses, loaded_state_fn);
if (status) {
ALOGE("%s:[%d] failed to stop session, err %d", __func__,
st_ses->sm_handle, status);
@@ -5467,7 +5478,6 @@
status = -EINVAL;
break;
}
- prepapre_second_stage_for_client(stc_ses);
stc_ses->state = ST_STATE_LOADED;
} else {
ALOGE("%s: received unexpected event, client state = %d",
@@ -5481,7 +5491,6 @@
if (status)
ALOGE("%s:[c%d] update sound_model failed %d", __func__,
stc_ses->sm_handle, status);
- stop_second_stage_for_client(stc_ses);
stc_ses->state = ST_STATE_IDLE;
} else {
ALOGE("%s: received unexpected event, client state = %d",
@@ -5574,7 +5583,7 @@
pthread_mutex_lock(&st_ses->lock);
DISPATCH_EVENT(st_ses, ev, status);
if (!status) {
- prepapre_second_stage_for_client(stc_ses);
+ prepare_second_stage_for_client(stc_ses);
stc_ses->state = ST_STATE_LOADED;
}
pthread_mutex_unlock(&st_ses->lock);