Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751879AbdGaInK (ORCPT ); Mon, 31 Jul 2017 04:43:10 -0400 Received: from mga02.intel.com ([134.134.136.20]:37584 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750979AbdGaInI (ORCPT ); Mon, 31 Jul 2017 04:43:08 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.40,441,1496127600"; d="scan'208";a="999057094" Subject: Re: Suspend-resume failure on Intel Eagle Lake Core2Duo To: Thomas Gleixner Cc: Martin Peres , jeffy.chen@rock-chips.com, linux-kernel@vger.kernel.org References: <4d6b511a-61d5-3c5e-a406-9f71d83670b6@linux.intel.com> <862ba91e-9185-89a8-da81-63bb42ea565d@intel.com> <7287f845-1012-51af-e696-99d26bcb9b7f@intel.com> <1d2c51f3-a655-2223-68a9-e6d700e7d8e1@intel.com> <412a3ab0-4ae5-b27a-0b2d-d2e03b27a999@intel.com> From: Tomi Sarvela Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Message-ID: Date: Mon, 31 Jul 2017 11:41:57 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1402 Lines: 41 On 31/07/17 11:29, Thomas Gleixner wrote: > On Mon, 31 Jul 2017, Tomi Sarvela wrote: >> On 31/07/17 10:45, Thomas Gleixner wrote: >>> 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. >> >> No such luck. ELK hangs in the suspend-test with earlier patch removed, this >> added. Checked again that the power-led is on, no serial output. >> >> Tree not pulled: still testing against the previous head -rc2, not current >> 4.13.0-rc3 > > Shouldn't make a difference. Can you please try the following: > > Offline CPU1 before invoking suspend. > > # echo 0 >/sys/devices/system/cpus/cpu1/offline Tested with your latest patch (irq_trace_state): echo 0 >/sys/devices/system/cpu/cpu1/online ./scripts/run-tests.sh -vt igt@gem_exec_suspend@basic-s3 -x devices No change, no wakey at all. Tomi -- Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo