aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Ahern <dsa@cumulusnetworks.com>2016-05-10 14:19:51 -0400
committerDavid S. Miller <davem@davemloft.net>2016-05-11 19:31:40 -0400
commit0b922b7a829c06e3b0790c58cd9ca026de86096e (patch)
treeb68c8a9b5148687c7bce7c6a5ab14e26a5c47050
parent74b20582ac389ee9f18a6fcc0eef244658ce8de0 (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.h1
-rw-r--r--net/ipv4/ip_input.c1
-rw-r--r--net/ipv4/ip_sockglue.c7
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 @@
36struct sock; 36struct sock;
37 37
38struct inet_skb_parm { 38struct 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;