From: Tom Rini <trini@kernel.crashing.org>
To: ganzinger@mvista.com
Cc: kgdb-bugreport@lists.sourceforge.net,
Kernel Mailing List <linux-kernel@vger.kernel.org>,
"Amit S. Kale" <amitkale@emsyssoft.com>
Subject: Re: [KGDB] Make kgdb get in sync with it's I/O drivers for the breakpoint
Date: Tue, 6 Apr 2004 14:04:59 -0700 [thread overview]
Message-ID: <20040406210459.GC31152@smtp.west.cox.net> (raw)
In-Reply-To: <40731A02.8090303@mvista.com>
On Tue, Apr 06, 2004 at 01:58:42PM -0700, George Anzinger wrote:
> Tom Rini wrote:
> >Hello. The following interdiff, vs current kgdb-2 CVS makes kgdb core
> >and I/O drivers get in sync in order to cause a breakpoint. This kills
> >off the init/main.c change, and makes way for doing things much earlier,
> >if other support exists. What would be left, tangentally, is some sort
> >of queue to register with, so we can handle the case of KGDBOE on a
> >pcmcia card. George? Amit? Comments ?
>
> Well a simple but dumb way is to poll using the timer list, i.e. set up a
> timer at the first entry were things "might" work and if the driver is not
> yet, do a timer to come back in 1 tick, and keep doing it for each tick
> until it is available. This puts it all on the kgdb side.
>
> The other way is with a call back list which would be managed by common OE
> code. This would put most of the code in that area. I tend to like call
> back lists that one registers for by passing in a structure which contains
> a "list_head" member. That way there is no memory allocation on either
> end. The manager, on a register call, just puts the new structure in its
> call back list. The struct would have the list_head member and a function
> member, and the function would be called with the struct address as its
> only parameter. This allows for an expanded struct if more complex info is
> needed.
Setting aside this problem for a minute (since what I posted does get
the job done, just not 100% clean), what do you think about the rest of
the changes?
--
Tom Rini
http://gate.crashing.org/~trini/
prev parent reply other threads:[~2004-04-06 21:10 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-04-05 23:30 [KGDB] Make kgdb get in sync with it's I/O drivers for the breakpoint Tom Rini
2004-04-06 14:51 ` [Kgdb-bugreport] " Stelian Pop
2004-04-06 14:57 ` Tom Rini
2004-04-06 15:30 ` Stelian Pop
2004-04-06 20:58 ` George Anzinger
2004-04-06 21:04 ` Tom Rini [this message]
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=20040406210459.GC31152@smtp.west.cox.net \
--to=trini@kernel.crashing.org \
--cc=amitkale@emsyssoft.com \
--cc=ganzinger@mvista.com \
--cc=kgdb-bugreport@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
/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