Re: [PATCH 0/21] KGDB: Request to merge KGDB

2007-10-24 Thread Pavel Machek
Hi!

> > 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.

_If_ the code is properly bisectable, it should be possible to merge
core now and then switch architectures lazily. (If it is not
bisectable, it should be fixed).
Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/21] KGDB: Request to merge KGDB

2007-10-19 Thread Russell King
On Thu, Oct 18, 2007 at 05:00:50PM +0100, Christoph Hellwig wrote:
> On Thu, Oct 18, 2007 at 04:54:29PM +0100, Russell King wrote:
> > What would be good is if some effort could be made by the utrace-
> > interested parties to make the transition to utrace much less painful.
> > For instance, I quite like the getregs/setregs abstractions, and it
> > looks like these *could* be self-contained in a single patch.  It
> > would be nice if we could move architectures over to this one a time.
> > Once that's in, that's one chunk of utrace merged.
> 
> Alexey Dobriyan had some patches to implement regsets, and these were
> posted here on lkml.

Given that not all architecture maintainers read lkml, and they are
patches affecting all *architectures*, wouldn't it have been a good
idea if they'd copied linux-arch?

-- 
Russell King
 Linux kernel2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/21] KGDB: Request to merge KGDB

2007-10-18 Thread Christoph Hellwig
On Thu, Oct 18, 2007 at 04:54:29PM +0100, Russell King wrote:
> What would be good is if some effort could be made by the utrace-
> interested parties to make the transition to utrace much less painful.
> For instance, I quite like the getregs/setregs abstractions, and it
> looks like these *could* be self-contained in a single patch.  It
> would be nice if we could move architectures over to this one a time.
> Once that's in, that's one chunk of utrace merged.

Alexey Dobriyan had some patches to implement regsets, and these were
posted here on lkml.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/21] KGDB: Request to merge KGDB

2007-10-18 Thread Russell King
On Thu, Oct 18, 2007 at 04:25:53PM +0100, Christoph Hellwig wrote:
> On Fri, Oct 19, 2007 at 12:23:10AM +0900, Paul Mundt wrote:
> > It would also be nice to balance this out with the utrace merge, as both
> > are going to cause quite a lot of pain on the architecture side.
> 
> I don't think a utrace merge as in one big merge is going to happen ever.
> It's just a too big patch doing too many things at once.  And the flag day
> for switching all architectures over is another blocker.

I agree with Christoph - the fact that *all* architectures have to be
either ptrace or utrace make it very very painful.

What would be good is if some effort could be made by the utrace-
interested parties to make the transition to utrace much less painful.
For instance, I quite like the getregs/setregs abstractions, and it
looks like these *could* be self-contained in a single patch.  It
would be nice if we could move architectures over to this one a time.
Once that's in, that's one chunk of utrace merged.

-- 
Russell King
 Linux kernel2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/21] KGDB: Request to merge KGDB

2007-10-18 Thread Christoph Hellwig
On Fri, Oct 19, 2007 at 12:23:10AM +0900, Paul Mundt wrote:
> It would also be nice to balance this out with the utrace merge, as both
> are going to cause quite a lot of pain on the architecture side.

I don't think a utrace merge as in one big merge is going to happen ever.
It's just a too big patch doing too many things at once.  And the flag day
for switching all architectures over is another blocker.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/21] KGDB: Request to merge KGDB

2007-10-18 Thread Paul Mundt
On Wed, Oct 17, 2007 at 09:20:25PM -0700, Andrew Morton wrote:
> On Mon, 15 Oct 2007 13:32:24 -0500 Jason Wessel <[EMAIL PROTECTED]> 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.
> 
It would also be nice to balance this out with the utrace merge, as both
are going to cause quite a lot of pain on the architecture side.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/21] KGDB: Request to merge KGDB

2007-10-17 Thread Andrew Morton
On Mon, 15 Oct 2007 13:32:24 -0500 Jason Wessel <[EMAIL PROTECTED]> 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.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 0/21] KGDB: Request to merge KGDB

2007-10-15 Thread Jason Wessel
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=shortlog;h=for_mm


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
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/