Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] util-linux: disable fallocate for avr32
Date: Sun, 3 Nov 2013 11:34:07 +0100	[thread overview]
Message-ID: <20131103113407.2b5cf76c@skate> (raw)
In-Reply-To: <1383472585-1876-1-git-send-email-spdawson@gmail.com>

Simon,

On Sun,  3 Nov 2013 09:56:25 +0000, spdawson at gmail.com wrote:
> From: Simon Dawson <spdawson@gmail.com>
> 
> The fallocate syscall is not implemented in the avr32 toolchain.
> 
> Fixes build failures such as the following.
> 
>   http://autobuild.buildroot.net/results/bc4/bc41a3fea20181526eb247ac910244aa2aa4c4c0/
> 
> Signed-off-by: Simon Dawson <spdawson@gmail.com>
> ---
>  package/util-linux/Config.in | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/package/util-linux/Config.in b/package/util-linux/Config.in
> index 41e0986..d56e631 100644
> --- a/package/util-linux/Config.in
> +++ b/package/util-linux/Config.in
> @@ -72,6 +72,7 @@ config BR2_PACKAGE_UTIL_LINUX_EJECT
>  
>  config BR2_PACKAGE_UTIL_LINUX_FALLOCATE
>  	bool "fallocate"
> +	depends on !BR2_avr32 # fallocate not implemented
>  	help
>  	  Preallocate space to a file
>  

Situation is a bit fuzzy here.

We have added a patch to uClibc that implements fallocate support:
package/uclibc/0.9.33.2/uclibc-0041-libc-add-posix_fallocate.patch. So
internal toolchains based on 0.9.33.2 should now have fallocate support.

However:

 * The AVR32 toolchain used in the autobuilders predates the addition
   of this patch, and therefore does not have the fallocate support
   (the autobuilders uses a Buildroot generated toolchain, but
   generated with Buildroot 2013.05).

 * The AVR32 support at the moment is using 0.9.33.2, but you told us
   that 0.9.33.2 is basically not working properly on AVR32, and that
   reverting to 0.9.31 was the only solution to get a working AVR32
   system. This would indeed mean that fallocate support would not be
   available on AVR32 (but it would more be related to the uClibc
   version rather than the architecture).

 * A larger problem that I have already raised several times is that
   adding feature patches to the uClibc package in Buildroot is causing
   issues with external toolchain support for uClibc toolchains. These
   external toolchains are very unlikely to carry the same backported
   feature patches as we do, and therefore the build will break with
   those external toolchains if Buildroot relies on uClibc having those
   features.

I'm Cc'ing Arnout, Peter, Yann and Thomas DS on this one, because we
really need to define a policy on this topic.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2013-11-03 10:34 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-03  9:56 [Buildroot] [PATCH] util-linux: disable fallocate for avr32 spdawson at gmail.com
2013-11-03 10:34 ` Thomas Petazzoni [this message]
2013-11-03 11:06   ` Yann E. MORIN
2013-11-03 11:17     ` Thomas Petazzoni
2013-11-04  6:40       ` Arnout Vandecappelle
2013-11-10 23:41 ` Peter Korsgaard

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=20131103113407.2b5cf76c@skate \
    --to=thomas.petazzoni@free-electrons.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox