Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-11 Thread Konstantin Belousov
On Wed, Sep 10, 2014 at 10:41:00AM -0400, John Baldwin wrote: > On Tuesday, September 09, 2014 11:58:13 AM Konstantin Belousov wrote: > > On Sat, Sep 06, 2014 at 04:04:49PM -0400, John Baldwin wrote: > > Move of the code to initialize CPU from identcpu() to initializecpu() > > seems to be a fix on

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-10 Thread John Baldwin
On Tuesday, September 09, 2014 11:58:13 AM Konstantin Belousov wrote: > On Sat, Sep 06, 2014 at 04:04:49PM -0400, John Baldwin wrote: > > @@ -692,6 +706,27 @@ > > > > #ifdef I586_CPU > > > > case CPU_586: > > switch (cpu_vendor_id) { > > > > + case CPU_VENDOR_AMD: > >

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-09 Thread Konstantin Belousov
On Sat, Sep 06, 2014 at 04:04:49PM -0400, John Baldwin wrote: > On Friday, September 05, 2014 10:44:05 AM John Baldwin wrote: > > On Friday, September 05, 2014 4:43:05 am Konstantin Belousov wrote: > > > There is one weird detail, not touched by your patch. Amd64 resume > > > path calls initialize

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-06 Thread John Baldwin
On Friday, September 05, 2014 10:44:05 AM John Baldwin wrote: > On Friday, September 05, 2014 4:43:05 am Konstantin Belousov wrote: > > There is one weird detail, not touched by your patch. Amd64 resume > > path calls initializecpu(), while i386 does not. I do not see any > > use for the call, th

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-06 Thread Konstantin Belousov
On Fri, Sep 05, 2014 at 10:44:05AM -0400, John Baldwin wrote: > On Friday, September 05, 2014 4:43:05 am Konstantin Belousov wrote: > > On Thu, Sep 04, 2014 at 10:50:25PM -0400, John Baldwin wrote: > > > On Tuesday, September 02, 2014 06:41:27 PM Konstantin Belousov wrote: > > > > On Tue, Sep 02, 2

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-05 Thread Bruce Evans
On Fri, 5 Sep 2014, John Baldwin wrote: We might also consider removing support for 486sx CPUs and requiring an on-CPU FPU for i386. If we do that we might able to use a common fpu.c which would be even nicer. You mean a common npx.c. The 'x' part of npx.c is much more descriptive now than i

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-05 Thread John Baldwin
On Friday, September 05, 2014 4:43:05 am Konstantin Belousov wrote: > On Thu, Sep 04, 2014 at 10:50:25PM -0400, John Baldwin wrote: > > On Tuesday, September 02, 2014 06:41:27 PM Konstantin Belousov wrote: > > > On Tue, Sep 02, 2014 at 11:00:57AM -0400, John Baldwin wrote: > > > > I thought about t

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-05 Thread Konstantin Belousov
On Thu, Sep 04, 2014 at 10:50:25PM -0400, John Baldwin wrote: > On Tuesday, September 02, 2014 06:41:27 PM Konstantin Belousov wrote: > > On Tue, Sep 02, 2014 at 11:00:57AM -0400, John Baldwin wrote: > > > I thought about that. I could easily make a parallel array, or perhaps > > > use a separate

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-04 Thread John Baldwin
On Tuesday, September 02, 2014 06:41:27 PM Konstantin Belousov wrote: > On Tue, Sep 02, 2014 at 11:00:57AM -0400, John Baldwin wrote: > > I thought about that. I could easily make a parallel array, or perhaps > > use a separate 'susppcb' structure that includes a pcb and the savefpu > > union and

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-02 Thread Konstantin Belousov
On Tue, Sep 02, 2014 at 11:00:57AM -0400, John Baldwin wrote: > On Saturday, August 30, 2014 3:58:09 pm Konstantin Belousov wrote: > > On Sat, Aug 30, 2014 at 05:48:38PM +, John Baldwin wrote: > > > Author: jhb > > > Date: Sat Aug 30 17:48:38 2014 > > > New Revision: 270850 > > > URL: http://sv

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-09-02 Thread John Baldwin
On Saturday, August 30, 2014 3:58:09 pm Konstantin Belousov wrote: > On Sat, Aug 30, 2014 at 05:48:38PM +, John Baldwin wrote: > > Author: jhb > > Date: Sat Aug 30 17:48:38 2014 > > New Revision: 270850 > > URL: http://svnweb.freebsd.org/changeset/base/270850 > > > > Log: > > Save and restor

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-08-30 Thread Konstantin Belousov
On Sat, Aug 30, 2014 at 05:48:38PM +, John Baldwin wrote: > Author: jhb > Date: Sat Aug 30 17:48:38 2014 > New Revision: 270850 > URL: http://svnweb.freebsd.org/changeset/base/270850 > > Log: > Save and restore FPU state across suspend and resume. In earlier revisions > of this patch, res

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-08-30 Thread Adrian Chadd
On 30 August 2014 10:58, John Baldwin wrote: > On Saturday, August 30, 2014 05:48:38 PM John Baldwin wrote: >> Author: jhb >> Date: Sat Aug 30 17:48:38 2014 >> New Revision: 270850 >> URL: http://svnweb.freebsd.org/changeset/base/270850 >> >> Log: >> Save and restore FPU state across suspend and

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-08-30 Thread Alexey Dokuchaev
On Sat, Aug 30, 2014 at 01:58:20PM -0400, John Baldwin wrote: > On Saturday, August 30, 2014 05:48:38 PM John Baldwin wrote: > > New Revision: 270850 > > URL: http://svnweb.freebsd.org/changeset/base/270850 > > > > Log: > > Save and restore FPU state across suspend and resume. In earlier > > re

Re: svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-08-30 Thread John Baldwin
On Saturday, August 30, 2014 05:48:38 PM John Baldwin wrote: > Author: jhb > Date: Sat Aug 30 17:48:38 2014 > New Revision: 270850 > URL: http://svnweb.freebsd.org/changeset/base/270850 > > Log: > Save and restore FPU state across suspend and resume. In earlier > revisions of this patch, resume

svn commit: r270850 - in head/sys: i386/i386 i386/include i386/isa x86/acpica

2014-08-30 Thread John Baldwin
Author: jhb Date: Sat Aug 30 17:48:38 2014 New Revision: 270850 URL: http://svnweb.freebsd.org/changeset/base/270850 Log: Save and restore FPU state across suspend and resume. In earlier revisions of this patch, resumectx() called npxresume() directly, but that doesn't work because resumect