Re: [PATCH] PM / suspend: show workqueues busy name in suspend flow

2016-06-29 Thread Tejun Heo
Hello, Roger. On Wed, Jun 29, 2016 at 11:54:11AM +0800, Roger Lu wrote: > Please allow me to elaborate my previous concern about printing > freezable workqueue info only in this case. > > The benefit of it is that debugger can quickly understand which > freezable workqueues block suspend flow and

Re: [PATCH] PM / suspend: show workqueues busy name in suspend flow

2016-06-28 Thread Roger Lu
Dear Tejun, On Tue, 2016-06-28 at 12:56 -0400, Tejun Heo wrote: > Hello, Roger. > > On Mon, Jun 27, 2016 at 05:44:05PM +0800, Roger Lu wrote: > > show_workqueue_state() is a better choice to me. However, only freezable > > workqueue is able to affect suspend flow. So, is there other mailing > > l

Re: [PATCH] PM / suspend: show workqueues busy name in suspend flow

2016-06-28 Thread Tejun Heo
Hello, Roger. On Mon, Jun 27, 2016 at 05:44:05PM +0800, Roger Lu wrote: > show_workqueue_state() is a better choice to me. However, only freezable > workqueue is able to affect suspend flow. So, is there other mailing > list discussing about showing freezable workqueue state only?? Maybe we > can

Re: [PATCH] PM / suspend: show workqueues busy name in suspend flow

2016-06-22 Thread Tejun Heo
Hello, On Wed, Jun 22, 2016 at 05:08:29PM +0800, Roger Lu wrote: > When suspend flow is aborted because of workqueues busy, > show workqueues busy name for debug purpose. > > Signed-off-by: Roger Lu > --- > include/linux/workqueue.h | 1 + > kernel/power/process.c| 3 +++ > kernel/workque