Skip to content

Check if plugins need to skip nscd regions#1194

Merged
xuyao0127 merged 2 commits intodmtcp:mainfrom
xuyao0127:fix_skip_memory
Mar 4, 2025
Merged

Check if plugins need to skip nscd regions#1194
xuyao0127 merged 2 commits intodmtcp:mainfrom
xuyao0127:fix_skip_memory

Conversation

@xuyao0127
Copy link
Copy Markdown
Collaborator

This commit fixes a bug when writing memory areas to disk. Previously, nscd areas are written to the disk before calling the dmtcp_skip_memory_region_ckpting() functions. Therefore, no matter if a plugin decides to skip the nscd area, the nscd area will always be written to the disk.

@xuyao0127 xuyao0127 requested review from gc00 and karya0 March 3, 2025 01:28
Copy link
Copy Markdown
Contributor

@gc00 gc00 left a comment

Choose a reason for hiding this comment

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

LGTM.

@karya0 says there's a cleaner way to do this. But I'm approving now, in order to not block progress. @karya0 , when you have more time, please do suggest any cleaner way that you may have.

@karya0
Copy link
Copy Markdown
Member

karya0 commented Mar 4, 2025

LGTM.

@karya0 says there's a cleaner way to do this. But I'm approving now, in order to not block progress. @karya0 , when you have more time, please do suggest any cleaner way that you may have.

I already suggested the cleaner way in the comment earlier : ). Re-elaborated it.

@xuyao0127: Feel free to merge it as is for now, but please make sure to create a separate PR to clean it.

@xuyao0127 xuyao0127 merged commit a74bac2 into dmtcp:main Mar 4, 2025
1 check passed
@xuyao0127 xuyao0127 deleted the fix_skip_memory branch March 17, 2025 18:57
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