Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp689374pxj; Tue, 18 May 2021 11:54:14 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxZF/2PtdOhi7vmZwQuFb77XzXlxpK/uLdF69YLWjUcba1WlZ6LqNzWpu1hSFFUqEWzWUMC X-Received: by 2002:aa7:d2d4:: with SMTP id k20mr8642986edr.384.1621364054640; Tue, 18 May 2021 11:54:14 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1621364054; cv=none; d=google.com; s=arc-20160816; b=twZ27n0tJczLlSuM5qYTIsRWPYtbe3+wj52aDNFcLmrklC1pIdwyDWfuWige5iouY5 lUKWLthGH2RdEuF0fr2oR2X2lizctHL8epRYxMXcAn12+kPZ5kbWbAjJqYtGUb+bG2KL 3KgPGnm/b4XU2tLrbww8ept22R5jv18LQoAeolFoi2xfiAZJg/KvVQkIuTvfFWABmC1C QwFQzZUZwEknVAYWwV2r+Pmp+CuMDQ6ENsVKNZPeV1lV6MoPTyKQEVNhT21hQ2zsd5l4 vNnGmG9mO98yAOV9DuDA/88ogrv6jHUXafUTbO+CP65LTXNvCtwzeRgb4IzGwEKKubgU m6lQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=ZXsRNWguWbVa0PtAmixG+uiM2qAEjYWOfHSwLuA4TyE=; b=m7nuVje2Wy4ZhAfx4JY9CVdUWOxydMWtHYXY6VWOkysntHcBijYfc526RncHFceAJI 8NTzDeItQxVyg/UpY7lVohvFQzbV5+MWaYKxhniWj2zKuejIc6hexW4NNmUBRh23rLFw xX4UpI/jLuQoA0P/mjb3FohUxmUVDI7XaA6XXPKrfegAT0MhmVFnmPqPlzRNfcoR48r5 3NKq85gEmCa3G8RqMQrEBaRSn8omTDHdImFQBvdcBJC8G2xBiAfY2OD7fahmOnm3E1/h usVxYYLKaUQM4JdgVt2nHVOsUqy8LpsKwQG4XxF6PjJodt9NaO481GacTRIHCU0qrJhr rZQQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@mail.ustc.edu.cn header.s=dkim header.b=EqztZO8D; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=mail.ustc.edu.cn Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v13si4100039eju.482.2021.05.18.11.53.37; Tue, 18 May 2021 11:54:13 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@mail.ustc.edu.cn header.s=dkim header.b=EqztZO8D; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=mail.ustc.edu.cn Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S244314AbhEQQRN (ORCPT + 99 others); Mon, 17 May 2021 12:17:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33456 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346372AbhEQQFv (ORCPT ); Mon, 17 May 2021 12:05:51 -0400 Received: from ustc.edu.cn (email6.ustc.edu.cn [IPv6:2001:da8:d800::8]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id BAEEBC0467E9 for ; Mon, 17 May 2021 07:45:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mail.ustc.edu.cn; s=dkim; h=Received:Date:From:To:Cc:Subject: Message-ID:In-Reply-To:References:MIME-Version:Content-Type: Content-Transfer-Encoding; bh=ZXsRNWguWbVa0PtAmixG+uiM2qAEjYWOfH SwLuA4TyE=; b=EqztZO8DWB8gA3sQFNBpU6PwbOgqEpdc/bTPL2yVNn47iZdzWK 6sUMGqvRSLsBbD3AJgkf9TbgcKVCzR5I7nqvqX4RUW6zShDmnorVMGrB5FceKPwd 03Xr7hymxSFVdqRy+q73W9cmxguTaVMm1fghbdBRHhLgd6qSfiDsfGVCk= Received: from xhacker (unknown [101.86.20.15]) by newmailweb.ustc.edu.cn (Coremail) with SMTP id LkAmygB3f0d+gaJgnBTyAA--.22620S2; Mon, 17 May 2021 22:45:18 +0800 (CST) Date: Mon, 17 May 2021 22:40:01 +0800 From: Jisheng Zhang To: Alex Ghiti Cc: Paul Walmsley , Palmer Dabbelt , Albert Ou , linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] riscv: mm: init: Consolidate vars, functions Message-ID: <20210517224001.42424097@xhacker> In-Reply-To: <66c8965d-494f-6c2f-0739-9bee13fc507c@ghiti.fr> References: <20210516211556.43c00055@xhacker> <20210516211922.2110232e@xhacker> <66c8965d-494f-6c2f-0739-9bee13fc507c@ghiti.fr> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-CM-TRANSID: LkAmygB3f0d+gaJgnBTyAA--.22620S2 X-Coremail-Antispam: 1UD129KBjvJXoW7ZFW8XF1fJw4ktw4kuw43Awb_yoW8JFWrpr 4FqFW5Ja1kArW7CasFqr4IgFyj9Fn3WFyrJw1UKr1fuFn8Grn8Xw4UWrW5u39FgrWxuF4Y vr4jy34qgw4jv37anT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDU0xBIdaVrnRJUUUy2b7Iv0xC_KF4lb4IE77IF4wAFF20E14v26r1j6r4UM7CY07I2 0VC2zVCF04k26cxKx2IYs7xG6rWj6s0DM7CIcVAFz4kK6r1j6r18M28lY4IEw2IIxxk0rw A2F7IY1VAKz4vEj48ve4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_tr0E3s1l84ACjcxK6xII jxv20xvEc7CjxVAFwI0_Gr1j6F4UJwA2z4x0Y4vEx4A2jsIE14v26rxl6s0DM28EF7xvwV C2z280aVCY1x0267AKxVW0oVCq3wAS0I0E0xvYzxvE52x082IY62kv0487Mc02F40EFcxC 0VAKzVAqx4xG6I80ewAv7VC0I7IYx2IY67AKxVWUXVWUAwAv7VC2z280aVAFwI0_Jr0_Gr 1lOx8S6xCaFVCjc4AY6r1j6r4UM4x0Y48IcVAKI48JMxAIw28IcxkI7VAKI48JMxC20s02 6xCaFVCjc4AY6r1j6r4UMI8I3I0E5I8CrVAFwI0_Jr0_Jr4lx2IqxVCjr7xvwVAFwI0_Jr I_JrWlx4CE17CEb7AF67AKxVWUAVWUtwCIc40Y0x0EwIxGrwCI42IY6xIIjxv20xvE14v2 6r1j6r1xMIIF0xvE2Ix0cI8IcVCY1x0267AKxVW8JVWxJwCI42IY6xAIw20EY4v20xvaj4 0_Zr0_Wr1UMIIF0xvEx4A2jsIE14v26r1j6r4UMIIF0xvEx4A2jsIEc7CjxVAFwI0_Gr0_ Gr1UYxBIdaVFxhVjvjDU0xZFpf9x07j5BMNUUUUU= X-CM-SenderInfo: xmv2xttqjtqzxdloh3xvwfhvlgxou0/ Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 16 May 2021 21:10:48 +0200 Alex Ghiti wrote: > Hi Jisheng, > > On 16/05/2021 15:19, Jisheng Zhang wrote: > > On Sun, 16 May 2021 21:15:56 +0800 Jisheng Zhang wrote: > > > >> From: Jisheng Zhang > >> > >> Consolidate the following items in init.c > >> > >> Staticize global vars as much as possible; > >> Add __initdata mark if the global var isn't needed after init > >> Add __init mark if the func isn't needed after init > >> Add __ro_after_init if the global var is read only after init > > > > Hi Alexandre, > > > > I think kernel_virt_addr can be removed, users can directly use > > the KERNEL_LINK_ADDR MACRO. Maybe I miss the reason of intrducing > > it in commit 2bfc6cd81bd17e43 ("riscv: Move kernel mapping outside of > > linear mapping"), could you please comment? > > > > Thanks in advance > > kernel_virt_addr will be used when I push my relocatable series since > then, the kernel virtual address will be different from > KERNEL_LINK_ADDR. I agree this was not needed in 2bfc6cd81bd17e43 > ("riscv: Move kernel mapping outside of linear mapping"), I'm a bit late > in pushing the relocatable series. > Thanks for your information. I think kernel_virt_addr could be __ro_after_init But I will keep it unchanged until your relocatable series. thanks