On Thu, 13 Jun 2019 at 09:54, Ben Pfaff <b...@ovn.org> wrote: > On Wed, Jun 12, 2019 at 09:48:59PM -0700, Gurucharan Shetty wrote: > > Commit e981a45a6cae4 (ovs-atomic: Add 64 bit apis.) > > added a few 64 bit apis (e.g: atomic_count_inc64). For C++, > > this invokes std::atomic_fetch_*_explicit() functions in > > lib/ovs-atomic-c++.h. > > > > The function overloading for 64 bit function fails without > > specifiying something like: > std::atomic_fetch_*_explicit<std::uint64_t>(). > > But it looks tricky to do this with macros. > > > > This patch tries to fix the compilation failures by calling atomic > > functions on the variables itself. > > > > Signed-off-by: Gurucharan Shetty <g...@ovn.org> > > If this works (I did not test it) then it looks nicer than the previous > code. Thank you! > > Acked-by: Ben Pfaff <b...@ovn.org> >
Thank you. I tested this and pushed to master. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
