On Fri, Jun 8, 2012 at 5:50 PM, Akinobu Mita <akinobu.mita@xxxxxxxxx> wrote: > lib/string.c | 36 ++++++++++++++++++++++++++++++++++++ Is lib/string.c the right place for this? I get a build error on the ia64 sim_defconfig: LD arch/ia64/hp/sim/boot/bootloader It fails because it pulls in lib/lib.a(string.o) to get some innocuous function like strcpy() ... but it also gets given memweight() which relies on __bitmap_weight() which it doesn't have, because it doesn't include lib/built-in.o (which is where bitmap.o, the definer of __bitmap_weight(), has been linked). Moving memweight() to lib/bitmap.c fixes the problem. But it isn't really clear that it belongs there either. Perhaps it should be its own file lib/memweight.c that gets included in lib/lib.a? -Tony -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
