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
next prev parent 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