git: a5c2d59ed3d6 - main - Expand comment explaining reasons for automatic swapoff on shutdown
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Fri, 03 Dec 2021 08:50:18 UTC
The branch main has been updated by kib: URL: https://cgit.FreeBSD.org/src/commit/?id=a5c2d59ed3d6365c26322f3e100467454774b501 commit a5c2d59ed3d6365c26322f3e100467454774b501 Author: Konstantin Belousov <kib@FreeBSD.org> AuthorDate: 2021-11-29 19:11:33 +0000 Commit: Konstantin Belousov <kib@FreeBSD.org> CommitDate: 2021-12-03 08:42:21 +0000 Expand comment explaining reasons for automatic swapoff on shutdown Reviewed by: alc, markj Sponsored by: The FreeBSD Foundation MFC after: 1 week Differential revision: https://reviews.freebsd.org/D33167 --- sys/kern/vfs_bio.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/sys/kern/vfs_bio.c b/sys/kern/vfs_bio.c index 61fe9a6e2bfd..acd16ecdf3fe 100644 --- a/sys/kern/vfs_bio.c +++ b/sys/kern/vfs_bio.c @@ -1458,9 +1458,16 @@ bufshutdown(int show_busybufs) printf("Final sync complete\n"); /* - * Unmount filesystems. Swapoff before unmount, - * because file-backed swap is non-operational after unmount - * of the underlying filesystem. + * Unmount filesystems and perform swapoff, to quiesce + * the system as much as possible. In particular, no + * I/O should be initiated from top levels since it + * might be abruptly terminated by reset, or otherwise + * erronously handled because other parts of the + * system are disabled. + * + * Swapoff before unmount, because file-backed swap is + * non-operational after unmount of the underlying + * filesystem. */ if (!KERNEL_PANICKED()) { swapoff_all();