This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: PATCH: Fix 32bit coredump read on Linux/AVX
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Mark Kettenis <mark dot kettenis at xs4all dot nl>
- Cc: GDB <gdb-patches at sourceware dot org>
- Date: Wed, 21 Apr 2010 07:32:30 -0700
- Subject: Re: PATCH: Fix 32bit coredump read on Linux/AVX
- References: <20100420032234.GA10544@intel.com> <20100420184339.GA18641@intel.com>
On Tue, Apr 20, 2010 at 11:43 AM, H.J. Lu <hongjiu.lu@intel.com> wrote:
> On Mon, Apr 19, 2010 at 08:22:34PM -0700, H.J. Lu wrote:
>> Hi,
>>
>> This patch:
>>
>> http://sourceware.org/ml/gdb-patches/2010-04/msg00276.html
>>
>> breaks 32bit coredump read on Linux/AVX since we only dump .reg and
>> .reg-xstate sections on AVX. But i386_linux_core_read_description
>> checks .reg2 and .reg-xfp sections first. Since there are no
>> .reg2 and .reg-xfp sections, NULL is returned and SSE target is used.
>> This patch changes the section check order to .reg-xstate, .reg-xfp,
>> .reg2. ?OK to install?
>>
>> Thanks.
>>
>
> .reg2 section has x87 regiters on i386 and SSE registers on amd64.
> Here is the updated patch to properly handle it. ?OK to install?
>
> Thanks.
>
>
> H.J.
> --
> 2010-04-20 ?H.J. Lu ?<hongjiu.lu@intel.com>
>
> ? ? ? ?PR corefiles/11523
> ? ? ? ?* amd64-linux-tdep.c (amd64_linux_core_read_description): Check
> ? ? ? ?XCR0 first.
>
> ? ? ? ?* i386-linux-tdep.c (i386_linux_core_read_xcr0): Return 0 if
> ? ? ? ?there is no .reg-xstate section.
> ? ? ? ?(i386_linux_core_read_description): Check XCR0 first.
>
Any suggestions to this patch? Mark, can you comment on this?
Thanks.
--
H.J.