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