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 CCDCDC46467 for ; Tue, 10 Jan 2023 14:47:56 +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:Mime-Version:References:In-Reply-To: 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=Mz0+rUfy6nGFjK9+IVw+rNPsJtLc9/fEVfKYQRWTzXw=; b=KhIrcZDnkwOXiS uqK6in5qGLDHgOkbLSyL1NAeXwvmEd0Wx9tmhAcPQ3r/px9LX3MofRk8q7YnzgHLGJhs0jtw5c43/ 5rOGg50qu5GPQN8LHuHdHx24VxrFwTALkPyit8VKyEKwVPCNggT/pD6C3NvHJmFcoiam9oM5JAND5 hx9RYu8KMTH+419glZmA+7zqtOp2IU61QQ5ye7no5dgSIhuI+OKglm/ZDtVnRfpMRwWLDTmNSvJ7P U5W4Hb0ExXbeil/zcE2Ph0JCAogqokkRi2LA3RxPBt+eMaS11HGtT7STNPF+VggqdSI89ePZ1QkwN HPyUwdkJbO5+6N2vWG2Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pFFtw-007O6q-7q; Tue, 10 Jan 2023 14:46:56 +0000 Received: from ams.source.kernel.org ([145.40.68.75]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pFFts-007O5e-LA for linux-arm-kernel@lists.infradead.org; Tue, 10 Jan 2023 14:46:54 +0000 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 C2666B816C6; Tue, 10 Jan 2023 14:46:50 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7740FC433D2; Tue, 10 Jan 2023 14:46:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1673362007; bh=IaikD1RTYaN6M6TGo0lVcnpxh3sUPsTrkuA870j4PL4=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=kPjEfiMiNWq5fLNk+oll30rgxZfthMc/ps5etPKXrknVOEBg6gzwdvidY8MKBKYP8 Udr030lMLCJpbUF2WteuSGJlVyArnUdZ4rkgZH+uevM6f6iSUbJNpb1CAXEvRiIAt9 1WBMUSqNgJM8kCfDif+8D2JUvQiBeLDHeRb79FAxTrM/gyQ1XwO46j95ODpxOyw/G+ pqjXPisHqhUa/O7joUHKuQLYUmSKMy0Z50Y3BHpHemxi/bTxNo8PGJd/NNj68oIMfO SXxbSiMIM9PnRKaBpWHOnHlKbvp1htTr6dcR3g+OJex6hnwET3daPaLXZ6dBXMcc+4 JVreBycgW9dGw== Date: Tue, 10 Jan 2023 23:46:43 +0900 From: Masami Hiramatsu (Google) To: Souradeep Chowdhury Cc: , Bjorn Andersson , , , "Sai Prakash Ranjan" , Sibi Sankar , Rajendra Nayak Subject: Re: [PATCH V0 1/1] bootconfig: Increase max size of bootconfig from 32 KB to 256 KB for DCC support Message-Id: <20230110234643.7bbd340ece99c28f25fe7ad7@kernel.org> In-Reply-To: References: <654357bcbfd3974072a558c494a51edafaa73e1a.1673261071.git.quic_schowdhu@quicinc.com> <20230110001820.5ca81344286f614ed4ccec77@kernel.org> X-Mailer: Sylpheed 3.8.0beta1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) Mime-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230110_064653_023378_A35E56AB X-CRM114-Status: GOOD ( 36.34 ) 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 Tue, 10 Jan 2023 17:26:07 +0530 Souradeep Chowdhury wrote: > > > On 1/9/2023 8:48 PM, Masami Hiramatsu (Google) wrote: > > On Mon, 9 Jan 2023 20:01:05 +0530 > > Souradeep Chowdhury wrote: > > > >> Increasing the memory size of bootconfig to be able to handle a max number of > >> 8192 nodes to be fitted in memory size of 256KB. > > > > Sorry, but you missed the 'xbc_node::data' stores the index of the data and > > that is uint16_t. So the XBC_DATA_MAX is fixed limitation. > > > > The number of nodes (XBC_NODE_MAX) can be expanded because I just decided it > > to keep the pre-compiled array size ~8KB. Maybe expanding it to 64KB just > > increase the size of kernel on init memory (and freed after boot). > > > > Could you tell me why you need such a big data for your DCC? > > > > Thank you, > > DCC is a debugging tool used in qcom which is needed to debug crashes > that can happen at boot-time. For debugging purposes a large number of > registers need to be configured in DCC driver which is to be fed via the > bootconfig file. For that we need to expand the nodes as well as memory > for using bootconfig. Hmm, how many registers does DCC usually use? And how big the bootconfig file is usually? I have no idea about that. > Can you let us know the changes that you suggest for doing the same? Is > it fine to just increase the XBC_NODE_MAX, do we also need to > change the uint16_t to u32 for proper storing of index values? Expanding the number of max nodes is easy, just increase the XBC_NODE_MAX (must be less than 64k). That will also increase the memory consumption during the boot time even if the bootconfig is small. Anyway, it will be freed after boot, so it maybe OK. But expanding the size of max bootconfig needs to change the type of the 'data' field to uint32_t (since that will be used for building bootconfig tool) and you also must confirm that `tools/bootconfig/bootconfig` can be built and pass the test-bootconfig.sh. Hmm, comparing with expanding the max number of XBC node, changing the 'data' type to uint32_t may not have much impact on memory consumption point of view, because it may increase only 20% of memory, but expanding the MAX_XBC_NODE always increases more than double. Thus, if we can accept increasing the number of node, it should be OK to change the 'data' type. BTW, I think now we don't need the ' __attribute__ ((__packed__))' for struct xbc_node. It was packed for reducing the size of array and able to pass 'compiled' bootconfig, but now it is just passed as a text data for safety. Thank you, > > > > > >> > >> Signed-off-by: Souradeep Chowdhury > >> --- > >> include/linux/bootconfig.h | 6 +++--- > >> 1 file changed, 3 insertions(+), 3 deletions(-) > >> > >> diff --git a/include/linux/bootconfig.h b/include/linux/bootconfig.h > >> index 1611f9d..64d233b 100644 > >> --- a/include/linux/bootconfig.h > >> +++ b/include/linux/bootconfig.h > >> @@ -55,11 +55,11 @@ struct xbc_node { > >> } __attribute__ ((__packed__)); > >> > >> #define XBC_KEY 0 > >> -#define XBC_VALUE (1 << 15) > >> -/* Maximum size of boot config is 32KB - 1 */ > >> +#define XBC_VALUE (1 << 18) > >> +/* Maximum size of boot config is 256KB - 1 */ > >> #define XBC_DATA_MAX (XBC_VALUE - 1) > >> > >> -#define XBC_NODE_MAX 1024 > >> +#define XBC_NODE_MAX 8192 > >> #define XBC_KEYLEN_MAX 256 > >> #define XBC_DEPTH_MAX 16 > >> > >> -- > >> 2.7.4 > >> > > > > -- Masami Hiramatsu (Google) _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel