Re: [ovs-dev] [PATCH] ofproto-dpif-upcall: Try lock for umap iteration during sweep.

2024-03-15 Thread Eelco Chaudron
Hi, Thanks for submitting a patch and researching the problem. I haven't had the chance to review your patch yet, as I still need to dedicate some time to review it, based on the complexity. However, if you've sent a new version in the meantime, please make sure to label it with a version

[ovs-dev] [PATCH] ofproto-dpif-upcall: Try lock for umap iteration during sweep.

2024-03-15 Thread LIU Yulong
A potential race condition happened with the following 3 threads: * PMD thread replaced the old_ukey and transitioned the state to UKEY_DELETED. * RCU thread is freeing the old_ukey mutex. * While the revalidator thread is trying to lock the old_ukey mutex. Then vswitchd process aborts at the

[ovs-dev] [PATCH] ofproto-dpif-upcall: try lock for umap iteration during sweep

2024-03-14 Thread LIU Yulong
A potential race condition happened with the following 3 threads: * PMD thread replaced the old_ukey and transitioned the state to UKEY_DELETED. * RCU thread is freeing the old_ukey mutex. * While the revalidator thread is trying to lock the old_ukey mutex. Then vswitchd process aborts at the

Re: [ovs-dev] [PATCH] ofproto-dpif-upcall: try lock for umap iteration during sweep

2024-03-13 Thread 0-day Robot
Bleep bloop. Greetings LIU Yulong, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: The subject summary should start with a capital. WARNING: The subject summary should

[ovs-dev] [PATCH] ofproto-dpif-upcall: try lock for umap iteration during sweep

2024-03-13 Thread LIU Yulong
A potential race condition happened with the following 3 threads: * PMD thread replaced the old_ukey and transitioned the state to UKEY_DELETED. * RCU thread is freeing the old_ukey mutex. * While the revalidator thread is trying to lock the old_ukey mutex. Then vswitchd process aborts at the