Commit 4f4cff0
committed
fix(runtime, build): the dlopen surface is walked, and one unwinder per process (2026.9.10.1)
A SYCL project built cleanly and then terminated with exit code 134 and no
exception text (#596). The trigger was in the ecosystem -- an adapter's farm
was missing one driver library -- but the reasons it presented as a silent
abort are two independent gaps in mcpp, and both are repaired here.
THE SURFACE NOTHING WALKED. `resolve_runtime_closure` is seeded with the
artifact and follows DT_NEEDED. A library a package publishes through
`runtime.library_dirs` exists precisely because something will dlopen it, so
no link edge names it and it is outside that closure BY CONSTRUCTION. Measured:
a farm of twenty-five libraries, two of which could not load at all, while the
build reported nothing -- because nothing had asked.
`inspect_dlopen_surface` reads each such library's own DT_NEEDED and resolves
it against the search path the artifact actually carries, separating three
states: resolved, present as a dangling link (the machine has no driver), and
absent (a packaging gap). Only the third is reported, and it is a warning: a
dangling link is the documented shape of a host driver that is not installed,
and failing there would turn a supported configuration into a failed build.
`runtime.dlopen_surface` in resolution.json carries the findings and both
denominators -- a surface that failed to build enumerates nothing, and "no
findings" must not read like "nothing was examined".
WHAT "THE SEARCH PATH THE ARTIFACT ACTUALLY CARRIES" TURNED OUT TO MEAN. Three
corrections, all of them false positives, none visible until a project with a
shared dependency was measured:
* `$ORIGIN` leads every artifact's DT_RPATH and a shared dependency is
deployed BESIDE the executable. `runtime_search_dirs` cannot carry that --
`$ORIGIN` is a property of each artifact, not of the plan -- so the
artifacts' own directories are added in `check_dlopen_surface`.
* A plan that produces no program has no surface to judge. The adapter
package is `kind = "lib"`; reporting a consumer's surface against an
archive's non-existent search path named a library the consumer resolves.
* A SONAME is not a filename. mcpp links `bin/libopencl.so` whose SONAME is
`libOpenCL.so.1`, and the alias appears later; `mcpp test` calls the check
twice and only the second call saw it. The SONAMEs this build produces are
read from the objects and passed in.
TWO UNWINDERS IN ONE PROCESS. A lane whose device compiler is configured
against libstdc++ puts libstdc++ on the link line while the artifact links
libc++ statically. `hide_static_cxx_runtime` skipped executables on the
premise that "ld exports only what a loaded object references, and mcpp passes
no -rdynamic" -- a correct premise with a wrong conclusion, because a loaded
libstdc++ DOES reference them. Measured: 89 exported symbols, 68 of them also
defined by libstdc++ or libgcc_s.
Ten of libgcc's eighteen `_Unwind_*` entry points came from the artifact and
eight stayed in libgcc_s, including the accessors libstdc++'s personality
routine calls. It read an LLVM libunwind context through libgcc's accessors,
recovered a meaningless IP, found no landing pad, and `__cxa_call_terminate`
ran past a handler three frames up; the verbose terminate handler's rethrow
then terminated as well, which is why nothing was printed.
Such a link now takes `--unwindlib=libgcc` and hides the static archives'
symbols. libgcc_s is in the process either way -- libstdc++ needs it -- so
this names a library rather than adding one, and the C++ runtime stays
embedded. A link with no second runtime on it is byte-for-byte unchanged.
Measured on one machine, same source, both situations:
before exit 134, no output
after `sycl: no usable device: ...` / `device unavailable`, exit 1
device `12 24 36 48` in both
`-Wl,--exclude-libs` alone was written here as a prediction from the mechanism
and then measured: exit 139 instead of 134. Hiding the exports makes libstdc++
bind to libgcc while the artifact's own libc++abi still calls its statically
linked libunwind, so the mismatch reproduces in the other direction. The
reason is recorded in the design record so the flag is not proposed again.
The duplicate-symbol warning now states that consequence when the conflicting
set includes the unwinder family, instead of describing it as one more copy
that is never called.
One documented claim was refuted along the way: the SYCL example states that a
missing device image is the one failure its island cannot turn into a return
code. Measured by compiling it for sm_90 and running on an sm_89 device -- it
was not outside the catches; no catch worked. The example's comment and README
are corrected.
Design record: .agents/docs/2026-09-09-dlopen-surface-and-two-unwinders.md
Ecosystem halves: openxlings/xim-pkgindex#796 #798, mcpplibs/mcpp-index#375
#377 #3781 parent 07086dd commit 4f4cff0
21 files changed
Lines changed: 1610 additions & 29 deletions
File tree
- .agents/docs
- docs
- zh
- examples/09-heterogeneous/sycl
- app
- src/kernels
- modules/versioning/src
- src
- build
- runtime
- tests/unit
Lines changed: 639 additions & 0 deletions
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
| 35 | + | |
35 | 36 | | |
36 | 37 | | |
37 | 38 | | |
| |||
44 | 45 | | |
45 | 46 | | |
46 | 47 | | |
| 48 | + | |
47 | 49 | | |
48 | 50 | | |
49 | 51 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
8 | 64 | | |
9 | 65 | | |
10 | 66 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
82 | 82 | | |
83 | 83 | | |
84 | 84 | | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
85 | 123 | | |
86 | 124 | | |
87 | 125 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
676 | 676 | | |
677 | 677 | | |
678 | 678 | | |
| 679 | + | |
| 680 | + | |
| 681 | + | |
| 682 | + | |
| 683 | + | |
| 684 | + | |
| 685 | + | |
| 686 | + | |
| 687 | + | |
| 688 | + | |
| 689 | + | |
| 690 | + | |
| 691 | + | |
| 692 | + | |
| 693 | + | |
| 694 | + | |
| 695 | + | |
| 696 | + | |
| 697 | + | |
| 698 | + | |
| 699 | + | |
| 700 | + | |
| 701 | + | |
679 | 702 | | |
680 | 703 | | |
681 | 704 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
67 | 67 | | |
68 | 68 | | |
69 | 69 | | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
70 | 104 | | |
71 | 105 | | |
72 | 106 | | |
| |||
0 commit comments