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

virtme-init: Start udev after mounting all filesystems #209

Merged
merged 1 commit into from
Dec 23, 2024

Conversation

arighi
Copy link
Owner

@arighi arighi commented Dec 23, 2024

Ensure udev is started only after all system filesystems have been mounted.

This improves the reliability of virtme-init, as the performance benefit from parallelizing filesystem mounts was negligible. For faster boot times, virtme-ng-init should be preferred, while virtme-init serves as a reliable fallback in case of issues, so prioritizing reliability over performance aligns with virtme-init's role.

This change also resolves networking failures that can occur when using virtme-init.

Ensure udev is started only after all system filesystems have been
mounted.

This improves the reliability of virtme-init, as the performance benefit
from parallelizing filesystem mounts was negligible. For faster boot
times, virtme-ng-init should be preferred, while virtme-init serves as a
reliable fallback in case of issues, so prioritizing reliability over
performance aligns with virtme-init's role.

This change also resolves networking failures that can occur when using
virtme-init.

Signed-off-by: Andrea Righi <[email protected]>
Copy link
Collaborator

@matttbe matttbe left a comment

Choose a reason for hiding this comment

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

Good idea, it makes sense to play it safe here with this init script.

@arighi arighi merged commit a721287 into main Dec 23, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants