Skip to content

Commit 0fa1a53

Browse files
Arnaldo Carvalho de MeloDavid S. Miller
authored andcommitted
[IPV6]: Introduce inet6_timewait_sock
Out of tcp6_timewait_sock, that now is just an aggregation of inet_timewait_sock and inet6_timewait_sock, using tw_ipv6_offset in struct inet_timewait_sock, that is common to the IPv6 transport protocols that use timewait sockets, like DCCP and TCP. tw_ipv6_offset plays the struct inet_sock pinfo6 role, i.e. for the generic code to find the IPv6 area in a timewait sock. Signed-off-by: Arnaldo Carvalho de Melo <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent b9750ce commit 0fa1a53

File tree

7 files changed

+41
-28
lines changed

7 files changed

+41
-28
lines changed

include/linux/ipv6.h

Lines changed: 21 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -348,26 +348,36 @@ static inline void inet_sk_copy_descendant(struct sock *sk_to,
348348

349349
#include <linux/tcp.h>
350350

351+
struct inet6_timewait_sock {
352+
struct in6_addr tw_v6_daddr;
353+
struct in6_addr tw_v6_rcv_saddr;
354+
};
355+
351356
struct tcp6_timewait_sock {
352-
struct tcp_timewait_sock tw_v6_sk;
353-
struct in6_addr tw_v6_daddr;
354-
struct in6_addr tw_v6_rcv_saddr;
357+
struct tcp_timewait_sock tcp6tw_tcp;
358+
struct inet6_timewait_sock tcp6tw_inet6;
355359
};
356360

357-
static inline struct tcp6_timewait_sock *tcp6_twsk(const struct sock *sk)
361+
static inline u16 inet6_tw_offset(const struct proto *prot)
362+
{
363+
return prot->twsk_obj_size - sizeof(struct inet6_timewait_sock);
364+
}
365+
366+
static inline struct inet6_timewait_sock *inet6_twsk(const struct sock *sk)
358367
{
359-
return (struct tcp6_timewait_sock *)sk;
368+
return (struct inet6_timewait_sock *)(((u8 *)sk) +
369+
inet_twsk(sk)->tw_ipv6_offset);
360370
}
361371

362-
static inline struct in6_addr *__tcp_v6_rcv_saddr(const struct sock *sk)
372+
static inline struct in6_addr *__inet6_rcv_saddr(const struct sock *sk)
363373
{
364374
return likely(sk->sk_state != TCP_TIME_WAIT) ?
365-
&inet6_sk(sk)->rcv_saddr : &tcp6_twsk(sk)->tw_v6_rcv_saddr;
375+
&inet6_sk(sk)->rcv_saddr : &inet6_twsk(sk)->tw_v6_rcv_saddr;
366376
}
367377

368-
static inline struct in6_addr *tcp_v6_rcv_saddr(const struct sock *sk)
378+
static inline struct in6_addr *inet6_rcv_saddr(const struct sock *sk)
369379
{
370-
return sk->sk_family == AF_INET6 ? __tcp_v6_rcv_saddr(sk) : NULL;
380+
return sk->sk_family == AF_INET6 ? __inet6_rcv_saddr(sk) : NULL;
371381
}
372382

373383
static inline int inet_v6_ipv6only(const struct sock *sk)
@@ -395,8 +405,8 @@ static inline struct raw6_sock *raw6_sk(const struct sock *sk)
395405
return NULL;
396406
}
397407

398-
#define __tcp_v6_rcv_saddr(__sk) NULL
399-
#define tcp_v6_rcv_saddr(__sk) NULL
408+
#define __inet6_rcv_saddr(__sk) NULL
409+
#define inet6_rcv_saddr(__sk) NULL
400410
#define tcp_twsk_ipv6only(__sk) 0
401411
#define inet_v6_ipv6only(__sk) 0
402412
#endif /* defined(CONFIG_IPV6) || defined(CONFIG_IPV6_MODULE) */

include/net/inet6_hashtables.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -110,10 +110,10 @@ static inline struct sock *
110110

111111
if(*((__u32 *)&(tw->tw_dport)) == ports &&
112112
sk->sk_family == PF_INET6) {
113-
const struct tcp6_timewait_sock *tcp6tw = tcp6_twsk(sk);
113+
const struct inet6_timewait_sock *tw6 = inet6_twsk(sk);
114114

115-
if (ipv6_addr_equal(&tcp6tw->tw_v6_daddr, saddr) &&
116-
ipv6_addr_equal(&tcp6tw->tw_v6_rcv_saddr, daddr) &&
115+
if (ipv6_addr_equal(&tw6->tw_v6_daddr, saddr) &&
116+
ipv6_addr_equal(&tw6->tw_v6_rcv_saddr, daddr) &&
117117
(!sk->sk_bound_dev_if || sk->sk_bound_dev_if == dif))
118118
goto hit;
119119
}

include/net/inet_timewait_sock.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,8 @@ struct inet_timewait_sock {
127127
__u16 tw_num;
128128
/* And these are ours. */
129129
__u8 tw_ipv6only:1;
130-
/* 31 bits hole, try to pack */
130+
/* 15 bits hole, try to pack */
131+
__u16 tw_ipv6_offset;
131132
int tw_timeout;
132133
unsigned long tw_ttd;
133134
struct inet_bind_bucket *tw_tb;

net/ipv4/inet_diag.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,12 +112,12 @@ static int inet_diag_fill(struct sk_buff *skb, struct sock *sk,
112112
r->idiag_inode = 0;
113113
#if defined(CONFIG_IPV6) || defined (CONFIG_IPV6_MODULE)
114114
if (r->idiag_family == AF_INET6) {
115-
const struct tcp6_timewait_sock *tcp6tw = tcp6_twsk(sk);
115+
const struct inet6_timewait_sock *tw6 = inet6_twsk(sk);
116116

117117
ipv6_addr_copy((struct in6_addr *)r->id.idiag_src,
118-
&tcp6tw->tw_v6_rcv_saddr);
118+
&tw6->tw_v6_rcv_saddr);
119119
ipv6_addr_copy((struct in6_addr *)r->id.idiag_dst,
120-
&tcp6tw->tw_v6_daddr);
120+
&tw6->tw_v6_daddr);
121121
}
122122
#endif
123123
nlh->nlmsg_len = skb->tail - b;

net/ipv4/tcp_minisocks.c

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -298,10 +298,12 @@ void tcp_time_wait(struct sock *sk, int state, int timeo)
298298
#if defined(CONFIG_IPV6) || defined(CONFIG_IPV6_MODULE)
299299
if (tw->tw_family == PF_INET6) {
300300
struct ipv6_pinfo *np = inet6_sk(sk);
301-
struct tcp6_timewait_sock *tcp6tw = tcp6_twsk((struct sock *)tw);
301+
struct inet6_timewait_sock *tw6;
302302

303-
ipv6_addr_copy(&tcp6tw->tw_v6_daddr, &np->daddr);
304-
ipv6_addr_copy(&tcp6tw->tw_v6_rcv_saddr, &np->rcv_saddr);
303+
tw->tw_ipv6_offset = inet6_tw_offset(sk->sk_prot);
304+
tw6 = inet6_twsk((struct sock *)tw);
305+
ipv6_addr_copy(&tw6->tw_v6_daddr, &np->daddr);
306+
ipv6_addr_copy(&tw6->tw_v6_rcv_saddr, &np->rcv_saddr);
305307
tw->tw_ipv6only = np->ipv6only;
306308
}
307309
#endif

net/ipv6/addrconf.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1195,7 +1195,7 @@ struct inet6_ifaddr * ipv6_get_ifaddr(struct in6_addr *addr, struct net_device *
11951195
int ipv6_rcv_saddr_equal(const struct sock *sk, const struct sock *sk2)
11961196
{
11971197
const struct in6_addr *sk_rcv_saddr6 = &inet6_sk(sk)->rcv_saddr;
1198-
const struct in6_addr *sk2_rcv_saddr6 = tcp_v6_rcv_saddr(sk2);
1198+
const struct in6_addr *sk2_rcv_saddr6 = inet6_rcv_saddr(sk2);
11991199
u32 sk_rcv_saddr = inet_sk(sk)->rcv_saddr;
12001200
u32 sk2_rcv_saddr = inet_rcv_saddr(sk2);
12011201
int sk_ipv6only = ipv6_only_sock(sk);

net/ipv6/tcp_ipv6.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -138,14 +138,14 @@ static int __tcp_v6_check_established(struct sock *sk, const __u16 lport,
138138

139139
/* Check TIME-WAIT sockets first. */
140140
sk_for_each(sk2, node, &(head + tcp_hashinfo.ehash_size)->chain) {
141-
const struct tcp6_timewait_sock *tcp6tw = tcp6_twsk(sk2);
141+
const struct inet6_timewait_sock *tw6 = inet6_twsk(sk2);
142142

143143
tw = inet_twsk(sk2);
144144

145145
if(*((__u32 *)&(tw->tw_dport)) == ports &&
146146
sk2->sk_family == PF_INET6 &&
147-
ipv6_addr_equal(&tcp6tw->tw_v6_daddr, saddr) &&
148-
ipv6_addr_equal(&tcp6tw->tw_v6_rcv_saddr, daddr) &&
147+
ipv6_addr_equal(&tw6->tw_v6_daddr, saddr) &&
148+
ipv6_addr_equal(&tw6->tw_v6_rcv_saddr, daddr) &&
149149
sk2->sk_bound_dev_if == sk->sk_bound_dev_if) {
150150
const struct tcp_timewait_sock *tcptw = tcp_twsk(sk2);
151151
struct tcp_sock *tp = tcp_sk(sk);
@@ -1663,14 +1663,14 @@ static void get_timewait6_sock(struct seq_file *seq,
16631663
{
16641664
struct in6_addr *dest, *src;
16651665
__u16 destp, srcp;
1666-
struct tcp6_timewait_sock *tcp6tw = tcp6_twsk((struct sock *)tw);
1666+
struct inet6_timewait_sock *tw6 = inet6_twsk((struct sock *)tw);
16671667
int ttd = tw->tw_ttd - jiffies;
16681668

16691669
if (ttd < 0)
16701670
ttd = 0;
16711671

1672-
dest = &tcp6tw->tw_v6_daddr;
1673-
src = &tcp6tw->tw_v6_rcv_saddr;
1672+
dest = &tw6->tw_v6_daddr;
1673+
src = &tw6->tw_v6_rcv_saddr;
16741674
destp = ntohs(tw->tw_dport);
16751675
srcp = ntohs(tw->tw_sport);
16761676

0 commit comments

Comments
 (0)