Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp3257745pxa; Tue, 18 Aug 2020 10:24:08 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyeAThaLoX4biIb1EMqxmk8pmx9oVbxdMgz2IiUqykI5pMIne9z1L+FPXnEzzqNib0xpvrM X-Received: by 2002:a05:6402:a59:: with SMTP id bt25mr21299942edb.71.1597771448472; Tue, 18 Aug 2020 10:24:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1597771448; cv=none; d=google.com; s=arc-20160816; b=JD9Kehw50YTP6dsW4YzzH9smqEcrIkbn32n96mhOlvU6DKertf3cVTWHush81B6aqL THkQ5p+V0ze6E5QMG8YSu9F6WoSS3kSKNx/l/5wW8PQgqUPtSqeWwB6Fpzmdj7wdT5RS Qg/NOiQnmfi7wPZve71P3qteYlBn8El5hwE5n8e6bZQtrz73hs2y4m+HwEalcr7mL3sJ 0iza9pVmOtWbDDsW15n06o6yCeZ3e13kEzhlXY1tumYkxb0tvVDfGCO1n7kXm0i4aNG7 b0oIPqkFNnzWAVitj8qgfNy+QjBTWh9F90sVGhq2x7OkxS07yGg4Vi7XC75TZIITB+nI oL5A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:date:cc:to:subject:from:references :in-reply-to:message-id; bh=i2bRNt0asuV13ksv9Papyn7VdKXZKmEsAVNSjhcIeL4=; b=pjgA22nfFbD9NTSIikVpmzZDjHtMvKtA2d393GLAEpmYuR6qzxXX/QFOCFOECA2QEz 2VJXL4zQiDNLwv87YRlU9c0XyThy0ahnk8I7RCw5oPP1xrkjzDbEPlaUnaKCxC+IWaE5 zSj0Rq11BMCoW4sljMH/7MJgTBu/7Fnbq9aN+BugAiD1+JOmn35IgbVnzbFDgwpZUOtI Y1CK1T4wpOaW1ChUVKNCo13qT1R1B+yBe40xCy9v3J/F/pqBfLjnpcGQfWKyxgmAkgs9 UkXMSvhA270e4N0lPBnl67HTtwB51Fe91QRcGb5vNM3lqYVEy9OeQL0QLlW6tTzN4liM 75pA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-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 a22si13566016eju.634.2020.08.18.10.23.44; Tue, 18 Aug 2020 10:24:08 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728811AbgHRRVK (ORCPT + 99 others); Tue, 18 Aug 2020 13:21:10 -0400 Received: from pegase1.c-s.fr ([93.17.236.30]:51116 "EHLO pegase1.c-s.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728506AbgHRRTV (ORCPT ); Tue, 18 Aug 2020 13:19:21 -0400 Received: from localhost (mailhub1-int [192.168.12.234]) by localhost (Postfix) with ESMTP id 4BWHhP6zbBz9vCxx; Tue, 18 Aug 2020 19:19:17 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at c-s.fr Received: from pegase1.c-s.fr ([192.168.12.234]) by localhost (pegase1.c-s.fr [192.168.12.234]) (amavisd-new, port 10024) with ESMTP id 1jnu3FO3pKDj; Tue, 18 Aug 2020 19:19:17 +0200 (CEST) Received: from messagerie.si.c-s.fr (messagerie.si.c-s.fr [192.168.25.192]) by pegase1.c-s.fr (Postfix) with ESMTP id 4BWHhP6Dnhz9vCxg; Tue, 18 Aug 2020 19:19:17 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by messagerie.si.c-s.fr (Postfix) with ESMTP id B49A88B7EE; Tue, 18 Aug 2020 19:19:19 +0200 (CEST) X-Virus-Scanned: amavisd-new at c-s.fr Received: from messagerie.si.c-s.fr ([127.0.0.1]) by localhost (messagerie.si.c-s.fr [127.0.0.1]) (amavisd-new, port 10023) with ESMTP id xE4_nO5j3Pco; Tue, 18 Aug 2020 19:19:19 +0200 (CEST) Received: from po17688vm.idsi0.si.c-s.fr (unknown [192.168.4.90]) by messagerie.si.c-s.fr (Postfix) with ESMTP id 6FC738B7EC; Tue, 18 Aug 2020 19:19:19 +0200 (CEST) Received: by po17688vm.idsi0.si.c-s.fr (Postfix, from userid 0) id 51A9E65CF4; Tue, 18 Aug 2020 17:19:19 +0000 (UTC) Message-Id: <3582e10a341d523c9c3f1ac925c3aaefc9d9293d.1597770847.git.christophe.leroy@csgroup.eu> In-Reply-To: References: From: Christophe Leroy Subject: [PATCH v2 07/25] powerpc/signal: Remove BUG_ON() in handler_signal functions To: Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman Cc: linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Date: Tue, 18 Aug 2020 17:19:19 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org There is already the same BUG_ON() check in do_signal() which is the only caller of handle_rt_signal64() handle_rt_signal32() and handle_signal32(). Remove those three redundant BUG_ON(). Signed-off-by: Christophe Leroy --- arch/powerpc/kernel/signal_32.c | 4 ---- arch/powerpc/kernel/signal_64.c | 2 -- 2 files changed, 6 deletions(-) diff --git a/arch/powerpc/kernel/signal_32.c b/arch/powerpc/kernel/signal_32.c index 7b291707eb31..8cbc9ac1343d 100644 --- a/arch/powerpc/kernel/signal_32.c +++ b/arch/powerpc/kernel/signal_32.c @@ -764,8 +764,6 @@ int handle_rt_signal32(struct ksignal *ksig, sigset_t *oldset, unsigned long msr = regs->msr; #endif - BUG_ON(tsk != current); - /* Set up Signal Frame */ /* Put a Real Time Context onto stack */ rt_sf = get_sigframe(ksig, get_tm_stackpointer(tsk), sizeof(*rt_sf), 1); @@ -1227,8 +1225,6 @@ int handle_signal32(struct ksignal *ksig, sigset_t *oldset, unsigned long msr = regs->msr; #endif - BUG_ON(tsk != current); - /* Set up Signal Frame */ frame = get_sigframe(ksig, get_tm_stackpointer(tsk), sizeof(*frame), 1); if (unlikely(frame == NULL)) diff --git a/arch/powerpc/kernel/signal_64.c b/arch/powerpc/kernel/signal_64.c index bfc939360bad..cae612bdde5f 100644 --- a/arch/powerpc/kernel/signal_64.c +++ b/arch/powerpc/kernel/signal_64.c @@ -822,8 +822,6 @@ int handle_rt_signal64(struct ksignal *ksig, sigset_t *set, unsigned long msr = regs->msr; #endif - BUG_ON(tsk != current); - frame = get_sigframe(ksig, get_tm_stackpointer(tsk), sizeof(*frame), 0); if (unlikely(frame == NULL)) goto badframe; -- 2.25.0