[ovs-dev] [PATCH] datapath-windows: Add Windows thread atomic APIs for x64 binaries.
Guru Shetty
guru at ovn.org
Fri Mar 25 17:58:09 UTC 2016
On 25 March 2016 at 10:41, Sorin Vinturis <svinturis at cloudbasesolutions.com>
wrote:
> This patch modifies atomic operations library export header to
> include Windows thread atomic APIs, ovs-atomic-msvc.h instead of
> ovs-atomic-pthreads.h, for both x86 and x64 binaries.
>
> Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
>
Have you gone through the ovs-atomic-msvc.h intrinsics carefully to make
sure that it will actually be atomic on 64 bit?
> ---
> lib/ovs-atomic.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/ovs-atomic.h b/lib/ovs-atomic.h
> index b38c9ef..b385b9c 100644
> --- a/lib/ovs-atomic.h
> +++ b/lib/ovs-atomic.h
> @@ -333,7 +333,7 @@
> #include "ovs-atomic-i586.h"
> #elif HAVE_GCC4_ATOMICS
> #include "ovs-atomic-gcc4+.h"
> - #elif _MSC_VER && _M_IX86 >= 500
> + #elif _WIN32
> #include "ovs-atomic-msvc.h"
> #else
> /* ovs-atomic-pthreads implementation is provided for portability.
> --
> 1.9.0.msysgit.0
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
More information about the dev
mailing list