Received: by 2002:a6b:fb09:0:0:0:0:0 with SMTP id h9csp3542941iog; Mon, 27 Jun 2022 19:29:30 -0700 (PDT) X-Google-Smtp-Source: AGRyM1sNZOVmVxzJ9+TiwDfuyfu3+73ROi2Vr6A2EVV2GO6DIZTjM9GBm+XHnoYpnhO4oYggf2ES X-Received: by 2002:a17:902:d488:b0:16a:158e:dd0b with SMTP id c8-20020a170902d48800b0016a158edd0bmr1280777plg.162.1656383370096; Mon, 27 Jun 2022 19:29:30 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1656383370; cv=none; d=google.com; s=arc-20160816; b=xvQPLbdGUUnoy6yNNY4XWD922x6nphEXTknFKRj8gDtzHk3CiyEcYIAvN5O4t1cc2w jCvRv6qqfuCUmhjEJslsfnBQl9teUOWeki/TuU2djyyaCaLEec7ZAfSi5nFnJ3B+sdSD la3iKSiD5lM+8HRRpvx7xxXo0TJWaOOewb5cAlFxgTx3uS4QSoo4CeHySj6z4YkowLmI y2JJi3Yk3BfV2OmhoGdr0mM+HbwGxpe4Y2QAX8/hC03/hhm7legsLv1gCfqKeJzLDY8e QEE7sT7GsscxxHXLIG3v4AkWFIETwaw3B7rtLEsnwB3WjMJZmmDOvL4hSdqs+fz5Z5nW xJRQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:from:subject:references:mime-version :message-id:in-reply-to:date:dkim-signature; bh=YzHIr0V9HMEpxWmEi0Xj1DJ+4d2XCpwecNoe7o3TDGc=; b=IbapUkSTAtuH+/SCDBktND4XeAjnLKBKHjH/oTU7b27ebfeR0XH3e0RFMS+TmbsA5n 3Erh+MDPBq+RNtzAuCDu2x73Cpe9jpe4fjBQS41rYvdo8+EBdFosh5PTPNQw3KigsVP9 WyRxp7ll91hlIfmdngDfm3X5HFjvhqb0NazQMk0Tm3lAfhqEneJsV89OU0m9KAX3aykS 6GwsJzTR7NvJCdS+uvdjzadotkF3EbxdO2upgcbset6Op3IJIzR6hH/9+imug5BgxbAc ACTuh40BM8fHW/THdwjxIDmlqXEjTG9mRcqkpKEWQ8lr8G0PSB+ooPuy/6GcvZtC33VP ndqQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20210112 header.b=WOYPYkzc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 77-20020a630050000000b003fc51453576si16185908pga.545.2022.06.27.19.29.16; Mon, 27 Jun 2022 19:29:30 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20210112 header.b=WOYPYkzc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239255AbiF1CDR (ORCPT + 99 others); Mon, 27 Jun 2022 22:03:17 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46200 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238911AbiF1CDL (ORCPT ); Mon, 27 Jun 2022 22:03:11 -0400 Received: from mail-yb1-xb49.google.com (mail-yb1-xb49.google.com [IPv6:2607:f8b0:4864:20::b49]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CA278BF42 for ; Mon, 27 Jun 2022 19:03:08 -0700 (PDT) Received: by mail-yb1-xb49.google.com with SMTP id b11-20020a5b008b000000b00624ea481d55so9766447ybp.19 for ; Mon, 27 Jun 2022 19:03:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=date:in-reply-to:message-id:mime-version:references:subject:from:to :cc; bh=YzHIr0V9HMEpxWmEi0Xj1DJ+4d2XCpwecNoe7o3TDGc=; b=WOYPYkzcX52RhlZvpifs4I6lPFXV+5C99yJcPGn4swZ412myqZhpEpJGG6JT8I+diP wmRTXCjvAFpcr/KomN65nvaJHvYHD4UymktEo8kdn5KBIufKB90oBi6uJgpha1QorqN4 FoHUev8RwrPNG+R9y5XCbQZGHm1ydbiuEHKmkqJMqr5AQVuz0Cn/swj+fCPxGk277wcm FeyAuVEaB9fajv1pTRajXKWxqwQb5RRy57WkXDveO+g9jxpPIug/uOD6UHEfjE7VM4eE 2vOtDCyMr9g/g/WN/lFcVpdMLesGVIWYzLk/qWic19C/MVsZG3elryvTosIN7O8GZ4ml x6ow== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:in-reply-to:message-id:mime-version :references:subject:from:to:cc; bh=YzHIr0V9HMEpxWmEi0Xj1DJ+4d2XCpwecNoe7o3TDGc=; b=WSjkbFF5xwXg8EkDyi8z0rNA13BKIBrGYajmTaPNgDC1X5NXrEaeV8f792lm6dmzec ++4LTl2O6UAESPbQP518iP9eQprn+lIuVof8S9x6f6oyjSOqkYinpmYDJR3Qy0QD/RTx 9HXBS/cTpT76OI2WG18opuzq7Rf7/ckpBI12zpHkudEXBoA4c8ZE/2EgAQAymR0AlYVD QWTQGcvssifjt3Ld+WNjMc7SGB4XyHwgfcBEW5rxchmKnyksXlqgcq8e210S6zZRhSmb 8QrvD/aW3pGpQxlU7T6Sc8mG9Q+hdMvZ6zvyPby3MNhKoct3u3NEpnwxUXPgOoCvrhry ytqA== X-Gm-Message-State: AJIora+4o1cSwLoWPb2T49OP/AKRv/nvo5i8sZcSrUKjRfEhtT1ML3e6 zB26+CHQwnhp0UmEDMtHc1b40G95qr5EPR4= X-Received: from saravanak.san.corp.google.com ([2620:15c:2d:3:1f27:a302:2101:1c82]) (user=saravanak job=sendgmr) by 2002:a81:12d3:0:b0:317:aa20:e4eb with SMTP id 202-20020a8112d3000000b00317aa20e4ebmr18821146yws.352.1656381787829; Mon, 27 Jun 2022 19:03:07 -0700 (PDT) Date: Mon, 27 Jun 2022 19:01:02 -0700 In-Reply-To: <20220628020110.1601693-1-saravanak@google.com> Message-Id: <20220628020110.1601693-2-saravanak@google.com> Mime-Version: 1.0 References: <20220628020110.1601693-1-saravanak@google.com> X-Mailer: git-send-email 2.37.0.rc0.161.g10f37bed90-goog Subject: [PATCH v1 1/2] driver core: Add probe_no_timeout flag for drivers From: Saravana Kannan To: Greg Kroah-Hartman , "Rafael J. Wysocki" , Laurentiu Tudor , Jiri Slaby , Michael Ellerman , Benjamin Herrenschmidt , Paul Mackerras , Joel Stanley , Andrew Jeffery , Florian Fainelli , Ray Jui , Scott Branden , Broadcom internal kernel review list , Nicolas Saenz Julienne , Al Cooper , Andy Shevchenko , Paul Cercueil , Vladimir Zapolskiy , Matthias Brugger , Thierry Reding , Jonathan Hunter , Kunihiko Hayashi , Masami Hiramatsu , Tobias Klauser , Russell King , Vineet Gupta , Richard Genoud , Nicolas Ferre , Alexandre Belloni , Claudiu Beznea , Alexander Shiyan , Baruch Siach , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Karol Gugala , Mateusz Holenko , Gabriel Somlo , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Taichi Sugaya , Takao Orito , Liviu Dudau , Sudeep Holla , Lorenzo Pieralisi , Andy Gross , Bjorn Andersson , Pali Rohar , Andreas Farber , Manivannan Sadhasivam , Krzysztof Kozlowski , Alim Akhtar , Laxman Dewangan , Palmer Dabbelt , Paul Walmsley , Orson Zhai , Baolin Wang , Chunyan Zhang , Patrice Chotard , Maxime Coquelin , Alexandre Torgue , "David S. Miller" , Hammer Hsieh , Peter Korsgaard , Timur Tabi , Michal Simek , Saravana Kannan Cc: Rob Herring , sascha hauer , peng fan , kevin hilman , ulf hansson , len brown , pavel machek , joerg roedel , will deacon , andrew lunn , heiner kallweit , eric dumazet , jakub kicinski , paolo abeni , linus walleij , hideaki yoshifuji , david ahern , kernel-team@android.com, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, iommu@lists.linux-foundation.org, netdev@vger.kernel.org, linux-gpio@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-serial@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, linux-rpi-kernel@lists.infradead.org, linux-mips@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-tegra@vger.kernel.org, linux-snps-arc@lists.infradead.org, linux-amlogic@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-actions@lists.infradead.org, linux-unisoc@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-riscv@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, sparclinux@vger.kernel.org, Ahmad Fatoum Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-9.6 required=5.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,USER_IN_DEF_DKIM_WL autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This flag only needs to be set for drivers of devices that meet all the following conditions: - Need to probe successfully before userspace init in started - Have optional suppliers - Can't wait for deferred_probe_timeout to expire fw_devlink=on uses this info, as needed, to ignore dependencies on supplier devices that have not been added or supplier devices that don't have any drivers. It's still up to the driver to decide which of the missing suppliers are optional or not. Fixes: 71066545b48e ("driver core: Set fw_devlink.strict=1 by default") Reported-by: Sascha Hauer Reported-by: Peng Fan Reported-by: Fabio Estevam Reported-by: Ahmad Fatoum Signed-off-by: Saravana Kannan --- drivers/base/base.h | 1 + drivers/base/core.c | 7 +++++++ drivers/base/dd.c | 3 +++ include/linux/device.h | 7 +++++++ include/linux/device/driver.h | 11 +++++++++++ 5 files changed, 29 insertions(+) diff --git a/drivers/base/base.h b/drivers/base/base.h index b3a43a164dcd..149822d2086f 100644 --- a/drivers/base/base.h +++ b/drivers/base/base.h @@ -193,6 +193,7 @@ extern void device_links_no_driver(struct device *dev); extern bool device_links_busy(struct device *dev); extern void device_links_unbind_consumers(struct device *dev); extern void fw_devlink_drivers_done(void); +extern void fw_devlink_probe_no_timeout(void); /* device pm support */ void device_pm_move_to_tail(struct device *dev); diff --git a/drivers/base/core.c b/drivers/base/core.c index ccdd5b4295de..8e18904a1584 100644 --- a/drivers/base/core.c +++ b/drivers/base/core.c @@ -54,6 +54,7 @@ static unsigned int defer_sync_state_count = 1; static DEFINE_MUTEX(fwnode_link_lock); static bool fw_devlink_is_permissive(void); static bool fw_devlink_drv_reg_done; +static bool fw_devlink_no_timeout; static bool fw_devlink_best_effort; /** @@ -969,6 +970,7 @@ static void device_links_missing_supplier(struct device *dev) static bool dev_is_best_effort(struct device *dev) { return (fw_devlink_best_effort && dev->can_match) || + (fw_devlink_no_timeout && dev->probe_no_timeout) || (dev->fwnode && (dev->fwnode->flags & FWNODE_FLAG_BEST_EFFORT)); } @@ -1688,6 +1690,11 @@ void fw_devlink_drivers_done(void) device_links_write_unlock(); } +void fw_devlink_probe_no_timeout(void) +{ + fw_devlink_no_timeout = true; +} + /** * wait_for_init_devices_probe - Try to probe any device needed for init * diff --git a/drivers/base/dd.c b/drivers/base/dd.c index e600dd2afc35..9b0ef2b6a058 100644 --- a/drivers/base/dd.c +++ b/drivers/base/dd.c @@ -324,6 +324,8 @@ static int deferred_probe_initcall(void) if (!IS_ENABLED(CONFIG_MODULES)) fw_devlink_drivers_done(); + else + fw_devlink_probe_no_timeout(); /* * Trigger deferred probe again, this time we won't defer anything @@ -734,6 +736,7 @@ static int __driver_probe_device(struct device_driver *drv, struct device *dev) return -EBUSY; dev->can_match = true; + dev->probe_no_timeout = drv->probe_no_timeout; pr_debug("bus: '%s': %s: matched device %s with driver %s\n", drv->bus->name, __func__, dev_name(dev), drv->name); diff --git a/include/linux/device.h b/include/linux/device.h index 424b55df0272..e6246b6cf6cf 100644 --- a/include/linux/device.h +++ b/include/linux/device.h @@ -536,6 +536,12 @@ struct device_physical_location { * @can_match: The device has matched with a driver at least once or it is in * a bus (like AMBA) which can't check for matching drivers until * other devices probe successfully. + * @probe_no_timeout: Set by driver core to indicate that this device's probe + * can't wait till driver_probe_timeout expires. This information + * is used by fw_devlink=on to avoid deferring the probe of this + * device to wait on supplier devices that haven't been added or + * probed successfully. + * See also: probe_no_timeout in struct driver. * @dma_coherent: this particular device is dma coherent, even if the * architecture supports non-coherent devices. * @dma_ops_bypass: If set to %true then the dma_ops are bypassed for the @@ -642,6 +648,7 @@ struct device { bool of_node_reused:1; bool state_synced:1; bool can_match:1; + bool probe_no_timeout:1; #if defined(CONFIG_ARCH_HAS_SYNC_DMA_FOR_DEVICE) || \ defined(CONFIG_ARCH_HAS_SYNC_DMA_FOR_CPU) || \ defined(CONFIG_ARCH_HAS_SYNC_DMA_FOR_CPU_ALL) diff --git a/include/linux/device/driver.h b/include/linux/device/driver.h index 7acaabde5396..2ce60e511504 100644 --- a/include/linux/device/driver.h +++ b/include/linux/device/driver.h @@ -55,6 +55,15 @@ enum probe_type { * @owner: The module owner. * @mod_name: Used for built-in modules. * @suppress_bind_attrs: Disables bind/unbind via sysfs. + * @probe_no_timeout: Set to true by drivers that bind to devices that meet all + * these conditions: + * - Need to probe successfully before userspace init in started + * - Have optional suppliers + * - Can't wait for deferred_probe_timeout to expire + * fw_devlink=on uses this info, as needed, to ignore dependencies + * on supplier devices that have not been added or supplier devices + * that don't have any drivers. It's still up to the driver to + * decide which of the missing suppliers are optional or not. * @probe_type: Type of the probe (synchronous or asynchronous) to use. * @of_match_table: The open firmware table. * @acpi_match_table: The ACPI match table. @@ -101,6 +110,8 @@ struct device_driver { const char *mod_name; /* used for built-in modules */ bool suppress_bind_attrs; /* disables bind/unbind via sysfs */ + bool probe_no_timeout; + enum probe_type probe_type; const struct of_device_id *of_match_table; -- 2.37.0.rc0.161.g10f37bed90-goog