linux-kernel - Re: [PATCH 08/17] tty: New RISC-V SBI console driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <mhng-1d53287f-4183-4f08-9dee-83379ee7504d@palmer-si-x1c4>
Date:   Wed, 12 Jul 2017 09:24:23 -0700 (PDT)
From:   Palmer Dabbelt <palmer@...belt.com>
To:     mpe@...erman.id.au
CC:     yamada.masahiro@...ionext.com, mmarek@...e.com,
        will.deacon@....com, peterz@...radead.org, boqun.feng@...il.com,
        mingo@...hat.com, daniel.lezcano@...aro.org, tglx@...utronix.de,
        jason@...edaemon.net, marc.zyngier@....com,
        gregkh@...uxfoundation.org, jslaby@...e.com, davem@...emloft.net,
        mchehab@...nel.org, sfr@...b.auug.org.au, fweisbec@...il.com,
        viro@...iv.linux.org.uk, mcgrof@...nel.org, dledford@...hat.com,
        bart.vanassche@...disk.com, sstabellini@...nel.org,
        daniel.vetter@...ll.ch, msalter@...hat.com,
        nicolas.dichtel@...nd.com, james.hogan@...tec.com,
        paul.gortmaker@...driver.com, linux@...ck-us.net,
        heiko.carstens@...ibm.com, schwidefsky@...ibm.com,
        linux-kernel@...r.kernel.org, patches@...ups.riscv.org,
        akpm@...ux-foundation.org, albert@...ive.com
Subject:     Re: [PATCH 08/17] tty: New RISC-V SBI console driver

On Wed, 12 Jul 2017 04:04:00 PDT (-0700), mpe@...erman.id.au wrote:
> Palmer Dabbelt <palmer@...belt.com> writes:
>
>> On Mon, 10 Jul 2017 23:21:07 PDT (-0700), mpe@...erman.id.au wrote:
>>> Palmer Dabbelt <palmer@...belt.com> writes:
>>>>
>>> ...
>>>> +#ifdef CONFIG_EARLY_PRINTK
>>>> +static void sbi_console_write(struct console *co, const char *buf,
>>>> +			      unsigned int n)
>>>> +{
>>>> +	int i;
>>>> +
>>>> +	for (i = 0; i < n; ++i) {
>>>> +		if (buf[i] == '\n')
>>>> +			sbi_console_putchar('\r');
>>>> +		sbi_console_putchar(buf[i]);
>>>> +	}
>>>> +}
>>>> +
>>>> +static struct console early_console_dev __initdata = {
>>>> +	.name	= "early",
>>>> +	.write	= sbi_console_write,
>>>> +	.flags	= CON_PRINTBUFFER | CON_BOOT,
>>>
>>> AFAICS you could add CON_ANYTIME here, which would mean this console
>>> would print output before the CPU is online.
>>>
>>> I think it doesn't currently matter because you call parse_early_param()
>>> from setup_arch(), at which point the boot CPU has been marked online.
>>>
>>> But if this console can actually work earlier then you might be better
>>> off just registering it unconditionally very early.
>>
>> That seems like a good idea.  I'm not familiar with how all this works, but
>> from my understanding of this early_initcall() should be sufficient to make
>> this work?  The only other driver that sets CON_ANYTIME and supports
>> EARLY_PRINTK is hvc_xen, but that installs a header to let init code register
>> the console directly.  The early_initcall mechanism seems cleaner if it does
>> the right thing.
>
> Unfortunately early_initcall is not very "early" :)  It's earlier than
> all the other initcalls, but it's late compared to most of the arch boot
> code.
>
> The early_param() will work better, ie. register the console earlier
> and increase the chance of you getting output from an early crash, than
> early_initcall. But it requires you to put earlyprintk on the command line.
>
> The best option is to just register the console as early as you can, ie.
> as soon as it can give you output. So somewhere in your setup_arch(), or
> even earlier (I haven't read your boot code).

Doing it that way would require either moving the TTY driver into arch code (it
was specifically suggested we move it out) or adding a header file to allow
setup_arch() to call into the driver (XEN does this, and we're doing it for our
timer, but it seems hacky).