[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080804062823.GA22015@atrey.karlin.mff.cuni.cz>
Date: Mon, 4 Aug 2008 08:28:23 +0200
From: Pavel Machek <pavel@...e.cz>
To: Ben Collins <ben.collins@...onical.com>
Cc: Greg KH <greg@...ah.com>, "Rafael J. Wysocki" <rjw@...k.pl>,
linux-kernel@...r.kernel.org,
pm list <linux-pm@...ts.linux-foundation.org>
Subject: Re: [PATCH 6/7] pm: Config option to disable handling of console during suspend/resume
Hi!
> > > diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
> > > index e7bea3e..6a042f8 100644
> > > --- a/Documentation/kernel-parameters.txt
> > > +++ b/Documentation/kernel-parameters.txt
> > > @@ -2247,6 +2247,11 @@ and is between 256 and 4096 characters. It is defined in the file
> > > vmpoff= [KNL,S390] Perform z/VM CP command after power off.
> > > Format: <command>
> > >
> > > + vt_switch= [PM] Switch vt's during suspend/resume
> > > + Format: { 0 | 1 }
> > > + Default is normally '1', but depends on value of
> > > + the PM_DISABLE_CONSOLE config option.
> > > +
> >
> > Why a build time config option? Why not just set the command line
> > option if you need this? And/or make it a sysctrl?
> >
> > That way we don't have to add yet-another-config-option for distros and
> > users to get wrong.
> >
> > thanks,
>
> Sysctl makes sense. I would rather not have to add a cmdline option to
> default installs (and on upgrades). I'll redo the patch that way and
> resend.
Maybe have acpi_sleep=no_vt_switch? THat is already settable from
userspace...?
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists