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 7C3E0C3DA7F for ; Mon, 5 Aug 2024 02:57:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id: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-Owner; bh=yrwIVSFERGwwt6j0moOGiSqtU4uf4f8TDmlxeI81KFs=; b=DJAuKVeBLOXxoF18ApHQ5kxGYo Zf0KCELcp5YuBQ5dRFkD6VdJBsoNzvJX9bmFgRlTerLx/uHJnO6so6QpJEcMCthExIjHbjA4y0THg ub0iaWv7i6c00pC/OBPh89PuZ5C8Ywkk97GEY8hFONiY4ub4BcfBg/DmLm9YMgOrl4yRP78PbEe/A kc7KLwdQwISflqG7HKan0rdQO0Izpmtrj5F06HRkZgQxoD7yrozNyicstAs5jOGww05lZJPc6X9WK PVGn9qqTeTJn/FPRbICvHiflWhE4Yks9nRbKDpMo3j0rZfWZqsJjj8mchXWI4CdvyoDpBfIWxEuCl y5+F7KGQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1santw-0000000EQWF-2gJQ; Mon, 05 Aug 2024 02:56:48 +0000 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1santQ-0000000EQO4-157K for linux-arm-kernel@lists.infradead.org; Mon, 05 Aug 2024 02:56:17 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1722826573; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=yrwIVSFERGwwt6j0moOGiSqtU4uf4f8TDmlxeI81KFs=; b=etlK/zfgnagDJsrO1ajibrA9ugvfY5axQKln+cBx2LvInAgPt3yPzqVu1RqeDP1wrsjNG/ 7n6Jpjhr4s9qEHsdozTxE5Md+XijROmHPSyLL/6BF5AhuOERPlkyne+eaSTwUXTvWcVJfy xHpVZl1mMi2bbnys4zOU853QMIrbD5w= Received: from mx-prod-mc-04.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-9-R5UQGAO3PwCsK9yP1GNeqg-1; Sun, 04 Aug 2024 22:56:10 -0400 X-MC-Unique: R5UQGAO3PwCsK9yP1GNeqg-1 Received: from mx-prod-int-04.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-04.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.40]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-04.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 7F6CA1955D42; Mon, 5 Aug 2024 02:56:08 +0000 (UTC) Received: from localhost (unknown [10.72.112.44]) by mx-prod-int-04.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 7E74019560AA; Mon, 5 Aug 2024 02:56:06 +0000 (UTC) Date: Mon, 5 Aug 2024 10:56:01 +0800 From: Baoquan He To: Jinjie Ruan Cc: "Russell King (Oracle)" , akpm@linux-foundation.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH] ARM: Support allocating crashkernel above 4G for LPAE Message-ID: References: <20240802092510.3915986-1-ruanjinjie@huawei.com> <6c0a45c3-08a2-f889-9e66-ab8aff66ae8c@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6c0a45c3-08a2-f889-9e66-ab8aff66ae8c@huawei.com> X-Scanned-By: MIMEDefang 3.0 on 10.30.177.40 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240804_195616_397430_7EEAE56A X-CRM114-Status: GOOD ( 23.01 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 08/05/24 at 09:23am, Jinjie Ruan wrote: > > > On 2024/8/2 19:01, Russell King (Oracle) wrote: > > On Fri, Aug 02, 2024 at 05:25:10PM +0800, Jinjie Ruan wrote: > >> As ARM LPAE feature support accessing memory beyond the 4G limit, define > >> HAVE_ARCH_CRASHKERNEL_RESERVATION_HIGH macro to support reserving crash > >> memory above 4G for ARM32 LPAE. > >> > >> No test because there is no LPAE ARM32 hardware. > > > > Why are you submitting patches for features you can't test? > > > > I'm not going to apply this without it being properly tested, because I > > don't believe that this will work in the generic case. > > > > If the crash kernel is located in memory outside of the lower 4GiB of > > address space, and there is no alias within physical address space > > for that memory, then there is *no* *way* for such a kernel to boot. > > I'm sorry that I released this patch without testing it. I actually > intended to bring up this issue for discussion. If anyone has the > environment to test it, that would be great. In the meantime, we could > have a discussion on the significance and relevance of this approach. I don't know arm32 and its LPAE. I know a little about x86_32 where crashkernel can only be reserved below 896M because of the virtual memory layout, and all memory above that is high memory which can't be used as kernel memory directly. So from this patch, arm32 is different than x86_32. > > > > > So, right now I believe this patch to be *fundamentally* wrong. > > >