On Friday, November 02, 2012 04:29:37 PM Jesse Barnes wrote: > On Fri, 02 Nov 2012 22:51:07 +0100 > "Rafael J. Wysocki" <rjw at sisk.pl> wrote: > > > On Friday, November 02, 2012 02:43:39 PM Jesse Barnes wrote: > > > I've lightly tested this with X and it definitely makes my > > > suspend/resume sequence a bit prettier. It should speed things up > > > trivally as well, but most of those gains come from other changes to the > > > i915 driver (posted earlier to intel-gfx). > > > > > > Any thoughts? > > > > I like the idea. > > > > > I suspect we'll have to be more defensive about the > > > resume configuration in case the BIOS did something weird, but overall I > > > think we should be able to do this one way or another. > > > > Perhaps patch [1/2] should be [2/2] and vice versa? :-) > > But then it wouldn't compile? I added the variable first, defaulting > to the current behavior, then made i915 support it and set the variable > to false there... At least, that's what I intended to do. OK So what happens if there are multiple graphics adapters in the system? Including such that aren't handled by i915? pm_vt_switch is global, so isn't there any problem with that? Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.