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

Commit 394844d6 authored by Tharun Kumar Merugu's avatar Tharun Kumar Merugu
Browse files

msm: adsprpc: print process kill failure only when subsystem is up



When trying to release process on DSP, print failure message only
when the subsystem is up, to avoid flooding of kernel logs for
daemons.

Change-Id: I8de5f68fba93505c9d4db65b13410ee9e16ba93e
Acked-by: default avatarThyagarajan Venkatanarayanan <venkatan@qti.qualcomm.com>
Signed-off-by: default avatarTharun Kumar Merugu <mtharu@codeaurora.org>
parent be77303e
Loading
Loading
Loading
Loading
+5 −5
Original line number Diff line number Diff line
@@ -2228,6 +2228,9 @@ static int fastrpc_release_current_dsp_process(struct fastrpc_file *fl)
	ioctl.crc = NULL;
	VERIFY(err, 0 == (err = fastrpc_internal_invoke(fl,
		FASTRPC_MODE_PARALLEL, 1, &ioctl)));
	if (err)
		pr_err("adsprpc: %s: releasing DSP process failed for %s, returned 0x%x",
					__func__, current->comm, err);
bail:
	return err;
}
@@ -2789,16 +2792,13 @@ static int fastrpc_file_free(struct fastrpc_file *fl)
	struct hlist_node *n = NULL;
	struct fastrpc_mmap *map = NULL, *lmap = NULL;
	struct fastrpc_perf *perf = NULL, *fperf = NULL;
	int cid, err = 0;
	int cid;

	if (!fl)
		return 0;
	cid = fl->cid;

	err = fastrpc_release_current_dsp_process(fl);
	if (err)
		pr_err("adsprpc: %s: releasing DSP process failed for %s, returned 0x%x",
					__func__, current->comm, err);
	(void)fastrpc_release_current_dsp_process(fl);

	spin_lock(&fl->apps->hlock);
	hlist_del_init(&fl->hn);