Re: [PATCH v3 00/11] tools/nolibc: shrink arch support

[Willy Tarreau]

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