-
Notifications
You must be signed in to change notification settings - Fork 26
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
(RHEL-44630) netif-naming-scheme: disable NAMING_BRIDGE_MULTIFUNCTION_SLOT #283
Conversation
Commit validationTracker - RHEL-44630 The following commits meet all requirements
Tracker validationSuccess🟢 Tracker RHEL-44630 has set desired product: Pull Request validationSuccess🟢 CI - All checks have passed Auto MergeSuccess🟢 Pull Request is not marked as draft and it's not blocked by |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like the wording of the manpage paragraph. It is difficult to understand in my opinion so I've tried to rewrite it for better clarity
Never generate a slot name when a PCI bridge is detected.
Since version "rhel-9.0", we have generated slot-based names for PCI multifunction devices, because we assumed that it is enough to use function numbers to distinguish between devices. However, name conflict can occur if these devices are not children of the same PCI bridge, e.g. there are multiple PCI bridges in the same slot.
This effectively reverts 66425da. The commit assumes that if the network interface has multifunctions, then the function fields of the pci devices under the same PCI bridge device are unique. But it seems not, at least on some setups. See issue #28929. Let's revert the change, and always refuse to set slot base naming if a PCI bridge is detected. Fixes #28929. (cherry picked from commit af7417a) Resolves: RHEL-44630
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
You are missing |
Good catch thanks! #284 |
This effectively reverts 66425da.
The commit assumes that if the network interface has multifunctions, then the function fields of the pci devices under the same PCI bridge device are unique.
But it seems not, at least on some setups. See issue #28929. Let's revert the change, and always refuse to set slot base naming if a PCI bridge is detected.
Fixes #28929.
(cherry picked from commit af7417a)
Resolves: RHEL-44630