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;
Date:   Sat, 15 Jul 2023 12:01:34 +0200
From:   Willy Tarreau <w@1wt.eu>
To:     Zhangjin Wu <falcon@tinylab.org>
Cc:     arnd@arndb.de, linux-kernel@vger.kernel.org,
        linux-kselftest@vger.kernel.org, thomas@t-8ch.de
Subject: Re: [PATCH v3 00/11] tools/nolibc: shrink arch support
Message-ID: <20230715100134.GD24086@1wt.eu>
References: <cover.1689150149.git.falcon@tinylab.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <cover.1689150149.git.falcon@tinylab.org>
User-Agent: Mutt/1.10.1 (2018-07-13)
Precedence: bulk

On Wed, Jul 12, 2023 at 05:15:28PM +0800, Zhangjin Wu wrote:
> Hi, Willy, Thomas
> 
> Here is the revision of the v2 arch support shrink patchset [1], it
> mainly applies suggestions from you.
> 
> It is based on the 20230710-nolibc-ser2-tom-syscall-configv4-report
> branch of nolibc repo.
> 
> Tested for all of the supported archs:
> 
>            arch/board | result
>           ------------|------------
>       arm/versatilepb | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.
>       arm/vexpress-a9 | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.
>              arm/virt | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.
>          aarch64/virt | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.
>               i386/pc | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.
>             x86_64/pc | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.
>          mipsel/malta | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.
>      loongarch64/virt | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.
>          riscv64/virt | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.
>          riscv32/virt | 151 test(s): 122 passed,   7 skipped,  22 failed => status: failure.
> s390x/s390-ccw-virtio | 151 test(s): 144 passed,   7 skipped,   0 failed => status: warning.

Thanks for all this work. So I've checked a few random archs and trust
your tests above to confirm they're correct ;-)  I'd just like to get
your confirmation regarding statx() support in 4.14 and 4.19, and likely
adjust _start_c() according to the last few tests. Also please do prepend
to the list the patch that adds the optimize("-Os") to fix _start, and we
should be good.

Thanks!
Willy