mbox series

[v2,0/2] Thread termination leak fixes

Message ID 20210511113358.2485-1-david.marchand@redhat.com (mailing list archive)
Headers
Series Thread termination leak fixes |

Message

David Marchand May 11, 2021, 11:33 a.m. UTC 
  After looking at the control thread API use in recent changes, I noticed
that some thread resources are not properly released.
This series fixes two net drivers.

Additional notes:
- I noticed that event/dlb2 never kills/terminates its service thread.
  It means unplug of this device (if supported) would leak the associated
  thread resources. I leave it to this driver maintainer to comment/fix.

- The vhost library also lets a service thread in the background but since
  there is no way to unitialize this library, there is no leak to fix from
  my pov.

Comments

Ferruh Yigit May 11, 2021, 9:41 p.m. UTC | #1 
On 5/11/2021 12:33 PM, David Marchand wrote:
> After looking at the control thread API use in recent changes, I noticed
> that some thread resources are not properly released.
> This series fixes two net drivers.
> 

Series applied to dpdk-next-net/main, thanks.

> Additional notes:
> - I noticed that event/dlb2 never kills/terminates its service thread.
>   It means unplug of this device (if supported) would leak the associated
>   thread resources. I leave it to this driver maintainer to comment/fix.
> 
> - The vhost library also lets a service thread in the background but since
>   there is no way to unitialize this library, there is no leak to fix from
>   my pov.
>