net: rtnetlink: use netif_oper_up instead of open code
authorJuhee Kang <claudiajkang@gmail.com>
Wed, 31 Aug 2022 12:58:45 +0000 (21:58 +0900)
committerJakub Kicinski <kuba@kernel.org>
Fri, 2 Sep 2022 03:09:23 +0000 (20:09 -0700)
The open code is defined as a new helper function(netif_oper_up) on netdev.h,
the code is dev->operstate == IF_OPER_UP || dev->operstate == IF_OPER_UNKNOWN.
Thus, replace the open code to netif_oper_up. This patch doesn't change logic.

Signed-off-by: Juhee Kang <claudiajkang@gmail.com>
Link: https://lore.kernel.org/r/20220831125845.1333-1-claudiajkang@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/core/rtnetlink.c

index 4b5b15c..f5e87fe 100644 (file)
@@ -866,14 +866,12 @@ static void set_operstate(struct net_device *dev, unsigned char transition)
                break;
 
        case IF_OPER_TESTING:
-               if (operstate == IF_OPER_UP ||
-                   operstate == IF_OPER_UNKNOWN)
+               if (netif_oper_up(dev))
                        operstate = IF_OPER_TESTING;
                break;
 
        case IF_OPER_DORMANT:
-               if (operstate == IF_OPER_UP ||
-                   operstate == IF_OPER_UNKNOWN)
+               if (netif_oper_up(dev))
                        operstate = IF_OPER_DORMANT;
                break;
        }