Received: by 2002:a05:6a10:2785:0:0:0:0 with SMTP id ia5csp269708pxb; Wed, 13 Jan 2021 03:15:11 -0800 (PST) X-Google-Smtp-Source: ABdhPJwZoMYWLx6q/575BfWoHEtAug9TovkCbW9PVMEoBirmBt8TNW9tbRGpDFHyU2c86xuJQKhj X-Received: by 2002:a17:906:aacd:: with SMTP id kt13mr1148120ejb.527.1610536511205; Wed, 13 Jan 2021 03:15:11 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1610536511; cv=none; d=google.com; s=arc-20160816; b=iL6cW2jJHeoFegDdTvF2mB2AoPvzD+Izcu7HVecgpaYSAOpggQcfU8gOlbZzxfJpDD 8z9Pc6MreQw2Orj1dovE5CJqVB0kGnxwegITh9taqIBuePrpDnBDTO3x6AvmJ7XawLB6 b7ybmE8yqN2M+DShtH+ypXc24w4a/Fnww9yVDZjirf8C1RxUIciY6gPJQomXHM95RVda 3NXXWJIQM9nE/vUyKkJ2rjCJLMGaNUPm8FDLAS1UdUz5fKi1HV02dggnh67mOOqfVq8F TxkRHPzaRJv1lqtX+b3ndhf3SiJAeB+TjVZSTdEewJCZON6veapbWid/J1IDfJu0HVKf VU8Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:message-id:user-agent:references:in-reply-to :subject:cc:to:from:date:content-transfer-encoding:mime-version; bh=S9k2h+9jLfNBJYQyHzw8hLvTRaEPuDIe4dpVkk0fP7c=; b=UFpZRySeHQmadJgDEwbpMkTqvWQCmrcASDF/VLkO30gxbx5U6RcZYffbBkNe8S5okp PCxq0s6/yHqqk3nlHIZbYdIJWBzxS8lk4H+ILB95S27loOOyW43Cy0k9jKGmHroUGxpr ptRlwT7dc2FT6qun4tRTAhb3YMXKfx6FXpDJDwRFyjoOmx9CDikULvHx6rqHLGxFqfEz JxmFeYo65LGM6i28mWnx/EXpb+g5ssYhNjc/gVUTgzEHYGZ3/JZxVe4tb8WJKhh/f1wB 2CETxv7Za+SH6AIiYRKHJfsjF98MwfsCCvZT1q57M3iE84CBay6LhEtA75nxgGCVA0E9 aOsQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id y26si757546ejb.149.2021.01.13.03.14.47; Wed, 13 Jan 2021 03:15:11 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728150AbhAMLMZ (ORCPT + 99 others); Wed, 13 Jan 2021 06:12:25 -0500 Received: from mail.kernel.org ([198.145.29.99]:41938 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728111AbhAMLMZ (ORCPT ); Wed, 13 Jan 2021 06:12:25 -0500 Received: from disco-boy.misterjones.org (disco-boy.misterjones.org [51.254.78.96]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 0C9B622286; Wed, 13 Jan 2021 11:11:44 +0000 (UTC) Received: from disco-boy.misterjones.org ([51.254.78.96] helo=www.loen.fr) by disco-boy.misterjones.org with esmtpsa (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.94) (envelope-from ) id 1kze3x-007EVt-Lk; Wed, 13 Jan 2021 11:11:41 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Wed, 13 Jan 2021 11:11:41 +0000 From: Marc Zyngier To: Greg Kroah-Hartman , Saravana Kannan Cc: "Rafael J. Wysocki" , kernel-team@android.com, linux-kernel@vger.kernel.org, Jisheng Zhang , Kevin Hilman , John Stultz , Nicolas Saenz Julienne Subject: Re: [PATCH v1 0/5] Enable fw_devlink=on by default In-Reply-To: References: <20201218031703.3053753-1-saravanak@google.com> User-Agent: Roundcube Webmail/1.4.9 Message-ID: X-Sender: maz@kernel.org X-SA-Exim-Connect-IP: 51.254.78.96 X-SA-Exim-Rcpt-To: gregkh@linuxfoundation.org, saravanak@google.com, rafael@kernel.org, kernel-team@android.com, linux-kernel@vger.kernel.org, Jisheng.Zhang@synaptics.com, khilman@baylibre.com, john.stultz@linaro.org, nsaenzjulienne@suse.de X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2021-01-07 20:05, Greg Kroah-Hartman wrote: > On Thu, Dec 17, 2020 at 07:16:58PM -0800, Saravana Kannan wrote: >> As discussed in LPC 2020, cyclic dependencies in firmware that >> couldn't >> be broken using logic was one of the last remaining reasons >> fw_devlink=on couldn't be set by default. >> >> This series changes fw_devlink so that when a cyclic dependency is >> found >> in firmware, the links between those devices fallback to permissive >> mode >> behavior. This way, the rest of the system still benefits from >> fw_devlink, but the ambiguous cases fallback to permissive mode. >> >> Setting fw_devlink=on by default brings a bunch of benefits >> (currently, >> only for systems with device tree firmware): >> * Significantly cuts down deferred probes. >> * Device probe is effectively attempted in graph order. >> * Makes it much easier to load drivers as modules without having to >> worry about functional dependencies between modules (depmod is still >> needed for symbol dependencies). >> >> Greg/Rafael, >> >> Can we get this pulled into 5.11-rc1 or -rc2 soon please? I expect to >> see some issues due to device drivers that aren't following best >> practices (they don't expose the device to driver core). Want to >> identify those early on and try to have them fixed before 5.11 >> release. >> See [1] for an example of such a case. > > Now queued up in my tree, will show up in linux-next in a few days, > let's see what breaks! :) > > And it is scheduled for 5.12-rc1, not 5.11, sorry. For the record, this breaks my rk3399 board, (NanoPC-T4) as no mass storage can be discovered (it lives on PCIe): (initramfs) find /sys -name 'waiting_for_supplier'| xargs grep .| egrep -v ':0$' /sys/devices/platform/ff3d0000.i2c/i2c-4/4-0022/waiting_for_supplier:1 /sys/devices/platform/f8000000.pcie/waiting_for_supplier:1 /sys/devices/platform/fe320000.mmc/waiting_for_supplier:1 /sys/devices/platform/sdio-pwrseq/waiting_for_supplier:1 /sys/devices/platform/ff3c0000.i2c/i2c-0/0-001b/waiting_for_supplier:1 Enabling the debug prints in device_links_check_suppliers(), I end up with the dump below (apologies for the size). This seems to all hang on the GPIO banks, but it is pretty unclear what is wrong with them. Happy to test things further. M. platform vcc3v3-sys: probe deferral - supplier vcc12v0-sys not ready platform vcc5v0-sys: probe deferral - supplier vcc12v0-sys not ready platform vcc1v8-s3: probe deferral - wait for supplier pmic@1b platform vcc3v0-sd: probe deferral - supplier vcc3v3-sys not ready platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform vbus-typec: probe deferral - supplier vcc5v0-sys not ready platform vcc5v0-host0: probe deferral - supplier vcc5v0-sys not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - wait for supplier pmic@1b platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - wait for supplier pmic@1b platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - wait for supplier pmic@1b platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - wait for supplier pmic@1b platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - wait for supplier pmic@1b platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - wait for supplier pmic@1b platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform fe320000.mmc: probe deferral - wait for supplier pmic@1b platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - wait for supplier pmic@1b platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform fe320000.mmc: probe deferral - wait for supplier pmic@1b platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - supplier 0-001b not ready platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform fe320000.mmc: probe deferral - wait for supplier gpio0@ff720000 platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - supplier 0-001b not ready platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform fe320000.mmc: probe deferral - wait for supplier gpio0@ff720000 platform fe300000.ethernet: probe deferral - supplier 0-001b not ready platform fe380000.usb: probe deferral - supplier ff770000.syscon:usb2-phy@e450 not ready platform fe3c0000.usb: probe deferral - supplier ff770000.syscon:usb2-phy@e460 not ready platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - supplier 0-001b not ready platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 platform fe3a0000.usb: probe deferral - supplier ff770000.syscon:usb2-phy@e450 not ready platform fe3e0000.usb: probe deferral - supplier ff770000.syscon:usb2-phy@e460 not ready platform fe320000.mmc: probe deferral - wait for supplier gpio0@ff720000 platform fe300000.ethernet: probe deferral - supplier 0-001b not ready platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - supplier 0-001b not ready platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 i2c 0-001b: probe deferral - wait for supplier gpio1@ff730000 platform fe320000.mmc: probe deferral - wait for supplier gpio0@ff720000 platform fe300000.ethernet: probe deferral - supplier 0-001b not ready platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - supplier 0-001b not ready platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 i2c 0-001b: probe deferral - wait for supplier gpio1@ff730000 platform fe320000.mmc: probe deferral - wait for supplier gpio0@ff720000 platform fe300000.ethernet: probe deferral - supplier 0-001b not ready platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - supplier 0-001b not ready platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 i2c 0-001b: probe deferral - wait for supplier gpio1@ff730000 platform fe320000.mmc: probe deferral - wait for supplier gpio0@ff720000 platform fe300000.ethernet: probe deferral - supplier 0-001b not ready platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - supplier 0-001b not ready platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 i2c 0-001b: probe deferral - wait for supplier gpio1@ff730000 platform fe320000.mmc: probe deferral - wait for supplier gpio0@ff720000 platform fe300000.ethernet: probe deferral - supplier 0-001b not ready platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 platform vcc1v8-s3: probe deferral - supplier 0-001b not ready platform vcca0v9-s3: probe deferral - supplier vcc1v8-s3 not ready platform vcca1v8-s3: probe deferral - supplier vcc1v8-s3 not ready platform f8000000.pcie: probe deferral - wait for supplier gpio2@ff780000 i2c 0-001b: probe deferral - wait for supplier gpio1@ff730000 platform fe320000.mmc: probe deferral - wait for supplier gpio0@ff720000 platform fe300000.ethernet: probe deferral - supplier 0-001b not ready platform sdio-pwrseq: probe deferral - wait for supplier gpio0@ff720000 -- Jazz is not dead. It just smells funny...