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

Commit fb66866b authored by Mohammed's avatar Mohammed Committed by Gerrit - the friendly Code Review server
Browse files

msm: ipa3: Reduced excessive logs from ipa 



Currently watchdog bark issue occurs due to
excessive logging from ipa. Ratelimit functions
are added inorder to reduce the logs.

Change-Id: I13c1ee235ea22dfd2d9473346e44a2c9a1d149ea
Acked-by: default avatarPooja Kumari <kumarip@qti.qualcomm.com>
Acked-by: default avatarPraveen Kurapati <pkurapat@qti.qualcomm.com>
Signed-off-by: default avatarMohammed Javid <mjavid@codeaurora.org>
parent edae206c

drivers/platform/msm/ipa/ipa_v2/ipa_qmi_service.h

+10 −0
Original line number Diff line number Diff line
@@ -55,6 +55,16 @@ 
			DEV_NAME " %s:%d " fmt, ## args); \

	} while (0)



#define IPAWANERR_RL(fmt, args...) \

	do { \

		pr_err_ratelimited_ipa(DEV_NAME " %s:%d " fmt, __func__,\

			__LINE__, ## args); \

		IPA_IPC_LOGGING(ipa_get_ipc_logbuf(), \

			DEV_NAME " %s:%d " fmt, ## args); \

		IPA_IPC_LOGGING(ipa_get_ipc_logbuf_low(), \

			DEV_NAME " %s:%d " fmt, ## args); \

	} while (0)



#define IPAWANINFO(fmt, args...) \

	do { \

		pr_info(DEV_NAME " %s:%d " fmt, __func__, __LINE__, ## args); \

drivers/platform/msm/ipa/ipa_v2/ipa_utils.c

+1 −1
Original line number Diff line number Diff line
@@ -903,7 +903,7 @@ int ipa2_get_wlan_stats(struct ipa_get_wdi_sap_stats *wdi_sap_stats) 
		ipa_ctx->uc_wdi_ctx.stats_notify(IPA_GET_WDI_SAP_STATS,

			wdi_sap_stats);

	} else {

		IPAERR("uc_wdi_ctx.stats_notify not registered\n");

		IPAERR_RL("uc_wdi_ctx.stats_notify not registered\n");

		return -EFAULT;

	}

	return 0;

drivers/platform/msm/ipa/ipa_v2/rmnet_ipa.c

+2 −1
Original line number Diff line number Diff line
@@ -3055,7 +3055,8 @@ int rmnet_ipa_query_tethering_stats_all( 
		rc = rmnet_ipa_query_tethering_stats_wifi(

			&tether_stats, data->reset_stats);

		if (rc) {

			IPAWANERR("wlan WAN_IOC_QUERY_TETHER_STATS failed\n");

			IPAWANERR_RL(

				"wlan WAN_IOC_QUERY_TETHER_STATS failed\n");

			return rc;

		}

		data->tx_bytes = tether_stats.ipv4_tx_bytes

drivers/platform/msm/ipa/ipa_v2/rmnet_ipa_fd_ioctl.c

+2 −2
Original line number Diff line number Diff line 
/* Copyright (c) 2013-2015, 2017, The Linux Foundation. All rights reserved.

/* Copyright (c) 2013-2015, 2018, 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
@@ -162,7 +162,7 @@ static long wan_ioctl(struct file *filp, unsigned int cmd, unsigned long arg) 
		}

		if (rmnet_ipa_poll_tethering_stats(

		(struct wan_ioctl_poll_tethering_stats *)param)) {

			IPAWANERR("WAN_IOCTL_POLL_TETHERING_STATS failed\n");

			IPAWANERR_RL("WAN_IOCTL_POLL_TETHERING_STATS failed\n");

			retval = -EFAULT;

			break;

		}

drivers/platform/msm/ipa/ipa_v3/ipa_qmi_service.h

+10 −0
Original line number Diff line number Diff line
@@ -59,6 +59,16 @@ 
				DEV_NAME " %s:%d " fmt, ## args); \

	} while (0)



#define IPAWANERR_RL(fmt, args...) \

	do { \

		pr_err_ratelimited_ipa(DEV_NAME " %s:%d " fmt, __func__,\

				__LINE__, ## args); \

		IPA_IPC_LOGGING(ipa_get_ipc_logbuf(), \

				DEV_NAME " %s:%d " fmt, ## args); \

		IPA_IPC_LOGGING(ipa_get_ipc_logbuf_low(), \

				DEV_NAME " %s:%d " fmt, ## args); \

	} while (0)



#define IPAWANINFO(fmt, args...) \

	do { \

		pr_info(DEV_NAME " %s:%d " fmt, __func__,\