Claude review: nova-core: mm: Select GPU_BUDDY for VRAM allocation

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Minor Style Issue:**
```diff
+	select GPU_BUDDY
+ 	select RUST_FW_LOADER_ABSTRACTIONS
+ 	select AUXILIARY_BUS
```

**Problem:** Select statements not alphabetically ordered (noted by Danilo Krummrich).

**Recommendation:**
```kconfig
	select AUXILIARY_BUS
	select GPU_BUDDY
	select RUST_FW_LOADER_ABSTRACTIONS
```

**Verdict:** Trivial fix, functionally correct.

---

## SUMMARY OF REQUIRED CHANGES

**Must Fix Before Merge:**
1. **Patch 1:** Fix `is_linked()` logic error - rename to `has_nodes()` with correct semantics
2. **Patch 1:** Move to `rust/kernel/ffi/` directory
3. **Patch 1:** Add explicit FFI-only documentation warnings
4. **Patch 2:** Change `alloc_blocks()` return type to `impl PinInit` (author acknowledged)
5. **Patch 2:** Pass `GpuBuddyParams` by value

**Should Fix:**
1. **Patch 1:** Document `Sync` soundness assumptions
2. **Patch 2:** Add TODO comment for mutex placement
3. **Patch 3:** Alphabetize Kconfig selects

**Overall Verdict:** Series shows mature engineering with careful safety considerations, but needs critical fixes before merge. The review discussion demonstrates good collaboration in addressing design concerns.

---
Generated by Claude Code Patch Reviewer