Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Aug 27, 2021

In the current version, udev rules are generated with hardcoded KERNEL udev entries for PCI devices that were found when the sdk_setup.sh script is run.

But, if the number of FPGA change, the udev rule is not updated without re-run the script.
This can be an issue if building an AMI on an instance with a single FPGA (or even no FPGA) and then using the AMI to start an instance type with more FPGA devices.

So, the udev rule was changed to dynamically detect devices independently to the initial configuration.

@deeppat deeppat self-requested a review September 14, 2021 14:04
Copy link
Contributor

@deeppat deeppat left a comment

Choose a reason for hiding this comment

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

Looks good!

@deeppat deeppat merged commit 1e17dcb into aws:master Sep 14, 2021
@ghost ghost deleted the udev-improvement branch September 14, 2021 14:39
kyyalama2 pushed a commit that referenced this pull request May 24, 2022
* Ensure udev rules works if devices changed since creation (#533)

* Add option to allow others instead of group in udev rules (#534)

Co-authored-by: xlz-jgoutin <[email protected]>
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.

2 participants