Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-07-18 Thread Tony Lindgren
* Andy Shevchenko [180718 14:54]: > On Wed, 2018-05-23 at 11:00 -0700, Tony Lindgren wrote: > > I thought you said earlier the issue was that you wanted to keep > > the console enabled all the time and never idle? > > Yes, for kernel console. > To be clear, if user supplies "console=ttySx" it kee

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-07-18 Thread Tony Lindgren
* Andy Shevchenko [180718 15:17]: > On Wed, 2018-05-23 at 10:58 -0700, Tony Lindgren wrote: > > > > OK yeah console.idle sounds good to me. We should default to a > > safe option. > > I'll see what we can do here. Like we discussed offline I think if we allow detaching and attaching kernel cons

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-07-18 Thread Andy Shevchenko
On Wed, 2018-05-23 at 10:58 -0700, Tony Lindgren wrote: > * Andy Shevchenko [180522 21:42]: > > On Thu, May 17, 2018 at 10:30 PM, Tony Lindgren > > wrote: > > > So how about add some "noidle" kernel command line parameter for > > > console > > > that calls > > > pm_runtime_forbid() and then you h

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-07-18 Thread Andy Shevchenko
On Wed, 2018-05-23 at 11:00 -0700, Tony Lindgren wrote: > * Andy Shevchenko [180522 21:54]: > > On Thu, May 17, 2018 at 10:48 PM, Tony Lindgren > > wrote: > > > * Andy Shevchenko [180517 16:38]: > > > > On Thu, May 17, 2018 at 4:48 PM, Tony Lindgren > > > > wrote: > > > > > * Sebastian Andrzej

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-23 Thread Tony Lindgren
* Tony Lindgren [180523 10:58]: > * Andy Shevchenko [180522 21:42]: > > Have you seen entire series which I keep here: > > https://bitbucket.org/andy-shev/linux/branch/topic/uart/rpm? > > Among other things it gets rid of those specific callbacks entirely. > > Well I was not Cc:ed on it, I brows

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-23 Thread Tony Lindgren
* Andy Shevchenko [180522 21:54]: > On Thu, May 17, 2018 at 10:48 PM, Tony Lindgren wrote: > > * Andy Shevchenko [180517 16:38]: > >> On Thu, May 17, 2018 at 4:48 PM, Tony Lindgren wrote: > >> > * Sebastian Andrzej Siewior [180516 10:49]: > > >> > The idea breaking PM seems silly to me consid

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-23 Thread Tony Lindgren
* Andy Shevchenko [180522 21:42]: > On Thu, May 17, 2018 at 10:30 PM, Tony Lindgren wrote: > > So how about add some "noidle" kernel command line parameter for console > > that calls > > pm_runtime_forbid() and then you have the UART permanently > > on. > > IIUC _forbid() can be overwritten via

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-22 Thread Andy Shevchenko
On Thu, May 17, 2018 at 10:48 PM, Tony Lindgren wrote: > * Andy Shevchenko [180517 16:38]: >> On Thu, May 17, 2018 at 4:48 PM, Tony Lindgren wrote: >> > * Sebastian Andrzej Siewior [180516 10:49]: >> > The idea breaking PM seems silly to me considering that we've had >> > it working for years

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-22 Thread Andy Shevchenko
On Thu, May 17, 2018 at 10:30 PM, Tony Lindgren wrote: > * Andy Shevchenko [180517 16:41]: >> On Thu, May 17, 2018 at 4:56 PM, Tony Lindgren wrote: >> > * Andy Shevchenko [180516 13:12]: >> >> On Wed, 2018-05-16 at 12:47 +0200, Sebastian Andrzej Siewior wrote: >> >> > But since I am on it. You

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-17 Thread Tony Lindgren
* Andy Shevchenko [180517 16:38]: > On Thu, May 17, 2018 at 4:48 PM, Tony Lindgren wrote: > > * Sebastian Andrzej Siewior [180516 10:49]: > >> On 2018-05-16 13:17:36 [+0300], Andy Shevchenko wrote: > >> > > The output is usually short so there > >> > > shouldn't be much benefit from using it. >

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-17 Thread Tony Lindgren
* Andy Shevchenko [180517 16:41]: > On Thu, May 17, 2018 at 4:56 PM, Tony Lindgren wrote: > > * Andy Shevchenko [180516 13:12]: > >> On Wed, 2018-05-16 at 12:47 +0200, Sebastian Andrzej Siewior wrote: > >> > But since I am on it. You have to enable runtime-PM for the UART. So > >> > what is the

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-17 Thread Tony Lindgren
* Tony Lindgren [180517 06:56]: > * Andy Shevchenko [180516 13:12]: > > On Wed, 2018-05-16 at 12:47 +0200, Sebastian Andrzej Siewior wrote: > > > But since I am on it. You have to enable runtime-PM for the UART. So > > > what is the problem if you simply don't enable it for the UART which > > > u

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-17 Thread Andy Shevchenko
On Thu, May 17, 2018 at 4:56 PM, Tony Lindgren wrote: > * Andy Shevchenko [180516 13:12]: >> On Wed, 2018-05-16 at 12:47 +0200, Sebastian Andrzej Siewior wrote: >> > But since I am on it. You have to enable runtime-PM for the UART. So >> > what is the problem if you simply don't enable it for the

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-17 Thread Andy Shevchenko
On Thu, May 17, 2018 at 4:48 PM, Tony Lindgren wrote: > * Sebastian Andrzej Siewior [180516 10:49]: >> On 2018-05-16 13:17:36 [+0300], Andy Shevchenko wrote: >> > > The output is usually short so there >> > > shouldn't be much benefit from using it. >> > >> > > I remember Tony wanted runtime-pm o

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-17 Thread Tony Lindgren
* Andy Shevchenko [180516 13:12]: > On Wed, 2018-05-16 at 12:47 +0200, Sebastian Andrzej Siewior wrote: > > But since I am on it. You have to enable runtime-PM for the UART. So > > what is the problem if you simply don't enable it for the UART which > > used as the kernel console? > > How do I kn

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-17 Thread Tony Lindgren
* Sebastian Andrzej Siewior [180516 10:49]: > On 2018-05-16 13:17:36 [+0300], Andy Shevchenko wrote: > > > The output is usually short so there > > > shouldn't be much benefit from using it. > > > > > I remember Tony wanted runtime-pm on the kernel console, too. And he > > > told me explicit how

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-16 Thread Andy Shevchenko
On Wed, 2018-05-16 at 12:47 +0200, Sebastian Andrzej Siewior wrote: > On 2018-05-16 13:17:36 [+0300], Andy Shevchenko wrote: > > > The output is usually short so there > > > shouldn't be much benefit from using it. > > > I remember Tony wanted runtime-pm on the kernel console, too. And > > > he > >

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-16 Thread Sebastian Andrzej Siewior
On 2018-05-16 13:17:36 [+0300], Andy Shevchenko wrote: > > The output is usually short so there > > shouldn't be much benefit from using it. > > > I remember Tony wanted runtime-pm on the kernel console, too. And he > > told me explicit how to test it so that it works. Once the UART goes > > into

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-16 Thread Andy Shevchenko
On Wed, May 16, 2018 at 1:08 PM, Sebastian Andrzej Siewior wrote: > On 2018-05-15 21:37:18 [+0300], Andy Shevchenko wrote: >> On Tue, 2018-05-15 at 21:34 +0300, Andy Shevchenko wrote: >> > Kernel console is sensitive to any kind of complex work needed to >> > print >> > out anything on it. One suc

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-16 Thread Sebastian Andrzej Siewior
On 2018-05-15 21:37:18 [+0300], Andy Shevchenko wrote: > On Tue, 2018-05-15 at 21:34 +0300, Andy Shevchenko wrote: > > Kernel console is sensitive to any kind of complex work needed to > > print > > out anything on it. One such case is emergency print during Oops. > > > > This series proposes to d

Re: [PATCH v1 0/3] console, serial8250: Disable PM and DMA ops

2018-05-15 Thread Andy Shevchenko
On Tue, 2018-05-15 at 21:34 +0300, Andy Shevchenko wrote: > Kernel console is sensitive to any kind of complex work needed to > print > out anything on it. One such case is emergency print during Oops. > > This series proposes to disable runtime PM and DMA operations on 8250 > serial console. > >