Trading off safety and performance in the kernel

Posted May 13, 2015 2:52 UTC (Wed) by krakensden (subscriber, #72039)
In reply to: Trading off safety and performance in the kernel by neilbrown
Parent article: Trading off safety and performance in the kernel

If suspend means "execute this script":

#!/bin/sh
sync
echo -n mem > /sys/power/state

then there's a window between writing to disk and suspending the system where the in-memory filesystem cache can get dirtied.

Trading off safety and performance in the kernel

Posted May 13, 2015 3:03 UTC (Wed) by neilbrown (subscriber, #359) [Link] (1 responses)

Thanks for being specific.

In the enter_state() function in kernel/power/suspend.c we have:

	trace_suspend_resume(TPS("sync_filesystems"), 0, true);
	printk(KERN_INFO "PM: Syncing filesystems ... ");
	sys_sync();
	printk("done.\n");
	trace_suspend_resume(TPS("sync_filesystems"), 0, false);

	pr_debug("PM: Preparing system for %s sleep\n", pm_states[state]);
	error = suspend_prepare(state);

and in suspend_prepare():

	pm_prepare_console();

	error = pm_notifier_call_chain(PM_SUSPEND_PREPARE);
	if (error)
		goto Finish;

	trace_suspend_resume(TPS("freeze_processes"), 0, true);
	error = suspend_freeze_processes();
	trace_suspend_resume(TPS("freeze_processes"), 0, false);

So the freezing of user-space processes happens *after* the sys_sync call. So that race is already present.

Trading off safety and performance in the kernel

Posted May 13, 2015 3:41 UTC (Wed) by chloe_zen (guest, #8258) [Link]

Given fuse, it rather has to be sync-then-suspend, doesn't it?

Trading off safety and performance in the kernel

Posted May 13, 2015 19:12 UTC (Wed) by flussence (guest, #85566) [Link] (3 responses)

A tangentially related aside: "echo -n" no longer works that way in recent versions of dash, which is the default /bin/sh on some distros. You'd need to use printf(1) instead.

Trading off safety and performance in the kernel

Posted May 13, 2015 22:41 UTC (Wed) by neilbrown (subscriber, #359) [Link]

> A tangentially related aside: "echo -n" no longer works that way in recent versions of dash,

I wonder where this habit of using "-n" came from. Just

echo mem > /sys/power/state

If any file in /sys requires "echo -n" then that is kernel bug. Please report it.

Trading off safety and performance in the kernel

Posted May 14, 2015 0:42 UTC (Thu) by mathstuf (subscriber, #69389) [Link] (1 responses)

-n on echo was never portable.

Trading off safety and performance in the kernel

Posted May 15, 2015 17:27 UTC (Fri) by flussence (guest, #85566) [Link]

I'm aware of that, but there's a huge amount of software in the wild convinced that it is: https://bugs.gentoo.org/show_bug.cgi?id=nonbash