|WARNING| [7/7] doc: add missing headers to doxy-api-index.md

[dpdklab at iol.unh.edu]

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

**1. Undefined variable `doc_dir` used instead of loop variable (drivers/meson.build:302)**

The code references `doc_dir` in the warning message, but the variable is `d` in the loop:
```python
if doc_dir not in doc_built_dirs
    warning('public header directory @0@ is not listed in @1@'.format(doc_dir,
        doc_build_path))
```
Should be:
```python
if d not in doc_built_dirs
    warning('public header directory @0@ is not listed in @1@'.format(d,
        doc_build_path))
```

**2. Undefined variable `doc_dir` used instead of loop variable (lib/meson.build:227)**

Same issue in `lib/meson.build`:
```python
if d not in doc_built_dirs
    warning('public header directory @0@ is not listed in @1@'.format(doc_dir,
        doc_build_path))
```
Should be:
```python
if d not in doc_built_dirs
    warning('public header directory @0@ is not listed in @1@'.format(d,
        doc_build_path))
```

### Warnings

**1. Missing feature or flag check for `fs.name()` (drivers/meson.build:293, lib/meson.build:214)**

The code calls `fs.name(h)` without verifying that the filesystem module methods are available. However, the check is already gated on `check_docs` which requires `meson.version().version_compare('>= 0.60.0')`, so this may be acceptable depending on when `fs.name()` was introduced in Meson. If `fs.name()` requires a newer Meson version, the condition should be adjusted.

---

## 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

### Critical Issues (Must Fix)
- **Patch 1/7**: Two instances of undefined variable `doc_dir` where loop variable `d` should be used (drivers/meson.build:302 and lib/meson.build:227). This will cause Meson build failures.

### Warnings
- **Patch 1/7**: Verify that `fs.name()` is available in Meson 0.60.0 (the minimum version checked). If not, the version requirement should be increased or the feature check should be more specific.

### Additional Notes
- The documentation fixes in patches 2-7 are straightforward and correct
- The series adds useful validation to catch documentation gaps
- After fixing the variable name bugs in patch 1, the series should provide the intended functionality