From nobody Sat Feb 7 21:23:48 2026 Received: from mail-pl1-f194.google.com (mail-pl1-f194.google.com [209.85.214.194]) (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 B56BD3596E9 for ; Fri, 6 Feb 2026 09:23:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.194 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770369822; cv=none; b=jpNR6eSqXGy6sJ0urjUdA1jkLRMqWdFMk6/xuN7JtkG7F+py6AXyNwotvKHpsk7I03CAQanOSIdlas5jlz+5pvb1FiHa8dPxks5aVtjg83nPe6p4fFNN7a405SO2ujs8hBn0vhtzamr4runyTqsC7FTUG0dusIvMAbqLQ66/Z7Y= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770369822; c=relaxed/simple; bh=oHrIAjiA3w8rr/43sMJV+DvF/OlfA20d1q3N81NAVV4=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=nlL7Ym/5Kp1HTy7Cwj3NsPla06YBRKgU2owQ/TDg93K85GdrkQExDk2PoqQkWMcmoIObgwtSOshiTqy+7cxdvAjMZnTU9H8epjsHRXSKDVE2QZek2H1iI7HvcK1BJqLrVX0LYbLyTCH1ZI6hwnZ0DGywOwxWPdBccdB5JaNBuMY= 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=l+W61SwE; arc=none smtp.client-ip=209.85.214.194 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="l+W61SwE" Received: by mail-pl1-f194.google.com with SMTP id d9443c01a7336-2a929245b6aso4588735ad.0 for ; Fri, 06 Feb 2026 01:23:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1770369822; x=1770974622; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=yuTzOBlCIALqpyQtHwJ+tH2O4xb49e/uTcQGRnE7ybI=; b=l+W61SwEe4A8hXGObDhNqGpYIF75xG36MbZipS38qFTOvRXGOH8HLrWwGXt1Erf033 EfVTdnkC+qn8rTfVrAgY1P/3nbV2uyeWURKgtsvpQ8k+koab7Gfs1q/GkgZVNo43z24p ajwRiAL0HxhQT6+dZ574ycOnRu3IFRp9V/PGgMp9ekJQhLfoYsK2PCE0mc7tM4kIOcMo eHNhynCoXzYRe4SGxe0n0/5Km84qtCvdQLR3WT4GE/k6Tocsi2uKFU+FZ3lpQOz4AoR8 mPU359KOc4wu0T1vAtXX1mEp9X1YkGD3kkjc0KZgrKK1kCRcn4ujxTE7w2kcK7POaVkU XpZQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1770369822; x=1770974622; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=yuTzOBlCIALqpyQtHwJ+tH2O4xb49e/uTcQGRnE7ybI=; b=ln5CwXzIT+KzWFiunGh5xQ1MP3J2c2ZtUNqJRGR/i6vYylfsC/PXlMiFYiqTJtvbud p0VIXZhroWo4bBfG8uTKjjrjR3QWmgp1YhSxKWM5olX0bKhPXGfqqWk3Tw9ZZglFEhGZ v+lQ0dduaOtETyMt2RTVE0fbQ23IjerBJkFJzX8KsNpIdFW4BIXubjFFwYFWeF3nGQP2 eGW2pkzlViPkxpdFM1nIKpVH8T71I/ZmQ0FMdyEWsyfT0xhsOqCEHZ25uwknB65eGTWM z3DCDmz6K9vLKnEp2gBqfUlx0udDonzRrKL4R3QomwCj4LxJ1WCYbj8wtOvzbKq6xT8M 804A== X-Forwarded-Encrypted: i=1; AJvYcCVG4r+0M8NYA4lzxKAG3kYab3p1PSi/TN0tJkH+bn7ahiawrpMOV9Vzb39VP/0mPFa6hL8P9nMpUHRv1qg=@vger.kernel.org X-Gm-Message-State: AOJu0Yy8Kpqg8/lLI+RRkMoUtXTZ2pmsTt0gp/A+gZt4N5YWwKxXvERb 10E8eWaNaKpXjaKq5tYupRHhN9VAXoXlv4B+ByCkCItfSUKSpORolAQz X-Gm-Gg: AZuq6aLCY/MiKfnpKh2jDO773ZxQs3mxsUR/uByjHs+81EpOXCFQqfPAhTCl/reNS9v Tri+9NIVlg2y9Ho1AFOIJA9FVj67+UsLwIqjqFmXtarO3QT08on49ZbRMDyQ9wMmmd6tHSTWp9l geSap8X4oAfNZ3tUEoRFZQaLfmnk/BUmuA+Z1vq1amGs7qSrZ0GyD1Ub5qHPyRa8a36k2ra/scw WrOmhZMPv5kf0lG8i2ub9TFfldSuvEzT7M8w5yawcQZTR6HZ0ZT3dwDIc+fgvNMnEP+wOY7Bb0F 2b4FlywjDOrt67wpux8+hU8b1OjvCqn4gwvSKM1sEj8Bg0Y+fCmCFwVvpopUeR2DJD2hdiV1kpK F60mb2v3WMzAKs5tbVNJ5uGPz7GDhlJgTNhzgOg0ryGPBLbBoJsVmUh62UqcFqZ7pfiBWLKi9Xk NUOkG03NeBkIoLm+gLyQXmcl7MbQJDI3buY+nloNbhUMM5BWeeiY/2bkhy1EwPUJ4d8DQ= X-Received: by 2002:a17:902:e84c:b0:295:62d:5004 with SMTP id d9443c01a7336-2a9516c4cb8mr25023275ad.26.1770369821942; Fri, 06 Feb 2026 01:23:41 -0800 (PST) Received: from eurus-Lenovo-Legion-Y70002021.. (n119237241079.netvigator.com. [119.237.241.79]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2a951c8578dsm19124415ad.32.2026.02.06.01.23.37 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 06 Feb 2026 01:23:41 -0800 (PST) From: Wang Jiayue To: rafael@kernel.org, lenb@kernel.org, pavel@kernel.org, gregkh@linuxfoundation.org, dakr@kernel.org Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, Wang Jiayue , Gui-Dong Han Subject: [PATCH v2] PM: sleep: wakeirq: Update outdated documentation comments Date: Fri, 6 Feb 2026 17:23:17 +0800 Message-Id: <20260206092317.148885-1-akaieurus@gmail.com> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" The comments claiming that dev_pm_*wake_irq*() helpers must be called with dev->power.lock held and only from rpm_suspend/resume paths are no longer correct, since pm_runtime_force_suspend/resume() call them lockless. Update the comments to reflect it. Reported-by: Gui-Dong Han Closes: https://lore.kernel.org/all/CAJZ5v0jN9fU9NdWqc-+F5hiSEP4JkR=3D_qcdG= zzHtk1i5tvCDbQ@mail.gmail.com/ Fixes: c46a0d5ae4f9 ("PM: runtime: Extend support for wakeirq for force_sus= pend|resume") Signed-off-by: Wang Jiayue --- v2: * Rephrase the comments to also cover the pm_runtime_force_suspend/resume() case, as suggested by Rafael J. Wysocki v1: * Initial fix just remove outdated comments --- drivers/base/power/wakeirq.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/base/power/wakeirq.c b/drivers/base/power/wakeirq.c index 8aa28c08b289..f5db37c4818e 100644 --- a/drivers/base/power/wakeirq.c +++ b/drivers/base/power/wakeirq.c @@ -270,8 +270,10 @@ EXPORT_SYMBOL_GPL(dev_pm_set_dedicated_wake_irq_revers= e); * otherwise try to disable already disabled wakeirq. The wake-up interrupt * starts disabled with IRQ_NOAUTOEN set. * - * Should be only called from rpm_suspend() and rpm_resume() path. - * Caller must hold &dev->power.lock to change wirq->status + * Should be called from rpm_suspend(), rpm_resume(), + * pm_runtime_force_suspend() or pm_runtime_force_resume(). + * Caller must hold &dev->power.lock or disable runtime PM to change + * wirq->status. */ void dev_pm_enable_wake_irq_check(struct device *dev, bool can_change_status) @@ -303,7 +305,8 @@ void dev_pm_enable_wake_irq_check(struct device *dev, * @cond_disable: if set, also check WAKE_IRQ_DEDICATED_REVERSE * * Disables wake-up interrupt conditionally based on status. - * Should be only called from rpm_suspend() and rpm_resume() path. + * Should be called from rpm_suspend(), rpm_resume(), + * pm_runtime_force_suspend() or pm_runtime_force_resume(). */ void dev_pm_disable_wake_irq_check(struct device *dev, bool cond_disable) { @@ -329,7 +332,7 @@ void dev_pm_disable_wake_irq_check(struct device *dev, = bool cond_disable) * enable wake IRQ after running ->runtime_suspend() which depends on * WAKE_IRQ_DEDICATED_REVERSE. * - * Should be only called from rpm_suspend() path. + * Should be called from rpm_suspend() or pm_runtime_force_suspend(). */ void dev_pm_enable_wake_irq_complete(struct device *dev) { --=20 2.34.1