X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=blobdiff_plain;f=TODO;h=a4643d7b831e42b408b8ef5dd609512810f65501;hp=5748289c435b68803b03def32b9d6e2d9e63036c;hb=88fae6e0441d4195e089434f07d3e7fd811d6297;hpb=f1c8f75b38def5b2e21a83b245c5392563fba7ef diff --git a/TODO b/TODO index 5748289c4..a4643d7b8 100644 --- a/TODO +++ b/TODO @@ -1,7 +1,17 @@ Bugfixes: -* remove MS_SHARED from src/core/execute.c and src/test/test-ns.c. They are always combined - with MS_REMOUNT, which currently does nothing in the kernel, but might which fail in the - future; https://bugzilla.redhat.com/show_bug.cgi?id=813563 +* there is nothing to warn about here :) + $ systemctl stop systemd-udevd.service systemd-udevd-kernel.socket systemd-udevd-control.socket + Warning: Stopping systemd-udevd.service, but it can still be activated by: + systemd-udevd-control.socket + systemd-udevd-kernel.socket + +* kill /etc/timezone handling entirely? What does it provide? + - /etc/localtime carries the same information already: + $ ls -l /etc/localtime; cat /etc/timezone + lrwxrwxrwx 1 root root 33 Jul 27 09:55 /etc/localtime -> /usr/share/zoneinfo/Europe/Berlin + Europe/Berlin + - systemd enforces /usr to be available at bootup, so we can + enforce the use of the symlink * check systemd-tmpfiles for selinux context hookup for mknod(), symlink() and similar @@ -20,33 +30,120 @@ Bugfixes: * we pull src/core/manager.h into src/shared/src/shared/path-lookup.c which is the wrong direction rename enum "ManagerRunningAs" to "SystemdRunningAs" and move it to shared/ +* crash happens when running a service as forking and then changing it to simple and reloading. + + Jul 09 18:20:57 mop systemd[1]: usbmuxd.service operation timed out. Terminating. + Jul 09 18:20:57 mop systemd[1]: Unit usbmuxd.service entered failed state. + Jul 09 18:22:24 mop systemd[1]: PID 21814 read from file /var/run/usbmuxd.pid does not exist. + Jul 09 18:22:24 mop systemd[1]: Unit usbmuxd.service entered failed state. + Jul 09 18:22:33 mop systemd[1]: Reloading. + Jul 09 18:22:37 mop systemd[1]: Assertion 's->type == SERVICE_FORKING' failed at src/core/service.c:3007, function service_sigchld_eve...Aborting. + Jul 09 18:22:37 mop systemd[1]: Caught , dumped core as pid 21865. + Jul 09 18:22:37 mop systemd[1]: Freezing execution. + Jul 09 18:22:37 mop [21866]: Process 21865 (systemd) dumped core. + +* support *static* (/run) hibernate inhibitors. All rpm -i actions should completely prevent any + sort of hibernate action until the next reboot. If the kernel or any other base tool is replaced + by rpm, the resume path might fail, the for resume needed kernel might even be uninstalled, and + the whole situation leads directly to data loss. + Features: -* load-fragment: when loading a unit file via a chain of symlinks - verify that it isn't masked via any of the names traversed. +* allow writing multiple conditions in unit files on one line -* journald: _BOOT_ID triggers too many collisions. +* journal: json output needs to be able to deal with multiple assignments of the same field -* journald: we currently rotate only after MaxUse+MaxFilesize has been reached. +* There's something wrong with escaping unit names: http://lists.freedesktop.org/archives/systemd-devel/2012-August/006292.html -* reexec journald across initrd transition +* cleanup ellipsation for log output in journalctl and systemctl status: have a sane way to disable ellipsation, and disable it by default when invoked in less/more -* nspawn: bind mount /var/log/journal from the host +* enforce limits on fds openened by socket units -* The current Storage=auto logic is borked, since people cannot mount - /var/log/journal via NFS since the mount point has to exist and we - already take that as signal to store our stuff there. +* explore multiple service instances per listening socket idea -* document that journal data is primarily ASCII, UTF-8 where necessary and binary only where nothing else makes sense. +* testing tool for socket activation: some binary that listens on a socket and passes it on using the usual socket activation protocol to some server. -* Document: - - PID 1 D-Bus API - - Journal C API manual pages - - scheduled shutdown API +* maybe make systemd-detect-virt suid? or use fscaps? + +* man: document in ExecStart= explicitly that we don't take shell command lines, only executable names with arguments + +* shutdown: don't read-only mount anything when running in container + +* nspawn: --read-only is not applied recursively to submounts + +* MountFlags=shared acts as MountFlags=slave right now. + +* ReadOnlyDirectories= is not applied recursively to submounts + +* drop PID 1 reloading, only do reexecing (difficult: Reload() + currently is properly synchronous, Reexec() is weird, because we + can't delay the response properly until we are back, so instead of + being properly synchronous we just keep open the fd and close it + when done. That means clients don't get a succesful method reply, + but much rather a disconnect on success. + +* document that service reload may be implemented as service reexec + +* remember which condition failed for services, not just the fact that something failed + +* use opterr = 0 for all getopt tools + +* properly handle loop back mounts via fstab, especially regards to fsck/passno + +* allow services with no ExecStart= but with an ExecStop= + +* add proper journal support to "systemctl --user status ..." + +* add _SYSTEMD_USER_UNIT= field to journal entries + +* remove Fedora /dev/null logic from localed.c, now that system-config-keyboard is gone + delete /etc/X11/xorg.conf.d/00-system-setup-keyboard.conf from spec file + +* journal: expose current disk usage + +* dracut-shutdown needs to be ordered before unmounting /boot + +* wiki: document new logind LockSessions() call + +* initialize the hostname from the fs label of /, if /etc/hostname does not exist? + +* logind: different policy actions for idle, suspend, shutdown blockers: allow idle blockers by default, don't allow suspend blockers by default + +* install README to /etc/rc.d/init.d (if support for that is enabled) helping people who use "ls" there to figure out which services exist. + +* logind: ignore inactive login screens when checking whether power key should be handled + +* rename "userspace" to "core-os" + +* systemctl: "Journal has been rotated since unit was started." message is misleading + +* syscall filter: add knowledge about compat syscalls + +* syscall filter: don't enforce no new privs? -* wiki: document logind's PreparingForShutdown, PreparingForSleep +* syscall filter: option to return EPERM rather than SIGSYS? -* support debian's console-setup logic +* syscall filter: port to libseccomp + +* logind: wakelock/opportunistic suspend support + +* switch-root: sockets need relabelling + +* systemd-analyze post-boot is broken for initrd + +* man: clarify that time-sync.target is not only sysv compat but also useful otherwise. Same for similar targets + +* .device aliases need to be implemented with the "following" logic, probably. + +* refuse taking lower-case variable names in sd_journal_send() and friends. + +* load-fragment: when loading a unit file via a chain of symlinks + verify that it isn't masked via any of the names traversed. + +* journald: we currently rotate only after MaxUse+MaxFilesize has been reached. + +* Document: + - PID 1 D-Bus API * introduce Type=pid-file @@ -60,30 +157,35 @@ Features: * efi: honour timezone efi variables for default timezone selection -* new dependency type to "group" services in a target +* efi: automatically mount EFI partition to /boot if no such entry exists in /etc/fstab and /boot is empty + gummiboot exports the EFI system partion (ESP) device: + /sys/firmware/efi/vars/LoaderDeviceIdentifier-4a67b082-0a4c-41cf-b6c7-440b29bb8c4f/data + Acpi(PNP0A03,0)/Pci(1F|2)/?/HD(Part1,Sig1FCBC57F-4BFC-4C2B-91A3-9C84FBCD9AF1) + '/' is the separator for the device path list + HD(Part1,Sig1FCBC57F-4BFC-4C2B-91A3-9C84FBCD9AF1) contains the GPT UUID of the ESP + +* read the bootloader performance data (raw TSC) in systemd-analyze + /sys/firmware/efi/vars/LoaderTicksExec-4a67b082-0a4c-41cf-b6c7-440b29bb8c4f/data + 19066159288 + /sys/firmware/efi/vars/LoaderTicksInit-4a67b082-0a4c-41cf-b6c7-440b29bb8c4f/data + 17442940316 + /sys/firmware/efi/vars/LoaderTicksStartMenu-4a67b082-0a4c-41cf-b6c7-440b29bb8c4f/data + (only set if the menu was active) * change Requires=basic.target to RequisiteOverride=basic.target -* exclude processes marked with argv[0][0]=@ from the normal service killing too - * support rd.luks.allow-discards= kernel cmdline params in cryptsetup generator * systemctl: when stopping a service which has triggres and warning about it actually check the TriggeredBy= deps fields -* journal: hook up with EFI firmware log, new kmsg logic +* journal: hook up with EFI firmware log * handle C-A-Del in logind, like the power/suspend buttons? * nspawn: make use of device cgroup contrller by default -* journalctl /dev/sda, journalctl --device=b12:8 (--device=n12, --device=+usb:1-1) - -* make use of /sys/power/wake_lock in inhibitors - * drop accountsservice's StandardOutput=syslog and Type=dbus fields -* make sure show-logs checks for utf8 validity, not ascii validity - * when breaking cycles drop sysv services first, then services from /run, then from /etc, then from /usr * readahead: when bumping /sys readahead variable save mtime and compare later to detect changes @@ -133,8 +235,6 @@ Features: * Add pretty name for seats in logind -* nspawn wants dev_setup() for /dev/fd/ and friends? - * selinux: merge systemd selinux access controls (dwalsh) * ConditionSecurity= should learn about IMA @@ -162,17 +262,12 @@ Features: * cleanup syslog 'priority' vs. 'level' wording -* journal: if mmap() fails for mapping window try to unmap a a few older maps - * dbus upstream still refers to dbus.target and shouldn't * when a service has the same env var set twice we actually store it twice and return that in systemctl show -p... We should only show the last setting * support container_ttys= -* journald: make configurable "store-on-var", "store-on-run", "dont-store", "auto" - (store-persistent, store-volatile?) - * introduce mix of BindTo and Requisite * journalctl: show multiline log messages sanely, expand tabs, and show all valid utf8 messages @@ -203,14 +298,6 @@ Features: * logind: add equivalent to sd_pid_get_owner_uid() to the D-Bus API -* write RPM spec macros for presets - -* journal: write man pages for API - -* journal: OR matches are borked - -* journal: extend hash tables as we go - * journal: API for looking for retrieving "all values of this field" * journal: deal nicely with byte-by-byte copied files, especially regards header @@ -219,15 +306,11 @@ Features: * journal: message catalog -* journal: forward-secure signatures - * document the exit codes when services fail before they are exec()ed -* rework namespace support, don't use pivot_root, and mount things after creating the namespace, not before - * systemctl journal command -* journalctl: --cursor support, priority filtering +* journalctl: --cursor support * systemctl status: show coredumps @@ -250,8 +333,6 @@ Features: * default to actual 32bit PIDs, via /proc/sys/kernel/pid_max -* add an option to make mounts private/shareable and so on, enable this for root by default - * be able to specify a forced restart of service A where service B depends on, in case B needs to be auto-respawned? @@ -322,6 +403,7 @@ Features: - resource control in systemd - inhibiting - testing with Harald's awesome test kit + - restart * allow port=0 in .socket units @@ -355,7 +437,7 @@ Features: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=99ee5315dac6211e972fa3f23bcc9a0343ff58c4 * implicitly import "defaults" settings file into all types -* exec settings override + * writable cgroups dbus properties for live changes * read config fragments for all units from /lib/systemd/system/foobar.service.d/ to override/extend specific settings @@ -415,16 +497,14 @@ Regularly: * Use PR_SET_PROCTITLE_AREA if it becomes available in the kernel -* %m in printf() instead of strerror(); +* %m in printf() instead of strerror(errno); * pahole * set_put(), hashmap_put() return values check. i.e. == 0 doesn't free()! +* use __secure_getenv() instead of getenv() where appropriate + Scheduled for removal (or fixing): * xxxOverridable dependencies - -* journald.conf: ImportKernel= - -* prefdm.service