[dpdk-dev] [PATCH v5 3/4] app/testpmd: add raw flow type to flow director
Wu, Jingjing
jingjing.wu at intel.com
Fri Oct 13 05:27:10 CEST 2017
> break;
> + case RTE_ETH_FLOW_RAW:
> + raw_packet_buff = open_ddp_package_file(res->filepath,
> + &raw_packet_size);
If you think open_ddp_package_file can be used for common binary file open,
could you rename it and then use it? Because it is not ddp file, right?
The buffer is allocated in open_ddp_package_file, after you use it, you need
to free it to avoid memory leak, or call close_ddp_package_file.
Even the ethdev patch is been rejected, and you may go to a private API way.
I think the comments are also worth for the raw package read.
Please pay attention in later version.
Thanks
Jingjing
More information about the dev
mailing list