Messages in this thread Patch in this message | | | Date | Mon, 23 Jul 2007 15:33:11 -0400 | From | Jeff Dike <> | Subject | [PATCH 1/3] UML - Fix string exporting on UML/i386 |
| |
In 2.6.23-rc1, i386 fiddled its string support such that UML started getting undefined references from modules. The UML asm/string.h was including the i386 string.h, which defined __HAVE_ARCH_STR*, but the corresponding implementations weren't being pulled in.
This is fixed by adding arch/i386/lib/string.h to the list of host architecture files to be pulled in to UML.
A complication is that the libc exports file assumed that the generic strlen and strstr weren't in use (i.e. __HAVE_ARCH_STR is defined), then they aren't exported. This is untrue for strlen, which is exported in either case, so this logic is not needed.
Signed-off-by: Jeff Dike <jdike@linux.intel.com> -- arch/um/os-Linux/user_syms.c | 5 +---- arch/um/sys-i386/Makefile | 2 +- 2 files changed, 2 insertions(+), 5 deletions(-)
Index: linux-2.6.17/arch/um/sys-i386/Makefile =================================================================== --- linux-2.6.17.orig/arch/um/sys-i386/Makefile 2007-07-23 14:58:46.000000000 -0400 +++ linux-2.6.17/arch/um/sys-i386/Makefile 2007-07-23 14:58:56.000000000 -0400 @@ -4,7 +4,7 @@ obj-y = bug.o bugs.o checksum.o delay.o obj-$(CONFIG_MODE_SKAS) += stub.o stub_segv.o -subarch-obj-y = lib/bitops.o lib/semaphore.o +subarch-obj-y = lib/bitops.o lib/semaphore.o lib/string.o subarch-obj-$(CONFIG_HIGHMEM) += mm/highmem.o subarch-obj-$(CONFIG_MODULES) += kernel/module.o Index: linux-2.6.17/arch/um/os-Linux/user_syms.c =================================================================== --- linux-2.6.17.orig/arch/um/os-Linux/user_syms.c 2007-07-23 14:58:18.000000000 -0400 +++ linux-2.6.17/arch/um/os-Linux/user_syms.c 2007-07-23 14:59:22.000000000 -0400 @@ -19,10 +19,7 @@ extern void *memmove(void *, const void extern void *memset(void *, int, size_t); extern int printf(const char *, ...); -/* If they're not defined, the export is included in lib/string.c.*/ -#ifdef __HAVE_ARCH_STRLEN -EXPORT_SYMBOL(strlen); -#endif +/* If it's not defined, the export is included in lib/string.c.*/ #ifdef __HAVE_ARCH_STRSTR EXPORT_SYMBOL(strstr); #endif - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |