Commit a10b24b8 authored Dec 10, 2020 by Antonio Quartulli Committed by David S. Miller Dec 10, 2020

vxlan: avoid double unlikely() notation when using IS_ERR()



The definition of IS_ERR() already applies the unlikely() notation
when checking the error status of the passed pointer. For this
reason there is no need to have the same notation outside of
IS_ERR() itself.

Clean up code by removing redundant notation.

Signed-off-by: Antonio Quartulli <a@unstable.cc>
Signed-off-by: David S. Miller <davem@davemloft.net>

parent a7105e34

drivers/net/vxlan.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -2477,7 +2477,7 @@ static struct dst_entry vxlan6_get_route(struct vxlan_dev vxlan,

		ndst = ipv6_stub->ipv6_dst_lookup_flow(vxlan->net, sock6->sock->sk,
		&fl6, NULL);
		if (unlikely(IS_ERR(ndst))) {
		if (IS_ERR(ndst)) {
		netdev_dbg(dev, "no route to %pI6\n", daddr);
		return ERR_PTR(-ENETUNREACH);
		}

Admin message