vxlan: avoid double unlikely() notation when using IS_ERR()
authorAntonio Quartulli <a@unstable.cc>
Thu, 10 Dec 2020 08:55:49 +0000 (09:55 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 10 Dec 2020 20:43:29 +0000 (12:43 -0800)
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>
drivers/net/vxlan.c

index e1e44d68ac4b4d276ae65912dfcb9eb2ff84d955..a8ad710629e69f9dac4d83eee38ba065780403f3 100644 (file)
@@ -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);
        }