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
