From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751858AbdGaHqI (ORCPT ); Mon, 31 Jul 2017 03:46:08 -0400 Received: from Galois.linutronix.de ([146.0.238.70]:36647 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750764AbdGaHqH (ORCPT ); Mon, 31 Jul 2017 03:46:07 -0400 Date: Mon, 31 Jul 2017 09:45:53 +0200 (CEST) From: Thomas Gleixner To: Tomi Sarvela cc: Martin Peres , jeffy.chen@rock-chips.com, linux-kernel@vger.kernel.org Subject: Re: Suspend-resume failure on Intel Eagle Lake Core2Duo In-Reply-To: Message-ID: References: <4d6b511a-61d5-3c5e-a406-9f71d83670b6@linux.intel.com> <67de95b4-a869-d897-e76c-17c974fb21a6@linux.intel.com> <862ba91e-9185-89a8-da81-63bb42ea565d@intel.com> <7287f845-1012-51af-e696-99d26bcb9b7f@intel.com> <1d2c51f3-a655-2223-68a9-e6d700e7d8e1@intel.com> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 31 Jul 2017, Tomi Sarvela wrote: > On 28/07/17 19:26, Thomas Gleixner wrote: > > Did you change anything else compared to the tests before ? > > I did check that the problem persisted in linus-HEAD before testing your > patch. The testing was done in order (reading from console logs I happen to > still have in one window): What I still do not understand is why this would affect the suspend path in any way. Can you remove the previous patch and apply the one below. If it resumes, please provide the data from the trace buffer again. Thanks, tglx 8<-------------- --- a/kernel/irq/internals.h +++ b/kernel/irq/internals.h @@ -459,3 +459,11 @@ static inline void irq_remove_debugfs_en { } #endif /* CONFIG_GENERIC_IRQ_DEBUGFS */ + +extern bool irq_suspend_resume; + +static inline void irq_trace_state(const char *what, struct irq_desc *desc) +{ + trace_printk("%s %d state %08x\n", what, irq_desc_get_irq(desc), + irqd_get(&desc->irq_data)); +} --- a/kernel/irq/pm.c +++ b/kernel/irq/pm.c @@ -14,6 +14,8 @@ #include "internals.h" +bool irq_suspend_resume; + bool irq_pm_check_wakeup(struct irq_desc *desc) { if (irqd_is_wakeup_armed(&desc->irq_data)) { @@ -120,6 +122,7 @@ void suspend_device_irqs(void) struct irq_desc *desc; int irq; + irq_suspend_resume = true; for_each_irq_desc(irq, desc) { unsigned long flags; bool sync; @@ -127,7 +130,9 @@ void suspend_device_irqs(void) if (irq_settings_is_nested_thread(desc)) continue; raw_spin_lock_irqsave(&desc->lock, flags); + irq_trace_state("presuspend", desc); sync = suspend_device_irq(desc); + irq_trace_state("postsuspend", desc); raw_spin_unlock_irqrestore(&desc->lock, flags); if (sync) @@ -150,8 +155,9 @@ static void resume_irq(struct irq_desc * /* Pretend that it got disabled ! */ desc->depth++; irq_state_set_disabled(desc); - irq_state_set_masked(desc); + resume: + irq_state_set_masked(desc); desc->istate &= ~IRQS_SUSPENDED; __enable_irq(desc); } @@ -172,9 +178,14 @@ static void resume_irqs(bool want_early) continue; raw_spin_lock_irqsave(&desc->lock, flags); + irq_trace_state("preresume", desc); resume_irq(desc); + irq_trace_state("postresume", desc); raw_spin_unlock_irqrestore(&desc->lock, flags); } + + if (!want_early) + irq_suspend_resume = false; } /**