|SUCCESS| [v2] config/arm: add Marvell CN20k
dpdklab at iol.unh.edu
dpdklab at iol.unh.edu
Tue Nov 3 00:02:17 CET 2026
Test-Label: ai-code-review
Test-Status: SUCCESS
https://dpdk.org/patch/172353
_AI Code Review_
=== Patch Review: series.patch (via Claude) ===
Review date: 2026-11-02
# DPDK Patch Review
## Patch Summary
Adds ARM cross-compilation configuration support for Marvell CN20k SoC (Neoverse N3 cores).
## ERRORS
None.
## WARNINGS
None.
## INFO
**CPU Part Number Verification:**
The patch sets `'part_number': '0xd8e'` for the Neoverse N3 core.
According to ARM documentation, the correct part number for Neoverse N3 is `0xd8e`.
This appears correct, but please verify against official ARM Technical Reference Manual
as part numbers must match hardware exactly for proper feature detection.
**Maximum NUMA Nodes:**
The patch sets `'RTE_MAX_NUMA_NODES'` to 1.
Verify this matches the actual CN20k hardware configuration.
Many server-class ARM SoCs support multi-socket configurations with multiple NUMA nodes.
**Maximum LCOREs:**
`'RTE_MAX_LCORE'` is set to 42.
This seems unusual (not a power of 2 or a typical core count).
Verify this matches the exact core count of CN20k SKUs.
Consider whether this should account for SMT or future SKU variants.
**Missing crypto march feature:**
Related SoC configs like `soc_cobalt100` include `'extra_march_features': ['crypto']`
to enable ARM Crypto Extensions.
If CN20k (Neoverse N3) supports the ARMv8 Crypto Extensions,
consider adding this field for optimal cryptographic performance.
**Documentation:**
No release notes update is included.
While adding a new SoC config is primarily a build system change,
consider whether this warrants a mention in release notes
as it enables support for a new platform family.
Review-Result: CLEAN
More information about the test-report
mailing list