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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B7879C433F5 for ; Mon, 13 Dec 2021 10:20:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242114AbhLMKUS (ORCPT ); Mon, 13 Dec 2021 05:20:18 -0500 Received: from ams.source.kernel.org ([145.40.68.75]:33204 "EHLO ams.source.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242169AbhLMKSS (ORCPT ); Mon, 13 Dec 2021 05:18:18 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 48C34B80FAF; Mon, 13 Dec 2021 10:18:15 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0A241C34600; Mon, 13 Dec 2021 10:18:09 +0000 (UTC) Date: Mon, 13 Dec 2021 10:18:06 +0000 From: Catalin Marinas To: john.p.donnelly@oracle.com Cc: Zhen Lei , Thomas Gleixner , Ingo Molnar , Borislav Petkov , x86@kernel.org, "H . Peter Anvin" , linux-kernel@vger.kernel.org, Dave Young , Baoquan He , Vivek Goyal , Eric Biederman , kexec@lists.infradead.org, Will Deacon , linux-arm-kernel@lists.infradead.org, Rob Herring , Frank Rowand , devicetree@vger.kernel.org, Jonathan Corbet , linux-doc@vger.kernel.org, Randy Dunlap , Feng Zhou , Kefeng Wang , Chen Zhou Subject: Re: [PATCH v16 00/11] support reserving crashkernel above 4G on arm64 kdump Message-ID: References: <20211123124646.1995-1-thunder.leizhen@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Fri, Dec 10, 2021 at 01:53:59PM -0600, john.p.donnelly@oracle.com wrote: > On 12/8/21 11:13 AM, Catalin Marinas wrote: > > On Tue, Nov 23, 2021 at 08:46:35PM +0800, Zhen Lei wrote: > > > Chen Zhou (10): > > > x86: kdump: replace the hard-coded alignment with macro CRASH_ALIGN > > > x86: kdump: make the lower bound of crash kernel reservation > > > consistent > > > x86: kdump: use macro CRASH_ADDR_LOW_MAX in functions > > > reserve_crashkernel() > > > x86: kdump: move xen_pv_domain() check and insert_resource() to > > > setup_arch() > > > x86: kdump: move reserve_crashkernel[_low]() into crash_core.c > > > arm64: kdump: introduce some macros for crash kernel reservation > > > arm64: kdump: reimplement crashkernel=X > > > x86, arm64: Add ARCH_WANT_RESERVE_CRASH_KERNEL config > > > of: fdt: Add memory for devices by DT property > > > "linux,usable-memory-range" > > > kdump: update Documentation about crashkernel > > > > > > Zhen Lei (1): > > > of: fdt: Aggregate the processing of "linux,usable-memory-range" > > > > Apart from a minor comment I made on patch 8 and some comments from Rob > > that need addressing, the rest looks fine to me. > > > > Ingo stated in the past that he's happy to ack the x86 changes as long > > as there's no functional change (and that's the case AFAICT). Ingo, does > > your conditional ack still stand? > > > > In terms of merging, I'm happy to take it all through the arm64 tree > > with acks from the x86 maintainers. Alternatively, with the change I > > mentioned for patch 8, the first 5 patches could be queued via the tip > > tree on a stable branch and I can base the rest of the arm64 on top. > > > > Thomas, Ingo, Peter, any preference? > > > > Thanks. > > If you notice the trend over the past year , some of additional review > requests are because the submitter had to rebase to the next version. Normally the reviews stand even if the series is rebased. You only drop them if there were non-trivial changes to the patches, not just because of a rebase. Ingo acked the x86 changes in the past but there were some functional changes to the minimum alignment, so the series had to be reworked and the ack not applied. https://lore.kernel.org/r/20200806133627.GB2077191@gmail.com > Can we get this acked and placed in a build so others can test and start > using it ? If Ingo's conditional ack (no functional changes, just a move) still stands over the five x86 patches now, I'm happy to queue them all via the arm64 tree. The latest series here: https://lore.kernel.org/r/20211210065533.2023-1-thunder.leizhen@huawei.com -- Catalin