From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933480AbcJRQ5y (ORCPT ); Tue, 18 Oct 2016 12:57:54 -0400 Received: from mail-wm0-f68.google.com ([74.125.82.68]:36323 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756686AbcJRQ5s (ORCPT ); Tue, 18 Oct 2016 12:57:48 -0400 Subject: Re: [Linux-kernel] [PATCH] tty: serial: 8250: add CON_CONSDEV to flags To: Ben Dooks , Matthew Leach , Greg Kroah-Hartman References: <1466522389-26943-1-git-send-email-matthew.leach@codethink.co.uk> <40b80b82-531d-d9e8-0b4d-bdd559ad5044@suse.cz> <1452ffb1-429b-65e8-3c5d-76a03ea4eb4a@codethink.co.uk> Cc: linux-kernel@lists.codethink.co.uk, linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org From: Jiri Slaby Message-ID: <048721f0-d3f1-c029-2349-044015ef9d8d@suse.cz> Date: Tue, 18 Oct 2016 18:57:45 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: <1452ffb1-429b-65e8-3c5d-76a03ea4eb4a@codethink.co.uk> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/18/2016, 06:23 PM, Ben Dooks wrote: > On 18/10/16 17:15, Jiri Slaby wrote: >> On 06/21/2016, 05:19 PM, Matthew Leach wrote: >>> When using the 8250 as a boot console and the main console results in >>> messages being printed twice. The console framework will only >>> unregister boot consoles if a new console is registered with the >>> CON_CONSDEV flag set. >>> >>> Set this flag for the univ8250 console to prevent double-registration. >>> >>> Signed-off-by: Matthew Leach >>> --- >>> drivers/tty/serial/8250/8250_core.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/tty/serial/8250/8250_core.c >>> b/drivers/tty/serial/8250/8250_core.c >>> index 2f4f5ee..7a7453d 100644 >>> --- a/drivers/tty/serial/8250/8250_core.c >>> +++ b/drivers/tty/serial/8250/8250_core.c >>> @@ -675,7 +675,7 @@ static struct console univ8250_console = { >>> .device = uart_console_device, >>> .setup = univ8250_console_setup, >>> .match = univ8250_console_match, >>> - .flags = CON_PRINTBUFFER | CON_ANYTIME, >>> + .flags = CON_PRINTBUFFER | CON_ANYTIME | CON_CONSDEV, >> >> This patch breaks console= parameter handling. Passing: >> console=ttyS0 console=tty0 >> no longer means /dev/console is tty0 and it should be as is documented. >> >> I have not investigated further yet. > > Ok, thanks. > I'll need to find Matthew Leach's new address as he's moved jobs. Oh sure: #define CON_CONSDEV (2) /* Last on the command line */ Why it should be set in struct console statically? The patch is obviously incorrect. And I do not understand from the vague commit log what the problem actually was. So for the time being, Greg, please revert it (d03516df8375873) and we can come with something better, when we understand the issue. Please CC stable (4.8+). thanks, -- js suse labs