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