|WARNING| dts: avoid Scapy MAC resolution in Rx split test

[dpdklab at iol.unh.edu]

Test-Label: ai-code-review
Test-Status: WARNING
https://dpdk.org/patch/165266

_AI Code Review_

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

# DPDK Patch Review

## Patch Summary
This patch modifies the DTS (DPDK Test Suite) Rx split test to avoid Scapy MAC resolution warnings during Sphinx documentation generation by providing a dummy MAC address when calculating the Ethernet header length.

---

## FINDINGS

### **Errors**
None

### **Warnings**
None

### **Info**

**1. Consider using a more explicit constant name**

The dummy MAC address `"00:00:00:00:00:00"` serves only to prevent Scapy's destination resolution. Consider using a named constant to make this intent clearer:

```python
# At module level, near other constants
_DUMMY_MAC = "00:00:00:00:00:00"  # Prevents Scapy MAC resolution during import
ETHER_HDR_LEN = len(Ether(dst=_DUMMY_MAC))
```

This makes it immediately clear to future readers that the MAC address value itself is insignificant and exists solely to suppress the warning.

---

## SUMMARY

**Correctness**: No issues identified. The change is safe - providing a dummy destination MAC to `Ether()` does not affect the header length calculation, which is the only value used from this expression.

**Style**: The patch follows Python and DPDK coding conventions. The use of a null MAC address is a reasonable solution to avoid Scapy's ARP resolution behavior during module import.

**Documentation**: The commit message clearly explains the problem (Sphinx warning during documentation build) and the solution (dummy MAC to prevent resolution). No additional documentation is needed for this internal test change.

**Testing**: This is a test infrastructure change that fixes a warning during doc generation. No additional testing required beyond verifying the warning is suppressed and the test still functions correctly.

The patch is acceptable as-is. The Info-level suggestion about using a named constant is optional and would only improve code clarity.