Re: [PATCH 06/16] assoc_array: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread David Howells
Pranith Kumar wrote: > static inline bool assoc_array_ptr_is_meta(const struct assoc_array_ptr *x) > { > - return (unsigned long)x & ASSOC_ARRAY_PTR_TYPE_MASK; > + return (unsigned long)lockless_dereference(x) & > + ASSOC_ARRAY_PTR_TYPE_MASK;

[PATCH 06/16] assoc_array: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. I replaced the inline functions dereferencing pointer 'x' to use lockless_dereference() because of which we do not need to litter the code with

[PATCH 06/16] assoc_array: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread Pranith Kumar
Recently lockless_dereference() was added which can be used in place of hard-coding smp_read_barrier_depends(). The following PATCH makes the change. I replaced the inline functions dereferencing pointer 'x' to use lockless_dereference() because of which we do not need to litter the code with

Re: [PATCH 06/16] assoc_array: Replace smp_read_barrier_depends() with lockless_dereference()

2014-11-13 Thread David Howells
Pranith Kumar bobby.pr...@gmail.com wrote: static inline bool assoc_array_ptr_is_meta(const struct assoc_array_ptr *x) { - return (unsigned long)x ASSOC_ARRAY_PTR_TYPE_MASK; + return (unsigned long)lockless_dereference(x) +