Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752372AbdGaOO5 (ORCPT ); Mon, 31 Jul 2017 10:14:57 -0400 Received: from mga05.intel.com ([192.55.52.43]:11644 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752132AbdGaOO4 (ORCPT ); Mon, 31 Jul 2017 10:14:56 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.40,442,1496127600"; d="scan'208";a="999153402" 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> <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: <5d8dfd5c-64c0-ae5e-865e-ebb28c7b4967@intel.com> Date: Mon, 31 Jul 2017 17:13:46 +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: 2000 Lines: 59 On 31/07/17 17:04, Thomas Gleixner wrote: > On Mon, 31 Jul 2017, Tomi Sarvela wrote: >> 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 > > So this "igt@gem_exec_suspend@basic-s3" thingy is that executing anything > extra aside of 'echo mem > /sys/power/state'? It's setting wakeup with rtcwake to +15 seconds, then suspending. Complete information glanceable from sources: https://cgit.freedesktop.org/xorg/app/intel-gpu-tools/tree/tests/gem_exec_suspend.c -> https://cgit.freedesktop.org/xorg/app/intel-gpu-tools/tree/lib/igt_aux.c:void igt_system_suspend_autoresume Tomi -- Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo