From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0D55533F384 for ; Tue, 17 Mar 2026 09:09:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773738581; cv=none; b=GHLWlWnFq7vbNtk4YZpG6wJymxhsDVHchpAnUpwkJoESdwA6ZdBKnSlGHg0BFXh/3kc1+EsphVD0I/mbDcKLLyd9ZVQvYIWgtP5pgFs/qwcs5IWpQRLyaxKFJUKrcv2GnoYVIYjSUTDEj9F6gpOL0ksvHGtoOT8W7c/K9Ny+YCU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773738581; c=relaxed/simple; bh=Ak+fkoI4kuQLOTzVMHsdfWZxTKkFz4NFB8J6dcF3eS4=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=FpitMLYITGjVhlareo8Lp64+OHnctCRqgjL0tJcndRhu9aaXiGNQ9KRdXAWQDYIlbM6ZXzt6OPh35GIIqFKTL6DcM1FRyNu2VKeZrxjtyF40M4Oru1f9laosJVu6ZuFglsxfwGupt5WqQBU6ZuY9SMawAM96/GI8+Ji4DGIQ8lw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=i7SW/bR0; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="i7SW/bR0" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 70463C4CEF7; Tue, 17 Mar 2026 09:09:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1773738580; bh=Ak+fkoI4kuQLOTzVMHsdfWZxTKkFz4NFB8J6dcF3eS4=; h=From:To:Cc:Subject:In-Reply-To:References:Date:From; b=i7SW/bR05M3EjYJgdOTRtR0PzK6x+w20Ft0042LLJ6uprnCdWX0HmDexl5MVUNu7l h+REqv0LK2fhaBMyoK7B3MAdNKQVF6DY8TZeUZUB7P8KyRHzWaEzvjE+ZdUNyJJeD1 RpGWZKX8DYZeoxLww7PNXtUy9YpT/CyjTZXXotLkjTqOOteuaAi8sKTk8jQyiTjCfl zx8ubORE5LoN6oLyWw6L0pPxyICvVFV+8ukD7WY+p0cwePLLHBmC8i/hW4k+CxYyed bs6/N+hEyhAYeTKpgjb8mk436rwyZpqIVa0T45Ne/jUVswC3aZdh3/SUoohUweuGLW +navEj6W7tgbQ== From: Pratyush Yadav To: Andrew Morton Cc: Pratyush Yadav , ranxiaokai627@163.com, graf@amazon.com, rppt@kernel.org, pasha.tatashin@soleen.com, jasonmiu@google.com, linux-kernel@vger.kernel.org, kexec@lists.infradead.org, ran.xiaokai@zte.com.cn, Breno Leitao Subject: Re: [PATCH] kho: fix child node parsing for debugfs in/sub_fdts In-Reply-To: <20260314154637.86a7d4f07800b59c00049814@linux-foundation.org> (Andrew Morton's message of "Sat, 14 Mar 2026 15:46:37 -0700") References: <20260309033530.244508-1-ranxiaokai627@163.com> <2vxzpl58f3oh.fsf@kernel.org> <20260314154637.86a7d4f07800b59c00049814@linux-foundation.org> Date: Tue, 17 Mar 2026 09:09:36 +0000 Message-ID: <2vxzse9yddpr.fsf@kernel.org> User-Agent: Gnus/5.13 (Gnus v5.13) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain On Sat, Mar 14 2026, Andrew Morton wrote: > On Fri, 13 Mar 2026 10:02:06 +0000 Pratyush Yadav wrote: > >> On Mon, Mar 09 2026, ranxiaokai627@163.com wrote: >> >> > From: Ran Xiaokai >> > >> > Commit e0c1731f5d57 ("kho: adopt radix tree for preserved memory tracking") >> > changed KHO_FDT_SUB_TREE_PROP_NAME from "fdt" to "preserved-data". However, >> > kho debugfs code still hard-coded the "fdt" string when parsing the >> > device tree, causing the debugfs node /debugfs/kho/in/sub_fdts/ >> > fail to parse child nodes correctly. >> > >> > Fix this by including the header file and using KHO_FDT_SUB_TREE_PROP_NAME >> > instead of the hard-coded string. >> > >> > Fixes: e0c1731f5d57 ("kho: adopt radix tree for preserved memory tracking") >> > Signed-off-by: Ran Xiaokai >> >> Breno's patch [0] also fixes this, but I'd rather have this go through >> hotfixes and get backported to stable, and Breno's series to build on >> top of this. >> >> [0] https://lore.kernel.org/linux-mm/20260309-kho-v8-4-c3abcf4ac750@debian.org/ > > Confused. This patch (kho: fix child node parsing for debugfs > in/sub_fdts) fixes the mm-unstable patch "kho: adopt radix tree for > preserved memory tracking". > > So what are you suggesting be added to -stable? > > (Please lay it all out very clearly, idenfifying patches by name!) Sorry, my bad for causing the confusion. I thought the radix tree patches ("kho: adopt radix tree for preserved memory tracking") already landed in mainline, but I went and checked and it seems that they haven't. They are still in mm-unstable. So we should just meld this patch ("kho: fix child node parsing for debugfs in/sub_fdts") into ("kho: adopt radix tree for preserved memory tracking"). There is nothing that needs to be added to -stable, and there is nothing that needs to go to hotfixes. Breno's patch ("kho: fix kho_in_debugfs_init() to handle non-FDT blobs") also fixes this bug, but then also adds some other changes with it. So Breno's patch will cause conflicts once applied on top of this patch, but they should be simple enough to resolve I think. -- Regards, Pratyush Yadav