|SUCCESS| ethdev: replace rte_memcpy with assignment

[dpdklab at iol.unh.edu]

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

_AI Code Review_

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

# DPDK Patch Review

## Patch Summary
The patch replaces `rte_memcpy()` with structure assignment in `rte_eth_dev_owner_get()` and removes the now-unused `<rte_memcpy.h>` include.

---

## Review Findings

###  Correctness
No correctness issues identified. The change is semantically equivalent and safe:
- Structure assignment is valid for `struct rte_eth_dev_owner`
- The lifetime and alignment of both source and destination are appropriate
- No risk of use-after-free, resource leaks, or race conditions introduced

###  C Coding Style
**Compliant.** The change follows DPDK guidelines:
- Structure assignment is preferred over `rte_memcpy()` in control-path code
- Include cleanup (removing unused `<rte_memcpy.h>`) is correct
- Code style is consistent with existing code

Per AGENTS.md "Appropriate Use of rte_memcpy()":
> For general use, standard `memcpy()` is preferred... Modern compilers optimize `memcpy()` effectively

Structure assignment is even better than `memcpy()` for fixed-size structures in non-fast-path code.

### Info: Documentation/Testing
This is a purely internal implementation change with no API or ABI impact:
- No public API signatures changed
- No new features added
- No deprecated API usage
- No release notes required (internal refactoring)
- No tests required (behavior unchanged)

---

## Summary
**No issues found.** The patch is correct and improves code quality by using idiomatic C structure assignment instead of an explicit copy operation in a control-path function. The removal of the unused include is appropriate cleanup.