From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3492836F428 for ; Sat, 29 Aug 2026 12:37:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.176 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788007055; cv=none; b=cFU+3UZKFKrban4Oz64NyQAua5zPzhKBJZ5iSEKuPzS7NAP7ENsndXHrzS/Cj73BuJ6j2uKXOIsYci4uUctJ5jYcd/JFIlGf48ItM3UOllyevDuYCPCVRk80jKvMOmUz4peFiCrsGVxmSLFjMvIQYfVuEzc5wA24pB0WgieJ1Is= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788007055; c=relaxed/simple; bh=jZ/lKIPH3msbV5HTdGIEOWuVbVvQFeF5aIS22QAKE1E=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=YKCmudRCtizL1TBzOVJwMWFAt0VXy9O57V8cQWCrkJs/HvS8td3MAvTTmpKaGROY/E6z2sqhSqUrpkCZYeTCcy29Y2+sJ7PN5bBGQjm+BiZmZnhLUXh24HUurkoFpjSbfedlsng/e64TkTOTfnDiRhwmo6tolrfltA1AIBn9l7g= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Nd/4LCly; arc=none smtp.client-ip=209.85.214.176 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Nd/4LCly" Received: by mail-pl1-f176.google.com with SMTP id d9443c01a7336-2d590f4c291so5328015ad.0 for ; Sat, 29 Aug 2026 05:37:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788007053; x=1788611853; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=jZ/lKIPH3msbV5HTdGIEOWuVbVvQFeF5aIS22QAKE1E=; b=Nd/4LClyODk27e6ZJMBxAQODmhe9JGZ/zkKakM7KucleOtUz1JPmzloeDtEsu6Vvvz 9f4j5K/wDRIsXd20eh697wIExGsuQv27K2y1eU/zpT1a1zi3XyyxA3MWc8ifQguTO4LK qiLjRZIgZaoTNrJkHV+gz3M2IORvGsvZj3T7UDD24pyFUw4ioTn8keryhHiX7jnYnIzm 7d9pCrTSZD5aJSf9AnYLX9+YZrN15diTiYJ09tGjifvFYg0hPdlWlmmSvMflqanznUDB 2ox2K3cfuIqq6f6RcPxUw6C7wsz9INHThU1yrJQVuQ3lQXOFlL6o9RY+VkZs9+tahMxw haDg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788007053; x=1788611853; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=jZ/lKIPH3msbV5HTdGIEOWuVbVvQFeF5aIS22QAKE1E=; b=rPbHbosXujIT1B19Xk7yHDwVzODoF9G26ZoQU1r50QUmMJslK8FWOgtceaHdxRMTDb RQPmLX5chYInqsAxeb9hFPEHZedgxXCWg+t5EREKOG36U7B93+t3HwPg6Kb09m+nfeha qZPVJN5ynST/lm228zQ94U7PcoJ0qY03NfHSLBtnseaCvZ8KmTnBpCKylUbM8AKm2KKT FMDkHvRlVZ1ZzpuRpOSIcDWZ/vPaLrldElaYrqPaEYJ25A4o6vztT3WuxrNeoKP3CFbI x5QB29E8igZnhvQYsxyZiLlwUHjEFwdT1K5owEOa27FgbLEnXBO6Quh8UCVS5CXSp2kG ttew== X-Gm-Message-State: AFuF++lvV2mxjnCt/AA9+0VvcgEmX9MLfEhkLIAzZtX1CWR2PriJzkSJ Agi9jcWazzgaKzBWpY4FgfZzJrWlLfabwsXS3e2U63EXQKkyT8VCrzUxA9nkjFVoNRI= X-Gm-Gg: AYBFou1o+rdzasNtPMVecjuwNokpVmYZrG7vlmqVD5RE85F95n1tAhrqU553Eq/CXCK 6tZjrUDOSjwaZzZQIwNH7Ea4C33rIM6oYyLRc6sQIPX5veVyC63/pGeUAXb1Z9MZoYLlFf0gBVC wxuLmOKyFy7lPDvwOlYQ92htBpqV58iCALp94JLrzc/2nA9F27u5n5tYppTAlCleLYGbdofROUk vznafKR7Rd/y31Ingr6e3F+N+9gHIT67WIFZLffLRoRJzkC2FTJQM+QYGqVjE1/X3El2Jdez65z r/h9o6HAbJ/b5twQV8KQJZ+xT6Fv39veqZ5rWZ+pnmxgo9f3g6CeyfcGsNn9j+YFAeQGquBR7Dm e+slyikfe+DRTCwl1MQGICfL9L0PhVc/luA+HZwOjxVdCT+x/AjVppwNWFwPdR8o8Qe+HnN+2jv aKBIQWKIkHFxLpsNl3uhct8E88TOe1uFj2A2AwU7kvxt2FdHXrebamFZOtKep/RzFTgYq4Gc9NJ eWszCuuefQqkXbdOaE1Cce0/EREmClN32HLlI8= X-Received: by 2002:a17:90b:17cd:b0:398:bad2:c10 with SMTP id 98e67ed59e1d1-398bad20e70mr1703035a91.6.1788007053352; Sat, 29 Aug 2026 05:37:33 -0700 (PDT) Received: from cachyos-aura ([45.112.149.37]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-142e0d1b38esm14008113c88.3.2026.08.29.05.37.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 29 Aug 2026 05:37:32 -0700 (PDT) From: Navon John Lukose To: linux-wireless@vger.kernel.org, miriam.rachel.korenblit@intel.com Cc: nika@nikableh.moe, emmanuel.grumbach@intel.com, helgaas@kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Navon John Lukose , stable@vger.kernel.org Subject: Re: [PATCH wireless 1/2] wifi: iwlwifi: pcie: arm the product reset at probe Date: Sat, 29 Aug 2026 18:07:25 +0530 Message-ID: <20260829123725.86014-1-navonjohnlukose@gmail.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260829100758.8E8491F000E9@smtp.kernel.org> References: <20260829095437.44716-1-navonjohnlukose@gmail.com> <20260829100758.8E8491F000E9@smtp.kernel.org> Precedence: bulk X-Mailing-List: linux-wireless@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Both findings are correct. Please drop this series; a v2 is coming. On the first: arming at probe does leave the mode selected for the lifetime of the driver, and the disarm in iwl_trans_pcie_removal_wk() cannot be relied on to undo it, because it fails in exactly the case that matters - the DSM is gated on the platform reading the device's PCI ID out of config space, so it is unavailable once the device is off the bus, and set_product_reset() ignores that failure when disarming. _RST is not gated the same way: it reads the mode variable directly, so a stale selection does execute a full product reset, Bluetooth off/on included, with no BT teardown and with the ME downgrade bypassed. I also need to retract something. The cover letter argued for stable on the grounds that "every _RST evaluation is already preceded by set_product_reset() setting the mode that reset wants, so arming at probe cannot alter the behaviour of any later reset". That is wrong. It holds only when the disarm succeeds, and the disarm cannot succeed on a device that is gone. The backport rationale as written does not stand. On the second: yes, it logs IWL_ERR on every probe on any platform without this DSM, and the commit message's claim that it is a no-op there is wrong. The two neighbouring functions, iwl_trans_pcie_check_product_reset_mode() and _status(), already return silently in the same situation, so the asymmetry looks unintended - and it is what hid the failed disarm above. v2 will instead select the mode from the suspend callback and clear it on resume, so it is only selected across the suspend window; guard the _RST call with pci_device_is_present(), which reads the same config register the platform's own gate does; and demote the log. Unrelated, but found while checking this: iwl_pcie_recheck_me_status() reads CSR_HW_IF_CONFIG_REG without a liveness check, so on a device that is off the bus it sees 0xffffffff, concludes IAMT_UP is set, and marks ME present on a machine that has none - which then downgrades every later product reset. I will send that separately. Patch 2/2 is substantively unchanged in v2. Thanks for the review. Navon