From owner-freebsd-hackers Wed May 2 15:13:38 2001 Delivered-To: freebsd-hackers@freebsd.org Received: from meow.osd.bsdi.com (meow.osd.bsdi.com [204.216.28.88]) by hub.freebsd.org (Postfix) with ESMTP id 158F237B422 for ; Wed, 2 May 2001 15:13:35 -0700 (PDT) (envelope-from jhb@FreeBSD.org) Received: from laptop.baldwin.cx (john@jhb-laptop.osd.bsdi.com [204.216.28.241]) by meow.osd.bsdi.com (8.11.2/8.11.2) with ESMTP id f42MCkG47845; Wed, 2 May 2001 15:13:16 -0700 (PDT) (envelope-from jhb@FreeBSD.org) Message-ID: X-Mailer: XFMail 1.4.0 on FreeBSD X-Priority: 3 (Normal) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8bit MIME-Version: 1.0 In-Reply-To: <20010502164749.E19547@guild.guildsoftware.com> Date: Wed, 02 May 2001 15:12:04 -0700 (PDT) From: John Baldwin To: Andy Sloane Subject: Re: new syscons screensaver Cc: freebsd-hackers@FreeBSD.org, Andrew Hesford , Nick Sayer Sender: owner-freebsd-hackers@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG On 02-May-01 Andy Sloane wrote: > On Wed, May 02, 2001 at 01:58:06PM -0700, Nick Sayer wrote: >> You might consider trying the apm_saver module instead, especially on >> laptops. I have found instances in the past of monitor/BIOS/video >> combinations where green didn't do the right thing but that the APM got >> right. > > Both apm_saver and green_saver are essentially (once you get past all the > screensaver setup code) one-line routines to turn off the display. I think > it would make sense to both tell the VGA chip to blank (green_saver) and > tell APM to shut down (apm_saver) with proper #ifdeffing around the APM > stuff. > > I also think it would make sense to do this inside dev/fb/splash.c (or > perhaps dev/syscons/syscons.c) instead of inside each screensaver, so that > it's automatically handled. And maybe a new 'vidcontrol' option to set the > blanking delay (as opposed to the screensaver delay) would be necessary. > There may be an even better place to put this, but I am mostly unfamiliar > with the kernel source tree and as such I won't attempt to write a patch > for this yet. You could just add new ioctl's to do this the same way that vidcontrol does settings now. This approach does sound the most sensible to me, FWIW, as it avoids duplicating code all over the place. -- John Baldwin -- http://www.FreeBSD.org/~jhb/ PGP Key: http://www.baldwin.cx/~john/pgpkey.asc "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-hackers" in the body of the message