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

Commit ab1e0a13 authored by Arnaldo Carvalho de Melo's avatar Arnaldo Carvalho de Melo Committed by David S. Miller
Browse files

[SOCK] proto: Add hashinfo member to struct proto 



This way we can remove TCP and DCCP specific versions of

sk->sk_prot->get_port: both v4 and v6 use inet_csk_get_port
sk->sk_prot->hash:     inet_hash is directly used, only v6 need
                       a specific version to deal with mapped sockets
sk->sk_prot->unhash:   both v4 and v6 use inet_hash directly

struct inet_connection_sock_af_ops also gets a new member, bind_conflict, so
that inet_csk_get_port can find the per family routine.

Now only the lookup routines receive as a parameter a struct inet_hashtable.

With this we further reuse code, reducing the difference among INET transport
protocols.

Eventually work has to be done on UDP and SCTP to make them share this
infrastructure and get as a bonus inet_diag interfaces so that iproute can be
used with these protocols.

net-2.6/net/ipv4/inet_hashtables.c:
  struct proto			     |   +8
  struct inet_connection_sock_af_ops |   +8
 2 structs changed
  __inet_hash_nolisten               |  +18
  __inet_hash                        | -210
  inet_put_port                      |   +8
  inet_bind_bucket_create            |   +1
  __inet_hash_connect                |   -8
 5 functions changed, 27 bytes added, 218 bytes removed, diff: -191

net-2.6/net/core/sock.c:
  proto_seq_show                     |   +3
 1 function changed, 3 bytes added, diff: +3

net-2.6/net/ipv4/inet_connection_sock.c:
  inet_csk_get_port                  |  +15
 1 function changed, 15 bytes added, diff: +15

net-2.6/net/ipv4/tcp.c:
  tcp_set_state                      |   -7
 1 function changed, 7 bytes removed, diff: -7

net-2.6/net/ipv4/tcp_ipv4.c:
  tcp_v4_get_port                    |  -31
  tcp_v4_hash                        |  -48
  tcp_v4_destroy_sock                |   -7
  tcp_v4_syn_recv_sock               |   -2
  tcp_unhash                         | -179
 5 functions changed, 267 bytes removed, diff: -267

net-2.6/net/ipv6/inet6_hashtables.c:
  __inet6_hash |   +8
 1 function changed, 8 bytes added, diff: +8

net-2.6/net/ipv4/inet_hashtables.c:
  inet_unhash                        | +190
  inet_hash                          | +242
 2 functions changed, 432 bytes added, diff: +432

vmlinux:
 16 functions changed, 485 bytes added, 492 bytes removed, diff: -7

/home/acme/git/net-2.6/net/ipv6/tcp_ipv6.c:
  tcp_v6_get_port                    |  -31
  tcp_v6_hash                        |   -7
  tcp_v6_syn_recv_sock               |   -9
 3 functions changed, 47 bytes removed, diff: -47

/home/acme/git/net-2.6/net/dccp/proto.c:
  dccp_destroy_sock                  |   -7
  dccp_unhash                        | -179
  dccp_hash                          |  -49
  dccp_set_state                     |   -7
  dccp_done                          |   +1
 5 functions changed, 1 bytes added, 242 bytes removed, diff: -241

/home/acme/git/net-2.6/net/dccp/ipv4.c:
  dccp_v4_get_port                   |  -31
  dccp_v4_request_recv_sock          |   -2
 2 functions changed, 33 bytes removed, diff: -33

/home/acme/git/net-2.6/net/dccp/ipv6.c:
  dccp_v6_get_port                   |  -31
  dccp_v6_hash                       |   -7
  dccp_v6_request_recv_sock          |   +5
 3 functions changed, 5 bytes added, 38 bytes removed, diff: -33

Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 9dc7f30e

include/net/inet6_hashtables.h

+1 −1
Original line number Diff line number Diff line
@@ -49,7 +49,7 @@ static inline int inet6_sk_ehashfn(const struct sock *sk) 
	return inet6_ehashfn(laddr, lport, faddr, fport);

}



extern void __inet6_hash(struct inet_hashinfo *hashinfo, struct sock *sk);

extern void __inet6_hash(struct sock *sk);



/*

 * Sockets in TCP_CLOSE state are _always_ taken out of the hash, so

include/net/inet_connection_sock.h

+3 −5
Original line number Diff line number Diff line
@@ -29,7 +29,6 @@ 
#undef INET_CSK_CLEAR_TIMERS



struct inet_bind_bucket;

struct inet_hashinfo;

struct tcp_congestion_ops;



/*
@@ -59,6 +58,8 @@ struct inet_connection_sock_af_ops { 
				int level, int optname,

				char __user *optval, int __user *optlen);

	void	    (*addr2sockaddr)(struct sock *sk, struct sockaddr *);

	int	    (*bind_conflict)(const struct sock *sk,

				     const struct inet_bind_bucket *tb);

};



/** inet_connection_sock - INET connection oriented sock
@@ -244,10 +245,7 @@ extern struct request_sock *inet_csk_search_req(const struct sock *sk, 
						const __be32 laddr);

extern int inet_csk_bind_conflict(const struct sock *sk,

				  const struct inet_bind_bucket *tb);

extern int inet_csk_get_port(struct inet_hashinfo *hashinfo,

			     struct sock *sk, unsigned short snum,

			     int (*bind_conflict)(const struct sock *sk,

						  const struct inet_bind_bucket *tb));

extern int inet_csk_get_port(struct sock *sk, unsigned short snum);



extern struct dst_entry* inet_csk_route_req(struct sock *sk,

					    const struct request_sock *req);

include/net/inet_hashtables.h

+9 −42
Original line number Diff line number Diff line
@@ -221,9 +221,9 @@ static inline int inet_sk_listen_hashfn(const struct sock *sk) 
}



/* Caller must disable local BH processing. */

static inline void __inet_inherit_port(struct inet_hashinfo *table,

				       struct sock *sk, struct sock *child)

static inline void __inet_inherit_port(struct sock *sk, struct sock *child)

{

	struct inet_hashinfo *table = sk->sk_prot->hashinfo;

	const int bhash = inet_bhashfn(inet_sk(child)->num, table->bhash_size);

	struct inet_bind_hashbucket *head = &table->bhash[bhash];

	struct inet_bind_bucket *tb;
@@ -235,15 +235,14 @@ static inline void __inet_inherit_port(struct inet_hashinfo *table, 
	spin_unlock(&head->lock);

}



static inline void inet_inherit_port(struct inet_hashinfo *table,

				     struct sock *sk, struct sock *child)

static inline void inet_inherit_port(struct sock *sk, struct sock *child)

{

	local_bh_disable();

	__inet_inherit_port(table, sk, child);

	__inet_inherit_port(sk, child);

	local_bh_enable();

}



extern void inet_put_port(struct inet_hashinfo *table, struct sock *sk);

extern void inet_put_port(struct sock *sk);



extern void inet_listen_wlock(struct inet_hashinfo *hashinfo);
@@ -266,41 +265,9 @@ static inline void inet_listen_unlock(struct inet_hashinfo *hashinfo) 
		wake_up(&hashinfo->lhash_wait);

}



extern void __inet_hash(struct inet_hashinfo *hashinfo, struct sock *sk);

extern void __inet_hash_nolisten(struct inet_hashinfo *hinfo, struct sock *sk);



static inline void inet_hash(struct inet_hashinfo *hashinfo, struct sock *sk)

{

	if (sk->sk_state != TCP_CLOSE) {

		local_bh_disable();

		__inet_hash(hashinfo, sk);

		local_bh_enable();

	}

}



static inline void inet_unhash(struct inet_hashinfo *hashinfo, struct sock *sk)

{

	rwlock_t *lock;



	if (sk_unhashed(sk))

		goto out;



	if (sk->sk_state == TCP_LISTEN) {

		local_bh_disable();

		inet_listen_wlock(hashinfo);

		lock = &hashinfo->lhash_lock;

	} else {

		lock = inet_ehash_lockp(hashinfo, sk->sk_hash);

		write_lock_bh(lock);

	}



	if (__sk_del_node_init(sk))

		sock_prot_inuse_add(sk->sk_prot, -1);

	write_unlock_bh(lock);

out:

	if (sk->sk_state == TCP_LISTEN)

		wake_up(&hashinfo->lhash_wait);

}

extern void __inet_hash_nolisten(struct sock *sk);

extern void inet_hash(struct sock *sk);

extern void inet_unhash(struct sock *sk);



extern struct sock *__inet_lookup_listener(struct net *net,

					   struct inet_hashinfo *hashinfo,
@@ -425,7 +392,7 @@ extern int __inet_hash_connect(struct inet_timewait_death_row *death_row, 
		struct sock *sk,

		int (*check_established)(struct inet_timewait_death_row *,

			struct sock *, __u16, struct inet_timewait_sock **),

		void (*hash)(struct inet_hashinfo *, struct sock *));

			       void (*hash)(struct sock *sk));

extern int inet_hash_connect(struct inet_timewait_death_row *death_row,

			     struct sock *sk);

#endif /* _INET_HASHTABLES_H */

include/net/sock.h

+3 −0
Original line number Diff line number Diff line
@@ -496,6 +496,7 @@ extern int sk_wait_data(struct sock *sk, long *timeo); 


struct request_sock_ops;

struct timewait_sock_ops;

struct inet_hashinfo;



/* Networking protocol blocks we attach to sockets.

 * socket layer -> transport layer interface
@@ -578,6 +579,8 @@ struct proto { 
	struct request_sock_ops	*rsk_prot;

	struct timewait_sock_ops *twsk_prot;



	struct inet_hashinfo	*hashinfo;



	struct module		*owner;



	char			name[32];

net/dccp/dccp.h

+0 −2
Original line number Diff line number Diff line
@@ -271,8 +271,6 @@ extern struct sk_buff *dccp_make_response(struct sock *sk, 


extern int	   dccp_connect(struct sock *sk);

extern int	   dccp_disconnect(struct sock *sk, int flags);

extern void	   dccp_hash(struct sock *sk);

extern void	   dccp_unhash(struct sock *sk);

extern int	   dccp_getsockopt(struct sock *sk, int level, int optname,

				   char __user *optval, int __user *optlen);

extern int	   dccp_setsockopt(struct sock *sk, int level, int optname,