Skip to content

Commit f6bc7d9

Browse files
Eric Dumazetdavem330
authored andcommitted
ipv6: avoid two atomics in ipv6_rthdr_rcv()
Use __in6_dev_get() instead of in6_dev_get()/in6_dev_put() Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 1be3b5f commit f6bc7d9

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

net/ipv6/exthdrs.c

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -312,6 +312,7 @@ static int ipv6_destopt_rcv(struct sk_buff *skb)
312312
Routing header.
313313
********************************/
314314

315+
/* called with rcu_read_lock() */
315316
static int ipv6_rthdr_rcv(struct sk_buff *skb)
316317
{
317318
struct inet6_skb_parm *opt = IP6CB(skb);
@@ -324,12 +325,9 @@ static int ipv6_rthdr_rcv(struct sk_buff *skb)
324325
struct net *net = dev_net(skb->dev);
325326
int accept_source_route = net->ipv6.devconf_all->accept_source_route;
326327

327-
idev = in6_dev_get(skb->dev);
328-
if (idev) {
329-
if (accept_source_route > idev->cnf.accept_source_route)
330-
accept_source_route = idev->cnf.accept_source_route;
331-
in6_dev_put(idev);
332-
}
328+
idev = __in6_dev_get(skb->dev);
329+
if (idev && accept_source_route > idev->cnf.accept_source_route)
330+
accept_source_route = idev->cnf.accept_source_route;
333331

334332
if (!pskb_may_pull(skb, skb_transport_offset(skb) + 8) ||
335333
!pskb_may_pull(skb, (skb_transport_offset(skb) +

0 commit comments

Comments
 (0)