diff options
| author | Coleman Kane <[email protected]> | 2021-07-26 13:55:55 -0400 |
|---|---|---|
| committer | Tony Hutter <[email protected]> | 2021-09-14 12:41:10 -0700 |
| commit | 4434baab1101a63f64c21eceb7e662dededb31a7 (patch) | |
| tree | 7e8799c96792f3ef6a066e3bc4920a350ba8be2b /config/kernel-vfs-set_page_dirty.m4 | |
| parent | 6385f4e70e62c5ee1cd5373ecbde5e2d33b22ec6 (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/kernel-vfs-set_page_dirty.m4')
| -rw-r--r-- | config/kernel-vfs-set_page_dirty.m4 | 34 |
1 files changed, 34 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]) + ]) +]) |