From patchwork Tue Jun 23 19:58:29 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Greg Kroah-Hartman X-Patchwork-Id: 191569 Delivered-To: patch@linaro.org Received: by 2002:a54:3249:0:0:0:0:0 with SMTP id g9csp1343216ecs; Tue, 23 Jun 2020 13:45:39 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxzpgLr5ZF9mR+/Jq0Gv1bfgQ/7AOkDEO84vlrJgjdWxuyhopquhE4mKafVLpyCIh+HONX3 X-Received: by 2002:aa7:cd4b:: with SMTP id v11mr24595440edw.356.1592945139914; Tue, 23 Jun 2020 13:45:39 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1592945139; cv=none; d=google.com; s=arc-20160816; b=0i201AxMdf7Yj2Houvrf8NEN44vPz5QM1WItwAAS/hmgVWlvVCHdsYjlPh5D1UWjlm 8jtrNsBGmRX89aLiN8RAN1Z+NNnBhfnJ7wo4D2fbk2JrdEqtKKWEKpKbGLR5RdNsZA2i kqV9EMYZrPZmDUy6KkV/r59RSlLVAw82t4EG29iVvE/IaNJ38EXTMtof2C0zkn/0uT+o TWdxO+wQc9G9rRIFFE9U9DDPZA1TaEIbNBT1tv/5MW/TNMW19zEUFPU+6vOqS6lGclRE Z7phAge4cBz7lk2qDz5pTvxu8Y3Gciv0DvEUU3D2QDiBIJFyHfpcu0gZZ9Xmks2sUzdy Icww== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=9hnzVE6GuKVYpftQ20I/sHVzYWR+qtdpnOODqp+CYlk=; b=guuVcd1gRSDhnCh0Dtn0+zf8MuwImu4+RRjiS3Wlewvt1brysJKBHrW02eBXM/QCvW 4tGZjXvfoYFCtODDwnPIwBHrMsCfK3FNtWCVhRQEugX88UYjsTKjPppz9UgrDsUmUSGr FJSHlpryY3lOOHoYRjA1lvicHto2550RULd7HEbVcMqvVxQyd+4ZzDEZXK8cq3QG4z+b g7j1OVa+p2PV/aEeIwnsVxynCuMZBbkoLg1MQnLo8695cmG/u+6vMfDcdeaNhNlLPkDb Gjvo8VySGyHaER3MR13EpwW17EN29yG0QoFO+oQXTzNbMFLQRb78NNE//Mnh2KZEgvWk +LRQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=vAk4yWbu; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id f13si3130014eje.212.2020.06.23.13.45.39; Tue, 23 Jun 2020 13:45:39 -0700 (PDT) Received-SPF: pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=vAk4yWbu; spf=pass (google.com: domain of stable-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=stable-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2392747AbgFWUph (ORCPT + 15 others); Tue, 23 Jun 2020 16:45:37 -0400 Received: from mail.kernel.org ([198.145.29.99]:43150 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2390937AbgFWUpg (ORCPT ); Tue, 23 Jun 2020 16:45:36 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 2B21520836; Tue, 23 Jun 2020 20:45:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1592945136; bh=DRFsV92GDkJasMMkP1EiCsfu8ldMUDEVDgx+9jRJSq0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=vAk4yWbubVfYRMObBPE7pvSJHHSmh2+hfvLdX4jYWmLfsSd9bY7S4+ZuBN5RP1rqK gEfjF3R5YHlMMjMJ9gFf4UrCtGZcNF5yGCAZZkc9gHVeQa3eqmUmhFoV1hUOQmUu5t 5l15eaA8bfJGLYP2dvkJbfh+Z8lXDtZ6fccjOVr4= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Josh Poimboeuf , clang-built-linux@googlegroups.com, Arnd Bergmann , David Teigland , Sasha Levin Subject: [PATCH 4.14 053/136] dlm: remove BUG() before panic() Date: Tue, 23 Jun 2020 21:58:29 +0200 Message-Id: <20200623195306.347715455@linuxfoundation.org> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20200623195303.601828702@linuxfoundation.org> References: <20200623195303.601828702@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Sender: stable-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org From: Arnd Bergmann [ Upstream commit fe204591cc9480347af7d2d6029b24a62e449486 ] Building a kernel with clang sometimes fails with an objtool error in dlm: fs/dlm/lock.o: warning: objtool: revert_lock_pc()+0xbd: can't find jump dest instruction at .text+0xd7fc The problem is that BUG() never returns and the compiler knows that anything after it is unreachable, however the panic still emits some code that does not get fully eliminated. Having both BUG() and panic() is really pointless as the BUG() kills the current process and the subsequent panic() never hits. In most cases, we probably don't really want either and should replace the DLM_ASSERT() statements with WARN_ON(), as has been done for some of them. Remove the BUG() here so the user at least sees the panic message and we can reliably build randconfig kernels. Fixes: e7fd41792fc0 ("[DLM] The core of the DLM for GFS2/CLVM") Cc: Josh Poimboeuf Cc: clang-built-linux@googlegroups.com Signed-off-by: Arnd Bergmann Signed-off-by: David Teigland Signed-off-by: Sasha Levin --- fs/dlm/dlm_internal.h | 1 - 1 file changed, 1 deletion(-) -- 2.25.1 diff --git a/fs/dlm/dlm_internal.h b/fs/dlm/dlm_internal.h index 748e8d59e6111..cb287df13a7aa 100644 --- a/fs/dlm/dlm_internal.h +++ b/fs/dlm/dlm_internal.h @@ -99,7 +99,6 @@ do { \ __LINE__, __FILE__, #x, jiffies); \ {do} \ printk("\n"); \ - BUG(); \ panic("DLM: Record message above and reboot.\n"); \ } \ }