Hi, > -----Original Message----- > From: Suanming Mou <suanmi...@mellanox.com> > Sent: Thursday, May 28, 2020 10:00 AM > To: Slava Ovsiienko <viachesl...@mellanox.com>; Matan Azrad > <ma...@mellanox.com>; dev@dpdk.org > Cc: Raslan Darawsheh <rasl...@mellanox.com>; sta...@dpdk.org > Subject: [PATCH v2] net/mlx5: fix secondary process resources release > > When secondary process starts, it will allocate its own process private > data, and also does remap to UAR register of the Tx queue. Once the > secondary process exits, these resources should be released accordingly. > And the shared resources owned by primary should not be touched. > > Currently, once one port in the secondary process spawn failed, all the > other spawned ports will also be released during process exits. However, > the mlx5_dev_close() function does not add the cases for secondary > process, it means call the mlx5_dev_close() function directly in > secondary process releases the resources it should not touch. > > Add the case for secondary process release to its own resources in > mlx5_dev_close() function to help it quits gracefully. > > Fixes: 942d13e6e7d1 ("net/mlx5: fix sharing context destroy order") > Fixes: 3a8207423a0f ("net/mlx5: close all ports on remove") > Cc: sta...@dpdk.org > > Signed-off-by: Suanming Mou <suanmi...@mellanox.com> > Acked-by: Matan Azrad <ma...@mellanox.com> > --- > v2: fix more issues in secondary process. > --- > drivers/net/mlx5/mlx5.c | 47 +++++++++++++++++++++++++++++++----- > --------
Patch applied to next-net-mlx, Kindest regards, Raslan Darawsheh