Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1687369

Re: [PATCH 0/2] Avoid crashes by early (boot) consoles using init memory

From "Fabio M. Di Nitto" <fdinitto@redhat.com>
Newsgroups linux.kernel
Subject Re: [PATCH 0/2] Avoid crashes by early (boot) consoles using init memory
Date 2017-07-14 15:00 +0200
Message-ID <u38mm-3Ag-7@gated-at.bofh.it> (permalink)
References <u38mm-3Ag-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Petr,

On 7/14/2017 2:51 PM, Petr Mladek wrote:
> Some early consoles have code and data in the init section. It makes some
> sense but this might cause problems when they are not replaced by
> the real console in time. The two patches fix the safequard and
> help to avoid the problems.
> 
> I though about removing keep_bootcon option completely. But it is useful
> at least for now.

Let´s just keep in mind that keep_bootcon was introduced only to debug
issues (read crashes or hangs) that could happen between disabling
bootconsole and enabling the real console. It shouldn´t be used for
anything else really.

If the new code can replace keep_bootcon, by all mean, go for it :-)

Cheers
Fabio

> There is not an easy way to disable a particular bootconsole
> when the related real console is installed. Instead all bootconsoles are
> removed when the preferred (last on the commandline) console is registered.
> But this is a bit cumbersome and non-intuitive.
> 
> In addition, the same problems might happen when the real console is
> registered using a deferred probe or when it is not registered at all
> for some reason.
> 
> The patchset is based on and inspired by the original patch from
> Matt Redfearn, see the discussion starting by the mail
> https://lkml.kernel.org/r/1499337481-19397-1-git-send-email-matt.redfearn@imgtec.com
> 
> Matt Redfearn (1):
>   printk/console: Always disable boot consoles that use init memory
>     before it is freed
> 
> Petr Mladek (1):
>   printk/console: Enhance the check for consoles using init memory
> 
>  kernel/printk/printk.c | 28 +++++++++++++++++-----------
>  1 file changed, 17 insertions(+), 11 deletions(-)
> 

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

Re: [PATCH 0/2] Avoid crashes by early (boot) consoles using init  memory "Fabio M. Di Nitto" <fdinitto@redhat.com> - 2017-07-14 15:00 +0200
  Re: [PATCH 0/2] Avoid crashes by early (boot) consoles using init  memory Petr Mladek <pmladek@suse.com> - 2017-07-14 16:40 +0200
    Re: [PATCH 0/2] Avoid crashes by early (boot) consoles using init  memory "Fabio M. Di Nitto" <fdinitto@redhat.com> - 2017-07-15 07:10 +0200

csiph-web