summaryrefslogtreecommitdiffstats
path: root/patches
diff options
context:
space:
mode:
authorMichael Olbrich <m.olbrich@pengutronix.de>2019-09-30 18:02:08 +0200
committerMichael Olbrich <m.olbrich@pengutronix.de>2019-09-30 18:02:11 +0200
commit6a89fda2431bbb08db7389b41eeb5895490fd95b (patch)
treee6062f720481ad92c9a580bf33f52cc5b8978c09 /patches
parenta7f1c593892d99d12d89ba57336123ccad4260dc (diff)
downloadptxdist-6a89fda2431bbb08db7389b41eeb5895490fd95b.tar.gz
ptxdist-6a89fda2431bbb08db7389b41eeb5895490fd95b.tar.xz
systemd: fix booting from NFS
For NFS and ext images that where created with genext2fs, readir() returns 'd_type == DT_UNKNOWN' for all files. This confused the unit load code and systemd started a incomplete 'default.target' with no login available. Add upstream fix to avoid this. Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
Diffstat (limited to 'patches')
-rw-r--r--patches/systemd-243-51-gfab6f010ac6c/0001-Handle-d_type-DT_UNKNOWN-correctly.patch90
-rw-r--r--patches/systemd-243-51-gfab6f010ac6c/series4
2 files changed, 94 insertions, 0 deletions
diff --git a/patches/systemd-243-51-gfab6f010ac6c/0001-Handle-d_type-DT_UNKNOWN-correctly.patch b/patches/systemd-243-51-gfab6f010ac6c/0001-Handle-d_type-DT_UNKNOWN-correctly.patch
new file mode 100644
index 000000000..01e72f125
--- /dev/null
+++ b/patches/systemd-243-51-gfab6f010ac6c/0001-Handle-d_type-DT_UNKNOWN-correctly.patch
@@ -0,0 +1,90 @@
+From: Michael Olbrich <m.olbrich@pengutronix.de>
+Date: Sun, 29 Sep 2019 13:43:00 +0200
+Subject: [PATCH] Handle d_type == DT_UNKNOWN correctly
+
+As documented in the man-page, readdir() may return a directory entry with
+d_type == DT_UNKNOWN. This must be handled for regular filesystems.
+
+dirent_ensure_type() is available to set d_type if necessary. Use it in
+some more places.
+
+Without this systemd will fail to boot correctly with nfsroot and some
+other filesystems.
+
+Closes #13609
+---
+ src/core/smack-setup.c | 2 ++
+ src/login/logind-dbus.c | 1 +
+ src/login/logind.c | 1 +
+ src/shared/bootspec.c | 1 +
+ src/shared/unit-file.c | 1 +
+ 5 files changed, 6 insertions(+)
+
+diff --git a/src/core/smack-setup.c b/src/core/smack-setup.c
+index b95e6239d4b2..e2d4c0d6e172 100644
+--- a/src/core/smack-setup.c
++++ b/src/core/smack-setup.c
+@@ -84,6 +84,7 @@ static int write_access2_rules(const char *srcdir) {
+ FOREACH_DIRENT(entry, dir, return 0) {
+ _cleanup_fclose_ FILE *policy = NULL;
+
++ dirent_ensure_type(dir, entry);
+ if (!dirent_is_file(entry))
+ continue;
+
+@@ -150,6 +151,7 @@ static int write_cipso2_rules(const char *srcdir) {
+ FOREACH_DIRENT(entry, dir, return 0) {
+ _cleanup_fclose_ FILE *policy = NULL;
+
++ dirent_ensure_type(dir, entry);
+ if (!dirent_is_file(entry))
+ continue;
+
+diff --git a/src/login/logind-dbus.c b/src/login/logind-dbus.c
+index 30b9a66334fa..679eb6a62ec0 100644
+--- a/src/login/logind-dbus.c
++++ b/src/login/logind-dbus.c
+@@ -1378,6 +1378,7 @@ static int flush_devices(Manager *m) {
+ struct dirent *de;
+
+ FOREACH_DIRENT_ALL(de, d, break) {
++ dirent_ensure_type(d, de);
+ if (!dirent_is_file(de))
+ continue;
+
+diff --git a/src/login/logind.c b/src/login/logind.c
+index 3a16e6c8712a..d889d7b9bec9 100644
+--- a/src/login/logind.c
++++ b/src/login/logind.c
+@@ -291,6 +291,7 @@ static int manager_enumerate_linger_users(Manager *m) {
+ FOREACH_DIRENT(de, d, return -errno) {
+ int k;
+
++ dirent_ensure_type(d, de);
+ if (!dirent_is_file(de))
+ continue;
+
+diff --git a/src/shared/bootspec.c b/src/shared/bootspec.c
+index b0c6e4885fa0..5e2aa28bcff4 100644
+--- a/src/shared/bootspec.c
++++ b/src/shared/bootspec.c
+@@ -475,6 +475,7 @@ static int boot_entries_find_unified(
+ _cleanup_free_ char *j = NULL, *osrelease = NULL, *cmdline = NULL;
+ _cleanup_close_ int fd = -1;
+
++ dirent_ensure_type(d, de);
+ if (!dirent_is_file(de))
+ continue;
+
+diff --git a/src/shared/unit-file.c b/src/shared/unit-file.c
+index 4a5f23e6c1bc..c7a6d6717244 100644
+--- a/src/shared/unit-file.c
++++ b/src/shared/unit-file.c
+@@ -279,6 +279,7 @@ int unit_file_build_name_map(
+ if (hashmap_contains(ids, de->d_name))
+ continue;
+
++ dirent_ensure_type(d, de);
+ if (de->d_type == DT_LNK) {
+ /* We don't explicitly check for alias loops here. unit_ids_map_get() which
+ * limits the number of hops should be used to access the map. */
diff --git a/patches/systemd-243-51-gfab6f010ac6c/series b/patches/systemd-243-51-gfab6f010ac6c/series
new file mode 100644
index 000000000..141c6b8ef
--- /dev/null
+++ b/patches/systemd-243-51-gfab6f010ac6c/series
@@ -0,0 +1,4 @@
+# generated by git-ptx-patches
+#tag:base --start-number 1
+0001-Handle-d_type-DT_UNKNOWN-correctly.patch
+# a8e3d3a17db7bad28e1af3d83d951007 - git-ptx-patches magic