Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <d03c7482-acde-4f33-a7fb-fa7c58fdc9a2@paulmck-laptop> <20230717182337.1098991-15-paulmck@kernel.org>
In-Reply-To: <20230717182337.1098991-15-paulmck@kernel.org>
From:   Feiyang Chen <chris.chenfeiyang@gmail.com>
Date:   Tue, 18 Jul 2023 10:56:06 +0800
Message-ID: <CACWXhK=Vc0kKpnOLRxgw-oP+QDJYt3AqD-kvcA1QkUitiW2Jng@mail.gmail.com>
Subject: Re: [PATCH rcu 15/18] torture: Add init-program support for loongarch
To:     "Paul E. McKenney" <paulmck@kernel.org>
Cc:     rcu@vger.kernel.org, linux-kernel@vger.kernel.org,
        kernel-team@meta.com, rostedt@goodmis.org,
        Feiyang Chen <chenfeiyang@loongson.cn>,
        Huacai Chen <chenhuacai@loongson.cn>, Willy Tarreau <w@1wt.eu>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Tue, Jul 18, 2023 at 2:23=E2=80=AFAM Paul E. McKenney <paulmck@kernel.or=
g> wrote:
>
> This commit adds the __loongarch__, __loongarch_lp64, and
> __loongarch_double_float targets to rcutorture's mkinitrd.sh
> script in order to allow nolibc init programs for loongarch.
>
> Cc: Feiyang Chen <chenfeiyang@loongson.cn>
> Cc: Huacai Chen <chenhuacai@loongson.cn>
> Cc: Willy Tarreau <w@1wt.eu>
> Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
> ---
>  tools/testing/selftests/rcutorture/bin/mkinitrd.sh | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/rcutorture/bin/mkinitrd.sh b/tools/t=
esting/selftests/rcutorture/bin/mkinitrd.sh
> index 3af3a86a0091..e7e23615dbe3 100755
> --- a/tools/testing/selftests/rcutorture/bin/mkinitrd.sh
> +++ b/tools/testing/selftests/rcutorture/bin/mkinitrd.sh
> @@ -63,7 +63,8 @@ ___EOF___
>  # build using nolibc on supported archs (smaller executable) and fall
>  # back to regular glibc on other ones.
>  if echo -e "#if __x86_64__||__i386__||__i486__||__i586__||__i686__" \
> -           "||__ARM_EABI__||__aarch64__||__s390x__\nyes\n#endif" \
> +           "||__ARM_EABI__||__aarch64__||__s390x__"\
> +          "||__loongarch__||__loongarch_lp64||__loongarch_double_float\n=
yes\n#endif" \

Hi, Paul,

Thank you for the patch, but I believe adding only the __loongarch__
should be sufficient.

Thanks,
Feiyang

>     | ${CROSS_COMPILE}gcc -E -nostdlib -xc - \
>     | grep -q '^yes'; then
>         # architecture supported by nolibc
> --
> 2.40.1