Re: [PATCH 1/9] tools/nolibc: move ioctl() to sys/ioctl.h

2025-05-23 Thread Mark Brown
On Fri, May 23, 2025 at 10:33:13AM +0200, Thomas Weißschuh wrote: > On 2025-05-21 19:49:39+0100, Mark Brown wrote: > > That seems to build with 869c788909b93a78ead1ca28c42b95eeb0779215 which > > is the current HEAD of: > > git://git.kernel.org/pub/scm/linux/kernel/git/nolibc/linux-nolibc.git >

Re: [PATCH 1/9] tools/nolibc: move ioctl() to sys/ioctl.h

2025-05-23 Thread Thomas Weißschuh
On 2025-05-21 19:49:39+0100, Mark Brown wrote: > On Wed, May 21, 2025 at 03:45:42PM +0200, Thomas Weißschuh wrote: > > > Or you could run your testsuite with the new nolibc for-next branch > > which should be fixed now. > > That seems to build with 869c788909b93a78ead1ca28c42b95eeb0779215 which >

Re: [PATCH 1/9] tools/nolibc: move ioctl() to sys/ioctl.h

2025-05-21 Thread Mark Brown
On Wed, May 21, 2025 at 03:45:42PM +0200, Thomas Weißschuh wrote: > Or you could run your testsuite with the new nolibc for-next branch > which should be fixed now. That seems to build with 869c788909b93a78ead1ca28c42b95eeb0779215 which is the current HEAD of: git://git.kernel.org/pub/scm/linu

Re: [PATCH 1/9] tools/nolibc: move ioctl() to sys/ioctl.h

2025-05-21 Thread Mark Brown
On Wed, May 21, 2025 at 03:45:42PM +0200, Thomas Weißschuh wrote: > On 2025-05-21 14:22:30+0100, Mark Brown wrote: > > Ah, you expect what's currently there to work - good. I noticed that > > the vDSO tests had a -I for the nolibc directory which made me think it > > was expected that it be there

Re: [PATCH 1/9] tools/nolibc: move ioctl() to sys/ioctl.h

2025-05-21 Thread Thomas Weißschuh
On 2025-05-21 14:22:30+0100, Mark Brown wrote: > On Wed, May 21, 2025 at 03:08:05PM +0200, Thomas Weißschuh wrote: > > On 2025-05-21 14:03:37+0100, Mark Brown wrote: > > > > > +/* make sure to include all global symbols */ > > > > +#include "nolibc.h" > > > > assumes that the nolibc include direc

Re: [PATCH 1/9] tools/nolibc: move ioctl() to sys/ioctl.h

2025-05-21 Thread Mark Brown
On Wed, May 21, 2025 at 03:08:05PM +0200, Thomas Weißschuh wrote: > On 2025-05-21 14:03:37+0100, Mark Brown wrote: > > > +/* make sure to include all global symbols */ > > > +#include "nolibc.h" > > assumes that the nolibc include directory is in the include path, or > > otherwise set up with a -

Re: [PATCH 1/9] tools/nolibc: move ioctl() to sys/ioctl.h

2025-05-21 Thread Thomas Weißschuh
On 2025-05-21 14:03:37+0100, Mark Brown wrote: > On Thu, May 15, 2025 at 09:57:47PM +0200, Thomas Weißschuh wrote: > > This is the location regular userspace expects this definition. > > > > Signed-off-by: Thomas Weißschuh > > This is in -next as b8c436bbef0f2a and breaks the build of all the ar

Re: [PATCH 1/9] tools/nolibc: move ioctl() to sys/ioctl.h

2025-05-21 Thread Mark Brown
On Thu, May 15, 2025 at 09:57:47PM +0200, Thomas Weißschuh wrote: > This is the location regular userspace expects this definition. > > Signed-off-by: Thomas Weißschuh This is in -next as b8c436bbef0f2a and breaks the build of all the arm64 selftests using nolibc: clang --target=aarch64-linux-