|WARNING| [7/7] doc: add missing headers to doxy-api-index.md
dpdklab at iol.unh.edu
dpdklab at iol.unh.edu
Thu Jul 2 22:43:12 CEST 2026
Test-Label: ai-code-review
Test-Status: WARNING
https://dpdk.org/patch/166498
_AI Code Review_
=== Patch Review: series.patch (via Claude) ===
Review date: 2026-07-02
# DPDK Patch Review
## Patch 1/7: doc: detect ignored public headers
### Errors
**PATCH 1: In `lib/meson.build`, line 228, wrong variable name in warning message:**
```
warning('public header directory @0@ is not listed in @1@'.format(doc_dir, doc_build_path))
```
The variable is named `d` in the foreach loop, not `doc_dir`. This will cause a build error if the warning triggers.
**Suggested fix:**
```diff
- warning('public header directory @0@ is not listed in @1@'.format(doc_dir,
+ warning('public header directory @0@ is not listed in @1@'.format(d,
doc_build_path))
```
### Warnings
**PATCH 1: Missing release notes for build infrastructure change:**
Release notes should be updated to describe the new documentation validation checks.
---
## Patch 2/7: doc: document rte_os.h
No issues found.
---
## Patch 3/7: doc: fix typos in rte_bus_pci.h
No issues found.
---
## Patch 4/7: doc: fix typos in rte_bus_vmbus.h
No issues found.
---
## Patch 5/7: doc: add missing globs to doxy-api.conf.in
No issues found.
---
## Patch 6/7: doc: add missing drivers to doxy-api.conf.in
No issues found.
---
## Patch 7/7: doc: add missing headers to doxy-api-index.md
No issues found.
---
## Summary
**Total findings: 2**
- **Errors: 1** (variable name bug in warning message)
- **Warnings: 1** (missing release notes)
The patch series correctly addresses documentation coverage gaps. The only correctness issue is the variable name error in patch 1 that would cause a build failure if the code path is executed. The patches are otherwise well-structured and follow DPDK coding style.
More information about the test-report
mailing list