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

RFC kernel-boot: Update udev rules and rdma_rename to handle auxiliary devices #1248

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tatyana-en
Copy link
Contributor

Update udev rules for irdma driver to set ID_RDMA_IWARP and ID_RDMA_ROCE environment vars based on the auxiliary device name. For the new rules to work correctly, a patch for the irdma driver is necessary to change the ib device parent from pci to auxiliary device.

Change the rdma_rename logic to recognize auxiliary devices and rename ib devices by_onboard or by_pci the same way it did before they were parented by auxiliary devices.

Signed-off-by: Tatyana Nikolova tatyana.e.nikolova@intel.com

…y devices

Update udev rules for irdma driver to set ID_RDMA_IWARP and ID_RDMA_ROCE
environment vars based on the auxiliary device name. For the new rules
to work correctly, a patch for the irdma driver is necessary to change
the ib device parent from pci to auxiliary device.

Change the rdma_rename logic to recognize auxiliary devices and
rename ib devices by_onboard or by_pci the same way
it did before they were parented by auxiliary devices.

Signed-off-by: Tatyana Nikolova <tatyana.e.nikolova@intel.com>
@tatyana-en
Copy link
Contributor Author

Hi, did you have a chance to review these changes? Do you have any comments?

@jgunthorpe
Copy link
Member

Thought I commented on the mailing list, send the kernel side as a non-rfc, it looks correct

@tatyana-en
Copy link
Contributor Author

Thank you, Jason. We are going to send a kernel patch soon and I am going to update this PR to drop the RFC. I was asking to check if you are okay with the user space rdma_rename changes.

@jgunthorpe
Copy link
Member

This is quite old now, what happened to this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants