app/testpmd: fix quit testpmd with vfs and pf
Checks
Commit Message
When testpmd startups with pf and vfs,this error occurs when quitting,
results in pf is released before vfs ,so the vf would access an
freed heap memory.
The solution is two steps:
1. Fetch the valid port value from RTE_ETH_FOREACH_DEV.
2. free the port in reverse order.
Signed-off-by: Ke Zhang <ke1x.zhang@intel.com>
---
app/test-pmd/testpmd.c | 28 +++++++++++++++++++++++-----
1 file changed, 23 insertions(+), 5 deletions(-)
@@ -3378,8 +3378,11 @@ detach_devargs(char *identifier)
void
pmd_test_exit(void)
{
+ unsigned int ethports[RTE_MAX_ETHPORTS];
+ unsigned int count = 0;
portid_t pt_id;
unsigned int i;
+ int index;
int ret;
if (test_done == 0)
@@ -3396,15 +3399,30 @@ pmd_test_exit(void)
#endif
if (ports != NULL) {
no_link_check = 1;
+ i = 0;
+
+ /* Fetch the valid port id from port list*/
RTE_ETH_FOREACH_DEV(pt_id) {
- printf("\nStopping port %d...\n", pt_id);
+ ethports[i] = pt_id;
+ i++;
+ }
+
+ count = i;
+ /*
+ * Free the port from Reverse order, as general,
+ * PF port < VF port, VF should be free before PF
+ * be free.
+ */
+ for (index = count - 1 ; index >= 0 ; index--) {
+ printf("\nStopping port %d...\n", ethports[index]);
fflush(stdout);
- stop_port(pt_id);
+ stop_port(ethports[index]);
}
- RTE_ETH_FOREACH_DEV(pt_id) {
- printf("\nShutting down port %d...\n", pt_id);
+
+ for (index = count - 1 ; index >= 0 ; index--) {
+ printf("\nShutting down port %d...\n", ethports[index]);
fflush(stdout);
- close_port(pt_id);
+ close_port(ethports[index]);
}
}