Hi, > -----Original Message----- > From: Gregory Etelson <[email protected]> > Sent: Thursday, November 3, 2022 12:44 PM > To: [email protected] > Cc: Gregory Etelson <[email protected]>; Matan Azrad > <[email protected]>; Raslan Darawsheh <[email protected]>; > [email protected]; Slava Ovsiienko <[email protected]>; Anatoly > Burakov <[email protected]>; Dmitry Kozlyuk > <[email protected]> > Subject: [PATCH] common/mlx5: fix shared mempool subscription > > MLX5 PMD counted each mempool subscribe invocation. The PMD expected > that the mempool subscription will be deleted after the mempool counter > dropped to 0. However, current PMD design unsubscribes mempool > callbacks only once. > As the result, the PMD destroyed mlx5_common_device but kept shared RX > subscription callback. EAL tried to activate that callback and crashed. > > The patch removes mempool subscriptions counter. > The PMD registers mempool subscription once only. An attempt to register > existing subscription returns EEXIST. > Also, the PMD expects to remove subscription when mempool unsubscribe > was activated. > > Fixes: 8ad97e4b3215 ("common/mlx5: fix multi-process mempool > registration") > > Cc: [email protected] > > Signed-off-by: Gregory Etelson <[email protected]> > Acked-by: Matan Azrad <[email protected]>
Patch applied to next-net-mlx, Kindest regards, Raslan Darawsheh
