Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 9 Jan 2012 22:22:14 +0200
From:      Kostik Belousov <kostikbel@gmail.com>
To:        Yamagi Burmeister <lists@yamagi.org>
Cc:        amd64@freebsd.org, arch@freebsd.org, marius@freebsd.org, flo@freebsd.org
Subject:   Re: AVX
Message-ID:  <20120109202214.GZ31224@deviant.kiev.zoral.com.ua>
In-Reply-To: <20120109103747.578d4e44.lists@yamagi.org>
References:  <20111226220756.GR50300@deviant.kiev.zoral.com.ua> <20120102063700.GF50300@deviant.kiev.zoral.com.ua> <20120108174112.50e030ba.lists@yamagi.org> <20120108195913.GI31224@deviant.kiev.zoral.com.ua> <20120109103747.578d4e44.lists@yamagi.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--9C8JZALFROMdO0HE
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Jan 09, 2012 at 10:37:47AM +0100, Yamagi Burmeister wrote:
> First, thank you for working on AVX, it's much appreciated.
>=20
> On Sun, 8 Jan 2012 21:59:13 +0200
> Kostik Belousov <kostikbel@gmail.com> wrote:
>=20
> > > CPU: Intel(R) Core(TM)2 Duo CPU     T6670  @ 2.20GHz (2194.55-MHz
> > > K8-class CPU) Origin =3D "GenuineIntel"  Id =3D 0x1067a  Family =3D 6=
  Model
> > >   =3D 17  Stepping =3D 10
> > > Features=3D0xbfebfbff<FPU,VME,DE,PSE,TSC,MSR,PAE,MCE,CX8,APIC,SEP,MTR=
R,PGE,
> > >   MCA,CMOV,PAT,PSE36,CLFLUSH,DTS,ACPI,MMX,FXSR,SSE,SSE2,SS,HTT,TM,PBE>
> > > Features2=3D0x408e3bd<SSE3,DTES64,MON,DS_CPL,VMX,EST,TM2,SSSE3,CX16,x=
TPR,
> > >   PDCM,SSE4.1,XSAVE>
> > > AMD Features=3D0x20100800<SYSCALL,NX,LM> AMD Features2=3D0x1<LAHF>=20
> > Is this Features excerpt from the patched kernel, or from pristine svn
> > sources ? If the later, please show me the Features from the patched
> > kernel.
>=20
> That was the output of the patched kernel.
I see. The issue was that cpu_feature2, which is decoded to print the
Features2 line, is retrieved much earlier then XCR0 is updated by
fpu initialization code. I added a cludge to reload cpu_feature2 if
XSAVE was indeed enabled. So now you should see OSXSAVE reported if
XSAVE is indeed enabled.

>=20
> > I thought that I correctly handled savectx, but apparently I did not.
> > The issue for sleep enter could be fixed by the avx.4.patch, I am not
> > sure about shutdown -r panic.
> >=20
> > http://people.freebsd.org/~kib/misc/avx.4.patch
>=20
> Both panics are gone. The system goes into suspend just fine and even
> resumes. And no more panics at reboot.
Very good, thank you for testing.

I decided to avoid use of __fillcontextx() for !x86 architectures for now,
since nobody seems to know what happens on sparc64. Florian, can you,
please, retest the defer_sig to see if it works now ?

I hope that this is commit candidate:
http://people.freebsd.org/~kib/misc/avx.5.patch

--9C8JZALFROMdO0HE
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (FreeBSD)

iEYEARECAAYFAk8LTHYACgkQC3+MBN1Mb4jcFgCeL5DpuUjELfEVHpjT3IS9h3VE
hC4AoM/xGLzjJD04zenoDKCWjlYwFLCy
=++UU
-----END PGP SIGNATURE-----

--9C8JZALFROMdO0HE--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120109202214.GZ31224>