Skip to content

refactor(rebinding)!: remove two-phase prepare/finalize model#777

Merged
jyao1 merged 1 commit intointel:mainfrom
MichalTarnacki:two-phase
Apr 7, 2026
Merged

refactor(rebinding)!: remove two-phase prepare/finalize model#777
jyao1 merged 1 commit intointel:mainfrom
MichalTarnacki:two-phase

Conversation

@MichalTarnacki
Copy link
Copy Markdown
Contributor

@MichalTarnacki MichalTarnacki commented Apr 3, 2026

refactor(rebinding)!: remove two-phase prepare/finalize model
Remove the two-phase rebinding model (PREPARE/FINALIZE) per GHCI 1.5:

  • Remove MIGTD_REBIND_OP_PREPARE/FINALIZE constants
  • Remove operation field from RebindingInfo struct
  • Adjust reserved field check (bytes 10-15 per spec)
  • Remove rebinding_old_finalize() and rebinding_new_finalize()
  • Simplify start_rebinding() to call prepare directly

BREAKING CHANGE: RebindingInfo no longer contains an operation field

@MichalTarnacki MichalTarnacki requested a review from jyao1 as a code owner April 3, 2026 06:21
@MichalTarnacki MichalTarnacki changed the title Two phase refactor(rebinding)!: remove two-phase prepare/finalize model Apr 3, 2026
Remove the two-phase rebinding model (PREPARE/FINALIZE) per GHCI 1.5:
- Remove MIGTD_REBIND_OP_PREPARE/FINALIZE constants
- Remove operation field from RebindingInfo struct
- Adjust reserved field check (bytes 10-15 per spec)
- Remove rebinding_old_finalize() and rebinding_new_finalize()
- Simplify start_rebinding() to call prepare directly

BREAKING CHANGE: RebindingInfo no longer contains an operation field

Co-authored-by: Grams, Stanislaw <stanislaw.grams@intel.com>
@jyao1 jyao1 merged commit 433dd79 into intel:main Apr 7, 2026
57 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