linux-kernel - Re: [patch] serial: add parameter to force skipping the test for the TXEN bug

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]

Date:	Tue, 21 Apr 2009 12:47:45 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Chuck Ebbert <cebbert@...hat.com>
Cc:	alan@...rguk.ukuu.org.uk, linux-kernel@...r.kernel.org
Subject: Re: [patch] serial: add parameter to force skipping the test for
 the TXEN bug

On Mon, 20 Apr 2009 18:05:49 -0400
Chuck Ebbert <cebbert@...hat.com> wrote:

> serial: add parameter to force skipping the test for the TXEN bug
> 
> Allow users to force skipping the TXEN test at init time. Applies
> to all serial ports. Intended for debugging only.
> 

<scratches head @ changelog>

> 
> Index: linux-2.6.29.noarch/drivers/serial/8250.c
> ===================================================================
> --- linux-2.6.29.noarch.orig/drivers/serial/8250.c
> +++ linux-2.6.29.noarch/drivers/serial/8250.c
> @@ -67,6 +67,8 @@ static int serial_index(struct uart_port
>  	return (serial8250_reg.minor - 64) + port->line;
>  }
>  
> +static unsigned int skip_txen_test; /* force skip of txen test at init time */
> +
>  /*
>   * Debugging.
>   */
> @@ -2097,7 +2099,7 @@ static int serial8250_startup(struct uar
>  	   is variable. So, let's just don't test if we receive
>  	   TX irq. This way, we'll never enable UART_BUG_TXEN.
>  	 */
> -	if (up->port.flags & UPF_NO_TXEN_TEST)
> +	if (skip_txen_test || up->port.flags & UPF_NO_TXEN_TEST)
>  		goto dont_test_tx_en;
>  
>  	/*
> @@ -3245,6 +3247,9 @@ MODULE_PARM_DESC(share_irqs, "Share IRQs
>  module_param(nr_uarts, uint, 0644);
>  MODULE_PARM_DESC(nr_uarts, "Maximum number of UARTs supported. (1-" __MODULE_STRING(CONFIG_SERIAL_8250_NR_UARTS) ")");
>  
> +module_param(skip_txen_test, uint, 0644);
> +MODULE_PARM_DESC(skip_txen_test, "Skip checking for the TXEN bug at init time");
> +
>  #ifdef CONFIG_SERIAL_8250_RSA
>  module_param_array(probe_rsa, ulong, &probe_rsa_count, 0444);
>  MODULE_PARM_DESC(probe_rsa, "Probe I/O ports for RSA");

Why is this useful?  What is the problem which you're solving here?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/