The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: George Anzinger <george@mvista.com>
To: Tom Rini <trini@kernel.crashing.org>
Cc: kgdb-bugreport@lists.sourceforge.net,
	Kernel Mailing List <linux-kernel@vger.kernel.org>,
	"Amit S. Kale" <amitkale@emsyssoft.com>,
	ganzinger@mvista.com
Subject: Re: [KGDB] Make kgdb get in sync with it's I/O drivers for the breakpoint
Date: Tue, 06 Apr 2004 13:58:42 -0700	[thread overview]
Message-ID: <40731A02.8090303@mvista.com> (raw)
In-Reply-To: <20040405233058.GV31152@smtp.west.cox.net>

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.
~
-- 
George Anzinger   george@mvista.com
High-res-timers:  http://sourceforge.net/projects/high-res-timers/
Preemption patch: http://www.kernel.org/pub/linux/kernel/people/rml


  parent reply	other threads:[~2004-04-06 21:00 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 [this message]
2004-04-06 21:04   ` Tom Rini

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=40731A02.8090303@mvista.com \
    --to=george@mvista.com \
    --cc=amitkale@emsyssoft.com \
    --cc=ganzinger@mvista.com \
    --cc=kgdb-bugreport@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=trini@kernel.crashing.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