Skip to content

unit: also prioritize input devices when triggering devices#25144

Merged
poettering merged 1 commit intosystemd:mainfrom
yuwata:unit-udev-trigger
Oct 26, 2022
Merged

unit: also prioritize input devices when triggering devices#25144
poettering merged 1 commit intosystemd:mainfrom
yuwata:unit-udev-trigger

Conversation

@yuwata
Copy link
Member

@yuwata yuwata commented Oct 26, 2022

As in most cases, tty device without input devices is meaningless.

This also swaps the priority of tty and net:

  • input devices are often connected under USB bus, hence may take slightly much time to be initialized. As, described in the above, in most cases it is allowed that tty devices are initialized just before input devices,
  • network configuration usually requires much time, e.g. DHCP or RA, hence it is better that network interfaces initialized. Then, network services can start DHCP client or friends earlier.

Fixes #24026.

As in most cases, tty device without input devices is meaningless.

This also swaps the priority of tty and net:
- input devices are often connected under USB bus, hence may take
  slightly much time to be initialized. As, described in the above,
  in most cases it is allowed that tty devices are initialized just
  before input devices,
- network configuration usually requires much time, e.g. DHCP or RA,
  hence it is better that network interfaces initialized. Then,
  network services can start DHCP client or friends earlier.

Fixes systemd#24026.
Copy link
Member

@bluca bluca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds like a regression introduced in 251, so we should merge this for 252 I think?

@keszybz
Copy link
Member

keszybz commented Oct 26, 2022

If the reporters report that this improves the situation for them, let's merge immediately. It is always possible that this again will make the situation better for some but worse for others, but it shouldn't break anything, and we're unlikely to get a lot of feedback until this is released anyway.

@bluca
Copy link
Member

bluca commented Oct 26, 2022

TEST-03-JOBS failed on arm, don't think I've seen that before?

@poettering poettering merged commit 403ca5b into systemd:main Oct 26, 2022
@yuwata yuwata deleted the unit-udev-trigger branch October 26, 2022 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Keyboard not available immediately in X11 / display manager after the boot on some machine

4 participants