diff options
author | David Ahern <dsa@cumulusnetworks.com> | 2016-05-10 14:19:51 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-05-11 19:31:40 -0400 |
commit | 0b922b7a829c06e3b0790c58cd9ca026de86096e (patch) | |
tree | b68c8a9b5148687c7bce7c6a5ab14e26a5c47050 | |
parent | 74b20582ac389ee9f18a6fcc0eef244658ce8de0 (diff) |
net: original ingress device index in PKTINFO
Applications such as OSPF and BFD need the original ingress device not
the VRF device; the latter can be derived from the former. To that end
add the skb_iif to inet_skb_parm and set it in ipv4 code after clearing
the skb control buffer similar to IPv6. From there the pktinfo can just
pull it from cb with the PKTINFO_SKB_CB cast.
The previous patch moving the skb->dev change to L3 means nothing else
is needed for IPv6; it just works.
Signed-off-by: David Ahern <dsa@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | include/net/ip.h | 1 | ||||
-rw-r--r-- | net/ipv4/ip_input.c | 1 | ||||
-rw-r--r-- | net/ipv4/ip_sockglue.c | 7 |
3 files changed, 8 insertions, 1 deletions
diff --git a/include/net/ip.h b/include/net/ip.h index 247ac82e9cf2..37165fba3741 100644 --- a/include/net/ip.h +++ b/include/net/ip.h | |||
@@ -36,6 +36,7 @@ | |||
36 | struct sock; | 36 | struct sock; |
37 | 37 | ||
38 | struct inet_skb_parm { | 38 | struct inet_skb_parm { |
39 | int iif; | ||
39 | struct ip_options opt; /* Compiled IP options */ | 40 | struct ip_options opt; /* Compiled IP options */ |
40 | unsigned char flags; | 41 | unsigned char flags; |
41 | 42 | ||
diff --git a/net/ipv4/ip_input.c b/net/ipv4/ip_input.c index 37375eedeef9..4b351af3e67b 100644 --- a/net/ipv4/ip_input.c +++ b/net/ipv4/ip_input.c | |||
@@ -478,6 +478,7 @@ int ip_rcv(struct sk_buff *skb, struct net_device *dev, struct packet_type *pt, | |||
478 | 478 | ||
479 | /* Remove any debris in the socket control block */ | 479 | /* Remove any debris in the socket control block */ |
480 | memset(IPCB(skb), 0, sizeof(struct inet_skb_parm)); | 480 | memset(IPCB(skb), 0, sizeof(struct inet_skb_parm)); |
481 | IPCB(skb)->iif = skb->skb_iif; | ||
481 | 482 | ||
482 | /* Must drop socket now because of tproxy. */ | 483 | /* Must drop socket now because of tproxy. */ |
483 | skb_orphan(skb); | 484 | skb_orphan(skb); |
diff --git a/net/ipv4/ip_sockglue.c b/net/ipv4/ip_sockglue.c index bdb222c0c6a2..5805762d7fc7 100644 --- a/net/ipv4/ip_sockglue.c +++ b/net/ipv4/ip_sockglue.c | |||
@@ -1193,7 +1193,12 @@ void ipv4_pktinfo_prepare(const struct sock *sk, struct sk_buff *skb) | |||
1193 | ipv6_sk_rxinfo(sk); | 1193 | ipv6_sk_rxinfo(sk); |
1194 | 1194 | ||
1195 | if (prepare && skb_rtable(skb)) { | 1195 | if (prepare && skb_rtable(skb)) { |
1196 | pktinfo->ipi_ifindex = inet_iif(skb); | 1196 | /* skb->cb is overloaded: prior to this point it is IP{6}CB |
1197 | * which has interface index (iif) as the first member of the | ||
1198 | * underlying inet{6}_skb_parm struct. This code then overlays | ||
1199 | * PKTINFO_SKB_CB and in_pktinfo also has iif as the first | ||
1200 | * element so the iif is picked up from the prior IPCB | ||
1201 | */ | ||
1197 | pktinfo->ipi_spec_dst.s_addr = fib_compute_spec_dst(skb); | 1202 | pktinfo->ipi_spec_dst.s_addr = fib_compute_spec_dst(skb); |
1198 | } else { | 1203 | } else { |
1199 | pktinfo->ipi_ifindex = 0; | 1204 | pktinfo->ipi_ifindex = 0; |