On Fri, Aug 19, 2005 at 12:27:39PM -0700, David S. Miller wrote:
> From: Ben Collins <[EMAIL PROTECTED]>
> Date: Fri, 19 Aug 2005 04:31:29 -0700
> 
> > The empty symbol warnings atleast are just a bug with
> > scripts/mod/modpost.c (the patch fabbione sent you awhile back Dave, for
> > STT_SPARC_REGISTER changes in glibc's elf.h).
> > 
> > I think the current_thread_info_reg went away with that patch aswell. If
> > it didn't, I have a fix for that one aswell.
> 
> That fix should be in current 2.6.x... Oh I see, it's still checking
> just STT_REGISTER.  Ben, can you resend that patch to me?


--- linux/scripts/mod/modpost.c 2005-06-17 21:48:29.000000000 +0200
+++ linux-source-2.6.12-2.6.12/scripts/mod/modpost.c    2005-06-30 
09:29:54.000000000 +0200
@@ -359,11 +359,16 @@
                /* ignore __this_module, it will be resolved shortly */
                if (strcmp(symname, MODULE_SYMBOL_PREFIX "__this_module") == 0)
                        break;
-#ifdef STT_REGISTER
+/* cope with newer glibc (2.3.4 or higher) STT_ definition in elf.h */
+#if defined(STT_REGISTER) || defined(STT_SPARC_REGISTER)
+/* add compatibility with older glibc */
+#ifndef STT_SPARC_REGISTER
+#define STT_SPARC_REGISTER STT_REGISTER
+#endif
                if (info->hdr->e_machine == EM_SPARC ||
                    info->hdr->e_machine == EM_SPARCV9) {
                        /* Ignore register directives. */
-                       if (ELF_ST_TYPE(sym->st_info) == STT_REGISTER)
+                       if (ELF_ST_TYPE(sym->st_info) == STT_SPARC_REGISTER)
                                break;
                }
 #endif

-- 
Ubuntu     - http://www.ubuntu.com/
Debian     - http://www.debian.org/
Linux 1394 - http://www.linux1394.org/
SwissDisk  - http://www.swissdisk.com/
-
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to