From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 08672C32771 for ; Mon, 26 Sep 2022 10:15:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=vqUBTDEpKr0vw/1D8BnSeJj87E7xJO7tFCWbcuYP59w=; b=O97WW81Vx05MMZ ZY7G/il/uw8p1aEGvUwg8PmvZBPEZs+vQDtPgqfqDbO76DM36evgO0ISLveCh4syFVF38AoOCk/Xd ktaVspw7JW/G1RKEWvnXFNT8ChrZ6m8+Hkxw3Ss6YgnGtpkGyiFeVoeSn8TLcLBVAXoPCJtpC/9EL xZH3qRaF43YkzCdnMZwsPF8ZaTGZoU2NDpuFMGejTnzNqOcjoAyCmwCC+ClDXetW7+ZtK4PZAHEQN GkvTiTL3vzzuS8+TlBPPOjjPnuN7JWFd7SsB7AouXJjim0q2tVisAgdmYHITnecs3iPinGMtBCzuW Zd+qM1tBgtBUeV4VWSLg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ocl7k-003zxT-6r; Mon, 26 Sep 2022 10:14:04 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:32c8:5054:ff:fe00:142]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ocl7f-003zr7-Bh for linux-arm-kernel@lists.infradead.org; Mon, 26 Sep 2022 10:14:01 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=2tkAUDQhqX2MUEa67IDFMo3NC6iPlLEfNVovQdjsDkQ=; b=Zp3G3RMI9vngezfV6YpVxFwi9y CHe1zfHC57xMs875E2OOSaAqL7MX3h0KraU11DTe2GCUK2a/xpjesT3frvle23GcxhIN2qeqFoVHF fcxOgXquwG0AxGHq7r8+oaOiTPQgvSCp8ENGae8gHZIHI6y2lhTXIBF2BmaUPALN6rV0+6qrRK/6u lZhGjN37MVsLKS+/GpoP+dN9iBFXTtB4T9yIgh68fcYs3Cx4JyYNeHIS1cTZmBTSee6mdYWWN4oj8 yE0ACi3T534ONw3rt28QGyqrcQaT1VGXSDq3NAUA6oZaAlciV6SH/9pk7Iee03cHmJz3apeGP0Ny7 NL6IN8UA==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:34494) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1ocl7Y-0000LO-2n; Mon, 26 Sep 2022 11:13:52 +0100 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1ocl7X-00054A-GG; Mon, 26 Sep 2022 11:13:51 +0100 Date: Mon, 26 Sep 2022 11:13:51 +0100 From: "Russell King (Oracle)" To: Kefeng Wang Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] ARM: mm: Provide better fault message for permission fault Message-ID: References: <20220919103845.100809-1-wangkefeng.wang@huawei.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220919103845.100809-1-wangkefeng.wang@huawei.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220926_031359_426468_8379A7B9 X-CRM114-Status: GOOD ( 11.99 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Sep 19, 2022 at 06:38:45PM +0800, Kefeng Wang wrote: > If there is a permission fault in __do_kernel_fault(), we only > print the generic "paging request" message which don't show > read, write or excute information, let's provide better fault > message for them. I don't like this change. With CPUs that do not have the ability to relocate the vectors to 0xffff0000, the vectors live at address 0, so NULL pointer dereferences can produce permission faults. I would much rather we did something similar to what x86 does: pr_alert("#PF: %s %s in %s mode\n", (error_code & X86_PF_USER) ? "user" : "supervisor", (error_code & X86_PF_INSTR) ? "instruction fetch" : (error_code & X86_PF_WRITE) ? "write access" : "read access", user_mode(regs) ? "user" : "kernel"); As we already print whether we're in user or kernel mode in the register dump, there's no need to repeat that. I think we just need an extra line to decode the FSR PF and write bits. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last! _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel