This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: PATCH: PR corefiles/11481: gcore doesn't work on i386 without SSE
> Date: Fri, 9 Apr 2010 09:01:01 -0700
> From: "H.J. Lu" <hjl.tools@gmail.com>
>
> On Fri, Apr 9, 2010 at 8:59 AM, Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
> >> Date: Fri, 9 Apr 2010 08:50:12 -0700
> >> From: "H.J. Lu" <hongjiu.lu@intel.com>
> >>
> >> Hi,
> >>
> >> This patch fixes gcore by not generating core regset sections which
> >> aren't supported by i386 without SSE. ?OK to install?
> >
> > Sorry, I think this is the wrong way to fix this.
> >
>
> Any suggestions?
Yup. This makes sure we always install a list of register notes that
matches the target description.
2010-04-09 Mark Kettenis <kettenis@gnu.org>
* i386-linux-tdep.c (i386_linux_regset_sections): Remove extended
register note sections.
(i386_linux_sse_regset_sections, i386_linux_avx_regset_sections):
New variables.
(i386_linux_init_abi): Install list of supported register note
sections that matches the target description.
Index: i386-linux-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/i386-linux-tdep.c,v
retrieving revision 1.74
diff -u -p -r1.74 i386-linux-tdep.c
--- i386-linux-tdep.c 8 Apr 2010 22:32:36 -0000 1.74
+++ i386-linux-tdep.c 9 Apr 2010 19:05:25 -0000
@@ -58,7 +58,19 @@ static struct core_regset_section i386_l
{
{ ".reg", 144, "general-purpose" },
{ ".reg2", 108, "floating-point" },
+ { NULL, 0 }
+};
+
+static struct core_regset_section i386_linux_sse_regset_sections[] =
+{
+ { ".reg", 144, "general-purpose" },
{ ".reg-xfp", 512, "extended floating-point" },
+ { NULL, 0 }
+};
+
+static struct core_regset_section i386_linux_avx_regset_sections[] =
+{
+ { ".reg", 144, "general-purpose" },
{ ".reg-xstate", I386_XSTATE_MAX_SIZE, "XSAVE extended state" },
{ NULL, 0 }
};
@@ -862,7 +874,12 @@ i386_linux_init_abi (struct gdbarch_info
svr4_fetch_objfile_link_map);
/* Install supported register note sections. */
- set_gdbarch_core_regset_sections (gdbarch, i386_linux_regset_sections);
+ if (tdesc_find_feature (tdesc, "org.gnu.gdb.i386.avx"))
+ set_gdbarch_core_regset_sections (gdbarch, i386_linux_avx_regset_sections);
+ else if (tdesc_find_feature (tdesc, "org.gnu.gdb.i386.sse"))
+ set_gdbarch_core_regset_sections (gdbarch, i386_linux_sse_regset_sections);
+ else
+ set_gdbarch_core_regset_sections (gdbarch, i386_linux_regset_sections);
set_gdbarch_core_read_description (gdbarch,
i386_linux_core_read_description);