Re: PATCH: 3/6 [3rd try]: Add AVX support (i386 changes)

H.J. Lu
2010-04-07T14:56:00Z

On Wed, Apr 7, 2010 at 3:13 AM, Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
>>
>> XSAVE is different from FXSAVE in some subtle ways, although
>> XSAVE memory layout is an extension to FXSAVE memory layout.
>> XSAVE has used or initialized states for SSE and AVX registers.
>> Most of the codes in i387_collect_xsave deal with used/initialized states.
>>
>> Please identify the duplication of code in i387_collect_xsave. I will take
>> a look.
>
> There is in if (gcore) { } else { } there, that has quite a bit of
> duplicated code.  I may be missing something, but the fact that
> i387_collect_xsave() does different things whether it is generating a
> core file or not seems to be undesirable and wrong to me.
>

I will take a look.

Thanks.

--
H.J.