Re: [PATCH 0/1] tools/nolibc: always disable stack protector for tests

From: Paul E. McKenney
Date: Sun Mar 19 2023 - 19:55:15 EST


On Sun, Mar 19, 2023 at 09:11:19PM +0100, Willy Tarreau wrote:
> Hi Paul,
>
> On Sun, Mar 19, 2023 at 11:05:15AM -0700, Paul E. McKenney wrote:
> > Queued and pushed, thank you both!
>
> Thank you!
>
> > Just to make sure I understand, you would like me to add this to the
> > serieses from Vincent, yourself, and Feiyang that are set up to go into
> > the next merge window, correct?
> >
> > If you instead would like me to push it to Linus directly into the v6.3
> > release as a regression fix late this coming week, please let me know.
>
> It would indeed be preferable for 6.3 as it fixes issues, but it's not
> critical at all, just an annoyance for some developers. So if you have a
> series of fixes pending to be sent soon, it would be nice to add it there.

Very well, here is what I currently have for nolibc, from newest to
oldest:

7712ec32cbda selftests/nolibc: Adjust indentation for Makefile
96218cfc79eb selftests/nolibc: Add support for LoongArch
cab6bbd00f2a tools/nolibc: Add support for LoongArch
496921e4724c tools/nolibc: Add statx() and make stat() rely on statx() if necessary
97a15c47b98c tools/nolibc: Include linux/fcntl.h and remove duplicate code
de06f2fe1aac tools/nolibc: check for S_I* macros before defining them
7fb54b517d44 selftests/nolibc: skip the chroot_root and link_dir tests when not privileged
296659c6e551 tools/nolibc: add getuid() and geteuid()
bd9a87d64994 tools/nolibc: add tests for the integer limits in stdint.h
2d1813a7ef67 tools/nolibc: enlarge column width of tests
5dbcbb2da574 tools/nolibc: add integer types and integer limit macros
111bb94f67eb tools/nolibc: add stdint.h
2dbf8575b2da tools/nolibc: Add gitignore to avoid git complaints about sysroot

I will take the new commit, rebase it to precede the first of the above,
so that the stack will look like this:

7712ec32cbda selftests/nolibc: Adjust indentation for Makefile
96218cfc79eb selftests/nolibc: Add support for LoongArch
cab6bbd00f2a tools/nolibc: Add support for LoongArch
496921e4724c tools/nolibc: Add statx() and make stat() rely on statx() if necessary
97a15c47b98c tools/nolibc: Include linux/fcntl.h and remove duplicate code
de06f2fe1aac tools/nolibc: check for S_I* macros before defining them
7fb54b517d44 selftests/nolibc: skip the chroot_root and link_dir tests when not privileged
296659c6e551 tools/nolibc: add getuid() and geteuid()
bd9a87d64994 tools/nolibc: add tests for the integer limits in stdint.h
2d1813a7ef67 tools/nolibc: enlarge column width of tests
5dbcbb2da574 tools/nolibc: add integer types and integer limit macros
111bb94f67eb tools/nolibc: add stdint.h
2dbf8575b2da tools/nolibc: Add gitignore to avoid git complaints about sysroot
???????????? tools/nolibc: always disable stack protector for tests

Then if tests, -next, and kernel test robot are kind, I will send a
pull request to Linus for the "always disable stack protector for tests"
commit. The rest I send for the upcoming merge window.

Please let me know if I am missing something.

Thanx, Paul