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

Commit 23e2fc20 authored by Ajay Agarwal's avatar Ajay Agarwal
Browse files

net: usbnet: Add module param to toggle LTE IPC logs 



IPC logging for LTE functions (RmNet/ECM) is hogging the CPU and
degrading the throughput. Add module parameter 'debug_mask' to
control these logs. By default, the value of ‘debug_mask’ is 0
which means no packets will be logged.

To enable all logs:
 echo 2 > /sys/module/usbnet/parameters/debug_mask

To enable only cv2x log:
 echo 1 > /sys/module/usbnet/parameters/debug_mask

To disable all logs (default setting on boot):
 echo 0 > /sys/module/usbnet/parameters/debug_mask

Change-Id: Id9497c95cbc8c76d0167cd858f7a36b6af12644d
Signed-off-by: default avatarAjay Agarwal <ajaya@codeaurora.org>
parent 179311b7

drivers/net/usb/usbnet.c

+14 −8
Original line number Diff line number Diff line
@@ -49,10 +49,6 @@ 


#define DRIVER_VERSION		"22-Aug-2005"



#define dbg_log_string(fmt, ...) \

	ipc_log_string(dev->ipc_log_ctxt,\

			"%s: " fmt, __func__, ##__VA_ARGS__)



/*-------------------------------------------------------------------------*/



/*
@@ -122,6 +118,17 @@ static int name_to_netdev_id(char *name) 
	return -EINVAL;

}



static int debug_mask;

module_param(debug_mask, int, 0644);

MODULE_PARM_DESC(debug_mask, "Control data packet IPC logging");



#define dbg_log_string(fmt, ...) do { \

if ((dev->netdev_id == USBNET_RMNET_USB1 && debug_mask == 1) || \

					debug_mask == 2) \

	ipc_log_string(dev->ipc_log_ctxt, "%s: " fmt, \

		       __func__, ##__VA_ARGS__); \

} while (0)



/*-------------------------------------------------------------------------*/



/* handles CDC Ethernet and many other network "bulk data" interfaces */
@@ -1746,7 +1753,6 @@ usbnet_probe (struct usb_interface *udev, const struct usb_device_id *prod) 
	int				status;

	const char			*name;

	struct usb_driver 	*driver = to_usb_driver(udev->dev.driver);

	int				netdev_id;



	/* usbnet already took usb runtime pm, so have to enable the feature

	 * for usb interface, otherwise usb_autopm_get_interface may return
@@ -1908,9 +1914,9 @@ usbnet_probe (struct usb_interface *udev, const struct usb_device_id *prod) 
	if (dev->driver_info->flags & FLAG_LINK_INTR)

		usbnet_link_change(dev, 0, 0);



	netdev_id = name_to_netdev_id(dev->net->name);

	if (netdev_id >= 0)

		dev->ipc_log_ctxt = usbnet_ipc_log_ctxt[netdev_id];

	dev->netdev_id = name_to_netdev_id(dev->net->name);

	if (dev->netdev_id >= 0)

		dev->ipc_log_ctxt = usbnet_ipc_log_ctxt[dev->netdev_id];



	return 0;

include/linux/usb/usbnet.h

+1 −0
Original line number Diff line number Diff line
@@ -86,6 +86,7 @@ struct usbnet { 
#		define EVENT_NO_IP_ALIGN	13



	void			*ipc_log_ctxt;

	int			netdev_id;

};



static inline struct usb_driver *driver_of(struct usb_interface *intf)