All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] glibc: needs kernel headers >= 4.5 on mips(64)
Date: Tue, 29 Aug 2017 23:09:15 +0200	[thread overview]
Message-ID: <20170829210915.GB3338@scaer> (raw)
In-Reply-To: <20170820144154.15347-1-romain.naour@gmail.com>

Romain, All,

On 2017-08-20 16:41 +0200, Romain Naour spake thusly:
> Since glibc 2.23 [1]:
> sysdeps/unix/sysv/linux/mips/configure.ac
> [...]
> if test -z "$arch_minimum_kernel"; then
>   if test x$libc_cv_mips_nan2008 = xyes; then
>     arch_minimum_kernel=4.5.0
>   fi
> fi
> 
> This mean that if nan2008 support is requested (i.e BR2_MIPS_NAN_2008
> in Buildroot) the minimum kernel headers version is 4.5.0 (not 3.2.0).
> 
> To simplify the dependancy handling in Buildroot we add this dependency
> for all mips(64), even if BR2_MIPS_NAN_2008 is not selected.
> 
> [1] https://sourceware.org/git/?p=glibc.git;a=commit;h=d5f2798a0ac9d5ad8ad7a506a2f840035135e2d2
> 
> Signed-off-by: Romain Naour <romain.naour@gmail.com>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> ---
> 
> This is for master.
> 
> I tried to use BR2_MIPS_NAN_2008 but it trigger a circular dependency in Kconfig.

After the discussion in this thread and on IRC, I've marked this patch
as "Changes Requested".

Now I just have to polish my own series that has a fix for that issue...

Thanks! ;-)

Regards,
Yann E. MORIN.

>  toolchain/toolchain-buildroot/Config.in | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/toolchain/toolchain-buildroot/Config.in b/toolchain/toolchain-buildroot/Config.in
> index f47001f..967da1a 100644
> --- a/toolchain/toolchain-buildroot/Config.in
> +++ b/toolchain/toolchain-buildroot/Config.in
> @@ -43,8 +43,9 @@ config BR2_TOOLCHAIN_BUILDROOT_UCLIBC
>  config BR2_TOOLCHAIN_BUILDROOT_GLIBC
>  	bool "glibc"
>  	depends on BR2_arm         || BR2_armeb      || BR2_aarch64 || \
> -		   BR2_aarch64_be  || BR2_i386       || BR2_mips    || \
> -		   BR2_mipsel      || BR2_mips64     || BR2_mips64el|| \
> +		   BR2_aarch64_be  || BR2_i386       || \
> +		   ((BR2_mips || BR2_mipsel || BR2_mips64 || BR2_mips64el) && \
> +		   BR2_TOOLCHAIN_HEADERS_AT_LEAST_4_5) || \
>  		   BR2_powerpc     || BR2_powerpc64  || BR2_powerpc64le || \
>  		   BR2_sh          || BR2_sparc64    || BR2_x86_64 || \
>  		   BR2_microblaze || BR2_nios2
> @@ -65,6 +66,11 @@ comment "glibc needs a toolchain w/ dynamic library, kernel headers >= 3.2"
>  	depends on BR2_USE_MMU
>  	depends on BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_2
>  
> +# glibc require kernel headers >= 4.5 if nan2008 support is selected.
> +comment "glibc on mips(64) needs a toolchain w/ headers >= 4.5"
> +	depends on BR2_mips || BR2_mipsel || BR2_mips64 || BR2_mips64el
> +	depends on !BR2_TOOLCHAIN_HEADERS_AT_LEAST_4_5
> +
>  config BR2_TOOLCHAIN_BUILDROOT_MUSL
>  	bool "musl"
>  	depends on BR2_aarch64	  || BR2_arm     || BR2_armeb     || BR2_i386 || \
> -- 
> 2.9.5
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

      parent reply	other threads:[~2017-08-29 21:09 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-20 14:41 [Buildroot] [PATCH] glibc: needs kernel headers >= 4.5 on mips(64) Romain Naour
2017-08-20 20:26 ` Thomas Petazzoni
2017-08-21 16:28   ` Yann E. MORIN
2017-08-21 21:10     ` Thomas Petazzoni
2017-08-22 16:34       ` Yann E. MORIN
2017-08-24 21:33         ` Romain Naour
2017-08-29 21:09 ` Yann E. MORIN [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170829210915.GB3338@scaer \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.