o af_decnet: remove MOD_{INC,DEC}_USE_COUNT

  
Now the core networking infrastructure will (finally) do that for the net
protocol families, its just a matter of setting the ->owner field in the
registered struct net_proto_family to THIS_MODULE.
parent a7591416
...@@ -469,24 +469,21 @@ static void dn_destruct(struct sock *sk) ...@@ -469,24 +469,21 @@ static void dn_destruct(struct sock *sk)
skb_queue_purge(&scp->other_receive_queue); skb_queue_purge(&scp->other_receive_queue);
dst_release(xchg(&sk->dst_cache, NULL)); dst_release(xchg(&sk->dst_cache, NULL));
MOD_DEC_USE_COUNT;
} }
struct sock *dn_alloc_sock(struct socket *sock, int gfp) struct sock *dn_alloc_sock(struct socket *sock, int gfp)
{ {
struct sock *sk;
struct dn_scp *scp; struct dn_scp *scp;
struct sock *sk = sk_alloc(PF_DECnet, gfp, sizeof(struct dn_sock),
dn_sk_cachep);
if ((sk = sk_alloc(PF_DECnet, gfp, sizeof(struct dn_sock), dn_sk_cachep)) == NULL) if (!sk)
goto no_sock; goto out;
scp = (struct dn_scp *)(sk + 1); DN_SK(sk) = scp = (struct dn_scp *)(sk + 1);
DN_SK(sk) = scp;
if (sock) { if (sock)
sock->ops = &dn_proto_ops; sock->ops = &dn_proto_ops;
}
sock_init_data(sock, sk); sock_init_data(sock, sk);
sk->backlog_rcv = dn_nsp_backlog_rcv; sk->backlog_rcv = dn_nsp_backlog_rcv;
...@@ -543,13 +540,8 @@ struct sock *dn_alloc_sock(struct socket *sock, int gfp) ...@@ -543,13 +540,8 @@ struct sock *dn_alloc_sock(struct socket *sock, int gfp)
scp->delack_fxn = dn_nsp_delayed_ack; scp->delack_fxn = dn_nsp_delayed_ack;
dn_start_slow_timer(sk); dn_start_slow_timer(sk);
out:
MOD_INC_USE_COUNT;
return sk; return sk;
no_sock:
return NULL;
} }
/* /*
...@@ -2238,6 +2230,7 @@ static int dn_get_info(char *buffer, char **start, off_t offset, int length) ...@@ -2238,6 +2230,7 @@ static int dn_get_info(char *buffer, char **start, off_t offset, int length)
static struct net_proto_family dn_family_ops = { static struct net_proto_family dn_family_ops = {
.family = AF_DECnet, .family = AF_DECnet,
.create = dn_create, .create = dn_create,
.owner = THIS_MODULE,
}; };
static struct proto_ops dn_proto_ops = { static struct proto_ops dn_proto_ops = {
...@@ -2304,7 +2297,7 @@ static int __init decnet_init(void) ...@@ -2304,7 +2297,7 @@ static int __init decnet_init(void)
* Requires an audit of the code to check for memory leaks and * Requires an audit of the code to check for memory leaks and
* initialisation problems etc. * initialisation problems etc.
*/ */
MOD_INC_USE_COUNT; try_module_get(THIS_MODULE);
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment