Commit 2aede851 authored by Rafael J. Wysocki's avatar Rafael J. Wysocki
Browse files

PM / Hibernate: Freeze kernel threads after preallocating memory 



There is a problem with the current ordering of hibernate code which
leads to deadlocks in some filesystems' memory shrinkers.  Namely,
some filesystems use freezable kernel threads that are inactive when
the hibernate memory preallocation is carried out.  Those same
filesystems use memory shrinkers that may be triggered by the
hibernate memory preallocation.  If those memory shrinkers wait for
the frozen kernel threads, the hibernate process deadlocks (this
happens with XFS, for one example).

Apparently, it is not technically viable to redesign the filesystems
in question to avoid the situation described above, so the only
possible solution of this issue is to defer the freezing of kernel
threads until the hibernate memory preallocation is done, which is
implemented by this change.

Unfortunately, this requires the memory preallocation to be done
before the "prepare" stage of device freeze, so after this change the
only way drivers can allocate additional memory for their freeze
routines in a clean way is to use PM notifiers.

Reported-by: default avatarChristoph <cr2005@u-club.de>
Signed-off-by: default avatarRafael J. Wysocki <rjw@sisk.pl>
parent 8f88893c

Documentation/power/devices.txt

+0 −4
Original line number Original line Diff line number Diff line
@@ -281,10 +281,6 @@ When the system goes into the standby or memory sleep state, the phases are: 
	time.)  Unlike the other suspend-related phases, during the prepare

	time.)  Unlike the other suspend-related phases, during the prepare

	phase the device tree is traversed top-down.

	phase the device tree is traversed top-down.





	In addition to that, if device drivers need to allocate additional

	memory to be able to hadle device suspend correctly, that should be

	done in the prepare phase.



	After the prepare callback method returns, no new children may be

	After the prepare callback method returns, no new children may be

	registered below the device.  The method may also prepare the device or

	registered below the device.  The method may also prepare the device or

	driver in some way for the upcoming system power transition (for

	driver in some way for the upcoming system power transition (for

include/linux/freezer.h

+3 −1
Original line number Original line Diff line number Diff line
@@ -49,6 +49,7 @@ extern int thaw_process(struct task_struct *p); 




extern void refrigerator(void);

extern void refrigerator(void);

extern int freeze_processes(void);

extern int freeze_processes(void);

extern int freeze_kernel_threads(void);

extern void thaw_processes(void);

extern void thaw_processes(void);





static inline int try_to_freeze(void)

static inline int try_to_freeze(void)
@@ -171,7 +172,8 @@ static inline void clear_freeze_flag(struct task_struct *p) {} 
static inline int thaw_process(struct task_struct *p) { return 1; }

static inline int thaw_process(struct task_struct *p) { return 1; }





static inline void refrigerator(void) {}

static inline void refrigerator(void) {}

static inline int freeze_processes(void) { BUG(); return 0; }

static inline int freeze_processes(void) { return -ENOSYS; }

static inline int freeze_kernel_threads(void) { return -ENOSYS; }

static inline void thaw_processes(void) {}

static inline void thaw_processes(void) {}





static inline int try_to_freeze(void) { return 0; }

static inline int try_to_freeze(void) { return 0; }

kernel/power/hibernate.c

+8 −4
Original line number Original line Diff line number Diff line
@@ -334,12 +334,16 @@ int hibernation_snapshot(int platform_mode) 
	if (error)

	if (error)

		goto Close;

		goto Close;





	error = dpm_prepare(PMSG_FREEZE);

	if (error)

		goto Complete_devices;



	/* Preallocate image memory before shutting down devices. */

	/* Preallocate image memory before shutting down devices. */

	error = hibernate_preallocate_memory();

	error = hibernate_preallocate_memory();

	if (error)

		goto Close;



	error = freeze_kernel_threads();

	if (error)

		goto Close;



	error = dpm_prepare(PMSG_FREEZE);

	if (error)

	if (error)

		goto Complete_devices;

		goto Complete_devices;

kernel/power/power.h

+2 −1
Original line number Original line Diff line number Diff line
@@ -228,7 +228,8 @@ extern int pm_test_level; 
#ifdef CONFIG_SUSPEND_FREEZER

#ifdef CONFIG_SUSPEND_FREEZER

static inline int suspend_freeze_processes(void)

static inline int suspend_freeze_processes(void)

{

{

	return freeze_processes();

	int error = freeze_processes();

	return error ? : freeze_kernel_threads();

}

}





static inline void suspend_thaw_processes(void)

static inline void suspend_thaw_processes(void)

kernel/power/process.c

+20 −10
Original line number Original line Diff line number Diff line
@@ -135,7 +135,7 @@ static int try_to_freeze_tasks(bool sig_only) 
}

}





/**

/**

 *	freeze_processes - tell processes to enter the refrigerator

 * freeze_processes - Signal user space processes to enter the refrigerator.

 */

 */

int freeze_processes(void)

int freeze_processes(void)

{

{
@@ -143,20 +143,30 @@ int freeze_processes(void) 




	printk("Freezing user space processes ... ");

	printk("Freezing user space processes ... ");

	error = try_to_freeze_tasks(true);

	error = try_to_freeze_tasks(true);

	if (error)

	if (!error) {

		goto Exit;

		printk("done.");

	printk("done.\n");

		oom_killer_disable();

	}

	printk("\n");

	BUG_ON(in_atomic());



	return error;

}



/**

 * freeze_kernel_threads - Make freezable kernel threads go to the refrigerator.

 */

int freeze_kernel_threads(void)

{

	int error;





	printk("Freezing remaining freezable tasks ... ");

	printk("Freezing remaining freezable tasks ... ");

	error = try_to_freeze_tasks(false);

	error = try_to_freeze_tasks(false);

	if (error)

	if (!error)

		goto Exit;

		printk("done.");

		printk("done.");





	oom_killer_disable();

 Exit:

	BUG_ON(in_atomic());

	printk("\n");

	printk("\n");

	BUG_ON(in_atomic());





	return error;

	return error;

}

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE