Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753254AbYCLOpT (ORCPT ); Wed, 12 Mar 2008 10:45:19 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751483AbYCLOpH (ORCPT ); Wed, 12 Mar 2008 10:45:07 -0400 Received: from outbound-dub.frontbridge.com ([213.199.154.16]:18721 "EHLO outbound7-dub-R.bigfish.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751471AbYCLOpF (ORCPT ); Wed, 12 Mar 2008 10:45:05 -0400 X-BigFish: VP X-MS-Exchange-Organization-Antispam-Report: OrigIP: 139.95.251.11;Service: EHS X-WSS-ID: 0JXMGYF-03-K43-01 Date: Wed, 12 Mar 2008 08:45:06 -0600 From: Jordan Crouse To: Pavel Machek Cc: akpm@linux-foundation.org, kernel list , Linux-pm mailing list , mm-commits@vger.kernel.org, dilinger@queued.net, adaplas@pol.net, dilinger@debian.org, rjw@sisk.pl Subject: Re: + pm-gxfb-add-hook-to-pm-console-layer-that-allows-disabling-of-suspend-vt-switch.patch added to -mm tree Message-ID: <20080312144506.GE1018@cosmic.amd.com> References: <200803120527.m2C5RmWl011967@imap1.linux-foundation.org> <20080312085149.GA3993@ucw.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20080312085149.GA3993@ucw.cz> User-Agent: Mutt/1.5.15+20070412 (2007-04-11) X-OriginalArrivalTime: 12 Mar 2008 14:43:51.0832 (UTC) FILETIME=[7D3A6180:01C8844F] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2571 Lines: 62 On 12/03/08 09:51 +0100, Pavel Machek wrote: > Hi! > > > Subject: PM/gxfb: add hook to PM console layer that allows disabling of suspend VT switch > > From: Andres Salomon > > > > Prior to suspend, we allocate and switch to a new VT; after suspend, we switch > > back to the original VT. This can be slow, and is completely unnecessary if > > the framebuffer we're using can restore video properly. > > > > This adds a hook that allows drivers to select whether or not to do this vt > > switch, and changes the gxfb driver to call this hook. It also adds a module > > param to gxfb to allow controlling of the vt switch (defaulting to no switch). > > > > (Note: I'm not convinced that console_sem is the best way to protect this, but > > we should probably have some form of locking..) > > I guess this is okay for now, but we probably want to make it more > elaborate in future. > > Console switch is there to make sure kernel (not X) owns the graphical > hardware. > > That is unneccessary for gxfb since X never owns graphics hardware, > good. (I do not see why it is optional, then. Do you really want to > see tty1?) Its the dream for the lxfb and gxfb to own the graphics hardware completely (at least, thats what Jim Gettys keeps pounding in to my brain), but we're not quite there yet. What we have here are framebuffer drivers that are intelligent enough to save and restore the state of the graphics engine, which is still a lot better then most other drivers. > Now, question is what happens with two graphics cards, one of them > driven by X. Fortunately that is uncommon. We would be screwed. But since the Geode has built in graphics logic, it doesn't make much sense to use an external card, so we'll mostly avoid this problem. But even if it happens, we'll have the VT switch available in a pinch. > Also, it would be nice to make the logic the other way around. Move vt > switching to the drivers that need it because X can be expected to own > the hardware. Conservation of energy. Its easier to make 3 or 4 drivers opt out of VT switching, rather then hacking 64 other drivers to opt in. Hopefully more people will fix their drivers and make this statement false.. :) Jordan -- Jordan Crouse Systems Software Development Engineer Advanced Micro Devices, Inc. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/