Thread (49 messages) 49 messages, 7 authors, 2017-02-03

Re: [PATCH v9 4/4] console: Make persistent scrollback a boot parameter

From: Adam Borowski <hidden>
Date: 2017-02-03 16:46:49
Also in: lkml

On Fri, Feb 03, 2017 at 05:04:15PM +0100, Manuel Schölling wrote:
On Thu, 2017-02-02 at 15:07 -0500, Paul Gortmaker wrote:
quoted
On Tue, Jan 10, 2017 at 4:28 PM, Manuel Schölling
[off-list ref] wrote:
quoted
The impact of the persistent scrollback feature on the code size is
rather small, so the config option is removed. The feature stays
disabled by default and can be enabled by using the boot command
line
parameter 'vgacon.scrollback_persistent=1' or by setting
VGACON_SOFT_SCROLLBACK_PERSISTENT_ENABLE_BY_DEFAULT=y.
[...]
quoted
+module_param_named(scrollback_persistent, scrollback_persistent,
bool, 0000);
+MODULE_PARM_DESC(scrollback_persistent, "Enable persistent
scrollback for all vga consoles");
Since this hasn't got widespread deployment yet and  only exists
in Greg's tree, can we please fix the above to use setup_param or
similar, since there is nothing modular about this code at all.
Not sure what you mean here.
If this is not the right may to declare it I'd be more than happy to
change this. But I could not find any function/macro named setup_param
[1].
It would be great if you could give me a hint what function to use
here!

[1] http://lxr.free-electrons.com/ident?i=setup_param
That shows only exact matches.

You want "git grep setup_param", which shows __setup_param() plus some
unrelated stuff.  I see only four uses in the kernel, but that's enough
to see how to use it.


Meow!
-- 
Autotools hint: to do a zx-spectrum build on a pdp11 host, type:
  ./configure --host=zx-spectrum --build=pdp11
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help