man/systemd-system.conf.xml

@@ -361,8 +361,7 @@
         <listitem><para>Configure the default value for the per-unit <varname>TasksMax=</varname> setting. See
         <citerefentry><refentrytitle>systemd.resource-control</refentrytitle><manvolnum>5</manvolnum></citerefentry>
         for details. This setting applies to all unit types that support resource control settings, with the exception
-        of slice units. Defaults to 15% of the minimum of <varname>kernel.pid_max=</varname>, <varname>kernel.threads-max=</varname>
-        and root cgroup <varname>pids.max</varname>.
+        of slice units. Defaults to 15% of the sysctl setting <varname>kernel.pid_max=</varname> or root cgroup <varname>pids.max</varname>.
         Kernel has a default value for <varname>kernel.pid_max=</varname> and an algorithm of counting in case of more than 32 cores.
         For example with the default <varname>kernel.pid_max=</varname>, <varname>DefaultTasksMax=</varname> defaults to 4915,
         but might be greater in other systems or smaller in OS containers.</para></listitem>

src/basic/fs-util.c

@@ -1614,7 +1614,7 @@ int path_is_encrypted(const char *path) {
         return blockdev_is_encrypted(p, 10 /* safety net: maximum recursion depth */);
 }
 
-int conservative_renameat(
+int conservative_rename(
                 int olddirfd, const char *oldpath,
                 int newdirfd, const char *newpath) {
 

src/basic/fs-util.h

@@ -143,7 +143,4 @@ int open_parent(const char *path, int flags, mode_t mode);
 
 int path_is_encrypted(const char *path);
 
-int conservative_renameat(int olddirfd, const char *oldpath, int newdirfd, const char *newpath);
-static inline int conservative_rename(const char *oldpath, const char *newpath) {
-        return conservative_renameat(AT_FDCWD, oldpath, AT_FDCWD, newpath);
-}
+int conservative_rename(int olddirfd, const char *oldpath, int newdirfd, const char *newpath);

src/libsystemd-network/sd-dhcp-lease.c

@@ -19,7 +19,6 @@
 #include "env-file.h"
 #include "fd-util.h"
 #include "fileio.h"
-#include "fs-util.h"
 #include "hexdecoct.h"
 #include "hostname-util.h"
 #include "in-addr-util.h"
@@ -1024,9 +1023,10 @@ int dhcp_lease_save(sd_dhcp_lease *lease, const char *lease_file) {
         if (r < 0)
                 goto fail;
 
-        r = conservative_rename(temp_path, lease_file);
-        if (r < 0)
+        if (rename(temp_path, lease_file) < 0) {
+                r = -errno;
                 goto fail;
+        }
 
         return 0;
 

src/network/networkd-dhcp4.c

@@ -36,6 +36,8 @@ static int dhcp4_release_old_lease(Link *link) {
 
         log_link_debug(link, "Removing old DHCPv4 address and routes.");
 
+        link_dirty(link);
+
         SET_FOREACH(route, link->dhcp_routes_old) {
                 k = route_remove(route, NULL, link, NULL);
                 if (k < 0)

src/network/networkd-dhcp6.c

@@ -160,6 +160,8 @@ static int dhcp6_pd_remove_old(Link *link, bool force) {
 
         log_link_debug(link, "Removing old DHCPv6 Prefix Delegation addresses and routes.");
 
+        link_dirty(link);
+
         SET_FOREACH(route, link->dhcp6_pd_routes_old) {
                 k = route_remove(route, NULL, link, NULL);
                 if (k < 0)
@@ -202,6 +204,8 @@ int dhcp6_pd_remove(Link *link) {
 
         log_link_debug(link, "Removing DHCPv6 Prefix Delegation addresses and routes.");
 
+        link_dirty(link);
+
         SET_FOREACH(route, link->dhcp6_pd_routes) {
                 k = route_remove(route, NULL, link, NULL);
                 if (k < 0)
@@ -348,6 +352,12 @@ static int dhcp6_pd_address_handler(sd_netlink *rtnl, sd_netlink_message *m, Lin
                         link_enter_failed(link);
                         return 1;
                 }
+
+                r = link_set_routes(link);
+                if (r < 0) {
+                        link_enter_failed(link);
+                        return 1;
+                }
         }
 
         return 1;
@@ -575,6 +585,8 @@ static int dhcp6_pd_prepare(Link *link) {
         if (!link_dhcp6_pd_is_enabled(link))
                 return 0;
 
+        link_dirty(link);
+
         link->dhcp6_pd_address_configured = false;
         link->dhcp6_pd_route_configured = false;
         link->dhcp6_pd_prefixes_assigned = true;
@@ -606,8 +618,14 @@ static int dhcp6_pd_finalize(Link *link) {
         if (link->dhcp6_pd_address_messages == 0) {
                 if (link->dhcp6_pd_prefixes_assigned)
                         link->dhcp6_pd_address_configured = true;
-        } else
+        } else {
                 log_link_debug(link, "Setting DHCPv6 PD addresses");
+                /* address_handler calls link_set_routes() and link_set_nexthop(). Before they are
+                 * called, the related flags must be cleared. Otherwise, the link becomes configured
+                 * state before routes are configured. */
+                link->static_routes_configured = false;
+                link->static_nexthops_configured = false;
+        }
 
         if (link->dhcp6_pd_route_messages == 0) {
                 if (link->dhcp6_pd_prefixes_assigned)
@@ -690,6 +708,8 @@ static int dhcp6_remove_old(Link *link, bool force) {
 
         log_link_debug(link, "Removing old DHCPv6 addresses and routes.");
 
+        link_dirty(link);
+
         SET_FOREACH(route, link->dhcp6_routes_old) {
                 k = route_remove(route, NULL, link, NULL);
                 if (k < 0)
@@ -724,6 +744,8 @@ static int dhcp6_remove(Link *link) {
 
         log_link_debug(link, "Removing DHCPv6 addresses and routes.");
 
+        link_dirty(link);
+
         SET_FOREACH(route, link->dhcp6_routes) {
                 k = route_remove(route, NULL, link, NULL);
                 if (k < 0)
@@ -940,6 +962,12 @@ static int dhcp6_address_handler(sd_netlink *rtnl, sd_netlink_message *m, Link *
                         link_enter_failed(link);
                         return 1;
                 }
+
+                r = link_set_routes(link);
+                if (r < 0) {
+                        link_enter_failed(link);
+                        return 1;
+                }
         }
 
         return 1;
@@ -1086,6 +1114,8 @@ static int dhcp6_lease_ip_acquired(sd_dhcp6_client *client, Link *link) {
         link->dhcp6_address_configured = false;
         link->dhcp6_route_configured = false;
 
+        link_dirty(link);
+
         while ((a = set_steal_first(link->dhcp6_addresses))) {
                 r = set_ensure_put(&link->dhcp6_addresses_old, &address_hash_ops, a);
                 if (r < 0)
@@ -1119,8 +1149,14 @@ static int dhcp6_lease_ip_acquired(sd_dhcp6_client *client, Link *link) {
 
         if (link->dhcp6_address_messages == 0)
                 link->dhcp6_address_configured = true;
-        else
+        else {
                 log_link_debug(link, "Setting DHCPv6 addresses");
+                /* address_handler calls link_set_routes() and link_set_nexthop(). Before they are
+                 * called, the related flags must be cleared. Otherwise, the link becomes configured
+                 * state before routes are configured. */
+                link->static_routes_configured = false;
+                link->static_nexthops_configured = false;
+        }
 
         if (link->dhcp6_route_messages == 0)
                 link->dhcp6_route_configured = true;

src/network/networkd-link.c

@@ -16,7 +16,6 @@
 #include "ethtool-util.h"
 #include "fd-util.h"
 #include "fileio.h"
-#include "fs-util.h"
 #include "ipvlan.h"
 #include "missing_network.h"
 #include "netlink-util.h"
@@ -597,13 +596,14 @@ void link_set_state(Link *link, LinkState state) {
         link->state = state;
 
         link_send_changed(link, "AdministrativeState", NULL);
-        link_dirty(link);
 }
 
 static void link_enter_unmanaged(Link *link) {
         assert(link);
 
         link_set_state(link, LINK_STATE_UNMANAGED);
+
+        link_dirty(link);
 }
 
 int link_stop_engines(Link *link, bool may_keep_dhcp) {
@@ -675,6 +675,8 @@ void link_enter_failed(Link *link) {
         link_set_state(link, LINK_STATE_FAILED);
 
         (void) link_stop_engines(link, false);
+
+        link_dirty(link);
 }
 
 static int link_join_netdevs_after_configured(Link *link) {
@@ -715,6 +717,8 @@ static void link_enter_configured(Link *link) {
         link_set_state(link, LINK_STATE_CONFIGURED);
 
         (void) link_join_netdevs_after_configured(link);
+
+        link_dirty(link);
 }
 
 void link_check_ready(Link *link) {
@@ -1601,8 +1605,6 @@ static int link_put_carrier(Link *link, Link *carrier, Hashmap **h) {
         if (r < 0)
                 return r;
 
-        link_dirty(link);
-
         return 0;
 }
 
@@ -1610,6 +1612,7 @@ static int link_new_bound_by_list(Link *link) {
         Manager *m;
         Link *carrier;
         int r;
+        bool list_updated = false;
 
         assert(link);
         assert(link->manager);
@@ -1627,13 +1630,20 @@ static int link_new_bound_by_list(Link *link) {
                         r = link_put_carrier(link, carrier, &link->bound_by_links);
                         if (r < 0)
                                 return r;
+
+                        list_updated = true;
                 }
         }
 
+        if (list_updated)
+                link_dirty(link);
+
         HASHMAP_FOREACH(carrier, link->bound_by_links) {
                 r = link_put_carrier(carrier, link, &carrier->bound_to_links);
                 if (r < 0)
                         return r;
+
+                link_dirty(carrier);
         }
 
         return 0;
@@ -1643,6 +1653,7 @@ static int link_new_bound_to_list(Link *link) {
         Manager *m;
         Link *carrier;
         int r;
+        bool list_updated = false;
 
         assert(link);
         assert(link->manager);
@@ -1660,13 +1671,20 @@ static int link_new_bound_to_list(Link *link) {
                         r = link_put_carrier(link, carrier, &link->bound_to_links);
                         if (r < 0)
                                 return r;
+
+                        list_updated = true;
                 }
         }
 
+        if (list_updated)
+                link_dirty(link);
+
         HASHMAP_FOREACH (carrier, link->bound_to_links) {
                 r = link_put_carrier(carrier, link, &carrier->bound_by_links);
                 if (r < 0)
                         return r;
+
+                link_dirty(carrier);
         }
 
         return 0;
@@ -1695,32 +1713,23 @@ static int link_new_carrier_maps(Link *link) {
 }
 
 static void link_free_bound_to_list(Link *link) {
-        bool updated = false;
         Link *bound_to;
 
-        assert(link);
-
-        while ((bound_to = hashmap_steal_first(link->bound_to_links))) {
-                updated = true;
+        HASHMAP_FOREACH (bound_to, link->bound_to_links) {
+                hashmap_remove(link->bound_to_links, INT_TO_PTR(bound_to->ifindex));
 
                 if (hashmap_remove(bound_to->bound_by_links, INT_TO_PTR(link->ifindex)))
                         link_dirty(bound_to);
         }
 
-        if (updated)
-                link_dirty(link);
-
         return;
 }
 
 static void link_free_bound_by_list(Link *link) {
-        bool updated = false;
         Link *bound_by;
 
-        assert(link);
-
-        while ((bound_by = hashmap_steal_first(link->bound_by_links))) {
-                updated = true;
+        HASHMAP_FOREACH (bound_by, link->bound_by_links) {
+                hashmap_remove(link->bound_by_links, INT_TO_PTR(bound_by->ifindex));
 
                 if (hashmap_remove(bound_by->bound_to_links, INT_TO_PTR(link->ifindex))) {
                         link_dirty(bound_by);
@@ -1728,17 +1737,26 @@ static void link_free_bound_by_list(Link *link) {
                 }
         }
 
-        if (updated)
-                link_dirty(link);
-
         return;
 }
 
 static void link_free_carrier_maps(Link *link) {
+        bool list_updated = false;
+
         assert(link);
 
+        if (!hashmap_isempty(link->bound_to_links)) {
                 link_free_bound_to_list(link);
+                list_updated = true;
+        }
+
+        if (!hashmap_isempty(link->bound_by_links)) {
                 link_free_bound_by_list(link);
+                list_updated = true;
+        }
+
+        if (list_updated)
+                link_dirty(link);
 
         return;
 }
@@ -1925,6 +1943,7 @@ static int link_enter_join_netdev(Link *link) {
 
         link_set_state(link, LINK_STATE_CONFIGURING);
 
+        link_dirty(link);
         link->enslaving = 0;
 
         if (link->network->bond) {
@@ -2247,8 +2266,9 @@ static int link_reconfigure_internal(Link *link, sd_netlink_message *m, bool for
         link_unref(set_remove(link->manager->links_requesting_uuid, link));
 
         /* Then, apply new .network file */
-        link->network = network_ref(network);
-        link_dirty(link);
+        r = network_apply(network, link);
+        if (r < 0)
+                return r;
 
         r = link_new_carrier_maps(link);
         if (r < 0)
@@ -2256,6 +2276,7 @@ static int link_reconfigure_internal(Link *link, sd_netlink_message *m, bool for
 
         link_set_state(link, LINK_STATE_INITIALIZED);
         link->activated = false;
+        link_dirty(link);
 
         /* link_configure_duid() returns 0 if it requests product UUID. In that case,
          * link_configure() is called later asynchronously. */
@@ -2369,8 +2390,9 @@ static int link_initialized_and_synced(Link *link) {
                                 log_link_debug(link, "Ignoring DHCP server for loopback link");
                 }
 
-                link->network = network_ref(network);
-                link_dirty(link);
+                r = network_apply(network, link);
+                if (r < 0)
+                        return r;
         }
 
         r = link_new_bound_to_list(link);
@@ -3176,9 +3198,10 @@ int link_save(Link *link) {
         if (r < 0)
                 goto fail;
 
-        r = conservative_rename(temp_path, link->state_file);
-        if (r < 0)
+        if (rename(temp_path, link->state_file) < 0) {
+                r = -errno;
                 goto fail;
+        }
 
         return 0;
 

src/network/networkd-manager.c

@@ -23,7 +23,6 @@
 #include "fd-util.h"
 #include "fileio.h"
 #include "firewall-util.h"
-#include "fs-util.h"
 #include "local-addresses.h"
 #include "netlink-util.h"
 #include "network-internal.h"
@@ -710,9 +709,10 @@ static int manager_save(Manager *m) {
         if (r < 0)
                 goto fail;
 
-        r = conservative_rename(temp_path, m->state_file);
-        if (r < 0)
+        if (rename(temp_path, m->state_file) < 0) {
+                r = -errno;
                 goto fail;
+        }
 
         if (m->operational_state != operstate) {
                 m->operational_state = operstate;

src/network/networkd-ndisc.c

@@ -118,7 +118,6 @@ static int ndisc_remove_old_one(Link *link, const struct in6_addr *router, bool
         NDiscDNSSL *dnssl;
         NDiscRDNSS *rdnss;
         int k, r = 0;
-        bool updated = false;
 
         assert(link);
         assert(router);
@@ -183,19 +182,12 @@ static int ndisc_remove_old_one(Link *link, const struct in6_addr *router, bool
                 }
 
         SET_FOREACH(rdnss, link->ndisc_rdnss)
-                if (rdnss->marked && IN6_ARE_ADDR_EQUAL(&rdnss->router, router)) {
+                if (rdnss->marked && IN6_ARE_ADDR_EQUAL(&rdnss->router, router))
                         free(set_remove(link->ndisc_rdnss, rdnss));
-                        updated = true;
-                }
 
         SET_FOREACH(dnssl, link->ndisc_dnssl)
-                if (dnssl->marked && IN6_ARE_ADDR_EQUAL(&dnssl->router, router)) {
+                if (dnssl->marked && IN6_ARE_ADDR_EQUAL(&dnssl->router, router))
                         free(set_remove(link->ndisc_dnssl, dnssl));
-                        updated = true;
-                }
-
-        if (updated)
-                link_dirty(link);
 
         return r;
 }
@@ -420,6 +412,12 @@ static int ndisc_address_handler(sd_netlink *rtnl, sd_netlink_message *m, Link *
                         link_enter_failed(link);
                         return 1;
                 }
+
+                r = link_set_routes(link);
+                if (r < 0) {
+                        link_enter_failed(link);
+                        return 1;
+                }
         }
 
         return 1;
@@ -934,7 +932,6 @@ static int ndisc_router_process_rdnss(Link *link, sd_ndisc_router *rt) {
         struct in6_addr router;
         NDiscRDNSS *rdnss;
         usec_t time_now;
-        bool updated = false;
         int n, r;
 
         assert(link);
@@ -996,13 +993,8 @@ static int ndisc_router_process_rdnss(Link *link, sd_ndisc_router *rt) {
                 if (r < 0)
                         return log_oom();
                 assert(r > 0);
-
-                updated = true;
         }
 
-        if (updated)
-                link_dirty(link);
-
         return 0;
 }
 
@@ -1027,7 +1019,6 @@ static int ndisc_router_process_dnssl(Link *link, sd_ndisc_router *rt) {
         uint32_t lifetime;
         usec_t time_now;
         NDiscDNSSL *dnssl;
-        bool updated = false;
         char **j;
         int r;
 
@@ -1087,13 +1078,8 @@ static int ndisc_router_process_dnssl(Link *link, sd_ndisc_router *rt) {
                 if (r < 0)
                         return log_oom();
                 assert(r > 0);
-
-                updated = true;
         }
 
-        if (updated)
-                link_dirty(link);
-
         return 0;
 }
 
@@ -1218,6 +1204,8 @@ static int ndisc_router_handler(Link *link, sd_ndisc_router *rt) {
         link->ndisc_addresses_configured = false;
         link->ndisc_routes_configured = false;
 
+        link_dirty(link);
+
         SET_FOREACH(na, link->ndisc_addresses)
                 if (IN6_ARE_ADDR_EQUAL(&na->router, &router.in6))
                         na->marked = true;
@@ -1256,9 +1244,16 @@ static int ndisc_router_handler(Link *link, sd_ndisc_router *rt) {
 
         if (link->ndisc_addresses_messages == 0)
                 link->ndisc_addresses_configured = true;
-        else
+        else {
                 log_link_debug(link, "Setting SLAAC addresses.");
 
+                /* address_handler calls link_set_routes() and link_set_nexthop(). Before they are
+                 * called, the related flags must be cleared. Otherwise, the link becomes configured
+                 * state before routes are configured. */
+                link->static_routes_configured = false;
+                link->static_nexthops_configured = false;
+        }
+
         if (link->ndisc_routes_messages == 0)
                 link->ndisc_routes_configured = true;
         else
@@ -1345,6 +1340,7 @@ void ndisc_vacuum(Link *link) {
         NDiscRDNSS *r;
         NDiscDNSSL *d;
         usec_t time_now;
+        bool updated = false;
 
         assert(link);
 
@@ -1353,12 +1349,19 @@ void ndisc_vacuum(Link *link) {
         time_now = now(clock_boottime_or_monotonic());
 
         SET_FOREACH(r, link->ndisc_rdnss)
-                if (r->valid_until < time_now)
+                if (r->valid_until < time_now) {
                         free(set_remove(link->ndisc_rdnss, r));
+                        updated = true;
+                }
 
         SET_FOREACH(d, link->ndisc_dnssl)
-                if (d->valid_until < time_now)
+                if (d->valid_until < time_now) {
                         free(set_remove(link->ndisc_dnssl, d));
+                        updated = true;
+                }
+
+        if (updated)
+                link_dirty(link);
 }
 
 void ndisc_flush(Link *link) {

src/network/networkd-network.c

@@ -736,6 +736,21 @@ int network_get(Manager *manager, unsigned short iftype, sd_device *device,
         return -ENOENT;
 }
 
+int network_apply(Network *network, Link *link) {
+        assert(network);
+        assert(link);
+
+        link->network = network_ref(network);
+
+        if (network->n_dns > 0 ||
+            !strv_isempty(network->ntp) ||
+            !ordered_set_isempty(network->search_domains) ||
+            !ordered_set_isempty(network->route_domains))
+                link_dirty(link);
+
+        return 0;
+}
+
 bool network_has_static_ipv6_configurations(Network *network) {
         Address *address;
         Route *route;

src/network/networkd-network.h

@@ -329,6 +329,7 @@ int network_get(Manager *manager, unsigned short iftype, sd_device *device,
                 const struct ether_addr *mac, const struct ether_addr *permanent_mac,
                 enum nl80211_iftype wlan_iftype, const char *ssid, const struct ether_addr *bssid,
                 Network **ret);
+int network_apply(Network *network, Link *link);
 void network_apply_anonymize_if_set(Network *network);
 
 bool network_has_static_ipv6_configurations(Network *network);

src/resolve/resolved-resolv-conf.c

@@ -360,7 +360,7 @@ int manager_write_resolv_conf(Manager *m) {
                         goto fail;
                 }
 
-                r = conservative_rename(temp_path_stub, PRIVATE_STUB_RESOLV_CONF);
+                r = conservative_rename(AT_FDCWD, temp_path_stub, AT_FDCWD, PRIVATE_STUB_RESOLV_CONF);
                 if (r < 0)
                         log_error_errno(r, "Failed to move new %s into place: %m", PRIVATE_STUB_RESOLV_CONF);
 
@@ -370,7 +370,7 @@ int manager_write_resolv_conf(Manager *m) {
                         log_error_errno(r, "Failed to symlink %s: %m", PRIVATE_STUB_RESOLV_CONF);
         }
 
-        r = conservative_rename(temp_path_uplink, PRIVATE_UPLINK_RESOLV_CONF);
+        r = conservative_rename(AT_FDCWD, temp_path_uplink, AT_FDCWD, PRIVATE_UPLINK_RESOLV_CONF);
         if (r < 0)
                 log_error_errno(r, "Failed to move new %s into place: %m", PRIVATE_UPLINK_RESOLV_CONF);
 

src/test/test-fs-util.c

@@ -847,37 +847,37 @@ static void test_conservative_rename(void) {
 
         /* Check that the hardlinked "copy" is detected */
         assert_se(link(p, q) >= 0);
-        assert_se(conservative_renameat(AT_FDCWD, q, AT_FDCWD, p) == 0);
+        assert_se(conservative_rename(AT_FDCWD, q, AT_FDCWD, p) == 0);
         assert_se(access(q, F_OK) < 0 && errno == ENOENT);
 
         /* Check that a manual copy is detected */
         assert_se(copy_file(p, q, 0, (mode_t) -1, 0, 0, COPY_REFLINK) >= 0);
-        assert_se(conservative_renameat(AT_FDCWD, q, AT_FDCWD, p) == 0);
+        assert_se(conservative_rename(AT_FDCWD, q, AT_FDCWD, p) == 0);
         assert_se(access(q, F_OK) < 0 && errno == ENOENT);
 
         /* Check that a manual new writeout is also detected */
         assert_se(write_string_file(q, "this is a test", WRITE_STRING_FILE_CREATE) >= 0);
-        assert_se(conservative_renameat(AT_FDCWD, q, AT_FDCWD, p) == 0);
+        assert_se(conservative_rename(AT_FDCWD, q, AT_FDCWD, p) == 0);
         assert_se(access(q, F_OK) < 0 && errno == ENOENT);
 
         /* Check that a minimally changed version is detected */
         assert_se(write_string_file(q, "this is a_test", WRITE_STRING_FILE_CREATE) >= 0);
-        assert_se(conservative_renameat(AT_FDCWD, q, AT_FDCWD, p) > 0);
+        assert_se(conservative_rename(AT_FDCWD, q, AT_FDCWD, p) > 0);
         assert_se(access(q, F_OK) < 0 && errno == ENOENT);
 
         /* Check that this really is new updated version */
         assert_se(write_string_file(q, "this is a_test", WRITE_STRING_FILE_CREATE) >= 0);
-        assert_se(conservative_renameat(AT_FDCWD, q, AT_FDCWD, p) == 0);
+        assert_se(conservative_rename(AT_FDCWD, q, AT_FDCWD, p) == 0);
         assert_se(access(q, F_OK) < 0 && errno == ENOENT);
 
         /* Make sure we detect extended files */
         assert_se(write_string_file(q, "this is a_testx", WRITE_STRING_FILE_CREATE) >= 0);
-        assert_se(conservative_renameat(AT_FDCWD, q, AT_FDCWD, p) > 0);
+        assert_se(conservative_rename(AT_FDCWD, q, AT_FDCWD, p) > 0);
         assert_se(access(q, F_OK) < 0 && errno == ENOENT);
 
         /* Make sure we detect truncated files */
         assert_se(write_string_file(q, "this is a_test", WRITE_STRING_FILE_CREATE) >= 0);
-        assert_se(conservative_renameat(AT_FDCWD, q, AT_FDCWD, p) > 0);
+        assert_se(conservative_rename(AT_FDCWD, q, AT_FDCWD, p) > 0);
         assert_se(access(q, F_OK) < 0 && errno == ENOENT);
 }