.github/workflows/build-test.sh

@@ -124,15 +124,10 @@ fi
 
 # This is added by default, and it is often broken, but we don't need anything from it
 sudo rm -f /etc/apt/sources.list.d/microsoft-prod.{list,sources}
-if grep -q 'VERSION_CODENAME=jammy' /usr/lib/os-release; then
-    sudo add-apt-repository -y --no-update ppa:upstream-systemd-ci/systemd-ci
-    sudo add-apt-repository -y --no-update --enable-source
-else
-    # add-apt-repository --enable-source does not work on deb822 style sources.
-    for f in /etc/apt/sources.list.d/*.sources; do
-        sudo sed -i "s/Types: deb/Types: deb deb-src/g" "$f"
-    done
-fi
+# add-apt-repository --enable-source does not work on deb822 style sources.
+for f in /etc/apt/sources.list.d/*.sources; do
+    sudo sed -i "s/Types: deb/Types: deb deb-src/g" "$f"
+done
 sudo apt-get -y update
 sudo apt-get -y build-dep systemd
 sudo apt-get -y install "${PACKAGES[@]}"
@@ -143,11 +138,6 @@ sudo apt-get -y install "${PACKAGES[@]}"
 pip3 install --user -r .github/workflows/requirements.txt --require-hashes --break-system-packages
 export PATH="$HOME/.local/bin:$PATH"
 
-if [[ -n "$CUSTOM_PYTHON" ]]; then
-    # If CUSTOM_PYTHON is set we need to pull jinja2 from pip, as a local interpreter is used
-    pip3 install --user --break-system-packages jinja2
-fi
-
 $CC --version
 meson --version
 ninja --version

.github/workflows/build-test.yml

@@ -19,13 +19,12 @@ jobs:
   build:
     runs-on: ${{ matrix.runner }}
     concurrency:
-      group: ${{ github.workflow }}-${{ toJSON(matrix.env) }}-${{ github.ref }}-${{ matrix.runner }}-${{ matrix.python-version }}
+      group: ${{ github.workflow }}-${{ toJSON(matrix.env) }}-${{ github.ref }}-${{ matrix.runner }}
       cancel-in-progress: true
     strategy:
       fail-fast: false
       matrix:
         runner: [ ubuntu-24.04 ]
-        python-version: [ '' ]
         env:
           - { COMPILER: "gcc",   COMPILER_VERSION: "11", LINKER: "bfd"  }
           - { COMPILER: "gcc",   COMPILER_VERSION: "13", LINKER: "mold" }
@@ -35,26 +34,13 @@ jobs:
         include:
           - env: { COMPILER: "gcc",   COMPILER_VERSION: "11", LINKER: "bfd"  }
             runner: [ ubuntu-24.04-ppc64le ]
-            python-version: ''
           - env: { COMPILER: "gcc",   COMPILER_VERSION: "13", LINKER: "mold"  }
             runner: [ ubuntu-24.04-arm ]
-            python-version: ''
           - env: { COMPILER: "clang", COMPILER_VERSION: "18", LINKER: "lld"  }
             runner: [ ubuntu-24.04-s390x ]
-            python-version: ''
-          # Do one run with the oldest supported python version to ensure there are no regressions
-          # It is only available on Jammy, which is useful as it will also provide a build check for
-          # older glibc
-          - env: { COMPILER: "gcc",   COMPILER_VERSION: "12", LINKER: "bfd", CUSTOM_PYTHON: "1"  }
-            runner: [ ubuntu-22.04 ]
-            python-version: '3.7'
     env: ${{ matrix.env }}
     steps:
       - name: Repository checkout
         uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
-      - name: Setup python
-        uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065
-        with:
-          python-version: ${{ matrix.python-version }}
       - name: Build check
         run: .github/workflows/build-test.sh

hwdb.d/60-keyboard.hwdb

@@ -285,6 +285,21 @@ evdev:atkbd:dmi:bvn*:bvr*:bd*:svnASUS:pn*:*
  KEYBOARD_KEY_ee=volumedown
  KEYBOARD_KEY_ef=mute
 
+# Asus ROG series
+# Prevents the keyboard from sending POWER and SLEEP key presses
+# when the keyboard goes to sleep.
+evdev:input:b*v0B05p*e0111*
+ KEYBOARD_KEY_10081=reserved
+ KEYBOARD_KEY_10082=reserved
+ KEYBOARD_KEY_70070=reserved
+ KEYBOARD_KEY_70071=reserved
+ KEYBOARD_KEY_70072=reserved
+ KEYBOARD_KEY_70073=reserved
+ KEYBOARD_KEY_70074=reserved
+ KEYBOARD_KEY_70075=reserved
+ KEYBOARD_KEY_70076=reserved
+ KEYBOARD_KEY_70077=reserved
+
 # Asus TF103C misses the home button in its PNP0C40 GPIO resources
 # causing the volume-button mappings to be off by one, correct this
 evdev:name:gpio-keys:phys:gpio-keys/input0:ev:3:dmi:*:svnASUSTeKCOMPUTERINC.:pnTF103C*:*

meson.build

@@ -1958,8 +1958,6 @@ endif
 
 conf.set10('HAVE_VMLINUX_H', use_provided_vmlinux_h or use_generated_vmlinux_h)
 
-conf.set10('ENABLE_SYSCTL_BPF', conf.get('HAVE_VMLINUX_H') == 1 and libbpf.version().version_compare('>= 0.7'))
-
 #####################################################################
 
 check_version_history_py   = files('tools/check-version-history.py')

profile.d/70-systemd-shell-extra.sh

@@ -13,16 +13,10 @@
 # are propagated into these environment variables by pam_systemd(8).
 
 if [ -n "${SHELL_PROMPT_PREFIX-}" ]; then
-    if [ -n "${BASH_VERSION-}" ] && [ "$PS1" = "\\s-\\v\\\$ " ]; then
-        PS1="[\u@\h \W]\\$ "
-    fi
     PS1="$SHELL_PROMPT_PREFIX$PS1"
 fi
 
 if [ -n "${SHELL_PROMPT_SUFFIX-}" ]; then
-    if [ -n "${BASH_VERSION-}" ] && [ "$PS1" = "\\s-\\v\\\$ " ]; then
-        PS1="[\u@\h \W]\\$ "
-    fi
     PS1="$PS1$SHELL_PROMPT_SUFFIX"
 fi
 

src/basic/filesystem-sets.py

@@ -307,7 +307,8 @@ def generate_fs_in_group():
     print('        switch (fs_group) {')
 
     for name, _, *filesystems in FILESYSTEM_SETS:
-        magics = sorted(set(sum((NAME_TO_MAGIC[fs] for fs in filesystems), [])))
+        magics = sorted(set(sum((NAME_TO_MAGIC[fs] for fs in filesystems),
+                                start=[])))
         enum = 'FILESYSTEM_SET_' + name[1:].upper().replace('-', '_')
         print(f'        case {enum}:')
         opts = '\n                    || '.join(f'F_TYPE_EQUAL(st->f_type, {magic})'
@@ -354,7 +355,7 @@ def magic_defines():
 
 def check():
     kernel_magics = set(magic_defines())
-    our_magics = set(sum(NAME_TO_MAGIC.values(), []))
+    our_magics = set(sum(NAME_TO_MAGIC.values(), start=[]))
     extra = kernel_magics - our_magics
     if extra:
         sys.exit(f"kernel knows additional filesystem magics: {', '.join(sorted(extra))}")

src/login/user-runtime-dir.c

@@ -210,11 +210,6 @@ static int apply_tmpfs_quota(
         assert(uid_is_valid(uid));
 
         STRV_FOREACH(p, paths) {
-                if (limit == UINT64_MAX && scale == UINT32_MAX) {
-                        log_debug("No disk quota on '%s' is requested.", *p);
-                        continue;
-                }
-
                 _cleanup_close_ int fd = open(*p, O_DIRECTORY|O_CLOEXEC);
                 if (fd < 0) {
                         log_warning_errno(errno, "Failed to open '%s' in order to set quota, ignoring: %m", *p);
@@ -289,8 +284,7 @@ static int apply_tmpfs_quota(
                         log_debug_errno(r, "Lacking privileges to set UID quota on %s, skipping: %m", *p);
                         continue;
                 } else if (r < 0) {
-                        log_warning_errno(r, "Failed to set disk quota limit to %s on %s for UID " UID_FMT ", ignoring: %m",
-                                          FORMAT_BYTES(v * QIF_DQBLKSIZE), *p, uid);
+                        log_warning_errno(r, "Failed to set disk quota limit to '%s' on %s for UID " UID_FMT ", ignoring: %m", FORMAT_BYTES(v), *p, uid);
                         continue;
                 }
 

src/network/bpf/sysctl-monitor/meson.build

@@ -1,6 +1,6 @@
 # SPDX-License-Identifier: LGPL-2.1-or-later
 
-if conf.get('ENABLE_SYSCTL_BPF') != 1
+if conf.get('HAVE_VMLINUX_H') != 1
         subdir_done()
 endif
 

src/network/meson.build

@@ -140,7 +140,7 @@ networkctl_sources = files(
 networkd_network_gperf_gperf = files('networkd-network-gperf.gperf')
 networkd_netdev_gperf_gperf = files('netdev/netdev-gperf.gperf')
 
-if conf.get('ENABLE_SYSCTL_BPF') == 1
+if conf.get('HAVE_VMLINUX_H') == 1
         systemd_networkd_extract_sources += sysctl_monitor_skel_h
 endif
 

src/network/networkd-manager.c

@@ -646,7 +646,7 @@ int manager_new(Manager **ret, bool test_mode) {
                 .dhcp_server_persist_leases = DHCP_SERVER_PERSIST_LEASES_YES,
                 .serialization_fd = -EBADF,
                 .ip_forwarding = { -1, -1, },
-#if ENABLE_SYSCTL_BPF
+#if HAVE_VMLINUX_H
                 .cgroup_fd = -EBADF,
 #endif
         };

src/network/networkd-manager.h

@@ -117,7 +117,7 @@ typedef struct Manager {
 
         /* sysctl */
         int ip_forwarding[2];
-#if ENABLE_SYSCTL_BPF
+#if HAVE_VMLINUX_H
         Hashmap *sysctl_shadow;
         sd_event_source *sysctl_event_source;
         struct ring_buffer *sysctl_buffer;
@@ -148,7 +148,7 @@ int manager_set_timezone(Manager *m, const char *timezone);
 int manager_reload(Manager *m, sd_bus_message *message);
 
 static inline Hashmap** manager_get_sysctl_shadow(Manager *manager) {
-#if ENABLE_SYSCTL_BPF
+#if HAVE_VMLINUX_H
         return &ASSERT_PTR(manager)->sysctl_shadow;
 #else
         return NULL;

src/network/networkd-sysctl.c

@@ -26,7 +26,7 @@
 #include "string-util.h"
 #include "sysctl-util.h"
 
-#if ENABLE_SYSCTL_BPF
+#if HAVE_VMLINUX_H
 
 #include "bpf-link.h"
 #include "bpf/sysctl-monitor/sysctl-monitor-skel.h"

src/network/networkd-sysctl.h

@@ -22,7 +22,7 @@ typedef enum IPReversePathFilter {
         _IP_REVERSE_PATH_FILTER_INVALID = -EINVAL,
 } IPReversePathFilter;
 
-#if ENABLE_SYSCTL_BPF
+#if HAVE_VMLINUX_H
 int manager_install_sysctl_monitor(Manager *manager);
 void manager_remove_sysctl_monitor(Manager *manager);
 int link_clear_sysctl_shadows(Link *link);

src/oom/test-oomd-util.c

@@ -68,8 +68,6 @@ TEST(oomd_cgroup_kill) {
         /* Create another cgroup below this one for the pids we forked off. We need this to be managed
          * by the test so that pid1 doesn't delete it before we can read the xattrs. */
         ASSERT_NOT_NULL(subcgroup = path_join(cgroup, "oomdkilltest"));
-        /* Always start clean, in case of repeated runs and failures */
-        ASSERT_OK(cg_trim(subcgroup, /* delete_root */ true));
         ASSERT_OK(cg_create(subcgroup));
 
         /* If we don't have permissions to set xattrs we're likely in a userns or missing capabilities */
@@ -108,8 +106,6 @@ TEST(oomd_cgroup_kill) {
                 ASSERT_OK(cg_get_xattr(subcgroup, "user.oomd_kill", &v, /* ret_size= */ NULL));
                 ASSERT_STREQ(v, i == 0 ? "2" : "4");
         }
-
-        ASSERT_OK(cg_trim(subcgroup, /* delete_root */ true));
 }
 
 TEST(oomd_cgroup_context_acquire_and_insert) {

src/shared/seccomp-util.c

@@ -1,9 +1,5 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-/* This needs to be included before the seccomp headers, otherwise missing syscalls will be defined in this
- * file and pass the ifdef, but they won't be defined in the seccomp headers so things like
- * SCMP_SYS(fchmodat2) will resolve as empty and fail the build with older glibc/libseccomp. */
-#include <sys/syscall.h>
 #include <fcntl.h>
 #include <linux/seccomp.h>
 #include <sched.h>

src/test/test-bpf-token.c

@@ -7,7 +7,7 @@
 #include "tests.h"
 
 static int intro(void) {
-#if defined(LIBBPF_MAJOR_VERSION) && (LIBBPF_MAJOR_VERSION > 1 || (LIBBPF_MAJOR_VERSION == 1 && LIBBPF_MINOR_VERSION >= 5))
+#if __LIBBPF_CURRENT_VERSION_GEQ(1, 5)
         _cleanup_close_ int bpffs_fd = open("/sys/fs/bpf", O_RDONLY);
         if (bpffs_fd < 0)
                 return log_error_errno(errno, "Failed to open '/sys/fs/bpf': %m");

src/test/test-calendarspec.c

@@ -190,16 +190,16 @@ TEST(calendar_spec_one) {
 TEST(calendar_spec_next) {
         test_next("2016-03-27 03:17:00", "", 12345, 1459048620000000);
         test_next("2016-03-27 03:17:00", "Europe/Berlin", 12345, 1459041420000000);
-        test_next("2016-03-27 03:17:00", "Europe/Helsinki", 12345, -1);
+        test_next("2016-03-27 03:17:00", "Europe/Kyiv", 12345, -1);
         test_next("2016-03-27 03:17:00 UTC", NULL, 12345, 1459048620000000);
         test_next("2016-03-27 03:17:00 UTC", "", 12345, 1459048620000000);
         test_next("2016-03-27 03:17:00 UTC", "Europe/Berlin", 12345, 1459048620000000);
-        test_next("2016-03-27 03:17:00 UTC", "Europe/Helsinki", 12345, 1459048620000000);
-        test_next("2016-03-27 03:17:00.420000001 UTC", "Europe/Helsinki", 12345, 1459048620420000);
-        test_next("2016-03-27 03:17:00.4200005 UTC", "Europe/Helsinki", 12345, 1459048620420001);
-        test_next("2015-11-13 09:11:23.42", "Europe/Helsinki", 12345, 1447398683420000);
-        test_next("2015-11-13 09:11:23.42/1.77", "Europe/Helsinki", 1447398683420000, 1447398685190000);
-        test_next("2015-11-13 09:11:23.42/1.77", "Europe/Helsinki", 1447398683419999, 1447398683420000);
+        test_next("2016-03-27 03:17:00 UTC", "Europe/Kyiv", 12345, 1459048620000000);
+        test_next("2016-03-27 03:17:00.420000001 UTC", "Europe/Kyiv", 12345, 1459048620420000);
+        test_next("2016-03-27 03:17:00.4200005 UTC", "Europe/Kyiv", 12345, 1459048620420001);
+        test_next("2015-11-13 09:11:23.42", "Europe/Kyiv", 12345, 1447398683420000);
+        test_next("2015-11-13 09:11:23.42/1.77", "Europe/Kyiv", 1447398683420000, 1447398685190000);
+        test_next("2015-11-13 09:11:23.42/1.77", "Europe/Kyiv", 1447398683419999, 1447398683420000);
         test_next("Sun 16:00:00", "Europe/Berlin", 1456041600123456, 1456066800000000);
         test_next("*-04-31", "", 12345, -1);
         test_next("2016-02~01 UTC", "", 12345, 1456704000000000);
@@ -219,7 +219,7 @@ TEST(calendar_spec_next) {
         test_next("2017-04-02 03:30:00 Pacific/Auckland", "", 12345, 1491060600000000);
         /* Confirm that timezones in the Spec work regardless of current timezone */
         test_next("2017-09-09 20:42:00 Pacific/Auckland", "", 12345, 1504946520000000);
-        test_next("2017-09-09 20:42:00 Pacific/Auckland", "Europe/Helsinki", 12345, 1504946520000000);
+        test_next("2017-09-09 20:42:00 Pacific/Auckland", "Europe/Kyiv", 12345, 1504946520000000);
         /* Check that we don't start looping if mktime() moves us backwards */
         test_next("Sun *-*-* 01:00:00 Europe/Dublin", "", 1616412478000000, 1617494400000000);
         test_next("Sun *-*-* 01:00:00 Europe/Dublin", "IST", 1616412478000000, 1617494400000000);

src/test/test-cgroup.c

@@ -51,12 +51,10 @@ TEST(cg_create) {
         _cleanup_free_ char *here = NULL;
         ASSERT_OK(cg_pid_get_path_shifted(0, NULL, &here));
 
-        /* cg_* will use path_simplify(), so use it here too otherwise when running in a container at the
-         * root it asserts with "/test-b != //test-b" */
-        _cleanup_free_ char *test_a = ASSERT_NOT_NULL(path_simplify(path_join(here, "/test-a"))),
-                            *test_b = ASSERT_NOT_NULL(path_simplify(path_join(here, "/test-b"))),
-                            *test_c = ASSERT_NOT_NULL(path_simplify(path_join(here, "/test-b/test-c"))),
-                            *test_d = ASSERT_NOT_NULL(path_simplify(path_join(here, "/test-b/test-d")));
+        _cleanup_free_ char *test_a = ASSERT_NOT_NULL(path_join(here, "/test-a")),
+                            *test_b = ASSERT_NOT_NULL(path_join(here, "/test-b")),
+                            *test_c = ASSERT_NOT_NULL(path_join(here, "/test-b/test-c")),
+                            *test_d = ASSERT_NOT_NULL(path_join(here, "/test-b/test-d"));
         char *path;
 
         log_info("Paths for test:\n%s\n%s", test_a, test_b);