From: Daniel Dragomir <daniel.dragomir@windriver.com>
To: openembedded-core@lists.openembedded.org
Subject: [OE-core][PATCH] wic/engine: warn about old host debugfs for standalone directory copy
Date: Thu, 18 Dec 2025 12:15:24 +0200 [thread overview]
Message-ID: <20251218101524.703244-1-daniel.dragomir@windriver.com> (raw)
When wic is used in standalone mode, it relies on host tools such as
debugfs. For directory host->image copies into ext* partitions, wic
uses scripted debugfs "-f" input with multiple mkdir/write commands.
Older host debugfs versions (< 1.47) may behave unreliably in this
mode and can silently miss files. This does not affect builds using
debugfs from OE where the version is known to be sufficiently new.
Add a debugfs version check and emit a warning when an older host
debugfs is detected. The warning is shown once per run and does not
alter execution.
Signed-off-by: Daniel Dragomir <daniel.dragomir@windriver.com>
---
scripts/lib/wic/engine.py | 37 +++++++++++++++++++++++++++++++++++++
1 file changed, 37 insertions(+)
diff --git a/scripts/lib/wic/engine.py b/scripts/lib/wic/engine.py
index 565a0db38a..28a973f6c4 100644
--- a/scripts/lib/wic/engine.py
+++ b/scripts/lib/wic/engine.py
@@ -220,6 +220,42 @@ def wic_list(args, scripts_path):
return False
+_DEBUGFS_VERSION = None
+_DEBUGFS_WARNED = False
+
+def debugfs_version_check(debugfs_path, min_ver=(1, 47, 0)):
+ global _DEBUGFS_VERSION, _DEBUGFS_WARNED
+
+ if _DEBUGFS_WARNED:
+ return
+
+ # Detect version once
+ if _DEBUGFS_VERSION is None:
+ out = ""
+ for flag in ("-V", "-v"):
+ try:
+ out = exec_cmd(f"{debugfs_path} {flag}")
+ break
+ except Exception:
+ continue
+
+ import re
+ m = re.search(r"(\d+)\.(\d+)\.(\d+)", out or "")
+ _DEBUGFS_VERSION = tuple(map(int, m.groups())) if m else None
+
+ ver = _DEBUGFS_VERSION
+
+ # Compare and warn once
+ if ver is not None and ver < min_ver:
+ logger.warning(
+ "debugfs %s detected (< 1.47): Directory copies into ext* partitions "
+ "via scripted debugfs (-f) may be unreliable. Consider using a newer "
+ "debugfs, for example via a native sysroot from a newer SDK.",
+ ".".join(map(str, ver)) if ver else "unknown"
+ )
+
+ _DEBUGFS_WARNED = True
+
class Disk:
def __init__(self, imagepath, native_sysroot, fstypes=('fat', 'ext')):
@@ -334,6 +370,7 @@ class Disk:
if self.partitions[pnum].fstype.startswith('ext'):
if isinstance(src, str): # host to image case
if os.path.isdir(src):
+ debugfs_version_check(self.debugfs)
base = os.path.abspath(src)
base_parent = os.path.dirname(base)
cmds = []
--
2.25.1
next reply other threads:[~2025-12-18 10:15 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-18 10:15 Daniel Dragomir [this message]
2025-12-18 11:34 ` [OE-core][PATCH] wic/engine: warn about old host debugfs for standalone directory copy Paul Barker
2025-12-18 11:46 ` Paul Barker
2025-12-18 13:48 ` Daniel Dragomir
2026-02-09 18:12 ` Ross Burton
2026-02-10 13:02 ` Daniel Dragomir
2026-01-09 17:45 ` [OE-core][PATCH v2] " Daniel Dragomir
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20251218101524.703244-1-daniel.dragomir@windriver.com \
--to=daniel.dragomir@windriver.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox