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

Storage simulation enhancements #176

Merged

Conversation

tonyastolfi
Copy link
Collaborator

No description provided.

@tonyastolfi tonyastolfi self-assigned this Dec 3, 2024
Copy link
Collaborator

@vidyasilai vidyasilai left a comment

Choose a reason for hiding this comment

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

LGTM! Just a nit comment and an open question.

src/llfs/page_cache_slot.cpp Show resolved Hide resolved
src/llfs/page_cache_slot_pinned_ref.hpp Show resolved Hide resolved
Add ScenarioRunner (testing); more static checks for cache slots/refs.

Fix header double-inclusion guard name.

CR feedback.
@tonyastolfi tonyastolfi force-pushed the storage-simulation-enhancements branch from 0db0caa to 4fa9d6d Compare December 4, 2024 22:03
@tonyastolfi tonyastolfi merged commit b1acd08 into mathworks:main Dec 6, 2024
4 checks passed
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