Commit 57ea3ab2 authored by Rafael J. Wysocki's avatar Rafael J. Wysocki

Merge branch 'pm-sleep'

Merge hibernation test mode fix for 6.4-rc1.

* pm-sleep:
  PM: hibernate: Do not get block device exclusively in test_resume mode
  PM: hibernate: Turn snapshot_test into global variable
parents bd446829 5904de0d
...@@ -64,6 +64,7 @@ enum { ...@@ -64,6 +64,7 @@ enum {
static int hibernation_mode = HIBERNATION_SHUTDOWN; static int hibernation_mode = HIBERNATION_SHUTDOWN;
bool freezer_test_done; bool freezer_test_done;
bool snapshot_test;
static const struct platform_hibernation_ops *hibernation_ops; static const struct platform_hibernation_ops *hibernation_ops;
...@@ -687,18 +688,22 @@ static int load_image_and_restore(void) ...@@ -687,18 +688,22 @@ static int load_image_and_restore(void)
{ {
int error; int error;
unsigned int flags; unsigned int flags;
fmode_t mode = FMODE_READ;
if (snapshot_test)
mode |= FMODE_EXCL;
pm_pr_dbg("Loading hibernation image.\n"); pm_pr_dbg("Loading hibernation image.\n");
lock_device_hotplug(); lock_device_hotplug();
error = create_basic_memory_bitmaps(); error = create_basic_memory_bitmaps();
if (error) { if (error) {
swsusp_close(FMODE_READ | FMODE_EXCL); swsusp_close(mode);
goto Unlock; goto Unlock;
} }
error = swsusp_read(&flags); error = swsusp_read(&flags);
swsusp_close(FMODE_READ | FMODE_EXCL); swsusp_close(mode);
if (!error) if (!error)
error = hibernation_restore(flags & SF_PLATFORM_MODE); error = hibernation_restore(flags & SF_PLATFORM_MODE);
...@@ -716,7 +721,6 @@ static int load_image_and_restore(void) ...@@ -716,7 +721,6 @@ static int load_image_and_restore(void)
*/ */
int hibernate(void) int hibernate(void)
{ {
bool snapshot_test = false;
unsigned int sleep_flags; unsigned int sleep_flags;
int error; int error;
...@@ -744,6 +748,9 @@ int hibernate(void) ...@@ -744,6 +748,9 @@ int hibernate(void)
if (error) if (error)
goto Exit; goto Exit;
/* protected by system_transition_mutex */
snapshot_test = false;
lock_device_hotplug(); lock_device_hotplug();
/* Allocate memory management structures */ /* Allocate memory management structures */
error = create_basic_memory_bitmaps(); error = create_basic_memory_bitmaps();
...@@ -940,6 +947,8 @@ static int software_resume(void) ...@@ -940,6 +947,8 @@ static int software_resume(void)
*/ */
mutex_lock_nested(&system_transition_mutex, SINGLE_DEPTH_NESTING); mutex_lock_nested(&system_transition_mutex, SINGLE_DEPTH_NESTING);
snapshot_test = false;
if (swsusp_resume_device) if (swsusp_resume_device)
goto Check_image; goto Check_image;
......
...@@ -59,6 +59,7 @@ asmlinkage int swsusp_save(void); ...@@ -59,6 +59,7 @@ asmlinkage int swsusp_save(void);
/* kernel/power/hibernate.c */ /* kernel/power/hibernate.c */
extern bool freezer_test_done; extern bool freezer_test_done;
extern bool snapshot_test;
extern int hibernation_snapshot(int platform_mode); extern int hibernation_snapshot(int platform_mode);
extern int hibernation_restore(int platform_mode); extern int hibernation_restore(int platform_mode);
......
...@@ -1518,9 +1518,13 @@ int swsusp_check(void) ...@@ -1518,9 +1518,13 @@ int swsusp_check(void)
{ {
int error; int error;
void *holder; void *holder;
fmode_t mode = FMODE_READ;
if (snapshot_test)
mode |= FMODE_EXCL;
hib_resume_bdev = blkdev_get_by_dev(swsusp_resume_device, hib_resume_bdev = blkdev_get_by_dev(swsusp_resume_device,
FMODE_READ | FMODE_EXCL, &holder); mode, &holder);
if (!IS_ERR(hib_resume_bdev)) { if (!IS_ERR(hib_resume_bdev)) {
set_blocksize(hib_resume_bdev, PAGE_SIZE); set_blocksize(hib_resume_bdev, PAGE_SIZE);
clear_page(swsusp_header); clear_page(swsusp_header);
...@@ -1547,7 +1551,7 @@ int swsusp_check(void) ...@@ -1547,7 +1551,7 @@ int swsusp_check(void)
put: put:
if (error) if (error)
blkdev_put(hib_resume_bdev, FMODE_READ | FMODE_EXCL); blkdev_put(hib_resume_bdev, mode);
else else
pr_debug("Image signature found, resuming\n"); pr_debug("Image signature found, resuming\n");
} else { } else {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment