Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b68aa9b3 authored by Pradosh Das's avatar Pradosh Das
Browse files

Revert "msm: adsprpc: Lock list before removing node" 



This reverts commit e1ec022f.

Change-Id: I844b624990b552baaa959c5ea053d29e2dfae26c
Signed-off-by: default avatarPradosh Das <prados@codeaurora.org>
parent e1ec022f

drivers/char/adsprpc.c

+6 −2
Original line number Diff line number Diff line
@@ -3198,13 +3198,17 @@ static int fastrpc_mmap_remove_ssr(struct fastrpc_file *fl) 
					kfree(ramdump_segments_rh);

				}

			}

			mutex_lock(&fl->map_mutex);

			fastrpc_mmap_free(match, 0);

			mutex_unlock(&fl->map_mutex);

		}

	} while (match);

	me->enable_ramdump = false;

bail:

	if (err && match) {

		mutex_lock(&fl->map_mutex);

		fastrpc_mmap_add(match);

		mutex_unlock(&fl->map_mutex);

	}

	return err;

}
@@ -4014,13 +4018,13 @@ static int fastrpc_channel_open(struct fastrpc_file *fl) 


	if (cid == ADSP_DOMAIN_ID && me->channel[cid].ssrcount !=

			 me->channel[cid].prevssrcount) {

		mutex_lock(&fl->map_mutex);

		mutex_unlock(&me->channel[cid].smd_mutex);

		err = fastrpc_mmap_remove_ssr(fl);

		if (err)

			pr_warn("adsprpc: %s: %s: failed to unmap remote heap for %s (err %d)\n",

					__func__, current->comm,

					me->channel[cid].subsys, err);

		mutex_unlock(&fl->map_mutex);

		mutex_lock(&me->channel[cid].smd_mutex);

		me->channel[cid].prevssrcount =

					me->channel[cid].ssrcount;

	}