diff options
| author | Coleman Kane <[email protected]> | 2021-07-26 13:55:55 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2021-07-26 10:55:55 -0700 |
| commit | 1c24bf966c373009f2be77438e8696aabf50a7e7 (patch) | |
| tree | 084098c3d61682fc78cc593b31a3ef13afece14c /config | |
| parent | f1ca7999bbf2f1c39e80cee176ba52fcca12ec05 (diff) | |
Linux 5.14 compat: explicity assign set_page_dirty
Kernel 5.14 introduced a change where set_page_dirty of
struct address_space_operations is no longer implicitly set to
__set_page_dirty_buffers(), which ended up resulting in a NULL
pointer deref in the kernel when it is attempted to be called.
This change sets .set_page_dirty in the structure to
__set_page_dirty_nobuffers(), which was introduced with the
related patch set. The breaking change was introduce in commit
0af573780b0b13fceb7fabd49dc1b073cee9a507 to torvalds/linux.git.
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: Coleman Kane <[email protected]>
Closes #12427
Diffstat (limited to 'config')
| -rw-r--r-- | config/kernel-vfs-set_page_dirty.m4 | 34 | ||||
| -rw-r--r-- | config/kernel.m4 | 2 |
2 files changed, 36 insertions, 0 deletions
diff --git a/config/kernel-vfs-set_page_dirty.m4 b/config/kernel-vfs-set_page_dirty.m4 new file mode 100644 index 000000000..a9d252e4e --- /dev/null +++ b/config/kernel-vfs-set_page_dirty.m4 @@ -0,0 +1,34 @@ +dnl # +dnl # Linux 5.14 adds a change to require set_page_dirty to be manually +dnl # wired up in struct address_space_operations. Determine if this needs +dnl # to be done. This patch set also introduced __set_page_dirty_nobuffers +dnl # declaration in linux/pagemap.h, so these tests look for the presence +dnl # of that function to tell the compiler to assign set_page_dirty in +dnl # module/os/linux/zfs/zpl_file.c +dnl # +AC_DEFUN([ZFS_AC_KERNEL_SRC_VFS_SET_PAGE_DIRTY_NOBUFFERS], [ + ZFS_LINUX_TEST_SRC([vfs_has_set_page_dirty_nobuffers], [ + #include <linux/pagemap.h> + #include <linux/fs.h> + + static const struct address_space_operations + aops __attribute__ ((unused)) = { + .set_page_dirty = __set_page_dirty_nobuffers, + }; + ],[]) +]) + +AC_DEFUN([ZFS_AC_KERNEL_VFS_SET_PAGE_DIRTY_NOBUFFERS], [ + dnl # + dnl # Linux 5.14 change requires set_page_dirty() to be assigned + dnl # in address_space_operations() + dnl # + AC_MSG_CHECKING([__set_page_dirty_nobuffers exists]) + ZFS_LINUX_TEST_RESULT([vfs_has_set_page_dirty_nobuffers], [ + AC_MSG_RESULT([yes]) + AC_DEFINE(HAVE_VFS_SET_PAGE_DIRTY_NOBUFFERS, 1, + [__set_page_dirty_nobuffers exists]) + ],[ + AC_MSG_RESULT([no]) + ]) +]) diff --git a/config/kernel.m4 b/config/kernel.m4 index 7196e66ca..5ea2286db 100644 --- a/config/kernel.m4 +++ b/config/kernel.m4 @@ -132,6 +132,7 @@ AC_DEFUN([ZFS_AC_KERNEL_TEST_SRC], [ ZFS_AC_KERNEL_SRC_SIGNAL_STOP ZFS_AC_KERNEL_SRC_SIGINFO ZFS_AC_KERNEL_SRC_SET_SPECIAL_STATE + ZFS_AC_KERNEL_SRC_VFS_SET_PAGE_DIRTY_NOBUFFERS AC_MSG_CHECKING([for available kernel interfaces]) ZFS_LINUX_TEST_COMPILE_ALL([kabi]) @@ -237,6 +238,7 @@ AC_DEFUN([ZFS_AC_KERNEL_TEST_RESULT], [ ZFS_AC_KERNEL_SIGNAL_STOP ZFS_AC_KERNEL_SIGINFO ZFS_AC_KERNEL_SET_SPECIAL_STATE + ZFS_AC_KERNEL_VFS_SET_PAGE_DIRTY_NOBUFFERS ]) dnl # |