On Mon, 9 Nov 2026 at 23:14, Stephen Hemminger <[email protected]> wrote: > > On Fri, 6 Nov 2026 15:57:29 +0100 > David Marchand <[email protected]> wrote: > > > memif_socket_remove_device() calls rte_intr_callback_unregister() on > > the listener socket interrupt handle. If memif_listener_handler() is > > active at that point, the unregister returns -EAGAIN and the code > > proceeds to close and free the handle while the handler is still > > running. > > > > Use rte_intr_callback_unregister_sync() which retries until the > > handler completes. > > > > Fixes: 7281511c2cfa ("net/memif: fix some interrupt FD leaks") > > > > Signed-off-by: David Marchand <[email protected]> > > Reviewed-by: Stephen Hemminger <[email protected]> > > --- > > Changes since v1: > > - added missing _sync in other call, > > > Applied to next-net but dropped the Fixes line since that commit > is after the last release. Fixes is not necessary during the merge > cycle.
I disagree. Fixes: tag are not *only* for backport purpose. Those tags also give useful information when looking back in git history, to understand why change X is done. -- David Marchand
