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

dummy initramfs required to boot mirage or custom kernel docs incorrect #5516

Closed
marmot1791 opened this issue Dec 14, 2019 · 6 comments
Closed
Labels
C: core C: doc diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. r4.2-host-stable r4.3-host-cur-test T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.

Comments

@marmot1791
Copy link

Qubes OS version
R4.0

Affected component(s) or functionality
Booting domU from a custom kernel installed in dom0

Brief summary
In the docs it states "All the files besides vmlinuz are optional." when installing customer kernels: https://www.qubes-os.org/doc/managing-vm-kernel/#kernel-files-structure

However, when testing a mirage firewall per this issue: mirage/qubes-mirage-firewall#70

A dummy initramfs file still appears to be required. Is the documentation incorrect or is this a bug?

To Reproduce
Steps to reproduce the behavior:
_1. Install a customer kernel with only vmlinuz in dom0 for use as a domU kernel
_2. Create a domU that uses the custom kernel in PV mode
_3. Start the qube --- get and erro about no initramfs found
_4. Create dummy initramfs and qube starts successfully

@marmot1791 marmot1791 added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. labels Dec 14, 2019
@marmarek
Copy link
Member

This is one of very few places where docs are ahead of the time (instead of outdated...). This is true for upcoming R4.1, but indeed in R4.0 dummy initramfs is still needed.

@andrewdavidwong andrewdavidwong added this to the Ongoing milestone Dec 15, 2019
marmot1791 added a commit to marmot1791/qubes-doc that referenced this issue Dec 15, 2019
@marmot1791
Copy link
Author

Thanks! Clarified this in the docs with pull request QubesOS/qubes-doc#907 which should close this issue.

@palainp
Copy link

palainp commented Nov 13, 2022

Currently initramfs is needed with Qubes 4.1 with this check: https://github.com/QubesOS/qubes-core-admin/blob/c177aacbe1afc35034826ef538c0e49ed40b6bc7/qubes/storage/kernels.py#L140

Is there a way to set something in the AppVM settings for booting without initramfs? (for example this file is not needed for qubes-mirage-firewall mirage/qubes-mirage-firewall#70).

@andrewdavidwong
Copy link
Member

@palainp: Please note that the issue tracker (qubes-issues) is not intended to be a place for fielding questions. Instead, we have other venues meant for asking questions, asking for help, and having discussions. (By contrast, the issue tracker is more of a technical tool intended to support our developers in their work.) Thank you for your understanding!

@palainp
Copy link

palainp commented Nov 14, 2022

Thanks for your reply and sorry for posting here, my intention was more related to the issue than a user question.

I more wanted to ask for re-openning the issue as the documentation state that initramfs is optional in Qubes 4.1 (in https://www.qubes-os.org/doc/managing-vm-kernels/#kernel-files-structure) but this seems to not be the case as the code check for the file existence (line 140 of kernel.py).

@DemiMarie DemiMarie reopened this Nov 14, 2022
@andrewdavidwong andrewdavidwong added the needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. label Nov 15, 2022
@andrewdavidwong andrewdavidwong removed this from the Non-release milestone Aug 13, 2023
@palainp
Copy link

palainp commented Jan 5, 2024

Hi, I opened a PR QubesOS/qubes-doc#1369 to update the documentation as initramfs is needed so far.

@andrewdavidwong andrewdavidwong added diagnosed Technical diagnosis has been performed (see issue comments). pr submitted A pull request has been submitted for this issue. and removed needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. labels Jan 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: core C: doc diagnosed Technical diagnosis has been performed (see issue comments). P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. r4.2-host-stable r4.3-host-cur-test T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists.
Projects
None yet
Development

No branches or pull requests

6 participants