summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWen Liang <liangwen12year@gmail.com>2022-08-21 21:15:37 -0400
committerThomas Haller <thaller@redhat.com>2022-09-07 15:51:56 +0200
commit72144946c90386cd6fea63d774e6628b93ca0dea (patch)
tree7d6a4d87cd3ac99b766f56e0b9a10e22906649a3
parent0e0ac364a1901b3e5bb56da4b37bc706555e5214 (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.c56
-rw-r--r--src/libnm-platform/nm-platform.h6
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. */ \