Re: [PATCH 1/8] kgdb: core API and gdb protocol handler

2008/2/9 Ray Lee <[EMAIL PROTECTED]>: > On Feb 9, 2008 9:27 AM, Christoph Hellwig <[EMAIL PROTECTED]> wrote: > > On Sat, Feb 09, 2008 at 07:35:07AM -0600, [EMAIL PROTECTED] wrote: > > > +#ifdef __BIG_ENDIAN > > > + *buf++ = hexchars[(tmp_s >> 12) & 0xf]; > > > + *buf++ = hex

Re: [PATCH 1/8] kgdb: core API and gdb protocol handler

On Feb 9, 2008 9:27 AM, Christoph Hellwig <[EMAIL PROTECTED]> wrote: > On Sat, Feb 09, 2008 at 07:35:07AM -0600, [EMAIL PROTECTED] wrote: > > +#ifdef __BIG_ENDIAN > > + *buf++ = hexchars[(tmp_s >> 12) & 0xf]; > > + *buf++ = hexchars[(tmp_s >> 8) & 0xf]; > > + *bu

Re: [PATCH 1/8] kgdb: core API and gdb protocol handler

On Sat, Feb 09, 2008 at 07:35:07AM -0600, [EMAIL PROTECTED] wrote: > index 000..24e0b6c > --- /dev/null > +++ b/include/asm-generic/kgdb.h > @@ -0,0 +1,105 @@ > +/* > + * include/asm-generic/kgdb.h Please don't mention the file name in the top-of-file comments. This information is redundant a

Re: [PATCH 1/8] kgdb: core API and gdb protocol handler

Hi Jason. > --- /dev/null > +++ b/kernel/kgdb.c > + > +struct debuggerinfo_struct { > + void*debuggerinfo; > + struct task_struct *task; > +} kgdb_info[NR_CPUS]; static? > + > +/* Is a host GDB connected to us? */ > +int kgdb_connected; >

Re: [PATCH 1/8] kgdb: core API and gdb protocol handler

[EMAIL PROTECTED] wrote: > --- /dev/null > +++ b/include/asm-generic/kgdb.h > @@ -0,0 +1,105 @@ > +/* > + * include/asm-generic/kgdb.h > + * > + * This provides the assembly level information so that KGDB can provide > + * a GDB that has been patched with enough information to know to stop > + * tr