Reviewed-by: Chengwen Feng <[email protected]>
On 8/22/2026 3:24 AM, Stephen Hemminger wrote:
> Don't need rte_memcpy for fixed size value.
>
> Signed-off-by: Stephen Hemminger <[email protected]>
> ---
> drivers/net/hns3/hns3_fdir.c | 2 +-
> drivers/net/hns3/hns3_flow.c | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/hns3/hns3_fdir.c b/drivers/net/hns3/hns3_fdir.c
> index 50572ae430..86dba4eb2e 100644
> --- a/drivers/net/hns3/hns3_fdir.c
> +++ b/drivers/net/hns3/hns3_fdir.c
> @@ -1066,7 +1066,7 @@ int hns3_fdir_filter_program(struct hns3_adapter *hns,
> return -ENOMEM;
> }
>
> - rte_memcpy(&node->fdir_conf, rule, sizeof(struct hns3_fdir_rule));
> + memcpy(&node->fdir_conf, rule, sizeof(struct hns3_fdir_rule));
> ret = hns3_insert_fdir_filter(hw, fdir_info, node);
> if (ret < 0) {
> rte_free(node);
> diff --git a/drivers/net/hns3/hns3_flow.c b/drivers/net/hns3/hns3_flow.c
> index f2d1e4ec3a..f7e4d339d3 100644
> --- a/drivers/net/hns3/hns3_flow.c
> +++ b/drivers/net/hns3/hns3_flow.c
> @@ -2588,8 +2588,8 @@ hns3_flow_query(struct rte_eth_dev *dev, struct
> rte_flow *flow,
> }
> rss_conf = (struct rte_flow_action_rss *)data;
> rss_rule = (struct hns3_rss_conf_ele *)flow->rule;
> - rte_memcpy(rss_conf, &rss_rule->filter_info.conf,
> - sizeof(struct rte_flow_action_rss));
> + memcpy(rss_conf, &rss_rule->filter_info.conf,
> + sizeof(struct rte_flow_action_rss));
> break;
> default:
> return rte_flow_error_set(error, ENOTSUP,
