linux-kernel - Re: [PATCHv5 0/3] Serial fixes

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] [day] [month] [year] [list]

Message-ID: <517AE1D3.5050702@ti.com>
Date:	Sat, 27 Apr 2013 01:51:39 +0530
From:	Sourav Poddar <sourav.poddar@...com>
To:	Sourav Poddar <sourav.poddar@...com>
CC:	<gregkh@...uxfoundation.org>, <tony@...mide.com>,
	<rmk+kernel@....linux.org.uk>, <khilman@...aro.org>,
	<linux-serial@...r.kernel.org>, <linux-omap@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>,
	Santosh Shilimkar <santosh.shilimkar@...com>,
	Felipe Balbi <balbi@...com>, Rajendra nayak <rnayak@...com>
Subject: Re: [PATCHv5 0/3] Serial fixes

On Saturday 27 April 2013 01:34 AM, Sourav Poddar wrote:
> Hi,
>
> This patch series contains fixes around the issue that
> the console UART should not idled on suspend while using
> "no_console_suspend" in bootargs.
>
> The approach thought of is to modify the serial core/serial driver to bypass
> runtime PM if the UART in contention is a console and we are using "no_console_suspend"
> in our bootargs.
>
> There were discussions about how to handle "no_idle_on_suspend" issue and all the
> discussions are as follows:
> https://lkml.org/lkml/2013/4/5/239
> https://lkml.org/lkml/2013/4/2/350
> https://lkml.org/lkml/2013/3/18/199
> https://lkml.org/lkml/2013/3/18/295
> Due to the amount of change in approach, I am posting
> this as a new series.
>
> v4->v5
> 1. Add comments
> 2. Formatting.
>
> v3->v4
> 1. check for console in runtime api.
>
> v2->v3
> 1. Use "-EBUSY" for no_console_suspend case
> 2. Bypass runtime PM only during suspend
> 3. Improve the commit log based on community suggestion.
>
> v1->v2
> 1. Remove the prepare/complete callback.
> 2. Adapt runtime PM callback to deal with the issue.
> 3. Fold patch(1,2) of previous series into 1.
> 4. Reordered the patch.
> 5. Change $subject and chage log for few patches.
>
> Cc: Santosh Shilimkar<santosh.shilimkar@...com>
> Cc: Felipe Balbi<balbi@...com>
> Cc: Rajendra nayak<rnayak@...com>
>
> Test info
> Omap4430sdp:
> - Tested wakeup from UART after suspend for dt and non dt case.
> Omap5430evm:
> - Tested wakeup from UART after suspend for dt case.
>
> These patches were the part of the bigger series[1]. Breaking them into
> two relevant series as they go through the different tree.
>
> [1]: http://lkml.org/lkml/2013/4/26/274
>
> This patches are based on 3.9-rc8
>
> The following changes since commit 824282ca7d250bd7c301f221c3cd902ce906d731:
>
>    Merge branch 'upstream' of git://git.linux-mips.org/pub/scm/ralf/upstream-linus (2013-04-22 15:00:59 -0700)
>
> are available in the git repository at:
>
>    git://gitorious.org/linux-connectivity/linux-connectivity.git serial_fix
>
> Sourav Poddar (2):
>        driver: tty: serial: Move "uart_console" def to core header file.
>        driver: serial: omap: prevent runtime PM for "no_console_suspend"
>
>   drivers/tty/serial/mpc52xx_uart.c |   10 ----------
>   drivers/tty/serial/omap-serial.c  |   34 +++++++++++++++++++++++++++++++++-
>   drivers/tty/serial/serial_core.c  |    6 ------
>   include/linux/serial_core.h       |    7 +++++++
>   4 files changed, 40 insertions(+), 17 deletion
Discard this cover..resend the cover with correct subject.
--
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/