mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Dmitry Torokhov <dtor_core@ameritech.net>
To: davem@redhat.com
Cc: linux-kernel@vger.kernel.org, vojtech@suze.cz
Subject: RE: Why pass pt_regs throughout the input system?
Date: Wed, 12 May 2004 11:50:26 -0700 (PDT)	[thread overview]
Message-ID: <20040512185026.28373.qmail@web80508.mail.yahoo.com> (raw)

Hi David,

First off sorry for breaking the thread but I am insteresed in the
topic and writing this mail via a web-based interface.

> On Wed, 12 May 2004 10:40:56 +0200
> Vojtech Pavlik <vojtech@suse.cz> wrote:
> 
> > Ask David S. Miller for details - I think the problem was with
> > simultaneous invocation of multiple pt_regs printouts.
> 
> That's correct, if i'm using multiple keyboards (say one i8042 based
> and one USB based) in order to get fancy debugging dumps, any scheme
> that saves away info at interrupt time simply will be inaccurate and
> not work.

I understand that but is the keyboard interrupt data is the most
interesting? I thought that the "middle" part of the call trace is
much more important as it shows what is happening to the system.
Or am I missing something?

As far as multiple keyboards issue going - SysRq is a debug tool and
I relly do not see you hitting SysRq on the two keyboards at the very
same time to mess up the call traces.

Anyway, my "problem" is the following: SysRq register dump and call
trace require keyboard.c event handler to be caled from hard interrupt
context which is not always feasible. If it is called from a tasklet
even if I save registers I will not be able to reconstruct the call
trace. I wonder if the following change is acceptable:

- SysRq posts a request for registers and call trace to be printed
- When next interrupt comes in (any interrupt) do_IRQ will check the
  flag and print registes and the trace. The check can be made pretty
  cheap as we can coarsely check if it is set without any locks and
  only if it is set do the fine-grained locking stuff so it processed
  only one. If we miss the flag in one interrupt it is not a big deal
  as it will be noticed when the next interrupt arrives.

This is assuming that we are actually interested in what system was
doing _besides_ processing keyboard interrupt.

Dmitry




             reply	other threads:[~2004-05-12 18:50 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-05-12 18:50 Dmitry Torokhov [this message]
2004-05-12 19:10 ` David S. Miller
  -- strict thread matches above, loose matches on Subject: below --
2004-05-12 20:01 Dmitry Torokhov
2004-05-12  4:04 Dmitry Torokhov
2004-05-12  8:40 ` Vojtech Pavlik
2004-05-12 18:15   ` David S. Miller

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=20040512185026.28373.qmail@web80508.mail.yahoo.com \
    --to=dtor_core@ameritech.net \
    --cc=davem@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=vojtech@suze.cz \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®