refine: Refine vm_exit

[junyu0312]

Summary by CodeRabbit

• Refactor
  • Reorganized internal module structure for improved code organization and maintainability. Import paths have been updated across multiple components to reflect the new module hierarchy.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3dc77ae and c6386f1.

📒 Files selected for processing (11)

• crates/vm-core/src/arch.rs
• crates/vm-core/src/arch/aarch64.rs
• crates/vm-core/src/arch/aarch64/vm_exit.rs
• crates/vm-core/src/arch/aarch64/vm_exit/smc.rs
• crates/vm-core/src/arch/vm_exit.rs
• crates/vm-core/src/arch/x86_64.rs
• crates/vm-core/src/arch/x86_64/vm_exit.rs
• crates/vm-core/src/virt/hvp.rs
• crates/vm-core/src/virt/hvp/vcpu.rs
• crates/vm-core/src/virt/kvm/vcpu/arch/aarch64.rs
• crates/vm-core/src/virt/kvm/vcpu/arch/x86_64.rs

💤 Files with no reviewable changes (2)

• crates/vm-core/src/arch.rs
• crates/vm-core/src/arch/vm_exit.rs

---

📝 Walkthrough

Walkthrough

The PR reorganizes the vm_exit module structure by relocating it from a parallel architecture-level module (arch/vm_exit/) to nested modules within each architecture-specific directory (arch/aarch64/vm_exit/ and arch/x86_64/vm_exit/). All affected import paths throughout the codebase are updated consistently to reflect this new structure.

Changes

Cohort / File(s)	Summary
Module Structure Reorganization crates/vm-core/src/arch.rs, crates/vm-core/src/arch/vm_exit.rs	Removed public vm_exit module exports from the arch-level module declarations, eliminating the previous centralized vm_exit re-export point.
Architecture-Specific Module Exports crates/vm-core/src/arch/aarch64.rs, crates/vm-core/src/arch/x86_64.rs	Added new public vm_exit module declarations nested within each architecture-specific module, establishing the new module path structure.
Internal Import Path Updates crates/vm-core/src/arch/aarch64/vm_exit.rs, crates/vm-core/src/arch/aarch64/vm_exit/smc.rs	Updated import paths to resolve dependencies from the new vm_exit module location within the aarch64 architecture directory.
Consumer Import Path Updates crates/vm-core/src/virt/hvp.rs, crates/vm-core/src/virt/hvp/vcpu.rs, crates/vm-core/src/virt/kvm/vcpu/arch/aarch64.rs, crates/vm-core/src/virt/kvm/vcpu/arch/x86_64.rs	Updated all references to VmExitReason, handle_vm_exit, and HandleVmExitResult to use the new nested module paths under each architecture directory.

Possibly related PRs

• feat: Refine API for smp #76: Modifies function signatures and implementations inside the aarch64 vm_exit module, directly affected by this module reorganization.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 The warren reorganized its tunnels today,
Moving vm_exit to each burrow's way.
From central paths to nested nests so neat,
Each arch now owns the module it will meet. 🏗️

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'Refine vm_exit' is vague and generic, using the non-descriptive term 'Refine' that doesn't convey the specific nature of the structural reorganization being performed.	Provide a more specific title that describes the actual changes, such as 'Move vm_exit module to architecture-specific directories' or 'Reorganize vm_exit module structure'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch refine

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}