anierin/msm-5.15
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

msm: adsprpc: move message for max fastrpc sessions outside mutex

Browse Source 
Move message for max fastrpc sessions outside channel mutex
to avoid holding mutex for a long period of time which can
cause kernel panic due to SSR holding same mutex.
Change message level to warning.

Change-Id: I354c15d2024d32774a729846f790f75082b4a0b5
Signed-off-by: Edgar Flores <edgarf@codeaurora.org>
This commit is contained in:
Edgar Flores
2021-09-17 17:51:20 -07:00
committed by Gerrit - the friendly Code Review server
parent 2943785dda
commit bc037aaa54
1 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
drivers/char/adsprpc.c
View File

@@ -5201,9 +5201,6 @@ static int fastrpc_session_alloc_locked(struct fastrpc_channel_ctx *chan,
}
if (idx >= chan->sesscount) {
err = -EUSERS;
ADSPRPC_ERR(
"max concurrent sessions limit (%d) already reached on %s err %d\n",
chan->sesscount, chan->subsys, err);
goto bail;
}
chan->session[idx].smmu.faults = 0;
@@ -5411,6 +5408,11 @@ static int fastrpc_session_alloc(struct fastrpc_channel_ctx *chan, int secure,
if (!*session)
err = fastrpc_session_alloc_locked(chan, secure, session);
mutex_unlock(&chan->smd_mutex);
if (err == -EUSERS) {
ADSPRPC_WARN(
"max concurrent sessions limit (%d) already reached on %s err %d\n",
chan->sesscount, chan->subsys, err);
}
return err;
}
@@ -6067,6 +6069,11 @@ static int fastrpc_get_info(struct fastrpc_file *fl, uint32_t *info)
err = fastrpc_session_alloc_locked(&fl->apps->channel[cid],
0, &fl->sctx);
mutex_unlock(&fl->apps->channel[cid].smd_mutex);
if (err == -EUSERS) {
ADSPRPC_WARN(
"max concurrent sessions limit (%d) already reached on %s err %d\n",
chan->sesscount, chan->subsys, err);
}
if (err)
goto bail;
}