RE: [PATCH v3 2/2] app/testpmd: fix aged flow destroy

2024-11-18 Thread Dariusz Sosnowski
sil-...@napatech.com > Cc: Gregory Etelson ; Alexander Kozyrev > ; dev@dpdk.org; sta...@dpdk.org; > ferruh.yi...@amd.com > Subject: [PATCH v3 2/2] app/testpmd: fix aged flow destroy > > External email: Use caution opening links or attachments > > > port_flow_destroy() fun

[PATCH v3 2/2] app/testpmd: fix aged flow destroy

2024-11-18 Thread Danylo Vodopianov
port_flow_destroy() function never assumed that rule array can be freed when it's executing, and port_flow_aged() just violated that assumption. In case of flow async create failure, it tries to do a cleanup, but it wrongly removes a 1st flow (with id 0). pf->id is not set at this moment and it al