Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

systemd: prevent unmounting storage via umount.target #3984

Merged
merged 2 commits into from
Nov 29, 2019

Conversation

HiassofT
Copy link
Member

/storage will be unmounted by systemd-shutdown, like /flash.

As the squashfs root has several symlinks to /storage it
should not be unmounted during the normal shutdown process.
This prevents the symlinked files/directories vanishing too early
and also removes the "unmount /storage failed" messages when
netbooting.

Drop-ins for the storage mount unit didn't work before (due to a systemd bug in combination with the order in which we mount filesystems in initramfs), so include the proposed fix which should hopefully make it into the next systemd version.

This removes the root mount dependency on systemd-journal.socket
and fixes drop-in loading for storage.mount.

See systemd/systemd#14040

Signed-off-by: Matthias Reichl <hias@horus.com>
/storage will be unmounted by systemd-shutdown, like /flash.

As the squashfs root has several symlinks to /storage it
should not be unmounted during the normal shutdown process.
This prevents the symlinked files/directories vanishing too early
and also removes the "unmount /storage failed" messages when
netbooting.

Signed-off-by: Matthias Reichl <hias@horus.com>
@HiassofT
Copy link
Member Author

The systemd fix has just been merged into systemd git systemd/systemd@bf7eedb so it should be safe to use and we can drop it on the next systemd bump

@MilhouseVH MilhouseVH merged commit 29479e5 into LibreELEC:master Nov 29, 2019
@HiassofT HiassofT deleted the le10-systemd-storage-deps branch January 16, 2021 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants