On Tue, Sep 25, 2012 at 01:10:27AM +0200, Sasha Levin wrote:
> On 09/25/2012 01:06 AM, Frederic Weisbecker wrote:
> > 2012/9/25 Sasha Levin <[email protected]>:
> >> On 09/25/2012 12:47 AM, Sasha Levin wrote:
> >>> - While I no longer see the warnings I've originally noticed, if I run
> >>> with Paul's last debug patch I see the following warning:
> >>
> >> Correction: Original warnings are still there, they just got buried in the
> >> huge spew that was caused by additional debug warnings
> >> so I've missed them initially.
> >
> > Are they the same? Could you send me your dmesg?
> >
> > Thanks.
> >
>
> Log is attached, you can go directly to 168.703017 when the warnings begin.
Sasha, sorry to burden you with more testing request.
Could you please try out this new branch? It includes some fixes after Wu
Fenguang and
Dan Carpenter reports (not related to your warnings though) and a patch on the
top
of the pile to ensure I diagnosed well the problem, which return immediately
from
rcu_user_*() APIs if we are in an interrupt.
This way we'll have a clearer view. I also would like to know if there are other
problems with the rcu user mode.
Thanks!
Branch is:
git://github.com/fweisbec/linux-dynticks.git
rcu/idle-for-v3.7-take4
Diff is:
diff --git a/arch/x86/kernel/traps.c b/arch/x86/kernel/traps.c
index cb20776..3789675 100644
--- a/arch/x86/kernel/traps.c
+++ b/arch/x86/kernel/traps.c
@@ -232,8 +232,8 @@ DO_ERROR_INFO(X86_TRAP_AC, SIGBUS, "alignment check",
alignment_check,
dotraplinkage void do_stack_segment(struct pt_regs *regs, long error_code)
{
exception_enter(regs);
- if (!notify_die(DIE_TRAP, "stack segment", regs, error_code,
- X86_TRAP_SS, SIGBUS) == NOTIFY_STOP) {
+ if (notify_die(DIE_TRAP, "stack segment", regs, error_code,
+ X86_TRAP_SS, SIGBUS) != NOTIFY_STOP) {
preempt_conditional_sti(regs);
do_trap(X86_TRAP_SS, SIGBUS, "stack segment", regs, error_code,
NULL);
preempt_conditional_cli(regs);
@@ -285,8 +285,8 @@ do_general_protection(struct pt_regs *regs, long error_code)
tsk->thread.error_code = error_code;
tsk->thread.trap_nr = X86_TRAP_GP;
- if (!notify_die(DIE_GPF, "general protection fault", regs,
error_code,
- X86_TRAP_GP, SIGSEGV) == NOTIFY_STOP)
+ if (notify_die(DIE_GPF, "general protection fault", regs,
error_code,
+ X86_TRAP_GP, SIGSEGV) != NOTIFY_STOP)
die("general protection fault", regs, error_code);
goto exit;
}
@@ -678,8 +678,8 @@ dotraplinkage void do_iret_error(struct pt_regs *regs, long
error_code)
info.si_errno = 0;
info.si_code = ILL_BADSTK;
info.si_addr = NULL;
- if (!notify_die(DIE_TRAP, "iret exception", regs, error_code,
- X86_TRAP_IRET, SIGILL) == NOTIFY_STOP) {
+ if (notify_die(DIE_TRAP, "iret exception", regs, error_code,
+ X86_TRAP_IRET, SIGILL) != NOTIFY_STOP) {
do_trap(X86_TRAP_IRET, SIGILL, "iret exception", regs,
error_code,
&info);
}
diff --git a/kernel/rcutree.c b/kernel/rcutree.c
index d249719..e0500c6 100644
--- a/kernel/rcutree.c
+++ b/kernel/rcutree.c
@@ -446,6 +446,9 @@ void rcu_user_enter(void)
WARN_ON_ONCE(!current->mm);
+ if (in_interrupt())
+ return;
+
local_irq_save(flags);
rdtp = &__get_cpu_var(rcu_dynticks);
if (!rdtp->ignore_user_qs && !rdtp->in_user) {
@@ -592,6 +595,9 @@ void rcu_user_exit(void)
unsigned long flags;
struct rcu_dynticks *rdtp;
+ if (in_interrupt())
+ return;
+
local_irq_save(flags);
rdtp = &__get_cpu_var(rcu_dynticks);
if (rdtp->in_user) {
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
