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

[Mark Kettenis <mark.kettenis@xs4all.nl>]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1230 bytes --]

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