* [PATCH RESEND v2] scsi: pm: Drop unneeded call to pm_runtime_mark_last_busy()
@ 2025-11-11 17:50 Nuno Sá via B4 Relay
2025-11-13 1:57 ` Martin K. Petersen
2025-11-20 4:16 ` Martin K. Petersen
0 siblings, 2 replies; 3+ messages in thread
From: Nuno Sá via B4 Relay @ 2025-11-11 17:50 UTC (permalink / raw)
To: linux-scsi; +Cc: James E.J. Bottomley, Martin K. Petersen
From: Nuno Sá <nuno.sa@analog.com>
There's no need to explicitly call pm_runtime_mark_last_busy() since
pm_runtime_autosuspend() is now doing it since commit 08071e64cb64
("PM: runtime: Mark last busy stamp in pm_runtime_autosuspend()")
Signed-off-by: Nuno Sá <nuno.sa@analog.com>
---
Changes in v2:
- Reference the commit introducing the change affecting this patch
- Link to v1: https://lore.kernel.org/r/20251027-scsi-pm-improv-v1-1-cb9f0bceb4be@analog.com
---
drivers/scsi/scsi_pm.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/scsi/scsi_pm.c b/drivers/scsi/scsi_pm.c
index d581613d87c7..2652fecbfe47 100644
--- a/drivers/scsi/scsi_pm.c
+++ b/drivers/scsi/scsi_pm.c
@@ -205,7 +205,6 @@ static int scsi_runtime_idle(struct device *dev)
/* Insert hooks here for targets, hosts, and transport classes */
if (scsi_is_sdev_device(dev)) {
- pm_runtime_mark_last_busy(dev);
pm_runtime_autosuspend(dev);
return -EBUSY;
}
---
base-commit: 8fec172c82c2b5f6f8e47ab837c1dc91ee3d1b87
change-id: 20251027-scsi-pm-improv-71477eb109c2
--
Thanks!
- Nuno Sá
--
Nuno Sá <nuno.sa@analog.com>
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH RESEND v2] scsi: pm: Drop unneeded call to pm_runtime_mark_last_busy()
2025-11-11 17:50 [PATCH RESEND v2] scsi: pm: Drop unneeded call to pm_runtime_mark_last_busy() Nuno Sá via B4 Relay
@ 2025-11-13 1:57 ` Martin K. Petersen
2025-11-20 4:16 ` Martin K. Petersen
1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2025-11-13 1:57 UTC (permalink / raw)
To: Nuno Sá via B4 Relay
Cc: linux-scsi, nuno.sa, James E.J. Bottomley, Martin K. Petersen
Nuno,
> There's no need to explicitly call pm_runtime_mark_last_busy() since
> pm_runtime_autosuspend() is now doing it since commit 08071e64cb64
> ("PM: runtime: Mark last busy stamp in pm_runtime_autosuspend()")
Applied to 6.19/scsi-staging, thanks!
--
Martin K. Petersen
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH RESEND v2] scsi: pm: Drop unneeded call to pm_runtime_mark_last_busy()
2025-11-11 17:50 [PATCH RESEND v2] scsi: pm: Drop unneeded call to pm_runtime_mark_last_busy() Nuno Sá via B4 Relay
2025-11-13 1:57 ` Martin K. Petersen
@ 2025-11-20 4:16 ` Martin K. Petersen
1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2025-11-20 4:16 UTC (permalink / raw)
To: linux-scsi, Nuno Sá; +Cc: Martin K . Petersen, James E.J. Bottomley
On Tue, 11 Nov 2025 17:50:17 +0000, Nuno Sá wrote:
> There's no need to explicitly call pm_runtime_mark_last_busy() since
> pm_runtime_autosuspend() is now doing it since commit 08071e64cb64
> ("PM: runtime: Mark last busy stamp in pm_runtime_autosuspend()")
>
>
Applied to 6.19/scsi-queue, thanks!
[1/1] scsi: pm: Drop unneeded call to pm_runtime_mark_last_busy()
https://git.kernel.org/mkp/scsi/c/1028258914f6
--
Martin K. Petersen
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-11-20 4:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-11 17:50 [PATCH RESEND v2] scsi: pm: Drop unneeded call to pm_runtime_mark_last_busy() Nuno Sá via B4 Relay
2025-11-13 1:57 ` Martin K. Petersen
2025-11-20 4:16 ` Martin K. Petersen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox