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

docs: add PCI passthrough removal instructions #146

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bryanvaz
Copy link

Adds instructions on how to remove a PCI passthrough device from a VM and add it back to the control domain.

Before submitting the pull request, you must agree with the following statements by checking both boxes with a 'x'.

  • "I accept that my contribution is placed under the CC BY-SA 2.0 license [1]."
  • "My contribution complies with the Developer Certificate of Origin [2]."

[1] https://creativecommons.org/licenses/by-sa/2.0/
[2] https://xcp-ng.org/docs/contributing.html#developer-certificate-of-origin-dco

docs/compute.md Outdated Show resolved Hide resolved
_Note: this command removes all PCI devices hidden from the control domain (`dom0`). If you have multiple devices, you can either: (a) add them back if they are inadvertently removed; or (b) use the previous `--set-dom0` command, to update the list of hidden PCI devices, excluding the device address that you want to add back to the control domain._

:::tip
You can use `cat /boot/efi/EFI/xenserver/grub.cfg` see a list of devices currently hidden under during a normal boot (`menuentry 'XCP-ng'`). The grub location will vary depending on how you setup your bootloader.
Copy link
Member

Choose a reason for hiding this comment

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

This is only true for UEFI hosts. The location differs for BIOS hosts.

Copy link
Member

Choose a reason for hiding this comment

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

Oh, I see you mentioned it, but as it comes after, it can be a bit confusing.

Copy link
Author

Choose a reason for hiding this comment

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

Yea, I don't know where the BIOS grub config is hiding.

Also on a separate note /opt/xensource/libexec/xen-cmdline --get-dom0 does not return anything on my UEFI system. This command ideally would be the best way to confirm this info.

Copy link
Member

Choose a reason for hiding this comment

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

/etc/grub.cfg is a symlink to the grub configuration in BIOS mode and /etc/grub-efi.cfg in UEFI mode. Can you update with this information?

Copy link
Member

Choose a reason for hiding this comment

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

The last comment was not addressed actually. I'll update the PR before we merge it.

Copy link
Member

@stormi stormi left a comment

Choose a reason for hiding this comment

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

Hi @bryanvaz! Could address the last comments?

@stormi
Copy link
Member

stormi commented Feb 1, 2022

Ping @bryanvaz. This PR is useful, but the comments still need addressing.

@stormi
Copy link
Member

stormi commented May 18, 2022

Pinging @bryanvaz again.

@stormi
Copy link
Member

stormi commented Jul 4, 2022

Hi @bryanvaz. It's been more than 7 months since you opened this pull request. I still consider it a useful addition but I can't merge it as is. If you don't have time to address the remaining comments, maybe you could ask on our forum for someone to finalize the guide for you?

@bryanvaz
Copy link
Author

@stormi Not sure what exactly needs to be changed or how you want it changed. I've resolved the conversations. I'll let you decide if you want to merge as is and have someone correct it later, or if it's better to just not have any PCIe passthrough removal instructions than ones with my extra tips based on lessons learned from mistakes I made when I attempted this for the first time.

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.

None yet

2 participants