"This is a request to merge KGDB into the mainline kernel," Jason Wessel announced, posting a series of patches aiming toward that goal. He continued, "as of right now KGDB is comprised of 21 different patches adding in the core api and docs first and then working up to add drivers and arch specific support to KGDB. The patches were broken down into logical pieces for review and comments." He went on to explain:
"The intent of the KGDB patches is to unify the KGDB support across all the architectures that elect to implement the KGDB functionality by providing a common core and an arch specific stub. For quite some time there has been different features and uses of KGDB across the most popular architectures. Having a common core that takes care of protocol parsing and the typical use case of software breakpoints should eliminate the inconsistencies across the archs as well as making it easier to add KGDB support to a new arch."
Andrew Morton, who has been supportive of getting a kernel debugger into the mainline kernel, explained that it was too late in the 2.6.24 review cycle to merge KGDB, meaning it would have to wait for 2.6.25 at the earliest, "this won't work very well. There's a lot of review work to be done here, and a lot of it by busy architecture maintainers. Expecting people to do all this review and test work late in the merge window when they're all madly scrambling to get their bugs^Wpatches into mainline is not reasonable. This should all have started a month ago. So we're looking at a 2.6.25 merge for this work."
From: Jason Wessel <jason.wessel@...>
Subject: [PATCH 0/21] KGDB: Request to merge KGDB
Date: Oct 15, 2:32 pm 2007
This is a request to merge KGDB into the mainline kernel.
These KGDB patches are against the tip of kernel of the tree on
October 15, 2007. I am continuing to update the for_mm branch against
the tip of the tree with the hope to merge KGDB into the kernel,
pending the review of the individual pieces.
http://git.kernel.org/?p=linux/kernel/git/jwessel/linux-2.6-kgdb.git;a=s...
As of right now KGDB is comprised of 21 different patches adding in
the core api and docs first and then working up to add drivers and
arch specific support to KGDB. The patches were broken down into
logical pieces for review and comments.
The intent of the KGDB patches is to unify the KGDB support across all
the architectures that elect to implement it KGDB functionality by
providing a common core and an arch specific stub. For quite some
time there has been different features and uses of KGDB across the
most popular architectures. Having a common core that takes care of
protocol parsing and the typical use case of software breakpoints
should eliminate the inconsistencies across the archs as well as
making it easier to add KGDB support to a new arch.
These KGDB patches include support for KGDB over ethernet and unify
the syntax and usage for setting up kgdb over rs232.
This series of KGDB patches covers the archs: i386, x86_64, ia64, arm,
mips, sh, ppc and powerpc.
After the merge of this patch set, any other arch can implement KGDB
support in the future by making use of the KGDB core by updating the
kgdb kconfig file and creating the arch specific stub.
Thanks,
Jason
-
From: Andrew Morton <akpm@...>
Subject: Re: [PATCH 0/21] KGDB: Request to merge KGDB
Date: Oct 18, 12:20 am 2007
On Mon, 15 Oct 2007 13:32:24 -0500 Jason Wessel wrote:
> This is a request to merge KGDB into the mainline kernel.
This won't work very well. There's a lot of review work to be done here,
and a lot of it by busy architecture maintainers. Expecting people to do
all this review and test work late in the merge window when they're all
madly scrambling to get their bugs^Wpatches into mainline is not reasonable.
This should all have started a month ago.
So we're looking at a 2.6.25 merge for this work. After this round of
review has settled, please upissue the patches and keep linux-arch (cc'ed
here) copied on everything, thanks.
-
From: Andrew Morton <akpm@...>
Subject: Re: [RFC] remove netpoll receive code
Date: Oct 18, 3:02 am 2007
On Wed, 17 Oct 2007 23:28:38 -0700 (PDT) David Miller wrote:
> From: Stephen Hemminger
> Date: Wed, 17 Oct 2007 13:21:31 -0700
>
> > I propose that we take out all the whole netpoll rx path. If/when
> > kgdb gets submitted a better and alternative receive path can be
> > added.
>
> I would like to kill the RX side handling of netpoll too,
> but I don't think that's reasonable as kgdb is actively
> being pushed for 2.6.25 inclusion.
>
> Andrew is likely to add it to his -mm tree soon and therefore kgdb
> will need to work properly now.
git-kgdb.patch has been in there for ages - maybe a year. Although I
disabled it a week or so ago due to the sheer number of rejects. Will
bring it back when things settle down a bit.
> The RX netpoll thing has a long precedence, it's been in the tree for
> a long time, so we are in some ways stuck with it until we have a
> complete replacement facility. That means we can't yank it out first
> and implement the replacement later.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Will it merge?
Will it get merged in 2.6.25?
Last I heard, Linus did not want a kernel debugger in Linux, because people who need kernel debuggers sucks, and shouldn't be doing kernel development.
Surprisingly (!), kernel
Surprisingly (!), kernel developers also have their personal opinions.
What Linus keeps saying is that he dislikes the idea of kernel debuggers in general — he never said he will veto its merge one once the obstacles have been worked out (splitting it into smaller patches, reviews, coding style, etc). Obviously there are lots of developers who do support merging KGDB.
I would like to see Linus
I would like to see Linus telling other kernel developers like Andrew Morton or others: "Hey, you are using a kernel debugger, so you shouldn't develop code for the kernel" cause that was what he said...
That's our Linus. A
That's our Linus. A self-admitted opinionated bastard. :)