Re: REGRESSION: 3.10-rc1: Dell Latitude e6410 hangs within 3seconds

From: Arend van Spriel
Date: Sat May 18 2013 - 03:15:06 EST


On 05/18/13 03:47, Paul E. McKenney wrote:
On Fri, May 17, 2013 at 10:49:16PM +0200, Arend van Spriel wrote:
On 05/14/2013 10:38 PM, Arend van Spriel wrote:
On 05/14/2013 10:34 PM, Rafael J. Wysocki wrote:
On Tuesday, May 14, 2013 03:03:58 PM Arend van Spriel wrote:
Laptop hangs pretty soon after booting. Workaround for me was to turn
off ACPI on kernel command line, ie. acpi=off

Attached is my kernel configuration.

Well, I have no idea what may be the source of this.

Me neither.

Any chance to bisect?

can try. Rebuilding kernels can be a drag :-p

Regards,
Arend

Hi Rafael,

The bisect result found commit c0f4dfd4f as culprit. I also verified
that the merge into Linus' tree introduced the issue upstream, ie.
6c24499 works and 1f889ec does not. I tried to revert the commit,
but that was a bit risky (conflict to resolve) and did not give me a
stable kernel. The only thing that works is selecting acpi=off. I
tried also nohz=off but that resulted in /init failure.

Regards,
Arend

---8<----------------------------------------------------------------

commit 1f889ec62c3f0d8913f3c32f9aff2a1e15099346
Merge: 6c24499 8fcfae3

commit 8fcfae31719c0a6c03f2cf63f815b46d378d8be4
Merge: d02a9a8 6d87669

commit 6d87669357936bffa1e8fea7a4e7743e76905736
Merge: 3f944ad 81e5949 910ee45

commit c0f4dfd4f90f1667d234d21f15153ea09a2eaa66
Author: Paul E. McKenney<paul.mckenney@xxxxxxxxxx>
Date: Fri Dec 28 11:30:36 2012 -0800

rcu: Make RCU_FAST_NO_HZ take advantage of numbered callbacks

Because RCU callbacks are now associated with the number
of the grace period that they must wait for, CPUs can now
take advance callbacks corresponding to grace periods that
ended while a given CPU was in dyntick-idle mode. This
eliminates the need to try forcing the RCU state machine
while entering idle, thus reducing the CPU intensiveness
of RCU_FAST_NO_HZ, which should increase its energy
efficiency.

Signed-off-by: Paul E. McKenney<paul.mckenney@xxxxxxxxxx>
Signed-off-by: Paul E. McKenney<paulmck@xxxxxxxxxxxxxxxxxx>

Hello, Arend,

Thank you for tracking this down! Could you please try out the following
patch?

Thanks, Paul

That did the trick. Made me bumping into another regression in the driver I am maintaining so more fun to do :-)

Looking at the commit message I guess it is already in some tree. Otherwise I would have said: "You may add

Tested-by: Arend van Spriel <arend@xxxxxxxxxxxx>"

------------------------------------------------------------------------

rcu: Fix comparison sense in rcu_needs_cpu()

Commit c0f4dfd4f (rcu: Make RCU_FAST_NO_HZ take advantage of numbered
callbacks) introduced a bug that can result in excessively long grace
periods. This bug reverse the senes of the "if" statement checking

Oh, there is a typo in the commit message: sense iso senes.

for lazy callbacks, so that RCU takes a lazy approach when there are
in fact non-lazy callbacks. This can result in excessive boot, suspend,
and resume times.

This commit therefore fixes the sense of this "if" statement.

Reported-by: Borislav Petkov<bp@xxxxxxxxx>
Reported-by: Bjørn Mork<bjorn@xxxxxxx>
Reported-by: Joerg Roedel<joro@xxxxxxxxxx>
Signed-off-by: Paul E. McKenney<paulmck@xxxxxxxxxxxxxxxxxx>
Tested-by: Bjørn Mork<bjorn@xxxxxxx>
Tested-by: Joerg Roedel<joro@xxxxxxxxxx>

Gr. AvS

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/