]> git.ipfire.org Git - thirdparty/systemd.git/blob - docs/HACKING.md
Merge pull request #32562 from Werkov/test-cgroup-opensuse
[thirdparty/systemd.git] / docs / HACKING.md
1 ---
2 title: Hacking on systemd
3 category: Contributing
4 layout: default
5 SPDX-License-Identifier: LGPL-2.1-or-later
6 ---
7
8 # Hacking on systemd
9
10 We welcome all contributions to systemd.
11 If you notice a bug or a missing feature, please feel invited to fix it, and submit your work as a
12 [GitHub Pull Request (PR)](https://github.com/systemd/systemd/pull/new).
13
14 Please make sure to follow our [Coding Style](/CODING_STYLE) when submitting patches.
15 Also have a look at our [Contribution Guidelines](/CONTRIBUTING).
16
17 When adding new functionality, tests should be added.
18 For shared functionality (in `src/basic/` and `src/shared/`) unit tests should be sufficient.
19 The general policy is to keep tests in matching files underneath `src/test/`,
20 e.g. `src/test/test-path-util.c` contains tests for any functions in `src/basic/path-util.c`.
21 If adding a new source file, consider adding a matching test executable.
22 For features at a higher level, tests in `src/test/` are very strongly recommended.
23 If that is not possible, integration tests in `test/` are encouraged.
24
25 ```shell
26 $ git config submodule.recurse true
27 $ git config fetch.recurseSubmodules on-demand
28 $ git config push.recurseSubmodules no
29 $ cp .git/hooks/pre-commit.sample .git/hooks/pre-commit
30 $ cp tools/git-submodule-update-hook.sh .git/hooks/post-rewrite
31 $ cp tools/git-submodule-update-hook.sh .git/hooks/post-checkout
32 ```
33
34 Please always test your work before submitting a PR.
35 For many of the components of systemd testing is straightforward as you can simply compile systemd and run the relevant tool from the build directory.
36
37 For some components (most importantly, systemd/PID 1 itself) this is not possible, however.
38 In order to simplify testing for cases like this we provide a set of `mkosi` config files directly in the source tree.
39 [mkosi](https://mkosi.systemd.io/)
40 is a tool for building clean OS images from an upstream distribution in combination with a fresh build of the project in the local working directory.
41 To make use of this, please install `mkosi` v19 or newer using your distribution's package manager or from the
42 [GitHub repository](https://github.com/systemd/mkosi).
43 `mkosi` will build an image for the host distro by default.
44 First, run `mkosi genkey` to generate a key and certificate to be used for secure boot and verity signing.
45 After that is done, it is sufficient to type `mkosi` in the systemd project directory to generate a disk image you can boot either in `systemd-nspawn` or in a UEFI-capable VM:
46
47 ```sh
48 $ sudo mkosi boot # nspawn still needs sudo for now
49 ```
50
51 or:
52
53 ```sh
54 $ mkosi qemu
55 ```
56
57 Every time you rerun the `mkosi` command a fresh image is built,
58 incorporating all current changes you made to the project tree.
59
60 By default a directory image is built.
61 This requires `virtiofsd` to be installed on the host.
62 To build a disk image instead which does not require `virtiofsd`, add the following to `mkosi.local.conf`:
63
64 ```conf
65 [Output]
66 Format=disk
67 ```
68
69 To boot in UEFI mode instead of using QEMU's direct kernel boot, add the following to `mkosi.local.conf`:
70
71 ```conf
72 [Host]
73 QemuFirmware=uefi
74 ```
75
76 To avoid having to build a new image all the time when iterating on a patch,
77 add the following to `mkosi.local.conf`:
78
79 ```conf
80 [Host]
81 RuntimeBuildSources=yes
82 ```
83
84 After enabling this setting, the source and build directories will be mounted to
85 `/work/src` and `/work/build` respectively when booting the image as a container
86 or virtual machine. To build the latest changes and re-install, run
87 `meson install -C /work/build --only-changed` in the container or virtual machine
88 and optionally restart the daemon(s) you're working on using
89 `systemctl restart <units>` or `systemctl daemon-reexec` if you're working on pid1
90 or `systemctl soft-reboot` to restart everything.
91
92 Aside from the image, the `mkosi.output` directory will also be populated with a
93 set of distribution packages. Assuming you're running the same distribution and
94 release as the mkosi image, you can install these rpms on your host or test
95 system as well for any testing or debugging that cannot easily be performed in a
96 VM or container.
97
98 By default, no debuginfo packages are produced. To produce debuginfo packages,
99 run mkosi with the `WITH_DEBUG` environment variable set to `1`:
100
101 ```sh
102 $ mkosi -E WITH_DEBUG=1 -f
103 ```
104
105 or configure it in `mkosi.local.conf`:
106
107 ```conf
108 [Content]
109 Environment=WITH_DEBUG=1
110 ```
111
112 Putting this all together, here's a series of commands for preparing a patch for systemd:
113
114 ```sh
115 $ git clone https://github.com/systemd/mkosi.git # If mkosi v19 or newer is not packaged by your distribution
116 $ ln -s $PWD/mkosi/bin/mkosi /usr/local/bin/mkosi # If mkosi v19 or newer is not packaged by your distribution
117 $ git clone https://github.com/systemd/systemd.git
118 $ cd systemd
119 $ git checkout -b <BRANCH> # where BRANCH is the name of the branch
120 $ vim src/core/main.c # or wherever you'd like to make your changes
121 $ mkosi -f qemu # (re-)build and boot up the test image in qemu
122 $ git add -p # interactively put together your patch
123 $ git commit # commit it
124 $ git push -u <REMOTE> # where REMOTE is your "fork" on GitHub
125 ```
126
127 And after that, head over to your repo on GitHub and click "Compare & pull request"
128
129 If you want to do a local build without mkosi,
130 most distributions also provide very simple and convenient ways to install most development packages necessary to build systemd:
131
132 ```sh
133 # Fedora
134 $ sudo dnf builddep systemd
135 # Debian/Ubuntu
136 $ sudo apt-get build-dep systemd
137 # Arch
138 $ sudo pacman -S devtools
139 $ pkgctl repo clone --protocol=https systemd
140 $ cd systemd
141 $ makepkg -seoc
142 ```
143
144 After installing the development packages, systemd can be built from source as follows:
145
146 ```sh
147 $ meson setup build <options>
148 $ ninja -C build
149 $ meson test -C build
150 ```
151
152 Happy hacking!
153
154 ## Templating engines in .in files
155
156 Some source files are generated during build. We use two templating engines:
157 * meson's `configure_file()` directive uses syntax with `@VARIABLE@`.
158
159 See the [Meson docs for `configure_file()`](https://mesonbuild.com/Reference-manual.html#configure_file) for details.
160
161 {% raw %}
162 * most files are rendered using jinja2, with `{{VARIABLE}}` and `{% if … %}`,
163 `{% elif … %}`, `{% else … %}`, `{% endif … %}` blocks. `{# … #}` is a jinja2 comment,
164 i.e. that block will not be visible in the rendered output.
165 `{% raw %} … `{% endraw %}`{{ '{' }}{{ '% endraw %' }}}` creates a block where jinja2 syntax is not interpreted.
166
167 See the [Jinja Template Designer Documentation](https://jinja.palletsprojects.com/en/3.1.x/templates/#synopsis) for details.
168
169 Please note that files for both template engines use the `.in` extension.
170
171 ## Developer and release modes
172
173 In the default meson configuration (`-Dmode=developer`),
174 certain checks are enabled that are suitable when hacking on systemd (such as internal documentation consistency checks).
175 Those are not useful when compiling for distribution and can be disabled by setting `-Dmode=release`.
176
177 ## Sanitizers in mkosi
178
179 See [Testing systemd using sanitizers](/TESTING_WITH_SANITIZERS) for more information on how to build with sanitizers enabled in mkosi.
180
181 ## Fuzzers
182
183 systemd includes fuzzers in `src/fuzz/` that use libFuzzer and are automatically run by [OSS-Fuzz](https://github.com/google/oss-fuzz) with sanitizers.
184 To add a fuzz target, create a new `src/fuzz/fuzz-foo.c` file with a `LLVMFuzzerTestOneInput` function and add it to the list in `src/fuzz/meson.build`.
185
186 Whenever possible, a seed corpus and a dictionary should also be added with new fuzz targets.
187 The dictionary should be named `src/fuzz/fuzz-foo.dict` and the seed corpus should be built and exported as `$OUT/fuzz-foo_seed_corpus.zip` in `tools/oss-fuzz.sh`.
188
189 The fuzzers can be built locally if you have libFuzzer installed by running `tools/oss-fuzz.sh`, or by running:
190
191 ```sh
192 CC=clang CXX=clang++ \
193 meson setup build-libfuzz -Dllvm-fuzz=true -Db_sanitize=address,undefined -Db_lundef=false \
194 -Dc_args='-fno-omit-frame-pointer -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION'
195 ninja -C build-libfuzz fuzzers
196 ```
197
198 Each fuzzer then can be then run manually together with a directory containing the initial corpus:
199
200 ```
201 export UBSAN_OPTIONS=print_stacktrace=1:print_summary=1:halt_on_error=1
202 build-libfuzz/fuzz-varlink-idl test/fuzz/fuzz-varlink-idl/
203 ```
204
205 Note: the `halt_on_error=1` UBSan option is especially important,
206 otherwise the fuzzer won't crash when undefined behavior is triggered.
207
208 You should also confirm that the fuzzers can be built and run using
209 [the OSS-Fuzz toolchain](https://google.github.io/oss-fuzz/advanced-topics/reproducing/#building-using-docker):
210
211 ```sh
212 path_to_systemd=...
213
214 git clone --depth=1 https://github.com/google/oss-fuzz
215 cd oss-fuzz
216
217 for sanitizer in address undefined memory; do
218 for engine in libfuzzer afl honggfuzz; do
219 ./infra/helper.py build_fuzzers --sanitizer "$sanitizer" --engine "$engine" \
220 --clean systemd "$path_to_systemd"
221
222 ./infra/helper.py check_build --sanitizer "$sanitizer" --engine "$engine" \
223 -e ALLOWED_BROKEN_TARGETS_PERCENTAGE=0 systemd
224 done
225 done
226
227 ./infra/helper.py build_fuzzers --clean --architecture i386 systemd "$path_to_systemd"
228 ./infra/helper.py check_build --architecture i386 -e ALLOWED_BROKEN_TARGETS_PERCENTAGE=0 systemd
229
230 ./infra/helper.py build_fuzzers --clean --sanitizer coverage systemd "$path_to_systemd"
231 ./infra/helper.py coverage --no-corpus-download systemd
232 ```
233
234 If you find a bug that impacts the security of systemd,
235 please follow the guidance in [CONTRIBUTING.md](/CONTRIBUTING) on how to report a security vulnerability.
236
237 For more details on building fuzzers and integrating with OSS-Fuzz, visit:
238
239 - [Setting up a new project - OSS-Fuzz](https://google.github.io/oss-fuzz/getting-started/new-project-guide/)
240 - [Tutorials - OSS-Fuzz](https://google.github.io/oss-fuzz/reference/useful-links/#tutorials)
241
242 ## Debugging binaries that need to run as root in vscode
243
244 When trying to debug binaries that need to run as root,
245 we need to do some custom configuration in vscode to have it try to run the applications as root and to ask the user for the root password when trying to start the binary.
246 To achieve this, we'll use a custom debugger path which points to a script that starts `gdb` as root using `pkexec`.
247 pkexec will prompt the user for their root password via a graphical interface.
248 This guide assumes the C/C++ extension is used for debugging.
249
250 First, create a file `sgdb` in the root of the systemd repository with the following contents and make it executable:
251
252 ```sh
253 #!/bin/sh
254 exec pkexec gdb "$@"
255 ```
256
257 Then, open launch.json in vscode, and set `miDebuggerPath` to `${workspaceFolder}/sgdb` for the corresponding debug configuration.
258 Now, whenever you try to debug the application, vscode will try to start gdb as root via pkexec which will prompt you for your password via a graphical interface.
259 After entering your password, vscode should be able to start debugging the application.
260
261 For more information on how to set up a debug configuration for C binaries,
262 please refer to the official vscode documentation [here](https://code.visualstudio.com/docs/cpp/launch-json-reference)
263
264 ## Debugging systemd with mkosi + vscode
265
266 To simplify debugging systemd when testing changes using mkosi, we're going to show how to attach [VSCode](https://code.visualstudio.com/)'s debugger to an instance of systemd running in a mkosi image using QEMU.
267
268 To allow VSCode's debugger to attach to systemd running in a mkosi image,
269 we have to make sure it can access the virtual machine spawned by mkosi where systemd is running.
270 After booting the image with `mkosi qemu`,
271 you should now be able to connect to it by running `mkosi ssh` from the same directory in another terminal window.
272
273 Now we need to configure VSCode.
274 First, make sure the C/C++ extension is installed.
275 If you're already using a different extension for code completion and other IDE features for C in VSCode,
276 make sure to disable the corresponding parts of the C/C++ extension in your VSCode user settings by adding the following entries:
277
278 ```json
279 "C_Cpp.formatting": "Disabled",
280 "C_Cpp.intelliSenseEngine": "Disabled",
281 "C_Cpp.enhancedColorization": "Disabled",
282 "C_Cpp.suggestSnippets": false,
283 ```
284
285 With the extension set up,
286 we can create the launch.json file in the .vscode/ directory to tell the VSCode debugger how to attach to the systemd instance running in our mkosi container/VM.
287 Create the file, and possibly the directory, and add the following contents:
288
289 ```json
290 {
291 "version": "0.2.0",
292 "configurations": [
293 {
294 "type": "cppdbg",
295 "program": "/usr/lib/systemd/systemd",
296 "processId": "${command:pickRemoteProcess}",
297 "request": "attach",
298 "name": "systemd",
299 "pipeTransport": {
300 "pipeProgram": "mkosi",
301 "pipeArgs": ["-C", "${workspaceFolder}", "ssh"],
302 "debuggerPath": "/usr/bin/gdb"
303 },
304 "MIMode": "gdb",
305 "sourceFileMap": {
306 "/work/src": {
307 "editorPath": "${workspaceFolder}",
308 "useForBreakpoints": false
309 },
310 }
311 }
312 ]
313 }
314 ```
315
316 Now that the debugger knows how to connect to our process in the container/VM and we've set up the necessary source mappings,
317 go to the "Run and Debug" window and run the "systemd" debug configuration.
318 If everything goes well, the debugger should now be attached to the systemd instance running in the container/VM.
319 You can attach breakpoints from the editor and enjoy all the other features of VSCode's debugger.
320
321 To debug systemd components other than PID 1,
322 set "program" to the full path of the component you want to debug and set "processId" to "${command:pickProcess}".
323 Now, when starting the debugger, VSCode will ask you the PID of the process you want to debug.
324 Run `systemctl show --property MainPID --value <component>`
325 in the container to figure out the PID and enter it when asked and VSCode will attach to that process instead.
326
327 ## Debugging systemd-boot
328
329 During boot, systemd-boot and the stub loader will output messages like `systemd-boot@0x0A` and `systemd-stub@0x0B`,
330 providing the base of the loaded code.
331 This location can then be used to attach to a QEMU session (provided it was run with `-s`).
332 See `debug-sd-boot.sh` script in the tools folder which automates this processes.
333
334 If the debugger is too slow to attach to examine an early boot code passage,
335 the call to `DEFINE_EFI_MAIN_FUNCTION()` can be modified to enable waiting.
336 As soon as the debugger has control, we can then run `set variable wait = 0` or `return` to continue.
337 Once the debugger has attached, setting breakpoints will work like usual.
338
339 To debug systemd-boot in an IDE such as VSCode we can use a launch configuration like this:
340 ```json
341 {
342 "name": "systemd-boot",
343 "type": "cppdbg",
344 "request": "launch",
345 "program": "${workspaceFolder}/build/src/boot/efi/systemd-bootx64.efi",
346 "cwd": "${workspaceFolder}",
347 "MIMode": "gdb",
348 "miDebuggerServerAddress": ":1234",
349 "setupCommands": [
350 { "text": "shell mkfifo /tmp/sdboot.{in,out}" },
351 { "text": "shell qemu-system-x86_64 [...] -s -serial pipe:/tmp/sdboot" },
352 { "text": "shell ${workspaceFolder}/tools/debug-sd-boot.sh ${workspaceFolder}/build/src/boot/efi/systemd-bootx64.efi /tmp/sdboot.out systemd-boot.gdb" },
353 { "text": "source /tmp/systemd-boot.gdb" },
354 ]
355 }
356 ```