On Thu, 11 May 2023 22:59:57 +0800 suntianyuan <suntiany...@baidu.com> wrote:
> The rte_vhost_driver_unregister() and vhost_user_read_cb() can be > called at the same time by 2 threads. reconn may be added back to > reconn_list by vhost_user_read_cb() after rte_vhost_driver_unregister() > removed from reconn_list. Then rte_vhost_driver_unregister free vsocket, > cause vhost_user_client_reconnect access invalid vsocket memory. > > Timeline is as below: > rte_vhost_driver_unregister thread execute vhost_user_remove_reconnect > vhost_user_read_cb thread execute vhost_user_start_client add reconn to > reconn_list > vhost_user_read_cb thread free conn > rte_vhost_driver_unregister thread cannot find conn, then execute > vhost_user_socket_mem_free > vhost_user_client_reconnect access invalid mem, crash > > Signed-off-by: suntianyuan <suntiany...@baidu.com> There is a test failure in the vf jumbo frames that needs evaluation. And this kind of patch needs ack from Maxime.
