All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: [Buildroot] [autobuild.buildroot.net] Your daily results for 2022-05-08
       [not found] <62789f76.1c69fb81.8311.5ad0SMTPIN_ADDED_MISSING@mx.google.com>
@ 2022-05-15  5:48 ` Petr Vorel
  2022-05-15 10:51   ` Fabrice Fontaine
  0 siblings, 1 reply; 3+ messages in thread
From: Petr Vorel @ 2022-05-15  5:48 UTC (permalink / raw)
  To: Thomas Petazzoni; +Cc: Fabrice Fontaine, Yann E. MORIN, Buildroot Mailing List

Hi all,

> Autobuilder failures
> ====================

>     arch     |             reason             |                                       url
> -------------+--------------------------------+---------------------------------------------------------------------------------
>    mips64    |     ltp-testsuite-20220121     | http://autobuild.buildroot.net/results/2dafec94b888960335115b44c281741dfd55840b

this uclibc 1.0.40  based toolchain is not able to detect fstatat() via configure, but
it's available:

configure:6615: checking for fstatat
configure:6615: buildroot/output/host/bin/mips64-buildroot-linux-uclibc-gcc -o conftest -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Os -g0 -Ibuildroot/output/host/bin/../mips64-buildroot-linux-uclibc/sysroot/usr/include/tirpc  -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE  conftest.c -Lbuildroot/output/host/bin/../mips64-buildroot-linux-uclibc/sysroot/usr/lib -ltirpc  -lfts >&5
buildroot/output/host/lib/gcc/mips64-buildroot-linux-uclibc/10.3.0/../../../../mips64-buildroot-linux-uclibc/bin/ld: /tmp/ccFw53nU.o: in function `main':
conftest.c:(.text.startup+0x10): undefined reference to `fstatat'
buildroot/output/host/lib/gcc/mips64-buildroot-linux-uclibc/10.3.0/../../../../mips64-buildroot-linux-uclibc/bin/ld: conftest.c:(.text.startup+0x18): undefined reference to `fstatat'

But even if I change include/config.h to
- /* #undef HAVE_FSTATAT */
+#define HAVE_FSTATAT 1

it fails to build:
output/build/ltp-testsuite-20220121/testcases/kernel/syscalls/fstatat/fstatat01.c:100: undefined reference to `fstatat'

That's strange, isn't it a bug in the toolchain?

Or is it really LTP bug? What would be missing then? If yes, it'd be nice to fix
it before upcoming LTP release (2 weeks).

Kind regards,
Petr
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [autobuild.buildroot.net] Your daily results for 2022-05-08
  2022-05-15  5:48 ` [Buildroot] [autobuild.buildroot.net] Your daily results for 2022-05-08 Petr Vorel
@ 2022-05-15 10:51   ` Fabrice Fontaine
  2022-05-15 13:56     ` Petr Vorel
  0 siblings, 1 reply; 3+ messages in thread
From: Fabrice Fontaine @ 2022-05-15 10:51 UTC (permalink / raw)
  To: Petr Vorel; +Cc: Yann E. MORIN, Thomas Petazzoni, Buildroot Mailing List

Hi Petr,

Le dim. 15 mai 2022 à 07:48, Petr Vorel <petr.vorel@gmail.com> a écrit :
>
> Hi all,
>
> > Autobuilder failures
> > ====================
>
> >     arch     |             reason             |                                       url
> > -------------+--------------------------------+---------------------------------------------------------------------------------
> >    mips64    |     ltp-testsuite-20220121     | http://autobuild.buildroot.net/results/2dafec94b888960335115b44c281741dfd55840b
>
> this uclibc 1.0.40  based toolchain is not able to detect fstatat() via configure, but
> it's available:
>
> configure:6615: checking for fstatat
> configure:6615: buildroot/output/host/bin/mips64-buildroot-linux-uclibc-gcc -o conftest -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Os -g0 -Ibuildroot/output/host/bin/../mips64-buildroot-linux-uclibc/sysroot/usr/include/tirpc  -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE  conftest.c -Lbuildroot/output/host/bin/../mips64-buildroot-linux-uclibc/sysroot/usr/lib -ltirpc  -lfts >&5
> buildroot/output/host/lib/gcc/mips64-buildroot-linux-uclibc/10.3.0/../../../../mips64-buildroot-linux-uclibc/bin/ld: /tmp/ccFw53nU.o: in function `main':
> conftest.c:(.text.startup+0x10): undefined reference to `fstatat'
> buildroot/output/host/lib/gcc/mips64-buildroot-linux-uclibc/10.3.0/../../../../mips64-buildroot-linux-uclibc/bin/ld: conftest.c:(.text.startup+0x18): undefined reference to `fstatat'
>
> But even if I change include/config.h to
> - /* #undef HAVE_FSTATAT */
> +#define HAVE_FSTATAT 1
>
> it fails to build:
> output/build/ltp-testsuite-20220121/testcases/kernel/syscalls/fstatat/fstatat01.c:100: undefined reference to `fstatat'
>
> That's strange, isn't it a bug in the toolchain?
>
> Or is it really LTP bug? What would be missing then? If yes, it'd be nice to fix
> it before upcoming LTP release (2 weeks).

I dig into this issue a while ago without finding a solution. Here is
what I found.
First the build failure is only raised with uclibc-ng on mips64.
Secondly, fstatat is defined as extern if __USE_FILE_OFFSET64 is not
defined [1] however its implementation is protected by "ifdef
__NR_fstatat64" [2]

__NR_fstatat64 is defined by:

#if __WORDSIZE == 64 && defined __NR_newfstatat
# define __NR_fstatat64 __NR_newfstatat
#endif

So, for an unknown reason, I assume that __NR_newfstatat is not defined.
In this case, uclibc-ng doesn't define any fallback [3] resulting in
the build failure.

>
> Kind regards,
> Petr

Best Regards,

Fabrice

[1] https://github.com/wbx-github/uclibc-ng/blob/ab1dd83bec59c9e65c31efd6e887182948f627be/include/sys/stat.h#L241
[2] https://github.com/wbx-github/uclibc-ng/blob/ab1dd83bec59c9e65c31efd6e887182948f627be/libc/sysdeps/linux/common/fstatat.c#L18
[3] https://github.com/wbx-github/uclibc-ng/blob/ab1dd83bec59c9e65c31efd6e887182948f627be/libc/sysdeps/linux/common/fstatat.c#L34
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [autobuild.buildroot.net] Your daily results for 2022-05-08
  2022-05-15 10:51   ` Fabrice Fontaine
@ 2022-05-15 13:56     ` Petr Vorel
  0 siblings, 0 replies; 3+ messages in thread
From: Petr Vorel @ 2022-05-15 13:56 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: Yann E. MORIN, Thomas Petazzoni, Buildroot Mailing List

Hi Fabrice,

> Hi Petr,

> Le dim. 15 mai 2022 à 07:48, Petr Vorel <petr.vorel@gmail.com> a écrit :

> > Hi all,

> > > Autobuilder failures
> > > ====================

> > >     arch     |             reason             |                                       url
> > > -------------+--------------------------------+---------------------------------------------------------------------------------
> > >    mips64    |     ltp-testsuite-20220121     | http://autobuild.buildroot.net/results/2dafec94b888960335115b44c281741dfd55840b

> > this uclibc 1.0.40  based toolchain is not able to detect fstatat() via configure, but
> > it's available:

> > configure:6615: checking for fstatat
> > configure:6615: buildroot/output/host/bin/mips64-buildroot-linux-uclibc-gcc -o conftest -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Os -g0 -Ibuildroot/output/host/bin/../mips64-buildroot-linux-uclibc/sysroot/usr/include/tirpc  -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE  conftest.c -Lbuildroot/output/host/bin/../mips64-buildroot-linux-uclibc/sysroot/usr/lib -ltirpc  -lfts >&5
> > buildroot/output/host/lib/gcc/mips64-buildroot-linux-uclibc/10.3.0/../../../../mips64-buildroot-linux-uclibc/bin/ld: /tmp/ccFw53nU.o: in function `main':
> > conftest.c:(.text.startup+0x10): undefined reference to `fstatat'
> > buildroot/output/host/lib/gcc/mips64-buildroot-linux-uclibc/10.3.0/../../../../mips64-buildroot-linux-uclibc/bin/ld: conftest.c:(.text.startup+0x18): undefined reference to `fstatat'

> > But even if I change include/config.h to
> > - /* #undef HAVE_FSTATAT */
> > +#define HAVE_FSTATAT 1

> > it fails to build:
> > output/build/ltp-testsuite-20220121/testcases/kernel/syscalls/fstatat/fstatat01.c:100: undefined reference to `fstatat'

> > That's strange, isn't it a bug in the toolchain?

> > Or is it really LTP bug? What would be missing then? If yes, it'd be nice to fix
> > it before upcoming LTP release (2 weeks).

> I dig into this issue a while ago without finding a solution. Here is
> what I found.
> First the build failure is only raised with uclibc-ng on mips64.
> Secondly, fstatat is defined as extern if __USE_FILE_OFFSET64 is not
> defined [1] however its implementation is protected by "ifdef
> __NR_fstatat64" [2]

> __NR_fstatat64 is defined by:

> #if __WORDSIZE == 64 && defined __NR_newfstatat
> # define __NR_fstatat64 __NR_newfstatat
> #endif

> So, for an unknown reason, I assume that __NR_newfstatat is not defined.
> In this case, uclibc-ng doesn't define any fallback [3] resulting in
> the build failure.

Thanks for confirming that it's bug/issue on uclibc-ng on mips64.
It'd be worth to at least report it to them.

Kind regards,
Petr


> > Kind regards,
> > Petr

> Best Regards,

> Fabrice

> [1] https://github.com/wbx-github/uclibc-ng/blob/ab1dd83bec59c9e65c31efd6e887182948f627be/include/sys/stat.h#L241
> [2] https://github.com/wbx-github/uclibc-ng/blob/ab1dd83bec59c9e65c31efd6e887182948f627be/libc/sysdeps/linux/common/fstatat.c#L18
> [3] https://github.com/wbx-github/uclibc-ng/blob/ab1dd83bec59c9e65c31efd6e887182948f627be/libc/sysdeps/linux/common/fstatat.c#L34
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-05-15 13:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <62789f76.1c69fb81.8311.5ad0SMTPIN_ADDED_MISSING@mx.google.com>
2022-05-15  5:48 ` [Buildroot] [autobuild.buildroot.net] Your daily results for 2022-05-08 Petr Vorel
2022-05-15 10:51   ` Fabrice Fontaine
2022-05-15 13:56     ` Petr Vorel

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.