From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754842AbcFUAlH (ORCPT ); Mon, 20 Jun 2016 20:41:07 -0400 Received: from ozlabs.org ([103.22.144.67]:40878 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753823AbcFUAk5 (ORCPT ); Mon, 20 Jun 2016 20:40:57 -0400 X-powerpc-patch-notification: thanks X-powerpc-patch-commit: 6262db7c088bbfc26480d10144cde70bbf077be3 In-Reply-To: <20160610035128.21087-1-boqun.feng@gmail.com> To: Boqun Feng , linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org From: Michael Ellerman Cc: Peter Zijlstra , Boqun Feng , Will Deacon , Alexander Graf , Paul Mackerras , "Suresh E. Warrier" , "Paul E. McKenney" Subject: Re: [v4] powerpc: spinlock: Fix spin_unlock_wait() Message-Id: <3rYTRV50CZz9t0w@ozlabs.org> Date: Tue, 21 Jun 2016 10:40:54 +1000 (AEST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 2016-10-06 at 03:51:28 UTC, Boqun Feng wrote: > There is an ordering issue with spin_unlock_wait() on powerpc, because > the spin_lock primitive is an ACQUIRE and an ACQUIRE is only ordering > the load part of the operation with memory operations following it. ... > > Suggested-by: "Paul E. McKenney" > Signed-off-by: Boqun Feng > Reviewed-by: "Paul E. McKenney" > [mpe: Inline the "nop" ll/sc loop and set EH=0, munge change log] > Signed-off-by: Michael Ellerman Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/6262db7c088bbfc26480d10144 cheers