On 5/25/23 13:51, Simon Horman wrote:
> On Sat, May 20, 2023 at 02:31:20AM +0200, Ilya Maximets wrote:
>> Tunnel config can be accessed by multiple threads at the same time and
>> it is supposed to be protected by the netdev_vport mutex.  However,
>> many functions are getting direct access to it via netdev API without
>> taking the mutex, creating a potential for various race conditions.
>>
>> Fix that by protecting the tunnel config with RCU.  The whole structure
>> is replaced on configuration changes.  Individual fields are never
>> updated and the structure itself is constant.  This way it can be safely
>> used by different threads within RCU grace period.
>>
>> Signed-off-by: Ilya Maximets <[email protected]>
> 
> Reviewed-by: Simon Horman <[email protected]>
> 

Thanks!  I applied the set.  Also backported down to 2.17
as these are mainly bug fixes.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to