src/basic/efivars.c

@@ -17,7 +17,6 @@
 #include "fileio.h"
 #include "io-util.h"
 #include "macro.h"
-#include "memory-util.h"
 #include "stdio-util.h"
 #include "strv.h"
 #include "time-util.h"
@@ -160,29 +159,12 @@ int efi_get_variable_string(const char *variable, char **p) {
         return 0;
 }
 
-static int efi_verify_variable(const char *variable, uint32_t attr, const void *value, size_t size) {
-        _cleanup_free_ void *buf = NULL;
-        size_t n;
-        uint32_t a;
-        int r;
-
-        assert(variable);
-        assert(value || size == 0);
-
-        r = efi_get_variable(variable, &a, &buf, &n);
-        if (r < 0)
-                return r;
-
-        return a == attr && memcmp_nn(buf, n, value, size) == 0;
-}
-
 int efi_set_variable(const char *variable, const void *value, size_t size) {
         struct var {
                 uint32_t attr;
                 char buf[];
         } _packed_ * _cleanup_free_ buf = NULL;
         _cleanup_close_ int fd = -1;
-        uint32_t attr = EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS;
         bool saved_flags_valid = false;
         unsigned saved_flags;
         int r;
@@ -192,12 +174,6 @@ int efi_set_variable(const char *variable, const void *value, size_t size) {
 
         const char *p = strjoina("/sys/firmware/efi/efivars/", variable);
 
-        /* size 0 means removal, empty variable would not be enough for that */
-        if (size > 0 && efi_verify_variable(variable, attr, value, size) > 0) {
-                log_debug("Variable '%s' is already in wanted state, skipping write.", variable);
-                return 0;
-        }
-
         /* Newer efivarfs protects variables that are not in an allow list with FS_IMMUTABLE_FL by default,
          * to protect them for accidental removal and modification. We are not changing these variables
          * accidentally however, hence let's unset the bit first. */
@@ -229,7 +205,7 @@ int efi_set_variable(const char *variable, const void *value, size_t size) {
                 goto finish;
         }
 
-        buf->attr = attr;
+        buf->attr = EFI_VARIABLE_NON_VOLATILE|EFI_VARIABLE_BOOTSERVICE_ACCESS|EFI_VARIABLE_RUNTIME_ACCESS;
         memcpy(buf->buf, value, size);
 
         r = loop_write(fd, buf, sizeof(uint32_t) + size, false);

src/network/networkd-dhcp6.c

@@ -150,6 +150,8 @@ static int dhcp6_pd_get_assigned_prefix(Link *link, const struct in6_addr *pd_pr
 }
 
 int dhcp6_pd_remove(Link *link, bool only_marked) {
+        Address *address;
+        Route *route;
         int k, r = 0;
 
         assert(link);
@@ -161,51 +163,45 @@ int dhcp6_pd_remove(Link *link, bool only_marked) {
         if (!only_marked)
                 link->dhcp6_pd_configured = false;
 
-        if (!link->network->dhcp6_pd_assign) {
-                Route *route;
+        SET_FOREACH(route, link->routes) {
+                if (route->source != NETWORK_CONFIG_SOURCE_DHCP6PD)
+                        continue;
+                if (only_marked && !route_is_marked(route))
+                        continue;
 
-                SET_FOREACH(route, link->routes) {
-                        if (route->source != NETWORK_CONFIG_SOURCE_DHCP6PD)
-                                continue;
-                        if (only_marked && !route_is_marked(route))
-                                continue;
+                if (link->radv)
+                        (void) sd_radv_remove_prefix(link->radv, &route->dst.in6, 64);
 
-                        if (link->radv)
-                                (void) sd_radv_remove_prefix(link->radv, &route->dst.in6, 64);
+                link_remove_dhcp6_pd_prefix(link, &route->dst.in6);
 
-                        link_remove_dhcp6_pd_prefix(link, &route->dst.in6);
+                k = route_remove(route);
+                if (k < 0)
+                        r = k;
 
-                        k = route_remove(route);
-                        if (k < 0)
-                                r = k;
+                route_cancel_request(route);
+        }
 
-                        route_cancel_request(route);
-                }
-        } else {
-                Address *address;
+        SET_FOREACH(address, link->addresses) {
+                struct in6_addr prefix;
 
-                SET_FOREACH(address, link->addresses) {
-                        struct in6_addr prefix;
+                if (address->source != NETWORK_CONFIG_SOURCE_DHCP6PD)
+                        continue;
+                if (only_marked && !address_is_marked(address))
+                        continue;
 
-                        if (address->source != NETWORK_CONFIG_SOURCE_DHCP6PD)
-                                continue;
-                        if (only_marked && !address_is_marked(address))
-                                continue;
+                prefix = address->in_addr.in6;
+                in6_addr_mask(&prefix, 64);
 
-                        prefix = address->in_addr.in6;
-                        in6_addr_mask(&prefix, 64);
+                if (link->radv)
+                        (void) sd_radv_remove_prefix(link->radv, &prefix, 64);
 
-                        if (link->radv)
-                                (void) sd_radv_remove_prefix(link->radv, &prefix, 64);
+                link_remove_dhcp6_pd_prefix(link, &prefix);
 
-                        link_remove_dhcp6_pd_prefix(link, &prefix);
+                k = address_remove(address);
+                if (k < 0)
+                        r = k;
 
-                        k = address_remove(address);
-                        if (k < 0)
-                                r = k;
-
-                        address_cancel_request(address);
-                }
+                address_cancel_request(address);
         }
 
         return r;
@@ -547,15 +543,15 @@ static int dhcp6_pd_prefix_distribute(
                 _cleanup_free_ char *buf = NULL;
                 struct in6_addr assigned_prefix;
 
-                if (!IN_SET(link->state, LINK_STATE_CONFIGURING, LINK_STATE_CONFIGURED))
-                        continue;
-
                 if (!link_dhcp6_pd_is_enabled(link))
                         continue;
 
                 if (link == dhcp6_link && !link->network->dhcp6_pd_assign)
                         continue;
 
+                if (IN_SET(link->state, LINK_STATE_FAILED, LINK_STATE_LINGER))
+                        continue;
+
                 if (assign_preferred_subnet_id != link_has_preferred_subnet_id(link))
                         continue;
 
@@ -581,7 +577,7 @@ static int dhcp6_pd_prefix_distribute(
 }
 
 static int dhcp6_pd_prepare(Link *link) {
-        if (!IN_SET(link->state, LINK_STATE_CONFIGURING, LINK_STATE_CONFIGURED))
+        if (IN_SET(link->state, LINK_STATE_FAILED, LINK_STATE_LINGER))
                 return 0;
 
         if (!link_dhcp6_pd_is_enabled(link))
@@ -596,7 +592,7 @@ static int dhcp6_pd_prepare(Link *link) {
 static int dhcp6_pd_finalize(Link *link) {
         int r;
 
-        if (!IN_SET(link->state, LINK_STATE_CONFIGURING, LINK_STATE_CONFIGURED))
+        if (IN_SET(link->state, LINK_STATE_FAILED, LINK_STATE_LINGER))
                 return 0;
 
         if (!link_dhcp6_pd_is_enabled(link))
@@ -844,17 +840,12 @@ static int dhcp6_pd_prefix_acquired(Link *dhcp6_link) {
                 return log_link_warning_errno(dhcp6_link, r, "Failed to get timestamp of DHCPv6 lease: %m");
 
         HASHMAP_FOREACH(link, dhcp6_link->manager->links_by_index) {
-                r = dhcp6_pd_prepare(link);
-                if (r < 0) {
-                        /* When failed on the upstream interface (i.e., the case link == dhcp6_link),
-                         * immediately abort the assignment of the prefixes. As, the all assigned
-                         * prefixes will be dropped soon in link_enter_failed(), and it is meaningless
-                         * to continue the assignment. */
-                        if (link == dhcp6_link)
-                                return r;
+                if (link == dhcp6_link)
+                        continue;
 
+                r = dhcp6_pd_prepare(link);
+                if (r < 0)
                         link_enter_failed(link);
-                }
         }
 
         for (sd_dhcp6_lease_reset_pd_prefix_iter(dhcp6_link->dhcp6_lease);;) {
@@ -926,13 +917,12 @@ static int dhcp6_pd_prefix_acquired(Link *dhcp6_link) {
         }
 
         HASHMAP_FOREACH(link, dhcp6_link->manager->links_by_index) {
-                r = dhcp6_pd_finalize(link);
-                if (r < 0) {
-                        if (link == dhcp6_link)
-                                return r;
+                if (link == dhcp6_link)
+                        continue;
 
+                r = dhcp6_pd_finalize(link);
+                if (r < 0)
                         link_enter_failed(link);
-                }
         }
 
         return 0;