Re: 2.5.31: modules don't work at all

From: Skip Ford (skip.ford@verizon.net)
Date: Sun Aug 11 2002 - 22:07:09 EST


Adam J. Richter wrote:
>
> I am also experiencing modules not working with CONFIG_PREEMPT
> set, and deactivating CONFIG_PREEMPT works around the problem for me too.
>
[snip]
>
> I already know that the error that trips insmod occurs at
> in modules.c, line 831, when qm_symbols gets an error from copy_to_user():
>
> for (; i < mod->nsyms ; ++i, ++s, vals += 2) {
> len = strlen(s->name)+1;
> if (len > bufsize)
> goto calc_space_needed;
>
> here------> if (copy_to_user(strings, s->name, len)
> || __put_user(s->value, vals+0)
> || __put_user(space, vals+1))
> return -EFAULT;
>
> strings += len;
> bufsize -= len;
> space += len;
> }
>
> The values of strings and s->name are similar in 2.5.30+preempt
> (works) and 2.5.31+preempt (does not work). strings is 0x08______, and
> s->name is 0xc0______.

If I back out this change to arch/i386/mm/fault.c then modules
successfully load. I have no idea if backing it out causes other
problems though.

diff -Nru a/arch/i386/mm/fault.c b/arch/i386/mm/fault.c
--- a/arch/i386/mm/fault.c Sat Aug 10 18:42:20 2002
+++ b/arch/i386/mm/fault.c Sat Aug 10 18:42:20 2002
@@ -181,10 +181,10 @@
         info.si_code = SEGV_MAPERR;
 
         /*
- * If we're in an interrupt or have no user
- * context, we must not take the fault..
+ * If we're in an interrupt, have no user context or are running in an
+ * atomic region then we must not take the fault..
          */
- if (in_interrupt() || !mm)
+ if (preempt_count() || !mm)
                 goto no_context;
 
         down_read(&mm->mmap_sem);

-- 
Skip
-
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/



This archive was generated by hypermail 2b29 : Thu Aug 15 2002 - 22:00:26 EST