On Tue, Jul 1, 2014 at 12:22 PM, Jesse Gross <je...@nicira.com> wrote: > On Tue, Jul 1, 2014 at 11:33 AM, Pritesh Kothari (pritkoth) > <pritk...@cisco.com> wrote: >> >> On Jul 1, 2014, at 11:18 AM, Pravin Shelar <pshe...@nicira.com> wrote: >> >>> why not just check for HAVE_U64_STATS_FETCH_BEGIN_IRQ? >> >> because of this which was added in 3.13, i guess: >> >> -#if LINUX_VERSION_CODE < KERNEL_VERSION(3,13,0) >> - >> -#if BITS_PER_LONG == 32 && defined(CONFIG_SMP) >> -# define u64_stats_init(syncp) seqcount_init(syncp.seq) >> -#else >> -# define u64_stats_init(syncp) do { } while (0) >> -#endif > > Yes, that's the reason why. Thanks to both of you for the reviews, I > pushed this to master and back to branch-2.1.
HAVE_U64_STATS_FETCH_BEGIN_IRQ should get defined for all kernel which defines u64_stats_fetch_begin_irq(). _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev