On 25 March 2016 at 10:41, Sorin Vinturis <svintu...@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 <svintu...@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@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
