From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 27FD5C433DB for ; Fri, 15 Jan 2021 14:54:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id E026F23370 for ; Fri, 15 Jan 2021 14:54:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729039AbhAOOyl (ORCPT ); Fri, 15 Jan 2021 09:54:41 -0500 Received: from mga02.intel.com ([134.134.136.20]:8787 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729116AbhAOOyk (ORCPT ); Fri, 15 Jan 2021 09:54:40 -0500 IronPort-SDR: d7lLysapGMcgvycqAHejactvKHX+QdmZv0RzmZYlc9a98kUgLkEN6DhNdD8aMFs08NF2TLJ2kt uv7oOrN2oO5g== X-IronPort-AV: E=McAfee;i="6000,8403,9864"; a="165642430" X-IronPort-AV: E=Sophos;i="5.79,349,1602572400"; d="scan'208";a="165642430" Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Jan 2021 06:52:54 -0800 IronPort-SDR: gCB9JG1048TU9p7aPAL+ZFO040M4t5L5Ucft7L6K2SrIjEiO+oWVa/17W3jonvu03UfQVyJloV aRQCCFcI0Luw== X-IronPort-AV: E=Sophos;i="5.79,349,1602572400"; d="scan'208";a="364602100" Received: from kshimanx-mobl.amr.corp.intel.com (HELO [10.212.104.192]) ([10.212.104.192]) by orsmga002-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Jan 2021 06:52:53 -0800 Subject: Re: ACPI scan regression -> Boot fail on Cherrytrail w/ 5.11-rc3 To: "Rafael J. Wysocki" Cc: Hans de Goede , ACPI Devel Mailing List , Mika Westerberg , "Rafael J. Wysocki" , Andy Shevchenko , "Rafael J. Wysocki" References: <792c89fd-88f2-b243-50df-21f3be1cc20c@linux.intel.com> <1bb7f766-4f29-55b1-882c-3d70cca0912f@linux.intel.com> <2473340.5W3TeCB90J@kreacher> From: Pierre-Louis Bossart Message-ID: Date: Fri, 15 Jan 2021 08:52:52 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <2473340.5W3TeCB90J@kreacher> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org >> Yep, those 'hacks' solve the boot problem on my device. I tried multiple >> times and it's completely reproducible. >> >>> And can you collect an acpidump from the device and either send it to me and Rafael >>> offlist, or upload it somewhere and send us a link ? >> >> will do > > In addition to what Hans asked for, can you please build the kernel with the > debug patch below applied (instead of the Hans' debug patch), try to boot > the affected machine with it and see what is missing with respect to booting > the kernel with the two problematic commits reverted? Sorry, not following. Are you asking me to apply the patch below as well as revert the two problematic commits? Or just the patch below? the boot process is stuck without the reverts and I don't have a serial link to see what happens (closed form-factor). > Also please send me the outout of "dmesg | grep "Enumeration" from the debug > kernel if possible. > > --- > drivers/acpi/scan.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Index: linux-pm/drivers/acpi/scan.c > =================================================================== > --- linux-pm.orig/drivers/acpi/scan.c > +++ linux-pm/drivers/acpi/scan.c > @@ -1951,7 +1951,7 @@ static acpi_status acpi_bus_check_add(ac > u32 count = acpi_scan_check_dep(handle); > /* Bail out if the number of recorded dependencies is not 0. */ > if (count > 0) { > - acpi_bus_scan_second_pass = true; > + acpi_handle_info(handle, "Enumeration skipped\n"); > return AE_CTRL_DEPTH; > } > } > > >