Skip to content

Commit 72168ff

Browse files
borkmannmehmetb0
authored andcommitted
net, team, bonding: Add netdev_base_features helper
BugLink: https://bugs.launchpad.net/bugs/2097332 [ Upstream commit d2516c3 ] Both bonding and team driver have logic to derive the base feature flags before iterating over their slave devices to refine the set via netdev_increment_features(). Add a small helper netdev_base_features() so this can be reused instead of having it open-coded multiple times. Signed-off-by: Daniel Borkmann <[email protected]> Cc: Nikolay Aleksandrov <[email protected]> Cc: Ido Schimmel <[email protected]> Cc: Jiri Pirko <[email protected]> Reviewed-by: Hangbin Liu <[email protected]> Reviewed-by: Nikolay Aleksandrov <[email protected]> Link: https://patch.msgid.link/[email protected] Signed-off-by: Paolo Abeni <[email protected]> Stable-dep-of: d064ea7 ("bonding: Fix initial {vlan,mpls}_feature set in bond_compute_features") Signed-off-by: Sasha Levin <[email protected]> Signed-off-by: Koichiro Den <[email protected]>
1 parent e34b1e8 commit 72168ff

File tree

3 files changed

+9
-5
lines changed

3 files changed

+9
-5
lines changed

drivers/net/bonding/bond_main.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1446,9 +1446,7 @@ static netdev_features_t bond_fix_features(struct net_device *dev,
14461446
struct slave *slave;
14471447

14481448
mask = features;
1449-
1450-
features &= ~NETIF_F_ONE_FOR_ALL;
1451-
features |= NETIF_F_ALL_FOR_ALL;
1449+
features = netdev_base_features(features);
14521450

14531451
bond_for_each_slave(bond, slave, iter) {
14541452
features = netdev_increment_features(features,

drivers/net/team/team_core.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2013,8 +2013,7 @@ static netdev_features_t team_fix_features(struct net_device *dev,
20132013
netdev_features_t mask;
20142014

20152015
mask = features;
2016-
features &= ~NETIF_F_ONE_FOR_ALL;
2017-
features |= NETIF_F_ALL_FOR_ALL;
2016+
features = netdev_base_features(features);
20182017

20192018
rcu_read_lock();
20202019
list_for_each_entry_rcu(port, &team->port_list, list) {

include/linux/netdev_features.h

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -261,4 +261,11 @@ static inline int find_next_netdev_feature(u64 feature, unsigned long start)
261261
NETIF_F_GSO_UDP_TUNNEL | \
262262
NETIF_F_GSO_UDP_TUNNEL_CSUM)
263263

264+
static inline netdev_features_t netdev_base_features(netdev_features_t features)
265+
{
266+
features &= ~NETIF_F_ONE_FOR_ALL;
267+
features |= NETIF_F_ALL_FOR_ALL;
268+
return features;
269+
}
270+
264271
#endif /* _LINUX_NETDEV_FEATURES_H */

0 commit comments

Comments
 (0)