On 3/13/23 21:14, Ilya Maximets wrote:
> On 3/13/23 16:28, Dumitru Ceara wrote:
>> ATOMIC_VAR_INIT has been removed in OVS since
>> https://github.com/openvswitch/ovs/commit/71ca8393b700 because it has a
>> trivial definition and was deprecated (to be removed) in newer
>> standards.
>>
>> Stop using it in OVN too.
>>
>> Reported-at: 
>> https://github.com/ovn-org/ovn/actions/runs/4394882419/jobs/7696233921#step:13:3361
>> Signed-off-by: Dumitru Ceara <[email protected]>
>> ---
>> V2:
>> - Addressed Ilya's comments:
>>   - use atomic_init() for (re)initialization of atomic variables.
>> ---
>>  lib/ovn-parallel-hmap.c | 4 ++--
>>  northd/northd.c         | 4 ++--
>>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> Thanks!  I didn't test, but LGTM.
> 
> Acked-by: Ilya Maximets <[email protected]>
> 

Thanks!  I applied this to the main branch.


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to