[ovs-dev] [PATCH] ovs-atomic: Reintroduce atomic_uint64_t and atomic_int64_t.

Ben Pfaff blp at ovn.org
Fri Oct 6 16:47:28 UTC 2017


On Wed, Oct 04, 2017 at 10:15:30AM +0200, Simon Horman wrote:
> On Fri, Sep 29, 2017 at 10:18:52AM -0700, Ben Pfaff wrote:
> > This is essentially a revert of commit e09d61c41b4f ("ovs-atomic: Remove
> > atomic_uint64_t and atomic_int64_t.")  My fear that some 32-bit platforms
> > did not support 64-bit integers seems overblown, because OVS 2.6.x uses
> > the 64-bit atomic_ullong and it is in Debian, which has tons of
> > architectures.
> > 
> > CC: Simon Horman <horms at verge.net.au>
> > Signed-off-by: Ben Pfaff <blp at ovn.org>
> 
> Hi Ben,
> 
> I do not have any recollection of the discussion around e09d61c41b4f,
> but this seems reasonable to me.
> 
> Acked-by: Simon Horman <simon.horman at netronome.com>

Thanks for the review.  I applied this to master.


More information about the dev mailing list