From: Mike Galbraith <umgwanakikbuti@gmail.com>
To: "Oza (Pawandeep) Oza" <oza@broadcom.com>
Cc: pawandeep oza <oza.contri.linux.kernel@gmail.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
malayasen rout <malayasen.rout@gmail.com>
Subject: Re: [KERNEL BUG] do_timer/tick_handover_do_timer 3.10.17
Date: Fri, 08 May 2015 07:12:11 +0200 [thread overview]
Message-ID: <1431061931.3168.41.camel@gmail.com> (raw)
In-Reply-To: <5C6899BCED92C94EBDCC00F80838E3D52113AB15@SJEXCHMB06.corp.ad.broadcom.com>
On Fri, 2015-05-08 at 04:16 +0000, Oza (Pawandeep) Oza wrote:
> So Mike, is this reason strong enough for you ?
Nope. I think you did the right thing in removing your dependency on
jiffies reliability in a dying box. You don't have to convince me of
anything though, CC timer subsystem maintainer, see what he says.
> I understand your point: solve the BUG, and I do tend to agree with you.
>
> But by design and implementation, the BUG() is just a beginning of the end for dying kernel.
> And what happens in between this 'the beginning' and 'the end' is not less important.
> (because say, on our platform we want to get clean RAMDUMP to analyze what happened, and for that we want to get clean reboot)
I don't see anybody else having any trouble getting crash dumps. I
spent yet another long day just yesterday, rummaging through one.
> Also,
> If somebody's design is to legally Crash the kernel (e.g. where kernel is actually not faulty).
> Then, I do expect that tick/timekeeping framework do its job as long as it can do, and it should do, because kernel is not faulty.
> But in this case it doesn’t handover jiffies incrementing job sanely.
It seems odd to me to use BUG() for what you appear to be using it for..
not that I know exactly what that it mind you, but when you said when
some other gizmo in your box has a problem you crash the kernel, my head
tilted to the side - surely there's a more controlled response possible
than poking the big red self destruct button ;-)
-Mike
next prev parent reply other threads:[~2015-05-08 5:12 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-06 17:27 [KERNEL BUG] do_timer/tick_handover_do_timer 3.10.17 pawandeep oza
2015-05-07 3:22 ` Mike Galbraith
2015-05-07 4:37 ` Oza (Pawandeep) Oza
2015-05-07 5:08 ` Mike Galbraith
2015-05-07 5:12 ` Oza (Pawandeep) Oza
2015-05-07 5:54 ` Mike Galbraith
2015-05-07 5:58 ` Oza (Pawandeep) Oza
2015-05-07 6:54 ` Mike Galbraith
2015-05-07 7:05 ` Oza (Pawandeep) Oza
2015-05-07 8:29 ` Mike Galbraith
2015-05-07 8:47 ` Oza (Pawandeep) Oza
2015-05-08 4:16 ` Oza (Pawandeep) Oza
2015-05-08 5:12 ` Mike Galbraith [this message]
2015-05-08 5:21 ` Oza (Pawandeep) Oza
2015-05-07 8:19 ` Oza (Pawandeep) Oza
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1431061931.3168.41.camel@gmail.com \
--to=umgwanakikbuti@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=malayasen.rout@gmail.com \
--cc=oza.contri.linux.kernel@gmail.com \
--cc=oza@broadcom.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.