Commit a4e61de6 authored by Ekansh Gupta's avatar Ekansh Gupta Committed by Greg Kroah-Hartman
Browse files

misc: fastrpc: Mark all sessions as invalid in cb_remove 



In remoteproc shutdown sequence, rpmsg_remove will get called which
would depopulate all the child nodes that have been created during
rpmsg_probe. This would result in cb_remove call for all the context
banks for the remoteproc. In cb_remove function, session 0 is
getting skipped which is not correct as session 0 will never become
available again. Add changes to mark session 0 also as invalid.

Fixes: f6f9279f ("misc: fastrpc: Add Qualcomm fastrpc basic driver model")
Cc: stable <stable@kernel.org>
Signed-off-by: default avatarEkansh Gupta <quic_ekangupt@quicinc.com>
Link: https://lore.kernel.org/r/20240108114833.20480-1-quic_ekangupt@quicinc.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 41bccc98

drivers/misc/fastrpc.c

+1 −1
Original line number Diff line number Diff line
@@ -2191,7 +2191,7 @@ static int fastrpc_cb_remove(struct platform_device *pdev) 
	int i;



	spin_lock_irqsave(&cctx->lock, flags);

	for (i = 1; i < FASTRPC_MAX_SESSIONS; i++) {

	for (i = 0; i < FASTRPC_MAX_SESSIONS; i++) {

		if (cctx->session[i].sid == sess->sid) {

			cctx->session[i].valid = false;

			cctx->sesscount--;