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

Feature: add --force, --enable-sriov to opae.io #3079

Merged
merged 3 commits into from
Jan 20, 2024

Conversation

fpgamatt
Copy link
Contributor

Add --force and --enable-sriov options to the subparser for opae.io init. The --force option enables controlling the force parameter to vfio_init() that was previously hard-coded to False.

The --enable-sriov option, when given, causes opae.io init to write 'Y' to /sys/module/vfio_pci/parameters/enable_sriov.

The default value for both options is False.

PR Title should start with one of the following tags: [Fix]/[Feature]/[Style]/[Update]

[Fix]- Bug Fix

[Feature]- for new feature

[Style]- Grammar or branding fix

[Update]-For an update to an existing feature

------------- Keep everything below this line -------------------------

Description

Describe the issue, update, change or fix and why

Collateral (docs, reports, design examples, case IDs):

  • Document Update Required? (Specify FIM/AFU/Scripts)

Tests added:

Tests run:

@fpgamatt fpgamatt self-assigned this Jan 19, 2024
@fpgamatt fpgamatt requested a review from a team as a code owner January 19, 2024 19:13
pcolberg
pcolberg previously approved these changes Jan 19, 2024
matthew-gerlach and others added 2 commits January 19, 2024 14:45
Add the missing parameters to the documention of a few OPAE
C++ methods.

Signed-off-by: Matthew Gerlach <matthew.gerlach@linux.intel.com>
Add --force and --enable-sriov options to the subparser for
opae.io init. The --force option enables controlling the force
parameter to vfio_init() that was previously hard-coded to False.

The --enable-sriov option, when given, causes opae.io init to
write 'Y' to /sys/module/vfio_pci/parameters/enable_sriov.

The default value for both options is False.

Signed-off-by: Tim Whisonant <tim.whisonant@intel.com>
Signed-off-by: Matthew Gerlach <matthew.gerlach@linux.intel.com>
@pcolberg
Copy link
Contributor

pcolberg commented Jan 19, 2024

The docs are still failing:

The conflict is caused by:
    The user requested sphinx==5.0.0
    …
    breathe 4.33.1 depends on Sphinx<5 and >=3.0

@fpgamatt fpgamatt force-pushed the tswhiston/opae.io_sriov_take2 branch 2 times, most recently from 7f65169 to 3ec47cd Compare January 19, 2024 23:46
Update to version 5.0.1 of sphinx which requires breathe 4.34.

Signed-off-by: Matthew Gerlach <matthew.gerlach@linux.intel.com>
@fpgamatt fpgamatt force-pushed the tswhiston/opae.io_sriov_take2 branch from 3ec47cd to 040053c Compare January 19, 2024 23:52
@fpgamatt
Copy link
Contributor Author

Yea! All checks have passed. Now I need an approval.

@fpgamatt
Copy link
Contributor Author

I only see the option to "Squash and merge" as enabled. Why can't I "Rebase and merge" because I was to keep the 3 commits separate.

@pcolberg
Copy link
Contributor

@fpgamatt Rebase and merge should be enabled now.

@fpgamatt
Copy link
Contributor Author

Thanks Peter! I see Rebase and merge.

@fpgamatt fpgamatt merged commit ee102f6 into master Jan 20, 2024
31 checks passed
@fpgamatt fpgamatt deleted the tswhiston/opae.io_sriov_take2 branch January 20, 2024 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants