This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH: PR corefiles/11467: amd64 gdb generates corrupted 32bit core file


> Date: Tue, 13 Apr 2010 10:39:24 -0700
> From: "H.J. Lu" <hjl.tools@gmail.com>
> 
> On Tue, Apr 13, 2010 at 10:26 AM, Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
> >> Date: Tue, 13 Apr 2010 10:17:57 -0700
> >> From: "H.J. Lu" <hongjiu.lu@intel.com>
> >>
> >> On Mon, Apr 12, 2010 at 06:22:25AM -0700, H.J. Lu wrote:
> >> > On Sun, Apr 11, 2010 at 01:52:50PM -0700, H.J. Lu wrote:
> >> > > Hi,
> >> > >
> >> > > Thanks for Mark's pointer. Solution is very simple. We just need to
> >> > > make sure that we call the right fill_gregset for 32bit executable
> >> > > on both Linux/x86-64 and Linux/i386. ?OK to install?
> >> > >
> >> > > Thanks.
> >> > >
> >> > >
> >> >
> >> > Small update to use tdep->gregset_reg_offset instead of
> >> > i386_linux_gregset_reg_offset. ?OK to install?
> >> >
> >>
> >>
> >> Here is the updated patch. It calls set_gdbarch_regset_from_core_section
> >> with i386_linux_regset_from_core_section. ?OK to install?
> >
> >
> > Sorry, no. ?You're adding far too much bloat.
> >
> 
> Can you be more specific?

Yup; sent an explanation as a reply to one of your earlier mail.  Just
wanted to make sure you didn't dig yourself an even bigger hole while
I was looking at some of the details here.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]