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=-8.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS 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 65237C433DB for ; Fri, 15 Jan 2021 15:23:38 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2947A238A0 for ; Fri, 15 Jan 2021 15:23:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728879AbhAOPXh (ORCPT ); Fri, 15 Jan 2021 10:23:37 -0500 Received: from cloudserver094114.home.pl ([79.96.170.134]:65032 "EHLO cloudserver094114.home.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731525AbhAOPXh (ORCPT ); Fri, 15 Jan 2021 10:23:37 -0500 Received: from 89-64-81-100.dynamic.chello.pl (89.64.81.100) (HELO kreacher.localnet) by serwer1319399.home.pl (79.96.170.134) with SMTP (IdeaSmtpServer 0.83.537) id b95aab54fdf6e60a; Fri, 15 Jan 2021 16:22:55 +0100 From: "Rafael J. Wysocki" To: Pierre-Louis Bossart Cc: "Rafael J. Wysocki" , Hans de Goede , ACPI Devel Mailing List , Mika Westerberg , "Rafael J. Wysocki" , Andy Shevchenko Subject: Re: ACPI scan regression -> Boot fail on Cherrytrail w/ 5.11-rc3 Date: Fri, 15 Jan 2021 16:22:54 +0100 Message-ID: <10742637.M1KgVVZ4YP@kreacher> In-Reply-To: References: <792c89fd-88f2-b243-50df-21f3be1cc20c@linux.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org On Friday, January 15, 2021 4:09:05 PM CET Pierre-Louis Bossart wrote: > > >>> 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? > > > > 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). > > > > The point is that the patch below may unstuck it, in which case it > > should be possible to find out what is missing with respect to the > > full successful boot. > > No luck. I tried twice with the patch below only, and the device is > still stuck after the 'Loading initial ramdisk ...'. Thanks! This means that skipping the enumeration of a certain device alone is problematic which is a surprise of sorts. Let's see what device that may be. Because the machine booted with the debug patch from Hans, it should also boot with the one below, so please do that and send the output of $ dmesg | grep Dependencies --- drivers/acpi/scan.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) Index: linux-pm/drivers/acpi/scan.c =================================================================== --- linux-pm.orig/drivers/acpi/scan.c +++ linux-pm/drivers/acpi/scan.c @@ -1950,10 +1950,8 @@ static acpi_status acpi_bus_check_add(ac if (type == ACPI_BUS_TYPE_DEVICE && check_dep) { 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; - return AE_CTRL_DEPTH; - } + if (count > 0) + acpi_handle_info(handle, "Dependencies found\n"); } acpi_add_single_object(&device, handle, type, sta); @@ -1961,8 +1959,7 @@ static acpi_status acpi_bus_check_add(ac return AE_CTRL_DEPTH; acpi_scan_init_hotplug(device); - if (!check_dep) - acpi_scan_dep_init(device); + acpi_scan_dep_init(device); out: if (!*adev_p)