Re: [PATCH v1 0/7] Flow API helpers enhancements

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 10/5/2017 10:49 AM, Adrien Mazarguil wrote:
> This series brings enhancements to various rte_flow helpers:
> 
> - Allow applications to use rte_flow_error_set() by making it part of the
>   public interface and documenting it as such.
> 
> - Address rte_flow_copy()'s limitations by replacing it with the more
>   versatile rte_flow_conv(). This new function allows retrieving other
>   properties such as item/action names, enabling testpmd to finally use it
>   and get rid of duplicated code.
> 
> - Add a script (gen-rte_flow_conv-h.sh) to help with generating the
>   resources used by rte_flow_conv(). Developers should run it when adding or
>   modifying pattern items or actions (done as part of this series to add the
>   missing "fuzzy" pattern item).
> 
> - Future plans for rte_flow_conv() include translating error codes to
>   human-readable messages, so applications do not have to make their own.
> 
> All these changes address concerns raised a couple of months ago [1]. Work
> on these patches actually started at the time but I was unable to complete
> and clean them up until recently.
> 
> [1] http://dpdk.org/ml/archives/dev/2017-July/070492.html
> 
> Adrien Mazarguil (7):
>   ethdev: expose flow API error helper
>   ethdev: replace flow API object copy function
>   ethdev: add flow API item/action name conversion
>   app/testpmd: rely on flow API conversion function
>   ethdev: enhance flow API item/action descriptions
>   ethdev: generate flow API conversion header
>   ethdev: update flow API conversion header

Hi Adrien,

This received too late for this release cycle, and changes in rte_flow
library may effect PMDs.

I suggest deferring the set to next release, what do you think?

Thanks,
ferruh