[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200808131102.34988.denys@visp.net.lb>
Date: Wed, 13 Aug 2008 11:02:34 +0300
From: Denys Fedoryshchenko <denys@...p.net.lb>
To: Jarek Poplawski <jarkao2@...il.com>
Cc: netdev@...r.kernel.org
Subject: Re: NMI lockup, 2.6.26 release
As soon as kernel reboot themself, it won't hurt me much.
With NMI watchdog i notice there was panic missing, so nmi_watchdog was
showing message and was not rebooting. It is fixed in next kernel and i patch
in my kernel - so i will not crash+freeze anymore i guess and will not need
to run to power switch at night.
It can be related to another problem (some corruption) which is not fixed yet,
so prefferably to show timer guys exact location of problem.
Maybe you can make some patch like:
+ if (q->next_watchdog < q->now || next_event <=
+ q->next_watchdog - PSCHED_TICKS_PER_SEC / (10 * HZ)) {
+ qdisc_watchdog_schedule(&q->watchdog, next_event);
+ q->next_watchdog = next_event;
+ } else {
something like BUG()
}
?
Probably also i will try to migrate to "rc" versions of kernel to see if
problem still exist there, a lot of changes done there... is HTB corruption
problem tracked finally and completely? I seen some discussions about it
recently...
On Wednesday 13 August 2008, Jarek Poplawski wrote:
> On Wed, Aug 13, 2008 at 10:28:11AM +0300, Denys Fedoryshchenko wrote:
> > Just as proposal, maybe we can catch situation when "things going wrong"
> > and panic? So we can forward some info to hrtimers guys?
> > If it is hrtimers bug...
>
> Yes, it would be the best, but I don't know how much I can "use" you
> and your clients for debugging this. So, of course, if it's possible
> you could simply edit this patch and try with increased values like
> (100 * HZ) or (1000 * HZ), or even something like:
>
> + if (q->next_watchdog < q->now || next_event <=
> + q->next_watchdog - 10) {
>
> Alas hrtimers guys didn't look like very interested, so the main
> concern should be doing this optimal in net at least.
>
> Jarek P.
>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists