Linux Media Controller development
 help / color / mirror / Atom feed
From: Haowen Tu <tuhaowen@uniontech.com>
To: laurent.pinchart@ideasonboard.com, rafael@kernel.org
Cc: gregkh@linuxfoundation.org, hansg@kernel.org,
	kernel@uniontech.com, lenb@kernel.org,
	linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
	linux-pm@vger.kernel.org, linux-usb@vger.kernel.org,
	mchehab@kernel.org, oneukum@suse.com, pavel@kernel.org,
	stern@rowland.harvard.edu, tuhaowen@uniontech.com
Subject: [PATCH v4 0/4] PM: hibernate: defer UVC streaming restart after snapshot
Date: Wed, 29 Jul 2026 15:11:19 +0800	[thread overview]
Message-ID: <cover.1785226280.git.tuhaowen@uniontech.com> (raw)
In-Reply-To: <20260723084633.GE648629@killaraus.ideasonboard.com>

During hibernation, after the memory snapshot has been created, the
kernel resumes devices with PMSG_THAW to write the hibernation image.
USB interface resume callbacks do not receive the PM message, so UVC
cannot distinguish that intermediate THAW resume from the later RESTORE
resume by using its local callback arguments.

This series keeps in_suspend consistent with the lifetime of snapshot
memory, exports pm_hibernation_snapshot_done(), adds a hibernation image
write failure notification, and uses both in uvcvideo to defer restarting
streaming during the THAW phase after snapshot creation.

Unlike v3, the UVC patch now handles the image write failure path.  The
driver records streams whose streaming restart was deferred.  If image
writeout fails, PM sends PM_HIBERNATION_IMAGE_WRITE_FAILED after
swsusp_write() returns and before userspace is thawed, and uvcvideo
restarts those deferred streams.  If the recovery restart fails, the
driver falls back to its existing streamoff error path so userspace is
not left waiting indefinitely.

Changes in v4:
- Add PM_HIBERNATION_IMAGE_WRITE_FAILED before userspace thaw on image
  write failure.
- Update uvcvideo to defer streaming restart instead of just skipping it.
- Recover deferred UVC streams on image write failure.
- Carry Rafael's Acked-by tags on the two PM patches from v3.

Haowen Tu (4):
  PM: hibernate: clear in_suspend before freeing the snapshot
  PM: hibernate: add pm_hibernation_snapshot_done() helper
  PM: hibernate: notify on image write failure
  media: uvcvideo: defer streaming restart after hibernation snapshot

 drivers/media/usb/uvc/uvc_driver.c | 56 +++++++++++++++++++++++++++++-
 drivers/media/usb/uvc/uvc_video.c  | 19 +++++++++-
 drivers/media/usb/uvc/uvcvideo.h   |  2 ++
 include/linux/suspend.h            |  3 ++
 kernel/power/hibernate.c           | 27 +++++++++++---
 5 files changed, 101 insertions(+), 6 deletions(-)

-- 
2.20.1

  reply	other threads:[~2026-07-29  7:11 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-28  8:05 [PATCH 0/2] PM: hibernate: add helper to detect image-write phase Haowen Tu
2026-04-28  8:05 ` [PATCH 1/2] PM: hibernate: add pm_hibernation_storing_image() helper Haowen Tu
2026-05-26 13:43   ` Rafael J. Wysocki
2026-04-28  8:05 ` [PATCH 2/2] media: uvcvideo: skip resume when writing hibernation image Haowen Tu
2026-04-28  8:36   ` Oliver Neukum
2026-04-28  8:58     ` Haowen Tu
2026-04-28  9:13   ` Laurent Pinchart
2026-04-29  1:15     ` Haowen Tu
2026-04-29  1:16     ` Haowen Tu
2026-05-28  8:18 ` [PATCH v2 0/2] PM: hibernate: skip UVC resume after snapshot Haowen Tu
2026-05-28  8:18   ` [PATCH v2 1/2] PM: hibernate: add pm_hibernation_snapshot_done() helper Haowen Tu
2026-06-01 18:22     ` Rafael J. Wysocki
2026-06-02  3:24       ` Haowen Tu
2026-06-18  1:31         ` [PATCH v3 0/3] PM: hibernate: skip UVC streaming restart after snapshot Haowen Tu
2026-06-18  1:31           ` [PATCH v3 1/3] PM: hibernate: clear in_suspend before freeing the snapshot Haowen Tu
2026-07-22 15:37             ` Rafael J. Wysocki (Intel)
2026-06-18  1:31           ` [PATCH v3 2/3] PM: hibernate: add pm_hibernation_snapshot_done() helper Haowen Tu
2026-07-22 15:40             ` Rafael J. Wysocki (Intel)
2026-06-18  1:31           ` [PATCH v3 3/3] media: uvcvideo: skip streaming restart after hibernation snapshot Haowen Tu
2026-07-22 20:06             ` Laurent Pinchart
2026-07-23  1:39               ` Haowen Tu
2026-07-23  8:46                 ` Laurent Pinchart
2026-07-29  7:11                   ` Haowen Tu [this message]
2026-07-29  7:11                     ` [PATCH v4 1/4] PM: hibernate: clear in_suspend before freeing the snapshot Haowen Tu
2026-07-29  7:11                     ` [PATCH v4 2/4] PM: hibernate: add pm_hibernation_snapshot_done() helper Haowen Tu
2026-07-29  7:11                     ` [PATCH v4 3/4] PM: hibernate: notify on image write failure Haowen Tu
2026-07-29  7:11                     ` [PATCH v4 4/4] media: uvcvideo: defer streaming restart after hibernation snapshot Haowen Tu
2026-07-29  7:29                       ` Hans de Goede
2026-07-29  7:56                         ` Haowen Tu
2026-05-28  8:18   ` [PATCH v2 2/2] media: uvcvideo: skip resume " Haowen Tu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1785226280.git.tuhaowen@uniontech.com \
    --to=tuhaowen@uniontech.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=hansg@kernel.org \
    --cc=kernel@uniontech.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=lenb@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=oneukum@suse.com \
    --cc=pavel@kernel.org \
    --cc=rafael@kernel.org \
    --cc=stern@rowland.harvard.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox