On Thu, 16 Nov 2017 09:37:48 -0600 Brian King <brking@xxxxxxxxxxxxxxxxxx> wrote: > Resending as the first attempt is not showing up in the list archive. > > This patch converts several network drivers to use smp_rmb > rather than read_barrier_depends. The initial issue was > discovered with ixgbe on a Power machine which resulted > in skb list corruption due to fetching a stale skb pointer. > More details can be found in the ixgbe patch description. Thanks for the fix Brian, I bet it was a tough debug. The only users in the entire kernel of read_barrier_depends() (not smp_read_barrier_depends) are the Intel network drivers. Wouldn't it be better for power to just fix read_barrier_depends to do the right thing on power? The question I'm not sure of the answer to is: Is it really the wrong barrier to be using or is the implementation in the kernel powerpc wrong? So I think the right thing might actually to be to: Fix arch powerpc read_barrier_depends to not be a noop, as the semantics of the read_barrier_depends seems to be sufficient to solve this problem, but it seems not to work for powerpc? Alex Duyck may have the most clarity on this problem, and possible solutions so I made sure to CC him.