[dpdk-dev] Flow Director - big endian handling

[Yaacov Hazan]

Hi JingJing,

I looked at your patch for flow director - app/testpmd: update flow director commands - a56335925919d26c81dec8accf31c39d2f790c5a.

It seems there is some mismatch in the handling of big endian between the filter and mask.
In the cmd_flow_director_filter_parsed function, which add the filter values, you called to rte_cpu_to_be_16 for the vlan_tci and ports values.
But in cmd_flow_director_mask_parsed function, which set the mask, you didn't called to rte_cpu_to_be_16 for those values (valn_tci & ports).

Does Intel's NICs (or Intel's PMDs) expected form application side to handle the big endian in different way for the filter values and the mask values?
If yes, it is very confusing from the application/user point of view.
I think that it is more make sense to leave the decision and handling of the big endian to the PMD layer, or at least to keep consistency for the expected handling in the application layer.

Thanks,
Yaacov.