Skip to content

Commit dc028da

Browse files
David Aherndavem330
authored andcommitted
inet: Move VRF table lookup to inlined function
Table lookup compiles out when VRF is not enabled. Signed-off-by: David Ahern <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent 808d28c commit dc028da

File tree

2 files changed

+25
-9
lines changed

2 files changed

+25
-9
lines changed

include/net/vrf.h

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,25 @@ static inline int vrf_dev_table(const struct net_device *dev)
8181
return tb_id;
8282
}
8383

84+
static inline int vrf_dev_table_ifindex(struct net *net, int ifindex)
85+
{
86+
struct net_device *dev;
87+
int tb_id = 0;
88+
89+
if (!ifindex)
90+
return 0;
91+
92+
rcu_read_lock();
93+
94+
dev = dev_get_by_index_rcu(net, ifindex);
95+
if (dev)
96+
tb_id = vrf_dev_table_rcu(dev);
97+
98+
rcu_read_unlock();
99+
100+
return tb_id;
101+
}
102+
84103
/* called with rtnl */
85104
static inline int vrf_dev_table_rtnl(const struct net_device *dev)
86105
{
@@ -125,6 +144,11 @@ static inline int vrf_dev_table(const struct net_device *dev)
125144
return 0;
126145
}
127146

147+
static inline int vrf_dev_table_ifindex(struct net *net, int ifindex)
148+
{
149+
return 0;
150+
}
151+
128152
static inline int vrf_dev_table_rtnl(const struct net_device *dev)
129153
{
130154
return 0;

net/ipv4/af_inet.c

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -450,15 +450,7 @@ int inet_bind(struct socket *sock, struct sockaddr *uaddr, int addr_len)
450450
goto out;
451451
}
452452

453-
if (sk->sk_bound_dev_if) {
454-
struct net_device *dev;
455-
456-
rcu_read_lock();
457-
dev = dev_get_by_index_rcu(net, sk->sk_bound_dev_if);
458-
if (dev)
459-
tb_id = vrf_dev_table_rcu(dev) ? : tb_id;
460-
rcu_read_unlock();
461-
}
453+
tb_id = vrf_dev_table_ifindex(net, sk->sk_bound_dev_if) ? : tb_id;
462454
chk_addr_ret = inet_addr_type_table(net, addr->sin_addr.s_addr, tb_id);
463455

464456
/* Not specified by any standard per-se, however it breaks too

0 commit comments

Comments
 (0)