From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932604AbcITPAN (ORCPT ); Tue, 20 Sep 2016 11:00:13 -0400 Received: from terminus.zytor.com ([198.137.202.10]:57726 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754668AbcITPAL (ORCPT ); Tue, 20 Sep 2016 11:00:11 -0400 Date: Tue, 20 Sep 2016 07:58:39 -0700 From: tip-bot for Jan Beulich Message-ID: Cc: torvalds@linux-foundation.org, hpa@zytor.com, paulmck@linux.vnet.ibm.com, tglx@linutronix.de, peterz@infradead.org, brgerst@gmail.com, akpm@linux-foundation.org, linux-kernel@vger.kernel.org, bp@alien8.de, mingo@kernel.org, JBeulich@suse.com, luto@kernel.org, jbeulich@suse.com, dvlasenk@redhat.com, jpoimboe@redhat.com Reply-To: peterz@infradead.org, tglx@linutronix.de, paulmck@linux.vnet.ibm.com, hpa@zytor.com, torvalds@linux-foundation.org, bp@alien8.de, linux-kernel@vger.kernel.org, brgerst@gmail.com, akpm@linux-foundation.org, luto@kernel.org, jbeulich@suse.com, JBeulich@suse.com, mingo@kernel.org, dvlasenk@redhat.com, jpoimboe@redhat.com In-Reply-To: <57E003CC0200007800110102@prv-mh.provo.novell.com> References: <57E003CC0200007800110102@prv-mh.provo.novell.com> To: linux-tip-commits@vger.kernel.org Subject: [tip:locking/core] locking/rwsem, x86: Drop a bogus cc clobber Git-Commit-ID: c907420fdaec78b17f59a6011cb5f9d6051c6a35 X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: c907420fdaec78b17f59a6011cb5f9d6051c6a35 Gitweb: http://git.kernel.org/tip/c907420fdaec78b17f59a6011cb5f9d6051c6a35 Author: Jan Beulich AuthorDate: Mon, 19 Sep 2016 07:27:08 -0600 Committer: Ingo Molnar CommitDate: Tue, 20 Sep 2016 08:26:41 +0200 locking/rwsem, x86: Drop a bogus cc clobber With the addition of uses of GCC's condition code outputs in commit: 35ccfb7114 ("x86, asm: Use CC_SET()/CC_OUT() in ") ... there's now an overlap of outputs and clobbers in __down_write_trylock(). Such overlaps are generally getting tagged with an error (occasionally even with an ICE). I can't really tell why plain GCC 6.2 doesn't detect this (judging by the code it is meant to), while the slightly modified one I use does. Since condition code clobbers are never necessary on x86 (other than perhaps for documentation purposes, which doesn't really get done consistently), remove it altogether rather than inventing something like CC_CLOBBER (to accompany CC_SET/CC_OUT). Signed-off-by: Jan Beulich Cc: Andrew Morton Cc: Andy Lutomirski Cc: Borislav Petkov Cc: Brian Gerst Cc: Denys Vlasenko Cc: H. Peter Anvin Cc: Josh Poimboeuf Cc: Linus Torvalds Cc: Paul E. McKenney Cc: Peter Zijlstra Cc: Thomas Gleixner Link: http://lkml.kernel.org/r/57E003CC0200007800110102@prv-mh.provo.novell.com Signed-off-by: Ingo Molnar --- arch/x86/include/asm/rwsem.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/include/asm/rwsem.h b/arch/x86/include/asm/rwsem.h index 8dbc762..3d33a71 100644 --- a/arch/x86/include/asm/rwsem.h +++ b/arch/x86/include/asm/rwsem.h @@ -154,7 +154,7 @@ static inline bool __down_write_trylock(struct rw_semaphore *sem) : "+m" (sem->count), "=&a" (tmp0), "=&r" (tmp1), CC_OUT(e) (result) : "er" (RWSEM_ACTIVE_WRITE_BIAS) - : "memory", "cc"); + : "memory"); return result; }