Mercurialcrosstool-ng / file revision
summary | shortlog | changelog | graph | tags | branches | files | changeset | file | latest | revisions | annotate | diff | raw | help
patches/linux/3.0.9/100-headers_install-fix-__packed-in-exported-kernel-head.patch
author "Yann E. MORIN" <yann.morin.1998@anciens.enib.fr>
Sun Jul 17 22:43:07 2011 +0200 (2011-07-17)
changeset 2893 a8a65758664f
parent 2735 f09ed6dd71a3
permissions -rw-r--r--
cc/gcc: do not use the core pass-2 to build the baremetal compiler

In case we build a baremetal compiler, use the standard passes:
- core_cc is used to build the C library;
- as such, it is meant to run on build, not host;
- the final compiler is meant to run on host;

As the current final compiler step can not build a baremetal compiler,
call the core backend from the final step.

NB: Currently, newlib is built during the start_files pass, so we have
to have a core compiler by then... Once we can build the baremetal
compiler from the final cc step, then we can move the newlib build to
the proper step, and then get rid of the core pass-1 static compiler...

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@anciens.enib.fr> 
     1 commit f210735fe2f17a6225432ee3d1239bcf23a8659c

     2 Author: Markus Trippelsdorf <markus@trippelsdorf.de>

     3 Date:   Fri Jun 24 15:51:00 2011 +0200

     4 

     5     headers_install: fix __packed in exported kernel headers

     6     

     7     checkpatch.pl warns about using __attribute__((packed)) in kernel

     8     headers: "__packed is preferred over __attribute__((packed))". If one

     9     follows that advice it could cause problems in the exported header

    10     files, because the outside world doesn't know about this shortcut.

    11     

    12     For example busybox will fail to compile:

    13      CC      miscutils/ubi_attach_detach.o

    14      In file included from miscutils/ubi_attach_detach.c:27:0:

    15      /usr/include/mtd/ubi-user.h:330:3: error: conflicting types for ‘__packed’

    16      /usr/include/mtd/ubi-user.h:314:3: note: previous declaration of ‘__packed’ was here

    17     ...

    18     

    19     Fix the problem by substituting __packed with __attribute__((packed)) in

    20     the header_install.pl script.

    21     

    22     Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

    23     CC: Joe Perches <joe@perches.com>

    24     Signed-off-by: Markus Trippelsdorf <markus@trippelsdorf.de>

    25     Signed-off-by: Michal Marek <mmarek@suse.cz>

    26 

    27 diff --git a/scripts/headers_install.pl b/scripts/headers_install.pl

    28 index efb3be1..48462be 100644

    29 --- a/scripts/headers_install.pl

    30 +++ b/scripts/headers_install.pl

    31 @@ -35,6 +35,7 @@ foreach my $file (@files) {

    32  		$line =~ s/([\s(])__iomem\s/$1/g;

    33  		$line =~ s/\s__attribute_const__\s/ /g;

    34  		$line =~ s/\s__attribute_const__$//g;

    35 +		$line =~ s/\b__packed\b/__attribute__((packed))/g;

    36  		$line =~ s/^#include <linux\/compiler.h>//;

    37  		$line =~ s/(^|\s)(inline)\b/$1__$2__/g;

    38  		$line =~ s/(^|\s)(asm)\b(\s|[(]|$)/$1__$2__$3/g;
crosstool-ng