[ovs-dev] [PATCH v2 6/8] netdev-dpdk: retry with queue action
Yuanhan Liu
yliu at fridaylinux.org
Mon Sep 18 02:21:40 UTC 2017
On Fri, Sep 15, 2017 at 06:30:06PM +0000, Darrell Ball wrote:
> That was my first reaction to remove such blind re-try. It could have
> been a good option if all the PMD driver report the error consistenly.
>
> And unfortunately, it's not true. For example, for MARK without QUEUE
> action, i40e reports RTE_FLOW_ERROR_TYPE_ACTION, which, IMO, is nothing
> wrong. While for mlx5, it reports RTE_FLOW_ERROR_TYPE_HANDLE. I don't
> know why it was set like this, and we may could fix this. But my point
> was, it's not that reliable to use rte_errno, at least it's true for now.
>
> [Darrell] I see what you mean looking at the drivers code and I imagine it makes you work difficult.
> It looks like only either RTE_FLOW_ERROR_TYPE_ACTION or RTE_FLOW_ERROR_TYPE_HANDLE
> is used though in these cases from what I saw; would it make sense to check for those specifically?
Yes, I think it's better than a blind retry (assuming it's failed due
to an unsupported protocol). Meanwhile, I will double check the DPDK
driver code.
--yliu
More information about the dev
mailing list