|SUCCESS| net/gve: advertise Rx scatter in GQI queue format

[dpdklab at iol.unh.edu]

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

_AI Code Review_

=== Patch Review: series.patch (via Claude) ===
Review date: 2026-10-19

# DPDK Patch Review

## ERRORS

None identified.

## WARNINGS

None identified.

## INFO

### Documentation Completeness

The features matrix update marks "Scattered Rx" as `P` (Partially supported).
According to `doc/guides/nics/features.rst`, features marked as partial should include a note in the limitations section of the driver guide explaining what is not supported.

The patch correctly advertises scatter only for GQI (not DQO), which justifies the `P` marking.
Consider adding a note to `doc/guides/nics/gve.rst` documenting that scatter Rx is available only in GQI queue format.

### Code Comment Quality

The comment explains the GQI multi-segment behavior clearly:
```c
/* GQI receives a frame larger than one Rx buffer as a chain of
 * buffers (gve_rx_burst follows GVE_RXF_PKT_CONT).
 */
```

This is good documentation of the conditional capability advertisement.

## SUMMARY

The patch correctly addresses the missing scatter Rx capability advertisement for GQI queue format.
The fix is minimal, well-targeted, and includes appropriate documentation updates (mailmap, features matrix).
The conditional logic is correct: only GQI supports scatter, so only GQI advertises it.

The bug this fixes is significant:
applications that check capabilities before enabling multi-segment Rx (VPP being the example given)
would fail to handle jumbo frames correctly on GQI gVNIC,
effectively capping usable MTU despite hardware support.

Review-Result: CLEAN