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

Commit 33b2f559 authored by sjur.brandeland@stericsson.com's avatar sjur.brandeland@stericsson.com Committed by David S. Miller
Browse files

caif: Bugfix debugfs directory name must be unique.



Race condition caused debugfs_create_dir() to fail due to duplicate
name. Use atomic counter to create unique directory name.

net_ratelimit() is introduced to limit debug printouts.

Signed-off-by: default avatarSjur Brændeland <sjur.brandeland@stericsson.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent c85c2951
Loading
Loading
Loading
Loading
+22 −12
Original line number Original line Diff line number Diff line
@@ -48,6 +48,7 @@ static struct dentry *debugfsdir;
#ifdef CONFIG_DEBUG_FS
#ifdef CONFIG_DEBUG_FS
struct debug_fs_counter {
struct debug_fs_counter {
	atomic_t caif_nr_socks;
	atomic_t caif_nr_socks;
	atomic_t caif_sock_create;
	atomic_t num_connect_req;
	atomic_t num_connect_req;
	atomic_t num_connect_resp;
	atomic_t num_connect_resp;
	atomic_t num_connect_fail_resp;
	atomic_t num_connect_fail_resp;
@@ -59,11 +60,11 @@ struct debug_fs_counter {
	atomic_t num_rx_flow_on;
	atomic_t num_rx_flow_on;
};
};
static struct debug_fs_counter cnt;
static struct debug_fs_counter cnt;
#define	dbfs_atomic_inc(v) atomic_inc(v)
#define	dbfs_atomic_inc(v) atomic_inc_return(v)
#define	dbfs_atomic_dec(v) atomic_dec(v)
#define	dbfs_atomic_dec(v) atomic_dec_return(v)
#else
#else
#define	dbfs_atomic_inc(v)
#define	dbfs_atomic_inc(v) 0
#define	dbfs_atomic_dec(v)
#define	dbfs_atomic_dec(v) 0
#endif
#endif


struct caifsock {
struct caifsock {
@@ -155,6 +156,7 @@ static int caif_queue_rcv_skb(struct sock *sk, struct sk_buff *skb)


	if (atomic_read(&sk->sk_rmem_alloc) + skb->truesize >=
	if (atomic_read(&sk->sk_rmem_alloc) + skb->truesize >=
		(unsigned)sk->sk_rcvbuf && rx_flow_is_on(cf_sk)) {
		(unsigned)sk->sk_rcvbuf && rx_flow_is_on(cf_sk)) {
		if (net_ratelimit())
			pr_debug("sending flow OFF (queue len = %d %d)\n",
			pr_debug("sending flow OFF (queue len = %d %d)\n",
					atomic_read(&cf_sk->sk.sk_rmem_alloc),
					atomic_read(&cf_sk->sk.sk_rmem_alloc),
					sk_rcvbuf_lowwater(cf_sk));
					sk_rcvbuf_lowwater(cf_sk));
@@ -168,6 +170,7 @@ static int caif_queue_rcv_skb(struct sock *sk, struct sk_buff *skb)
		return err;
		return err;
	if (!sk_rmem_schedule(sk, skb->truesize) && rx_flow_is_on(cf_sk)) {
	if (!sk_rmem_schedule(sk, skb->truesize) && rx_flow_is_on(cf_sk)) {
		set_rx_flow_off(cf_sk);
		set_rx_flow_off(cf_sk);
		if (net_ratelimit())
			pr_debug("sending flow OFF due to rmem_schedule\n");
			pr_debug("sending flow OFF due to rmem_schedule\n");
		dbfs_atomic_inc(&cnt.num_rx_flow_off);
		dbfs_atomic_inc(&cnt.num_rx_flow_off);
		caif_flow_ctrl(sk, CAIF_MODEMCMD_FLOW_OFF_REQ);
		caif_flow_ctrl(sk, CAIF_MODEMCMD_FLOW_OFF_REQ);
@@ -838,7 +841,7 @@ static int caif_connect(struct socket *sock, struct sockaddr *uaddr,
	sock->state = SS_CONNECTING;
	sock->state = SS_CONNECTING;
	sk->sk_state = CAIF_CONNECTING;
	sk->sk_state = CAIF_CONNECTING;


	/* Check priority value coming from socket */
	/* Check priority value comming from socket */
	/* if priority value is out of range it will be ajusted */
	/* if priority value is out of range it will be ajusted */
	if (cf_sk->sk.sk_priority > CAIF_PRIO_MAX)
	if (cf_sk->sk.sk_priority > CAIF_PRIO_MAX)
		cf_sk->conn_req.priority = CAIF_PRIO_MAX;
		cf_sk->conn_req.priority = CAIF_PRIO_MAX;
@@ -942,6 +945,7 @@ static int caif_release(struct socket *sock)


	dbfs_atomic_inc(&cnt.num_disconnect);
	dbfs_atomic_inc(&cnt.num_disconnect);


	WARN_ON(IS_ERR(cf_sk->debugfs_socket_dir));
	if (cf_sk->debugfs_socket_dir != NULL)
	if (cf_sk->debugfs_socket_dir != NULL)
		debugfs_remove_recursive(cf_sk->debugfs_socket_dir);
		debugfs_remove_recursive(cf_sk->debugfs_socket_dir);


@@ -1047,7 +1051,7 @@ static void caif_sock_destructor(struct sock *sk)
	caif_assert(sk_unhashed(sk));
	caif_assert(sk_unhashed(sk));
	caif_assert(!sk->sk_socket);
	caif_assert(!sk->sk_socket);
	if (!sock_flag(sk, SOCK_DEAD)) {
	if (!sock_flag(sk, SOCK_DEAD)) {
		pr_info("Attempt to release alive CAIF socket: %p\n", sk);
		pr_debug("Attempt to release alive CAIF socket: %p\n", sk);
		return;
		return;
	}
	}
	sk_stream_kill_queues(&cf_sk->sk);
	sk_stream_kill_queues(&cf_sk->sk);
@@ -1058,6 +1062,7 @@ static void caif_sock_destructor(struct sock *sk)
static int caif_create(struct net *net, struct socket *sock, int protocol,
static int caif_create(struct net *net, struct socket *sock, int protocol,
			int kern)
			int kern)
{
{
	int num;
	struct sock *sk = NULL;
	struct sock *sk = NULL;
	struct caifsock *cf_sk = NULL;
	struct caifsock *cf_sk = NULL;
	static struct proto prot = {.name = "PF_CAIF",
	static struct proto prot = {.name = "PF_CAIF",
@@ -1120,14 +1125,16 @@ static int caif_create(struct net *net, struct socket *sock, int protocol,
	cf_sk->conn_req.protocol = protocol;
	cf_sk->conn_req.protocol = protocol;
	/* Increase the number of sockets created. */
	/* Increase the number of sockets created. */
	dbfs_atomic_inc(&cnt.caif_nr_socks);
	dbfs_atomic_inc(&cnt.caif_nr_socks);
	num = dbfs_atomic_inc(&cnt.caif_sock_create);
#ifdef CONFIG_DEBUG_FS
#ifdef CONFIG_DEBUG_FS
	if (!IS_ERR(debugfsdir)) {
	if (!IS_ERR(debugfsdir)) {

		/* Fill in some information concerning the misc socket. */
		/* Fill in some information concerning the misc socket. */
		snprintf(cf_sk->name, sizeof(cf_sk->name), "cfsk%d",
		snprintf(cf_sk->name, sizeof(cf_sk->name), "cfsk%d", num);
				atomic_read(&cnt.caif_nr_socks));


		cf_sk->debugfs_socket_dir =
		cf_sk->debugfs_socket_dir =
			debugfs_create_dir(cf_sk->name, debugfsdir);
			debugfs_create_dir(cf_sk->name, debugfsdir);

		debugfs_create_u32("sk_state", S_IRUSR | S_IWUSR,
		debugfs_create_u32("sk_state", S_IRUSR | S_IWUSR,
				cf_sk->debugfs_socket_dir,
				cf_sk->debugfs_socket_dir,
				(u32 *) &cf_sk->sk.sk_state);
				(u32 *) &cf_sk->sk.sk_state);
@@ -1171,6 +1178,9 @@ static int __init caif_sktinit_module(void)
		debugfs_create_u32("num_sockets", S_IRUSR | S_IWUSR,
		debugfs_create_u32("num_sockets", S_IRUSR | S_IWUSR,
				debugfsdir,
				debugfsdir,
				(u32 *) &cnt.caif_nr_socks);
				(u32 *) &cnt.caif_nr_socks);
		debugfs_create_u32("num_create", S_IRUSR | S_IWUSR,
				debugfsdir,
				(u32 *) &cnt.caif_sock_create);
		debugfs_create_u32("num_connect_req", S_IRUSR | S_IWUSR,
		debugfs_create_u32("num_connect_req", S_IRUSR | S_IWUSR,
				debugfsdir,
				debugfsdir,
				(u32 *) &cnt.num_connect_req);
				(u32 *) &cnt.num_connect_req);