From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752671AbdK0NFm (ORCPT ); Mon, 27 Nov 2017 08:05:42 -0500 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:42736 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752246AbdK0NFg (ORCPT ); Mon, 27 Nov 2017 08:05:36 -0500 Date: Mon, 27 Nov 2017 14:05:25 +0100 (CET) From: Sebastian Ott X-X-Sender: sebott@schleppi To: Will Deacon cc: Peter Zijlstra , Ingo Molnar , Martin Schwidefsky , Heiko Carstens , linux-kernel@vger.kernel.org Subject: Re: [bisected] system hang after boot In-Reply-To: <20171127124506.GB30679@arm.com> References: <20171122182659.GA22648@arm.com> <20171122202217.GO3326@worktop> <20171127114947.GA30679@arm.com> <20171127124506.GB30679@arm.com> User-Agent: Alpine 2.21 (LFD 202 2017-01-01) Organization: =?ISO-8859-15?Q?=22IBM_Deutschland_Research_&_Development_GmbH?= =?ISO-8859-15?Q?_=2F_Vorsitzende_des_Aufsichtsrats=3A_Martina?= =?ISO-8859-15?Q?_Koederitz_Gesch=E4ftsf=FChrung=3A_Dirk_Wittkopp?= =?ISO-8859-15?Q?_Sitz_der_Gesellschaft=3A_B=F6blingen_=2F_Reg?= =?ISO-8859-15?Q?istergericht=3A_Amtsgericht_Stuttgart=2C_HRB_2432?= =?ISO-8859-15?Q?94=22?= MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-TM-AS-GCONF: 00 x-cbid: 17112713-0040-0000-0000-000003F2F790 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 17112713-0041-0000-0000-000025F5D151 Message-Id: X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-11-27_06:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=3 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 lowpriorityscore=0 impostorscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1709140000 definitions=main-1711270182 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 27 Nov 2017, Will Deacon wrote: > Sebastian: could you try the diff below, please? If that fixes s390, then > we can debate the merits of GENERIC_LOCKBREAK independently of fixing this > issue. > > Thanks, > > Will > > --->8 > > diff --git a/kernel/locking/spinlock.c b/kernel/locking/spinlock.c > index 1fd1a7543cdd..0ebb253e2199 100644 > --- a/kernel/locking/spinlock.c > +++ b/kernel/locking/spinlock.c > @@ -68,8 +68,8 @@ void __lockfunc __raw_##op##_lock(locktype##_t *lock) \ > \ > if (!(lock)->break_lock) \ > (lock)->break_lock = 1; \ > - while ((lock)->break_lock) \ > - arch_##op##_relax(&lock->raw_lock); \ > + \ > + arch_##op##_relax(&lock->raw_lock); \ > } \ > (lock)->break_lock = 0; \ > } \ > @@ -88,8 +88,8 @@ unsigned long __lockfunc __raw_##op##_lock_irqsave(locktype##_t *lock) \ > \ > if (!(lock)->break_lock) \ > (lock)->break_lock = 1; \ > - while ((lock)->break_lock) \ > - arch_##op##_relax(&lock->raw_lock); \ > + \ > + arch_##op##_relax(&lock->raw_lock); \ > } \ > (lock)->break_lock = 0; \ > return flags; \ I tested that on top of 4.15.0-rc1 and it worked! Sebastian