Date: Thu, 1 Oct 1998 16:07:39 -0500 From: Jonathan Lemon <jlemon@americantv.com> To: Mike Smith <mike@smith.net.au> Cc: sos@FreeBSD.ORG, adhir@worldbank.org, yokota@FreeBSD.ORG, cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/i386/include console.h src/sys/i386/isa scvesactl.c scvidctl.c syscons.c syscons.h vesa.c videoio.c Message-ID: <19981001160739.01031@right.PCS> In-Reply-To: <199810012059.NAA01433@dingo.cdrom.com>; from Mike Smith on Oct 10, 1998 at 01:59:53PM -0700 References: <19981001154049.29400@right.PCS> <199810012059.NAA01433@dingo.cdrom.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Oct 10, 1998 at 01:59:53PM -0700, Mike Smith wrote: > > On Oct 10, 1998 at 10:32:33PM +0200, Søren Schmidt wrote: > > > In reply to Mike Smith who wrote: > > > > > > It works here, but I still have a patch in kern_clock, but I dont think > > > its needed anymore... > > > > > > > There were some significant VM86-and-SMP fixes committed yesterday. > > > > I'd be inclined to try it again, certainly. > > > > > > > > > Any idea if this works in SMP now? > > > > I have one more patch to vm86bios.s, which I'll commit in a moment. > > The patch to kern_clock.c is needed if you are using the statistics > > clock. (Actually, Tor suggested patching machine/cpu.h, in order to > > make it dependent on opt_vm86.h, so there is no overhead on the alpha.) > > Is the issue likely to apply to SMP alpha systems? If so, it's > probably a reasonable investment in the future. No, it's specific to vm86, not SMP. Calling the BIOS implies that we are running in usermode, and that there is a current process context. This is not true if, for example, syscons happens to make a VESA call when the processor is in the idle loop. This causes problems with the clock code, which says, "okay, you were in user mode, I'm going to dereference p-> now to update some stats." :-( -- Jonathan
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?19981001160739.01031>