From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1423249AbcFMTpi (ORCPT ); Mon, 13 Jun 2016 15:45:38 -0400 Received: from mx2.suse.de ([195.135.220.15]:33175 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753527AbcFMTpd (ORCPT ); Mon, 13 Jun 2016 15:45:33 -0400 Date: Mon, 13 Jun 2016 12:45:23 -0700 From: Davidlohr Bueso To: Pan Xinhui Cc: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, arnd@arndb.de, peterz@infradead.org, waiman.long@hp.com Subject: Re: [PATCH] locking/qspinlock: Use atomic_sub_return_release in queued_spin_unlock Message-ID: <20160613194523.GA2094@linux-80c1.suse> References: <1464943094-3129-1-git-send-email-xinhui.pan@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: <1464943094-3129-1-git-send-email-xinhui.pan@linux.vnet.ibm.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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) >Signed-off-by: Pan Xinhui 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().