From: Daniel Mack <daniel@caiaq.de>
To: David Newall <davidn@davidnewall.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
Clemens Ladisch <clemens@ladisch.de>,
Greg Kroah-Hartman <gregkh@suse.de>,
Andrew Morton <akpm@linux-foundation.org>,
Geert Uytterhoeven <geert@linux-m68k.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] vt: make the default cursor shape configurable
Date: Mon, 9 Nov 2009 19:07:17 +0100 [thread overview]
Message-ID: <20091109180717.GD14091@buzzloop.caiaq.de> (raw)
In-Reply-To: <4AF8582B.2000403@davidnewall.com>
On Tue, Nov 10, 2009 at 04:28:03AM +1030, David Newall wrote:
> Daniel Mack wrote:
> > I had that previously and the response was that there is no reason to
> > define that setting per-boot process, as it is rather unlikely that you
> > want to change it from one boot to the next. Hence, it should be set at
> > compile time.
> >
>
> It should be set as a boot parameter, since this is fixed at system
> installation and then left untouched.
I'd be fine with either way. As long as it disables the blinking cursor.
> It seems like adding cruft to the kernel that is just as effectively
> available at run-time. Where does it end? Do we eventually add bash to
> the kernel?
Well, all I need is a way to disable the cursor at boot time, as we
don't want it to be in our boot screen on a embedded system. It is not
only ruin the aesthetics but also gives the wrong assumption that the
system awaits input. There is no userspace at this point, so that's no
option.
My first approach was adding a __setup variable to select that, but it
wasn't beloved by the people get_maintainer.pl spit out. So I made it
compile-time definable, and Clemens extended that to a more powerful
interface of settings.
I don't really have a strong opinion on which way to go, all I need is a
cursor-free system startup :) And I would of course prefer that patch to
be in mainline.
Daniel
next prev parent reply other threads:[~2009-11-09 18:07 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-05 9:17 [PATCH] fbcon: make cursor display conditional Daniel Mack
2009-11-06 8:16 ` Clemens Ladisch
2009-11-06 14:39 ` Andrea Righi
2009-11-06 16:45 ` Daniel Mack
2009-11-09 8:35 ` Clemens Ladisch
2009-11-09 9:10 ` Daniel Mack
2009-11-09 9:15 ` [PATCH] vt: make the default cursor shape configurable Clemens Ladisch
2009-11-09 9:53 ` Alan Cox
2009-11-09 11:26 ` Clemens Ladisch
2009-11-09 11:46 ` Alan Cox
2009-11-09 12:21 ` Clemens Ladisch
2009-11-09 14:50 ` Daniel Mack
2009-11-09 17:58 ` David Newall
2009-11-09 18:07 ` Daniel Mack [this message]
2009-11-09 22:09 ` Daniel Mack
2009-11-10 12:05 ` David Newall
2009-11-10 12:30 ` Daniel Mack
2009-11-10 12:36 ` David Newall
2009-11-10 12:39 ` Daniel Mack
2009-11-10 21:09 ` Pavel Machek
2009-11-10 23:26 ` Andrew Morton
2009-11-11 6:56 ` Clemens Ladisch
2009-11-11 7:54 ` Pavel Machek
2009-11-12 22:05 ` Andrew Morton
2009-11-13 7:28 ` Clemens Ladisch
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20091109180717.GD14091@buzzloop.caiaq.de \
--to=daniel@caiaq.de \
--cc=akpm@linux-foundation.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=clemens@ladisch.de \
--cc=davidn@davidnewall.com \
--cc=geert@linux-m68k.org \
--cc=gregkh@suse.de \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox