|SUCCESS| [v2,6/6] vhost: simplify some descriptor handling

[dpdklab at iol.unh.edu]

Test-Label: ai-code-review
Test-Status: SUCCESS
https://dpdk.org/patch/169089

_AI Code Review_

=== Patch Review: series.patch (via Claude) ===
Review date: 2026-09-04

# DPDK Vhost Async Datapath Removal Review

## Patch 1/6: vhost: decouple guest pages population from async datapath

### Errors

None.

### Warnings

None.

### Info

- The patch introduces `RTE_VHOST_USER_MAP_POPULATE` as an alias for `RTE_VHOST_USER_ASYNC_COPY` to decouple MAP_POPULATE behavior from async datapath before removing async support. This is a preparatory refactoring step with no functional change.

---

## Patch 2/6: examples/vhost: stop using async datapath

### Errors

None.

### Warnings

None.

### Info

- The vhost example is updated to stop using async datapath APIs before their removal in patch 4. The removal of `allow_experimental_apis = true` is correct since the example no longer uses experimental APIs after dropping async support.

---

## Patch 3/6: examples/vdpa: remove dependency on PCI library

### Errors

None.

### Warnings

None.

---

## Patch 4/6: vhost: drop async datapath

### Errors

None.

### Warnings

None.

### Info

- This is the main patch removing all async datapath code. The removal is comprehensive and includes API functions, internal structures, and helper code. Release notes are properly updated documenting the removal of experimental APIs.

---

## Patch 5/6: vhost: rename packed layout helpers for batches

### Errors

None.

### Warnings

None.

---

## Patch 6/6: vhost: simplify some descriptor handling

### Errors

None.

### Warnings

None.

### Info

- Post-async-removal cleanup that inlines descriptor copy logic and removes the intermediate `sync_fill_seg()` helper. The elimination of `buf_iova` tracking in the descriptor-to-mbuf path is a valid optimization since IOVA is only needed for logging in the mbuf-to-descriptor direction.

---

## Summary

All six patches are clean with no correctness bugs, style violations, or process compliance issues. The series properly removes the experimental async datapath feature following DPDK policy (experimental APIs can be removed), updates documentation and release notes, and includes cleanup patches to simplify code post-removal.