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

specification: Clear all ATC cache when unbinding a TDI #121

Merged
merged 1 commit into from
May 14, 2024

Conversation

sameo
Copy link
Collaborator

@sameo sameo commented May 14, 2024

No description provided.

8. The TSM removes all C-IOMMU, TVM G-stage mappings and the MMIO gpa -> hpa
mapping for the interface.
8. The TSM removes all C-IOMMU mappings and all cached C-IOMMU ATC mappings, the
TVM G-stage mappings and the MMIO gpa -> hpa mappings for the interface.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Note: Add a glossary entry for ATC

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good catch. Done.

@@ -923,8 +923,8 @@ Unbinding an interface from a TVM goes through the following steps:
belongs to a bound interface, or if it does not belong to the MMIO gpa -> hpa
mapping the TSM initially created for the interface at
xref:binding-flow[TDI binding time].
8. The TSM removes all C-IOMMU, TVM G-stage mappings and the MMIO gpa -> hpa
mapping for the interface.
8. The TSM removes all C-IOMMU mappings and all cached C-IOMMU ATC mappings, the
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe add full name for ATC?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done as well, thanks.

@sameo sameo force-pushed the topic/tdi-unbinding branch from 0926b67 to 99e5f52 Compare May 14, 2024 09:17
Signed-off-by: Samuel Ortiz <sameo@rivosinc.com>
@sameo sameo force-pushed the topic/tdi-unbinding branch from 99e5f52 to e1ba3a5 Compare May 14, 2024 09:19
@sameo sameo merged commit 357ed7b into riscv-non-isa:main May 14, 2024
2 checks passed
Copy link
Collaborator

@rsahita rsahita left a comment

Choose a reason for hiding this comment

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

lgtm!

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.

3 participants