On 6/9/21 3:09 PM, Gaetan Rivet wrote:
> The atomic exchange operation is a useful primitive that should be
> available as well.  Most compiler already expose or offer a way

s/compiler/compilers/

> to use it, but a single symbol needs to be defined.
> 
> Signed-off-by: Gaetan Rivet <[email protected]>
> Reviewed-by: Eli Britstein <[email protected]>
> ---
>  lib/ovs-atomic-c++.h      |  3 +++
>  lib/ovs-atomic-clang.h    |  5 +++++
>  lib/ovs-atomic-gcc4+.h    |  5 +++++
>  lib/ovs-atomic-gcc4.7+.h  |  5 +++++
>  lib/ovs-atomic-i586.h     |  5 +++++
>  lib/ovs-atomic-locked.h   |  9 +++++++++
>  lib/ovs-atomic-msvc.h     | 22 ++++++++++++++++++++++
>  lib/ovs-atomic-pthreads.h |  5 +++++
>  lib/ovs-atomic-x86_64.h   |  5 +++++
>  lib/ovs-atomic.h          |  8 +++++++-
>  10 files changed, 71 insertions(+), 1 deletion(-)
> 

Other than this typo, it looks good to me:

Reviewed-by: Maxime Coquelin <[email protected]>

Thanks,
Maxime

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

Reply via email to