All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] vt: disable console blanking by default
@ 2014-06-18  9:27 Tomasz Torcz
  2014-06-22 16:37 ` Pavel Machek
  2014-06-25 12:39 ` One Thousand Gnomes
  0 siblings, 2 replies; 7+ messages in thread
From: Tomasz Torcz @ 2014-06-18  9:27 UTC (permalink / raw)
  To: Greg Kroah-Hartman; +Cc: Jiri Slaby, Tomasz Torcz, linux-kernel


>From ea7cf47e3230eda63aa6c46092719437f9bbae8e Mon Sep 17 00:00:00 2001
From: Tomasz Torcz <tomek@pipebreaker.pl>
Date: Wed, 18 Jun 2014 11:18:06 +0200
Subject: [PATCH] vt: disable console blanking by default

  Remove default 10 minute blank interval. Instead: never blank by
  default.
  "Screensaving" is no longer useful.  Today it only provides
  obstacle when interacting with text console, especially through
  remote lights-out management solutions.

Signed-off-by: Tomasz Torcz <tomek@pipebreaker.pl>
---
 Documentation/kernel-parameters.txt | 4 ++--
 drivers/tty/vt/vt.c                 | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
index 6eaa9cd..0618648 100644
--- a/Documentation/kernel-parameters.txt
+++ b/Documentation/kernel-parameters.txt
@@ -710,8 +710,8 @@ bytes respectively. Such letter suffixes can also be entirely omitted.
 		For now, only VisioBraille is supported.
 
 	consoleblank=	[KNL] The console blank (screen saver) timeout in
-			seconds. Defaults to 10*60 = 10mins. A value of 0
-			disables the blank timer.
+			seconds. A value of 0 disables the blank timer.
+			Defaults to never blank.
 
 	coredump_filter=
 			[KNL] Change the default value for
diff --git a/drivers/tty/vt/vt.c b/drivers/tty/vt/vt.c
index 5e0f6ff..dbd4f23 100644
--- a/drivers/tty/vt/vt.c
+++ b/drivers/tty/vt/vt.c
@@ -176,7 +176,7 @@ int console_blanked;
 
 static int vesa_blank_mode; /* 0:none 1:suspendV 2:suspendH 3:powerdown */
 static int vesa_off_interval;
-static int blankinterval = 10*60;
+static int blankinterval;
 core_param(consoleblank, blankinterval, int, 0444);
 
 static DECLARE_WORK(console_work, console_callback);
-- 
1.9.3

-- 
Tomasz Torcz                Only gods can safely risk perfection,
xmpp: zdzichubg@chrome.pl     it's a dangerous thing for a man.  -- Alia


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] vt: disable console blanking by default
  2014-06-18  9:27 [PATCH] vt: disable console blanking by default Tomasz Torcz
@ 2014-06-22 16:37 ` Pavel Machek
  2014-06-25 11:41   ` Tomasz Torcz
  2014-06-25 12:39 ` One Thousand Gnomes
  1 sibling, 1 reply; 7+ messages in thread
From: Pavel Machek @ 2014-06-22 16:37 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Jiri Slaby, linux-kernel

On Wed 2014-06-18 11:27:31, Tomasz Torcz wrote:
> 
> >From ea7cf47e3230eda63aa6c46092719437f9bbae8e Mon Sep 17 00:00:00 2001
> From: Tomasz Torcz <tomek@pipebreaker.pl>
> Date: Wed, 18 Jun 2014 11:18:06 +0200
> Subject: [PATCH] vt: disable console blanking by default
> 
>   Remove default 10 minute blank interval. Instead: never blank by
>   default.
>   "Screensaving" is no longer useful.  Today it only provides
>   obstacle when interacting with text console, especially through
>   remote lights-out management solutions.
> 
> Signed-off-by: Tomasz Torcz <tomek@pipebreaker.pl>

Dunno. You'll kill an LCD if you let same image there for too
long. Make it 10hours?

								Pavel

>  Documentation/kernel-parameters.txt | 4 ++--
>  drivers/tty/vt/vt.c                 | 2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/kernel-parameters.txt b/Documentation/kernel-parameters.txt
> index 6eaa9cd..0618648 100644
> --- a/Documentation/kernel-parameters.txt
> +++ b/Documentation/kernel-parameters.txt
> @@ -710,8 +710,8 @@ bytes respectively. Such letter suffixes can also be entirely omitted.
>  		For now, only VisioBraille is supported.
>  
>  	consoleblank=	[KNL] The console blank (screen saver) timeout in
> -			seconds. Defaults to 10*60 = 10mins. A value of 0
> -			disables the blank timer.
> +			seconds. A value of 0 disables the blank timer.
> +			Defaults to never blank.
>  
>  	coredump_filter=
>  			[KNL] Change the default value for
> diff --git a/drivers/tty/vt/vt.c b/drivers/tty/vt/vt.c
> index 5e0f6ff..dbd4f23 100644
> --- a/drivers/tty/vt/vt.c
> +++ b/drivers/tty/vt/vt.c
> @@ -176,7 +176,7 @@ int console_blanked;
>  
>  static int vesa_blank_mode; /* 0:none 1:suspendV 2:suspendH 3:powerdown */
>  static int vesa_off_interval;
> -static int blankinterval = 10*60;
> +static int blankinterval;
>  core_param(consoleblank, blankinterval, int, 0444);
>  
>  static DECLARE_WORK(console_work, console_callback);

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] vt: disable console blanking by default
  2014-06-22 16:37 ` Pavel Machek
@ 2014-06-25 11:41   ` Tomasz Torcz
  2014-06-29 20:35     ` Pavel Machek
  0 siblings, 1 reply; 7+ messages in thread
From: Tomasz Torcz @ 2014-06-25 11:41 UTC (permalink / raw)
  To: linux-kernel; +Cc: Jiri Slaby, Tomasz Torcz, Greg Kroah-Hartman, Pavel Machek

On Sun, Jun 22, 2014 at 06:37:29PM +0200, Pavel Machek wrote:
> On Wed 2014-06-18 11:27:31, Tomasz Torcz wrote:
> > 
> > >From ea7cf47e3230eda63aa6c46092719437f9bbae8e Mon Sep 17 00:00:00 2001
> > From: Tomasz Torcz <tomek@pipebreaker.pl>
> > Date: Wed, 18 Jun 2014 11:18:06 +0200
> > Subject: [PATCH] vt: disable console blanking by default
> > 
> >   Remove default 10 minute blank interval. Instead: never blank by
> >   default.
> >   "Screensaving" is no longer useful.  Today it only provides
> >   obstacle when interacting with text console, especially through
> >   remote lights-out management solutions.
> > 
> > Signed-off-by: Tomasz Torcz <tomek@pipebreaker.pl>
> 
> Dunno. You'll kill an LCD if you let same image there for too
> long. Make it 10hours?

  There is no practical difference between 10 minutes and 10 hours.  Both
are more than none.  This means that If I need to access console, it
will probably be blanked.  For some servers, console access is needed
less than once per year, if ever.  But IF the console need to be accessed,
there's a strong reason to it (some kind of fault), and having to press
anything to unblank may be invasive.

 I would prefer to fix the issue at the source, instead of sticking "consoleblank=0"
in grub2.cfg on houndreds of boxes.  BTW, kernel already contains "consoleblank=0" in 
arch/arc/boot/dts/nsimosci.dts and arch/arm/boot/dts/omap3-lilly-a83x.dtsi 

  Is LCD damage for currently produced LCDs real issue, still?

-- 
Tomasz Torcz                        To co nierealne -- tutaj jest normalne.
xmpp: zdzichubg@chrome.pl          Ziomale na życie mają tu patenty specjalne.


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] vt: disable console blanking by default
  2014-06-18  9:27 [PATCH] vt: disable console blanking by default Tomasz Torcz
  2014-06-22 16:37 ` Pavel Machek
@ 2014-06-25 12:39 ` One Thousand Gnomes
  2014-06-25 19:21   ` Arkadiusz Miskiewicz
  1 sibling, 1 reply; 7+ messages in thread
From: One Thousand Gnomes @ 2014-06-25 12:39 UTC (permalink / raw)
  To: Tomasz Torcz; +Cc: Greg Kroah-Hartman, Jiri Slaby, linux-kernel

On Wed, 18 Jun 2014 11:27:31 +0200
Tomasz Torcz <tomek@pipebreaker.pl> wrote:

> 
> >From ea7cf47e3230eda63aa6c46092719437f9bbae8e Mon Sep 17 00:00:00 2001
> From: Tomasz Torcz <tomek@pipebreaker.pl>
> Date: Wed, 18 Jun 2014 11:18:06 +0200
> Subject: [PATCH] vt: disable console blanking by default
> 
>   Remove default 10 minute blank interval. Instead: never blank by
>   default.
>   "Screensaving" is no longer useful.  Today it only provides
>   obstacle when interacting with text console, especially through
>   remote lights-out management solutions.

Screensaving is extremely useful as is power management, and power prices
are going up not down. We don't customise the upstream kernel just
because J Random User happens to want his defaults.

There are a couple of things you can sensibly do

1. Just turn it off in software. It's run time configurable via setterm

2. If you have a reliable way of detecting the presence of a remote
console device and that device supports reporting when a connection is
made/broken you could submit a patch to make it unblank on connect.

Alan

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] vt: disable console blanking by default
  2014-06-25 12:39 ` One Thousand Gnomes
@ 2014-06-25 19:21   ` Arkadiusz Miskiewicz
  2014-06-26 21:35     ` One Thousand Gnomes
  0 siblings, 1 reply; 7+ messages in thread
From: Arkadiusz Miskiewicz @ 2014-06-25 19:21 UTC (permalink / raw)
  To: linux-kernel
  Cc: One Thousand Gnomes, Tomasz Torcz, Greg Kroah-Hartman, Jiri Slaby

On Wednesday 25 of June 2014, One Thousand Gnomes wrote:
> On Wed, 18 Jun 2014 11:27:31 +0200
> 
> Tomasz Torcz <tomek@pipebreaker.pl> wrote:
> > >From ea7cf47e3230eda63aa6c46092719437f9bbae8e Mon Sep 17 00:00:00 2001
> > 
> > From: Tomasz Torcz <tomek@pipebreaker.pl>
> > Date: Wed, 18 Jun 2014 11:18:06 +0200
> > Subject: [PATCH] vt: disable console blanking by default
> > 
> >   Remove default 10 minute blank interval. Instead: never blank by
> >   default.
> >   "Screensaving" is no longer useful.  Today it only provides
> >   obstacle when interacting with text console, especially through
> >   remote lights-out management solutions.
> 
> Screensaving is extremely useful as is power management, and power prices
> are going up not down. We don't customise the upstream kernel just
> because J Random User happens to want his defaults.
> 
> There are a couple of things you can sensibly do
> 
> 1. Just turn it off in software. It's run time configurable via setterm
> 
> 2. If you have a reliable way of detecting the presence of a remote
> console device and that device supports reporting when a connection is
> made/broken you could submit a patch to make it unblank on connect.

3. unblank & disable blanking on any oops? (probably problematic)

Anyway I dislike blanking here, too.

> Alan


-- 
Arkadiusz Miśkiewicz, arekm / maven.pl

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] vt: disable console blanking by default
  2014-06-25 19:21   ` Arkadiusz Miskiewicz
@ 2014-06-26 21:35     ` One Thousand Gnomes
  0 siblings, 0 replies; 7+ messages in thread
From: One Thousand Gnomes @ 2014-06-26 21:35 UTC (permalink / raw)
  To: Arkadiusz Miskiewicz
  Cc: linux-kernel, Tomasz Torcz, Greg Kroah-Hartman, Jiri Slaby

> 3. unblank & disable blanking on any oops? (probably problematic)

We already try to do that.

Alan

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] vt: disable console blanking by default
  2014-06-25 11:41   ` Tomasz Torcz
@ 2014-06-29 20:35     ` Pavel Machek
  0 siblings, 0 replies; 7+ messages in thread
From: Pavel Machek @ 2014-06-29 20:35 UTC (permalink / raw)
  To: linux-kernel, Jiri Slaby, Greg Kroah-Hartman

On Wed 2014-06-25 13:41:09, Tomasz Torcz wrote:
> On Sun, Jun 22, 2014 at 06:37:29PM +0200, Pavel Machek wrote:
> > On Wed 2014-06-18 11:27:31, Tomasz Torcz wrote:
> > > 
> > > >From ea7cf47e3230eda63aa6c46092719437f9bbae8e Mon Sep 17 00:00:00 2001
> > > From: Tomasz Torcz <tomek@pipebreaker.pl>
> > > Date: Wed, 18 Jun 2014 11:18:06 +0200
> > > Subject: [PATCH] vt: disable console blanking by default
> > > 
> > >   Remove default 10 minute blank interval. Instead: never blank by
> > >   default.
> > >   "Screensaving" is no longer useful.  Today it only provides
> > >   obstacle when interacting with text console, especially through
> > >   remote lights-out management solutions.
> > > 
> > > Signed-off-by: Tomasz Torcz <tomek@pipebreaker.pl>
> > 
> > Dunno. You'll kill an LCD if you let same image there for too
> > long. Make it 10hours?
> 
>   There is no practical difference between 10 minutes and 10 hours.  Both
> are more than none.  This means that If I need to access console, it
> will probably be blanked.  For some servers, console access is needed
> less than once per year, if ever.  But IF the console need to be accessed,
> there's a strong reason to it (some kind of fault), and having to press
> anything to unblank may be invasive.
> 
>  I would prefer to fix the issue at the source, instead of sticking "consoleblank=0"
> in grub2.cfg on houndreds of boxes.  BTW, kernel already contains "consoleblank=0" in 
> arch/arc/boot/dts/nsimosci.dts and arch/arm/boot/dts/omap3-lilly-a83x.dtsi 
> 
>   Is LCD damage for currently produced LCDs real issue, still?

Probably yes. Plus, people are still running CRTs and blanked screen
still eats significantly less power. So "consoleblank=0" seems right
solution for you, and current default still sounds right.
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2014-06-29 20:35 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-18  9:27 [PATCH] vt: disable console blanking by default Tomasz Torcz
2014-06-22 16:37 ` Pavel Machek
2014-06-25 11:41   ` Tomasz Torcz
2014-06-29 20:35     ` Pavel Machek
2014-06-25 12:39 ` One Thousand Gnomes
2014-06-25 19:21   ` Arkadiusz Miskiewicz
2014-06-26 21:35     ` One Thousand Gnomes

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.