On Fri, 03 Jun 2016, Pan Xinhui wrote:
The existing version uses a heavy barrier while only release semantics is required. So use atomic_sub_return_release instead. Suggested-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx> Signed-off-by: Pan Xinhui <xinhui.pan@xxxxxxxxxxxxxxxxxx>
I just noticed this change in -tip and, while I know that saving a barrier in core spinlock paths is perhaps a worthy exception, I cannot help but wonder if this is the begging of the end for smp__{before,after}_atomic(). -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html