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