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

Commit 105012c5 authored by Sivan Reinstein's avatar Sivan Reinstein
Browse files

msm: ipa: set SSR TAG process timeout to 150msec



According to experiments, 20msecs timeout is insufficient
for TAG process. This change increases the timeout in
a SSR scenario to 150msec.

Change-Id: Ie5300d0fef5a3d5e4f772fe9ca70a1212c79ad46
Acked-by: default avatarDavid Arinzon <darinzon@qti.qualcomm.com>
Signed-off-by: default avatarSivan Reinsten <sivanr@codeaurora.org>
parent 18c43109
Loading
Loading
Loading
Loading
+3 −7
Original line number Diff line number Diff line
/* Copyright (c) 2012-2014, The Linux Foundation. All rights reserved.
/* Copyright (c) 2012-2015, The Linux Foundation. All rights reserved.
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License version 2 and
@@ -48,7 +48,7 @@

#define IPA_AGGR_MAX_STR_LENGTH (10)

#define CLEANUP_TAG_PROCESS_TIMEOUT 20
#define CLEANUP_TAG_PROCESS_TIMEOUT 150

#define IPA_AGGR_STR_IN_BYTES(str) \
	(strnlen((str), IPA_AGGR_MAX_STR_LENGTH - 1) + 1)
@@ -1486,7 +1486,7 @@ static int ipa_q6_set_ex_path_dis_agg(void)
		}
	}

	/* Will wait 20msecs for IPA tag process completion */
	/* Will wait 150msecs for IPA tag process completion */
	retval = ipa_tag_process(desc, num_descs,
				 msecs_to_jiffies(CLEANUP_TAG_PROCESS_TIMEOUT));
	if (retval) {
@@ -1497,10 +1497,6 @@ static int ipa_q6_set_ex_path_dis_agg(void)
				kfree(desc[index].user1);
			retval = -EINVAL;
		}

		/* Ignoring TAG process timeout */
		if (retval == -ETIME)
			retval = 0;
	}

	kfree(desc);
+2 −1
Original line number Diff line number Diff line
@@ -4230,7 +4230,8 @@ int ipa_tag_process(struct ipa_desc desc[],
	IPADBG("waiting for TAG response\n");
	res = wait_for_completion_timeout(&comp->comp, timeout);
	if (res == 0) {
		IPAERR("timeout for waiting for TAG response\n");
		IPAERR("timeout (%lu msec) on waiting for TAG response\n",
			timeout);
		WARN_ON(1);
		if (atomic_dec_return(&comp->cnt) == 0)
			kfree(comp);