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

Limit number of NICs an instance can have attached #810

Closed
jmpesp opened this issue Mar 25, 2022 · 3 comments
Closed

Limit number of NICs an instance can have attached #810

jmpesp opened this issue Mar 25, 2022 · 3 comments

Comments

@jmpesp
Copy link
Contributor

jmpesp commented Mar 25, 2022

#771 makes the arbitrary decision that an instance can have a maximum of eight disks attached to an instance, but leaves as a TODO the question of what the maximum should be for NICs.

This issue tracks adding a similar limit for NICs.

@bnaecker
Copy link
Collaborator

Good idea! Is this during instance creation, or a limit to also be respected when creating a new NIC for an existing instance?

@rmustacc
Copy link

Good idea! Is this during instance creation, or a limit to also be respected when creating a new NIC for an existing instance?

This is a permanent limit of 8 each for both disks and NICs for a given instance at this time. The reason for this is a bit of a result from the decisions in RFD 135 Virtual Storage Interfaces. As a part of that, we started with basically having a virtual machine that is leveraging PCI -- critically not PCIe, and the associated ACPI PCI hotplug mechanism. The latter works by taking a single PCI bus and creating one hotpluggable slot per device on that bus. The PCI device is afforded 5 bits in the general PCI scheme, giving us a total of 32 devices. rom there, it's 8 NICs, 8 volumes to start with. The remaining devices are getting used up by other things as well (e.g. USB, VGA, etc.) though more of those can be multi-function devices.

@bnaecker
Copy link
Collaborator

Closed by #816

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

No branches or pull requests

3 participants