diff mbox series

[RFC] systemd: extract dependencies from .note.dlopen ELF segments

Message ID 20241002090759.1220383-1-ross.burton@arm.com
State New
Headers show
Series [RFC] systemd: extract dependencies from .note.dlopen ELF segments | expand

Commit Message

Ross Burton Oct. 2, 2024, 9:07 a.m. UTC
First, this is not the final implementation, this is very much a RFC and
prototype.

Some binaries don't dynamically link to libraries, but instead at runtime
dlopen() them. This means extra work for distributions as the dependencies
are not detected automatically, so libraries may be missing.

systemd is one such project which does this, and in an attempt to solve
the packaging problem it also embeds the names of the libraries that can
potentially be opened at runtime into ELF notes.  These can be read to
generate package dependencies.

For example:

packages/cortexa57-poky-linux/systemd/libsystemd-shared: RRECOMMENDS: added "libkmod (['>= 33']) libzstd (['>= 1.5.6'])"
packages/cortexa57-poky-linux/systemd/libsystemd: RRECOMMENDS: added "libzstd (['>= 1.5.6'])"

I expect this code to be changed drastically before merging. Whilst
systemd is the main user of his approach right now, I expect to see it
used in more places in the future so there's a good argument to merge it
into the core shlibs code.  Also it currently manually extracts and parses
the data, whereas maybe we should incorporate pyelftools into meta/lib/oe
and use that to parse ELF files across all of OE.

Signed-off-by: Ross Burton <ross.burton@arm.com>
---
 meta/recipes-core/systemd/dlopen-deps.inc  | 67 ++++++++++++++++++++++
 meta/recipes-core/systemd/systemd_256.6.bb |  2 +
 2 files changed, 69 insertions(+)
 create mode 100644 meta/recipes-core/systemd/dlopen-deps.inc
diff mbox series

Patch

diff --git a/meta/recipes-core/systemd/dlopen-deps.inc b/meta/recipes-core/systemd/dlopen-deps.inc
new file mode 100644
index 00000000000..5fcaa4518d3
--- /dev/null
+++ b/meta/recipes-core/systemd/dlopen-deps.inc
@@ -0,0 +1,67 @@ 
+PACKAGEFUNCS =+ "package_generate_dlopen_deps"
+
+python package_generate_dlopen_deps() {
+    # https://systemd.io/ELF_DLOPEN_METADATA/
+
+    import struct, json
+
+    def extract_segment(filename, segment):
+        """
+        Return the named segment from the ELF.
+        """
+        import tempfile, subprocess
+
+        with tempfile.NamedTemporaryFile() as f:
+            cmd = [d.getVar("OBJCOPY"), "--dump-section", f"{segment}={f.name}", filename]
+            subprocess.run(cmd, check=True)
+            return f.read()
+
+    def parse(buffer, is_little):
+        deps = []
+        offset = 0
+        while offset < len(buffer):
+            format = f"{'<' if is_little else '>'}iii"
+            name_size, desc_size, note_type = struct.unpack_from(format, buffer, offset)
+            offset += struct.calcsize(format)
+
+            format = f"{name_size}s0i{desc_size}s0i"
+            if note_type == 0x407c0c0a:
+                name_b, desc_b = struct.unpack_from(format, buffer, offset)
+                name = name_b.strip(b"\x00").decode("ascii")
+                if name == "FDO":
+                    desc = desc_b.strip(b"\x00").decode("utf-8")
+                    deps.append(*json.loads(desc))
+            offset += struct.calcsize(format)
+        return deps
+
+    dep_map = {
+        "required": "RDEPENDS",
+        "recommended": "RRECOMMENDS",
+        "suggested": "RSUGGESTS"
+    }
+
+    shlibs = oe.package.read_shlib_providers(d)
+
+    for pkg, files in pkgfiles.items():
+        # TODO: skip -dbg?
+        for f in files:
+            if cpath.islink(f):
+                continue
+
+            if f.endswith(".so") or ".so." in f:
+                try:
+                    elf = oe.qa.ELFFile(f)
+                    elf.open()
+                    for dep in parse(extract_segment(f, ".note.dlopen"), elf.isLittleEndian()):
+                        dependency = dep_map[dep["priority"]]
+                        for soname in dep["soname"]:
+                            if soname in shlibs:
+                                # TODO don't just take first
+                                package_deps = list(shlibs[soname].values())[0]
+                                bb.note(f"{pkg}: adding {dependency} via dlopen on {package_deps[0]}")
+                                d.appendVar(f"{dependency}:{pkg}", f" {package_deps[0]} (>= {package_deps[1]})")
+                            else:
+                                bb.warn(f"cannot find {soname}")
+                except oe.qa.NotELFFileError:
+                    pass
+}
diff --git a/meta/recipes-core/systemd/systemd_256.6.bb b/meta/recipes-core/systemd/systemd_256.6.bb
index 68f15ab065d..17aae3a6ce4 100644
--- a/meta/recipes-core/systemd/systemd_256.6.bb
+++ b/meta/recipes-core/systemd/systemd_256.6.bb
@@ -907,3 +907,5 @@  pkg_postinst:udev-hwdb () {
 pkg_prerm:udev-hwdb () {
 	rm -f $D${sysconfdir}/udev/hwdb.bin
 }
+
+require dlopen-deps.inc