diff options
author | Wen Liang <liangwen12year@gmail.com> | 2022-08-21 21:15:37 -0400 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2022-09-07 15:51:56 +0200 |
commit | 72144946c90386cd6fea63d774e6628b93ca0dea (patch) | |
tree | 7d6a4d87cd3ac99b766f56e0b9a10e22906649a3 | |
parent | 0e0ac364a1901b3e5bb56da4b37bc706555e5214 (diff) |
Revert "platform: add the `a_no_auto_noprefixroute` flag"
This flag won't be used. Instead we will pass a flag to
nm_platform_ip_route_sync() to disable addition of the prefix route
flag.
This reverts commit bd84ae4dc5a1e2b44a7471c3b0b4c3428b789a04.
-rw-r--r-- | src/libnm-platform/nm-platform.c | 56 | ||||
-rw-r--r-- | src/libnm-platform/nm-platform.h | 6 |
2 files changed, 23 insertions, 39 deletions
diff --git a/src/libnm-platform/nm-platform.c b/src/libnm-platform/nm-platform.c index c023506439..ff635b33fe 100644 --- a/src/libnm-platform/nm-platform.c +++ b/src/libnm-platform/nm-platform.c @@ -4529,20 +4529,18 @@ next_plat:; nm_platform_ip4_broadcast_address_from_addr(&known_address->a4), lifetime, preferred, - known_address->ax.a_no_auto_noprefixroute ? 0 : IFA_F_NOPREFIXROUTE, + IFA_F_NOPREFIXROUTE, known_address->a4.label)) success = FALSE; } else { - if (!nm_platform_ip6_address_add( - self, - ifindex, - known_address->a6.address, - known_address->a6.plen, - known_address->a6.peer_address, - lifetime, - preferred, - (known_address->ax.a_no_auto_noprefixroute ? 0 : IFA_F_NOPREFIXROUTE) - | known_address->a6.n_ifa_flags)) + if (!nm_platform_ip6_address_add(self, + ifindex, + known_address->a6.address, + known_address->a6.plen, + known_address->a6.peer_address, + lifetime, + preferred, + IFA_F_NOPREFIXROUTE | known_address->a6.n_ifa_flags)) success = FALSE; } } @@ -6790,7 +6788,6 @@ nm_platform_ip4_address_to_string(const NMPlatformIP4Address *address, char *buf " src %s" "%s" /* a_acd_not_ready */ "%s" /* a_force_commit */ - "%s" /* a_no_auto_noprefixroute */ "", s_address, address->plen, @@ -6809,8 +6806,7 @@ nm_platform_ip4_address_to_string(const NMPlatformIP4Address *address, char *buf str_label, nmp_utils_ip_config_source_to_string(address->addr_source, s_source, sizeof(s_source)), address->a_acd_not_ready ? " ip4acd-not-ready" : "", - address->a_force_commit ? " force-commit" : "", - address->a_no_auto_noprefixroute ? " no-auto-noprefixroute" : ""); + address->a_force_commit ? " force-commit" : ""); g_free(str_peer); return buf; } @@ -6931,7 +6927,6 @@ nm_platform_ip6_address_to_string(const NMPlatformIP6Address *address, char *buf len, "%s/%d lft %s pref %s%s%s%s%s src %s" "%s" /* a_force_commit */ - "%s" /* a_no_auto_noprefixroute */ "", s_address, address->plen, @@ -6942,8 +6937,7 @@ nm_platform_ip6_address_to_string(const NMPlatformIP6Address *address, char *buf str_dev, _to_string_ifa_flags(address->n_ifa_flags, s_flags, sizeof(s_flags)), nmp_utils_ip_config_source_to_string(address->addr_source, s_source, sizeof(s_source)), - address->a_force_commit ? " force-commit" : "", - address->a_no_auto_noprefixroute ? " no-auto-noprefixroute" : ""); + address->a_force_commit ? " force-commit" : ""); g_free(str_peer); return buf; } @@ -8606,8 +8600,7 @@ nm_platform_ip4_address_hash_update(const NMPlatformIP4Address *obj, NMHashState NM_HASH_COMBINE_BOOLS(guint8, obj->use_ip4_broadcast_address, obj->a_acd_not_ready, - obj->a_force_commit, - obj->a_no_auto_noprefixroute)); + obj->a_force_commit)); nm_hash_update_strarr(h, obj->label); } @@ -8656,7 +8649,6 @@ nm_platform_ip4_address_cmp(const NMPlatformIP4Address *a, NM_CMP_FIELD(a, b, broadcast_address); NM_CMP_FIELD_UNSAFE(a, b, a_acd_not_ready); NM_CMP_FIELD_UNSAFE(a, b, a_force_commit); - NM_CMP_FIELD_UNSAFE(a, b, a_no_auto_noprefixroute); } return 0; } @@ -8666,18 +8658,17 @@ nm_platform_ip4_address_cmp(const NMPlatformIP4Address *a, void nm_platform_ip6_address_hash_update(const NMPlatformIP6Address *obj, NMHashState *h) { - nm_hash_update_vals( - h, - obj->ifindex, - obj->addr_source, - obj->timestamp, - obj->lifetime, - obj->preferred, - obj->n_ifa_flags, - obj->plen, - obj->address, - obj->peer_address, - NM_HASH_COMBINE_BOOLS(guint8, obj->a_force_commit, obj->a_no_auto_noprefixroute)); + nm_hash_update_vals(h, + obj->ifindex, + obj->addr_source, + obj->timestamp, + obj->lifetime, + obj->preferred, + obj->n_ifa_flags, + obj->plen, + obj->address, + obj->peer_address, + NM_HASH_COMBINE_BOOLS(guint8, obj->a_force_commit)); } int @@ -8721,7 +8712,6 @@ nm_platform_ip6_address_cmp(const NMPlatformIP6Address *a, NM_CMP_FIELD(a, b, n_ifa_flags); NM_CMP_FIELD(a, b, addr_source); NM_CMP_FIELD_UNSAFE(a, b, a_force_commit); - NM_CMP_FIELD_UNSAFE(a, b, a_no_auto_noprefixroute); } return 0; } diff --git a/src/libnm-platform/nm-platform.h b/src/libnm-platform/nm-platform.h index e1715ad9eb..2570a47dda 100644 --- a/src/libnm-platform/nm-platform.h +++ b/src/libnm-platform/nm-platform.h @@ -339,12 +339,6 @@ typedef enum { * should be configured. */ \ bool a_force_commit : 1; \ \ - /* nm_platform_ip_address_sync() likes to add IFA_F_NOPREFIXROUTE flag for all - * addresses, regardless of a_ifi_flags property. By setting this boolean, that - * automatism can be suppressed, and the noprefixroute flag does not get added - * automatically. */ \ - bool a_no_auto_noprefixroute : 1; \ - \ /* Don't have a bitfield as last field in __NMPlatformIPAddress_COMMON. It would then * be unclear how the following fields get merged. We could also use a zero bitfield, * but instead we just have there the uint8 field. */ \ |