From patchwork Mon Jul 10 10:47:04 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Harry Austen X-Patchwork-Id: 117848 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:9f45:0:b0:3ea:f831:8777 with SMTP id v5csp4938946vqx; Mon, 10 Jul 2023 04:03:39 -0700 (PDT) X-Google-Smtp-Source: APBJJlGZgiQ6oTNvtVTUVKAEhb08HpfV3MycRwDGuWKOsWpW1buwOApzcdEla/LRsZhpXR381Caw X-Received: by 2002:a05:6808:2117:b0:3a3:f72c:793e with SMTP id r23-20020a056808211700b003a3f72c793emr3735784oiw.31.1688987018851; Mon, 10 Jul 2023 04:03:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1688987018; cv=none; d=google.com; s=arc-20160816; b=Hh6Fyfm4IATIgnd8z6lOcgxoo+vmx4FlmrCEsuXxgjdV48nAVm5z3Ilhs7kXN5XOVh anfb9flrl8nhScz2dT4XDxTJbXlQikyhtkdQIt29kwE0sDQP0wEaSFxp5sZWKUAmo41U SezpJVI2ANvb2C47HaHF6kqcNg6d+oSKDOcoDtBbMfiUHlvp+7mMWs1XllY2b5hDADZL hDbMN5t8Nyyg7OIg8OOBAmWi6VttbaqiAIN5wQvxVcA0miON7imoN4WlsYHkZOUcs4z1 jIqCWIABAwX86tmAdBp3Ih82cxG43D9tQjCK/ntHfmOKwdo1OAreE1ZH64kWw4HkTGzQ 0C0A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :feedback-id:message-id:subject:cc:from:to:dkim-signature:date; bh=JUSxIhlyp5Rsy2QluQCpjPVJMXty3tKlls6ZhMmED8c=; fh=S7+p+0KGJtgQ3/6xTIKa3/1oTdesEkcMmD5Lw5nFylU=; b=c466pu2Ki7E0JBgA6+MLXydx8Udq+gO9ZRv9d9Vrx+BizhLwqyRTruZp8wBiZniHdK uTPxmVZB6uI11L9gTut0lhBoSiGmz5l1g+2Wc4JAbZ6/iJG0pTvud3Mw4IBZ7xbR80P3 uINYWTQ98ecWSiRtkKlNd2yzHIGSiJlXtldPm18EaQz669qrDM0sU+2nUO/Z6B8DsEsk FV9z+UxsDtiY9JGw2iaeLsvlBf7lD0eXpj2I9KEYwWNyI+1+ovoHLWKqMEcAyVCMVZ/u FvPMhP8MY4v+ipxHR5a/KcMh+sFpWDLG5NBWZEOYg9Q8knuAWRkQH6EZH+eYLCw4Z67I Okxw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@protonmail.com header.s=protonmail3 header.b=oGK35t+2; 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=QUARANTINE sp=QUARANTINE dis=NONE) header.from=protonmail.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id k5-20020a632405000000b0054f904a0a0dsi2244420pgk.624.2023.07.10.04.03.25; Mon, 10 Jul 2023 04:03:38 -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=@protonmail.com header.s=protonmail3 header.b=oGK35t+2; 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=QUARANTINE sp=QUARANTINE dis=NONE) header.from=protonmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230248AbjGJKrO (ORCPT + 99 others); Mon, 10 Jul 2023 06:47:14 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50538 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229522AbjGJKrM (ORCPT ); Mon, 10 Jul 2023 06:47:12 -0400 Received: from mail-40134.protonmail.ch (mail-40134.protonmail.ch [185.70.40.134]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 92405B5; Mon, 10 Jul 2023 03:47:10 -0700 (PDT) Date: Mon, 10 Jul 2023 10:47:04 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=protonmail.com; s=protonmail3; t=1688986028; x=1689245228; bh=JUSxIhlyp5Rsy2QluQCpjPVJMXty3tKlls6ZhMmED8c=; h=Date:To:From:Cc:Subject:Message-ID:Feedback-ID:From:To:Cc:Date: Subject:Reply-To:Feedback-ID:Message-ID:BIMI-Selector; b=oGK35t+2jrXUNI7a8SJ7Is0Dge8JAq+HiFi8NW5VHDdxm6M5ZHBMmjd1QYqt27v6x l5TI5rV29lpRsdVb+aDSpjmTyP8ZP7QWFBmlgEmm4IA4OGmNdceNQnME2CH5Qhd6YV tTzMV7HPbfxlhMmcUymsZKC8K5mdWVLpbr5Yy0efHWYhYaw+tybokV5M3ZckKrcnTt o5gD1OKC2QDZrjz71KUtYFIOgWjSvxcMxzcRQzWpQ5GWWUqp/rpQjVmCwMf5VYVBaS GakmffrhJ+lWtDRMLy6VnFV3TZ3iqpF1SLth5ydmVAV6PMAh2gEnqGhFrSfeeEnkg1 45kko/82wvJCg== To: linux-wireless@vger.kernel.org From: Harry Austen Cc: linux-kernel@vger.kernel.org, Gregory Greenman , Kalle Valo , Johannes Berg , Miri Korenblit , Avraham Stern , Daniel Lezcano , Zhang Rui , Harry Austen Subject: [PATCH] wifi: iwlwifi: mvm: enable thermal zone only when firmware is loaded Message-ID: <20230710104626.8399-1-hpausten@protonmail.com> Feedback-ID: 53116287:user:proton MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,RCVD_IN_MSPIKE_H5, RCVD_IN_MSPIKE_WL,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham 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 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1771031252524827755 X-GMAIL-MSGID: 1771031252524827755 In iwl_mvm_thermal_zone_register(), when registering a thermal zone, the thermal subsystem will evaluate its temperature. But iwl_mvm_tzone_get_temp() fails at this time because iwl_mvm_firmware_running() returns false. And that's why many users report that they see "thermal thermal_zoneX: failed to read out thermal zone (-61)" message during wifi driver probing. This patch attempts to fix this by delaying enabling of the thermal zone until after the firmware has been loaded/initialized. It also gets disabled when going into suspend. Signed-off-by: Harry Austen --- .../net/wireless/intel/iwlwifi/mvm/mac80211.c | 18 ++++++++++++++++++ drivers/net/wireless/intel/iwlwifi/mvm/tt.c | 9 +-------- 2 files changed, 19 insertions(+), 8 deletions(-) -- 2.41.0 diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c index ce7905faa08f..a47d29a64dd4 100644 --- a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c +++ b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c @@ -1187,6 +1187,17 @@ int iwl_mvm_mac_start(struct ieee80211_hw *hw) mutex_unlock(&mvm->mutex); +#ifdef CONFIG_THERMAL + /* Needs to be done outside of mutex guarded section to prevent deadlock, since enabling + * the thermal zone calls the .get_temp() callback, which attempts to acquire the mutex. + */ + if (!ret) { + ret = thermal_zone_device_enable(mvm->tz_device.tzone); + if (ret) + IWL_DEBUG_TEMP(mvm, "Failed to enable thermal zone (err = %d)\n", ret); + } +#endif + iwl_mvm_mei_set_sw_rfkill_state(mvm); return ret; @@ -1282,6 +1293,7 @@ void __iwl_mvm_mac_stop(struct iwl_mvm *mvm) void iwl_mvm_mac_stop(struct ieee80211_hw *hw) { struct iwl_mvm *mvm = IWL_MAC80211_GET_MVM(hw); + int ret; flush_work(&mvm->async_handlers_wk); flush_work(&mvm->add_stream_wk); @@ -1307,6 +1319,12 @@ void iwl_mvm_mac_stop(struct ieee80211_hw *hw) iwl_mvm_mei_set_sw_rfkill_state(mvm); +#ifdef CONFIG_THERMAL + ret = thermal_zone_device_disable(mvm->tz_device.tzone); + if (ret) + IWL_DEBUG_TEMP(mvm, "Failed to disable thermal zone (err = %d)\n", ret); +#endif + mutex_lock(&mvm->mutex); __iwl_mvm_mac_stop(mvm); mutex_unlock(&mvm->mutex); diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/tt.c b/drivers/net/wireless/intel/iwlwifi/mvm/tt.c index 157e96fa23c1..964d2d011c6b 100644 --- a/drivers/net/wireless/intel/iwlwifi/mvm/tt.c +++ b/drivers/net/wireless/intel/iwlwifi/mvm/tt.c @@ -680,7 +680,7 @@ static struct thermal_zone_device_ops tzone_ops = { static void iwl_mvm_thermal_zone_register(struct iwl_mvm *mvm) { - int i, ret; + int i; char name[16]; static atomic_t counter = ATOMIC_INIT(0); @@ -707,13 +707,6 @@ static void iwl_mvm_thermal_zone_register(struct iwl_mvm *mvm) return; } - ret = thermal_zone_device_enable(mvm->tz_device.tzone); - if (ret) { - IWL_DEBUG_TEMP(mvm, "Failed to enable thermal zone\n"); - thermal_zone_device_unregister(mvm->tz_device.tzone); - return; - } - /* 0 is a valid temperature, * so initialize the array with S16_MIN which invalid temperature */