LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Jason Wessel <jason.wessel@windriver.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: kgdb-bugreport@lists.sourceforge.net, linux-kernel@vger.kernel.org
Subject: Re: kgdb 2.6.28 merge plans
Date: Wed, 22 Oct 2008 05:07:33 -0500	[thread overview]
Message-ID: <48FEFB65.3090902@windriver.com> (raw)
In-Reply-To: <20081022093413.23245ab6@lxorguk.ukuu.org.uk>

Alan Cox wrote:
>>       tty: Fix sparse static warning for tty_driver_lookup_tty
>>       kgdboc, tty: Add the rx polling call back capability
>>       kgdboc, 8250: rx polling hook for the 8250 driver
>>       kgdboc, amba-pl011: rx polling hook for the amba-pl011 driver
>
> NAK.

The kgdb merge request queue has had any serial and tty changes
removed.

The only changes that remain are to the kgdb core for the
purpose of moving the blackfin off its existing kgdb stub over to the
common kgdb core API.

>
> In addition to which random hooks in random mixtures of uart drivers are
> not a practical sensible way to manage general purpose interfaces to
> debug tools.
>

This is a discussion that needs to be hashed out.  It is a question of
where would you like it discussed (IE: new thread in lmkl,
linux-serial@...)?

It is a discussion more about general polled vs interrupt driven
serial/tty APIs, not all of which are used for debuggers.  I am open
to help fix/change/extend/refactor the existing kernel's CONSOLE_POLL
API or even deprecate it in favor of something else so long as it
leads to an interface which can allow the sharing of the hardware
among various contexts with early_printk, console, tty, debugger,
etc...

If it was all obvious and easy, it would have already been done.  Rome
was not built in a day after all. :-)

Cheers,
Jason.

  reply	other threads:[~2008-10-22 10:08 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-21 19:14 Jason Wessel
2008-10-21 19:14 ` [PATCH 1/7] kgdb: Add the ability to schedule a breakpoint via a tasklet Jason Wessel
2008-10-21 19:14   ` [PATCH 2/7] tty: Fix sparse static warning for tty_driver_lookup_tty Jason Wessel
2008-10-21 19:14     ` [PATCH 3/7] kgdboc, tty: Add the rx polling call back capability Jason Wessel
2008-10-21 19:14       ` [PATCH 4/7] kgdboc, 8250: Add the rx polling hook to 8250 driver Jason Wessel
2008-10-21 19:14         ` [PATCH 5/7] kgdboc, tty: use tty open to start low level drivers Jason Wessel
2008-10-21 19:14           ` [PATCH 6/7] kgdb: kgdboc console poll hooks for serial_txx9 uart Jason Wessel
2008-10-21 19:14             ` [PATCH 7/7] kgdb: Make mem access function weak in kgdb.c andkgdb.h Jason Wessel
2008-10-21 21:49           ` [PATCH 5/7] kgdboc, tty: use tty open to start low level drivers Jason Wessel
2008-10-21 22:47           ` Alan Cox
2008-10-22  1:54             ` Jason Wessel
2008-10-21 22:49 ` kgdb 2.6.28 merge plans Alan Cox
2008-10-22  2:09   ` Jason Wessel
2008-10-22  8:34     ` Alan Cox
2008-10-22 10:07       ` Jason Wessel [this message]
2008-10-22 12:25         ` Alan Cox

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=48FEFB65.3090902@windriver.com \
    --to=jason.wessel@windriver.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=kgdb-bugreport@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --subject='Re: kgdb 2.6.28 merge plans' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).