[PATCH] frv: support module exception tables
authorDavid Howells <dhowells@redhat.com>
Sun, 8 Jan 2006 09:01:19 +0000 (01:01 -0800)
committerLinus Torvalds <torvalds@g5.osdl.org>
Mon, 9 Jan 2006 04:13:36 +0000 (20:13 -0800)
Fix the exception table handling so that modules exceptions are dealt with.

Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
arch/frv/mm/extable.c

index 41be1128dc6433b5779fd27d4a21f63c5a2fe12a..caacf030ac753f438026d8c642458bba7c07518e 100644 (file)
@@ -43,7 +43,7 @@ static inline unsigned long search_one_table(const struct exception_table_entry
  */
 unsigned long search_exception_table(unsigned long pc)
 {
-       unsigned long ret = 0;
+       const struct exception_table_entry *extab;
 
        /* determine if the fault lay during a memcpy_user or a memset_user */
        if (__frame->lr == (unsigned long) &__memset_user_error_lr &&
@@ -55,9 +55,10 @@ unsigned long search_exception_table(unsigned long pc)
                 */
                return (unsigned long) &__memset_user_error_handler;
        }
-       else if (__frame->lr == (unsigned long) &__memcpy_user_error_lr &&
-                (unsigned long) &memcpy <= pc && pc < (unsigned long) &__memcpy_end
-                ) {
+
+       if (__frame->lr == (unsigned long) &__memcpy_user_error_lr &&
+           (unsigned long) &memcpy <= pc && pc < (unsigned long) &__memcpy_end
+           ) {
                /* the fault occurred in a protected memset
                 * - we search for the return address (in LR) instead of the program counter
                 * - it was probably during a copy_to/from_user()
@@ -65,27 +66,10 @@ unsigned long search_exception_table(unsigned long pc)
                return (unsigned long) &__memcpy_user_error_handler;
        }
 
-#ifndef CONFIG_MODULES
-       /* there is only the kernel to search.  */
-       ret = search_one_table(__start___ex_table, __stop___ex_table - 1, pc);
-       return ret;
-
-#else
-       /* the kernel is the last "module" -- no need to treat it special */
-       unsigned long flags;
-       struct module *mp;
+       extab = search_exception_tables(pc);
+       if (extab)
+               return extab->fixup;
 
-       spin_lock_irqsave(&modlist_lock, flags);
-
-       for (mp = module_list; mp != NULL; mp = mp->next) {
-               if (mp->ex_table_start == NULL || !(mp->flags & (MOD_RUNNING | MOD_INITIALIZING)))
-                       continue;
-               ret = search_one_table(mp->ex_table_start, mp->ex_table_end - 1, pc);
-               if (ret)
-                       break;
-       }
+       return 0;
 
-       spin_unlock_irqrestore(&modlist_lock, flags);
-       return ret;
-#endif
 } /* end search_exception_table() */