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

Make rd.udev.debug work in systemd-based initrds #2576

Closed
wants to merge 1 commit into from

Conversation

dtardon
Copy link
Contributor

@dtardon dtardon commented Nov 30, 2023

This makes rd.udev.debug work in systemd-based initrds too.

Changes

A systemd generator is added that sets log level of systemd-udevd.service to debug if rd.udev.debug is present on the kernel command line.

Note

rd.udev.info is ignored as that's the default log level of udevd. The existence of this option makes some sense in the legacy init.sh, as it sets the log level of udevd to err by default, but I don't think we want to go this way...

Checklist

  • I have tested it locally
  • I have reviewed and updated any documentation if relevant
  • I am providing new code and test(s) for it

This functionality was lost when dracut was ported to systemd. Let's
restore it.

Note: rd.udev.info is ignored as that's the default log level of udevd.
@github-actions github-actions bot added the modules Issue tracker for all modules label Nov 30, 2023
@LaszloGombos
Copy link
Collaborator

Lint error :-)

-getargbool 0 rd.udev.debug -d -y rdudevdebug && cat > "$1" <<EOF
+getargbool 0 rd.udev.debug -d -y rdudevdebug && cat > "$1" << EOF

@dtardon
Copy link
Contributor Author

dtardon commented Nov 30, 2023

Oops, sorry. Apparently I wasn't fully awake yet when I tried this in the morning... The option is handled (in modules.d/98dracut-systemd/dracut-cmdline.sh), I just didn't find it.

@dtardon dtardon closed this Nov 30, 2023
@dtardon dtardon deleted the rd.udev.debug branch November 30, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
modules Issue tracker for all modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants