[BUG] Linux 3.0 commit 3627924acf70a broke include/mtd/ubi_user.h.

From: Rob Landley
Date: Sun Aug 14 2011 - 13:45:49 EST


"make headers_install" copies include/mtd/ubi_user.h to userspace, where
things like busybox link against it. With 3.0, building defconfig
busybox gets this error:

In file included from miscutils/ubi_tools.c:63:
/home/landley/aboriginal/aboriginal/build/simple-cross-compiler-i686/bin/../include/mtd/ubi-user.h:329:
error: conflicting types for '__packed'
/home/landley/aboriginal/aboriginal/build/simple-cross-compiler-i686/bin/../include/mtd/ubi-user.h:313:
error: previous declaration of '__packed' was here

This is due to the Linux 3.0 commit in the title:

UBI: use __packed instead of __attribute__((packed))

This relies on __packed being defined in:

include/linux/compiler-gcc.h:
#define __packed __attribute__((packed))

Which is not exported to userspace.

I.E. this "cleanup" broke the ability for userspace to use this header,
even though the header is exported to userspace by headers_install.

Rob
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/