|SUCCESS| [2/2] net/ice: fix DCF minimum API version log

dpdklab at iol.unh.edu dpdklab at iol.unh.edu
Mon Nov 9 07:49:28 CET 2026


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

_AI Code Review_

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

# Review of DPDK Patch Series

## PATCH 1/2: net/iavf: fix minimum API version log

### Correctness

This patch corrects a copy-paste error in a log format string. The second format argument should be `VIRTCHNL_VERSION_MINOR_START` but was incorrectly `VIRTCHNL_VERSION_MAJOR_START`, causing the error message to display the major version number twice instead of showing major.minor.

The fix is correct and does not introduce any bugs.

### Style and Compliance

No style issues identified. The change is minimal and correctly placed.

### Documentation

This is a logging fix that does not require release notes or documentation updates.

---

## PATCH 2/2: net/ice: fix DCF minimum API version log

### Correctness

This patch fixes an identical copy-paste error in the DCF (Device Config Function) code path. The second format argument should be `ICE_CPF_VIRTCHNL_VERSION_MINOR_START` but was incorrectly `ICE_CPF_VIRTCHNL_VERSION_MAJOR_START`.

The fix is correct and does not introduce any bugs.

### Style and Compliance

No style issues identified. The change is minimal and correctly placed.

### Documentation

This is a logging fix that does not require release notes or documentation updates.

---

## Series Assessment

Both patches are straightforward bug fixes that correct logging output. They do not change behavior, API, ABI, or data flow--only the diagnostic message printed when a version check fails. The fixes are appropriate for stable backport (correctly tagged with `Cc: stable at dpdk.org`).

No errors or warnings identified.

Review-Result: CLEAN


More information about the test-report mailing list