Skip to content

Commit

Permalink
fix(dracut): rework timeout for devices added via --mount and --add-d…
Browse files Browse the repository at this point in the history
…evice

Currently, dracut adds a systemd dropin that sets an infinite timeout _only_ for
the underlying persistent devices of each host device that must be active in the
initrd. E.g.:

```
localhost:~ # dracut -f --stdlog 3 --install "/mnt" --mount "/dev/sda1 /mnt btrfs rw,relatime"
localhost:~ # lsinitrd | grep timeout.conf
-rw-r--r--   2 root     root            0 Oct 28 14:21 etc/systemd/system/dev-disk-by\x2duuid-0d1b24c2\x2df112\x2d48ef\x2d8442\x2d1001cffc92f0.device.d/timeout.conf
localhost:~ # ls -l /dev/disk/by-uuid/0d1b24c2-f112-48ef-8442-1001cffc92f0
lrwxrwxrwx 1 root root 10 Oct 28 11:43 /dev/disk/by-uuid/0d1b24c2-f112-48ef-8442-1001cffc92f0 -> ../../sda1
```

It can be verified in the emergency shell:

```
sh-5.2# systemctl show --property JobRunningTimeoutUSec /dev/disk/by-uuid/0d1b24c2-f112-48ef-8442-1001cffc92f0
JobRunningTimeoutUSec=infinity
sh-5.2# systemctl show --property JobRunningTimeoutUSec /dev/sda1
JobRunningTimeoutUSec=1min 30s
```

This can cause a problem if a user application expects to mount /dev/sda1 in the
initrd (that's why it was explicitly added `--mount "/dev/sda1 ..."`), but for
some reason the link is not created within 1min 30s.
  • Loading branch information
Antonio Alvarez Feijoo authored and Conan-Kudo committed Nov 10, 2024
1 parent 93e85e2 commit c79fc8f
Show file tree
Hide file tree
Showing 3 changed files with 36 additions and 10 deletions.
8 changes: 4 additions & 4 deletions dracut-functions.sh
Original file line number Diff line number Diff line change
Expand Up @@ -607,9 +607,9 @@ for_each_host_dev_and_slaves_all() {
local _dev
local _ret=1
[[ "${host_devs[*]}" ]] || return 2
[[ "${host_devs[*]}" ]] || [[ "${user_devs[*]}" ]] || return 2
for _dev in "${host_devs[@]}"; do
for _dev in "${host_devs[@]}" "${user_devs[@]}"; do
[[ -b $_dev ]] || continue
if check_block_and_slaves_all "$_func" "$(get_maj_min "$_dev")"; then
_ret=0
Expand All @@ -622,9 +622,9 @@ for_each_host_dev_and_slaves() {
local _func="$1"
local _dev
[[ "${host_devs[*]}" ]] || return 2
[[ "${host_devs[*]}" ]] || [[ "${user_devs[*]}" ]] || return 2
for _dev in "${host_devs[@]}"; do
for _dev in "${host_devs[@]}" "${user_devs[@]}"; do
[[ -b $_dev ]] || continue
check_block_and_slaves "$_func" "$(get_maj_min "$_dev")" && return 0
done
Expand Down
20 changes: 15 additions & 5 deletions dracut.sh
Original file line number Diff line number Diff line change
Expand Up @@ -328,6 +328,16 @@ push_host_devs() {
done
}

# Fills up user_devs stack variable and makes sure there are no duplicates
push_user_devs() {
local _dev
for _dev in "$@"; do
[[ -z $_dev ]] && continue
[[ " ${user_devs[*]} " == *" $_dev "* ]] && return
user_devs+=("$_dev")
done
}

check_conf_file() {
if grep -H -e '^[^#]*[+]=\("[^ ]\|.*[^ ]"\)' "$@"; then
printf '\ndracut[W]: <key>+=" <values> ": <values> should have surrounding white spaces!\n' >&2
Expand Down Expand Up @@ -1698,7 +1708,7 @@ for line in "${fstab_lines[@]}"; do
push_host_devs "$mp"
done
fi
push_host_devs "$dev"
push_user_devs "$dev"
host_fs_types["$dev"]="$3"
done
Expand All @@ -1710,12 +1720,12 @@ for f in $add_fstab; do
done
for dev in $add_device; do
push_host_devs "$dev"
push_user_devs "$dev"
done
if ((${#add_device_l[@]})); then
add_device+=" ${add_device_l[*]} "
push_host_devs "${add_device_l[@]}"
push_user_devs "${add_device_l[@]}"
fi
if [[ $hostonly ]] && [[ $hostonly_default_device != "no" ]]; then
Expand Down Expand Up @@ -1841,7 +1851,7 @@ _get_fs_type() {
return 1
}
for dev in "${host_devs[@]}"; do
for dev in "${host_devs[@]}" "${user_devs[@]}"; do
_get_fs_type "$dev"
check_block_and_slaves_all _get_fs_type "$(get_maj_min "$dev")"
done
Expand All @@ -1867,7 +1877,7 @@ export initdir dracutbasedir \
omit_drivers mdadmconf lvmconf root_devs \
use_fstab fstab_lines libdirs fscks nofscks ro_mnt \
stdloglvl sysloglvl fileloglvl kmsgloglvl logfile \
host_fs_types host_devs swap_devs sshkey add_fstab \
host_fs_types host_devs user_devs swap_devs sshkey add_fstab \
DRACUT_VERSION \
prefix filesystems drivers \
hostonly_cmdline loginstall \
Expand Down
18 changes: 17 additions & 1 deletion modules.d/99base/module-setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ install() {

## save host_devs which we need bring up
if [[ $hostonly_cmdline == "yes" ]]; then
if [[ -n ${host_devs[*]} ]]; then
if [[ -n ${host_devs[*]} ]] || [[ -n ${user_devs[*]} ]]; then
dracut_need_initqueue
fi
if [[ -f $initdir/lib/dracut/need-initqueue ]] || ! dracut_module_included "systemd"; then
Expand Down Expand Up @@ -105,6 +105,22 @@ install() {
*) ;;
esac
done

for _dev in "${user_devs[@]}"; do

case "$_dev" in
/dev/?*) wait_for_dev "$_dev" 0 ;;
*) ;;
esac

_pdev=$(get_persistent_dev "$_dev")
[[ $_dev == "$_pdev" ]] && continue

case "$_pdev" in
/dev/?*) wait_for_dev "$_pdev" 0 ;;
*) ;;
esac
done
)
fi
fi
Expand Down

0 comments on commit c79fc8f

Please sign in to comment.