summaryrefslogtreecommitdiff
path: root/sys-fs/zfs-utils/files
diff options
context:
space:
mode:
Diffstat (limited to 'sys-fs/zfs-utils/files')
-rw-r--r--sys-fs/zfs-utils/files/kernel-4.18.patch1393
1 files changed, 0 insertions, 1393 deletions
diff --git a/sys-fs/zfs-utils/files/kernel-4.18.patch b/sys-fs/zfs-utils/files/kernel-4.18.patch
deleted file mode 100644
index ef8d648d..00000000
--- a/sys-fs/zfs-utils/files/kernel-4.18.patch
+++ /dev/null
@@ -1,1393 +0,0 @@
-From 3cf6af50e0bc02b294bf974a8e653e12c40b5263 Mon Sep 17 00:00:00 2001
-From: Brian Behlendorf <behlendorf1@llnl.gov>
-Date: Sun, 12 Aug 2018 18:22:03 -0400
-Subject: [PATCH] Linux 4.18 compat: inode timespec -> timespec64
-
-Commit torvalds/linux@95582b0 changes the inode i_atime, i_mtime,
-and i_ctime members form timespec's to timespec64's to make them
-2038 safe. As part of this change the current_time() function was
-also updated to return the timespec64 type.
-
-Resolve this issue by introducing a new inode_timespec_t type which
-is defined to match the timespec type used by the inode. It should
-be used when working with inode timestamps to ensure matching types.
-
-The timestruc_t type under Illumos was used in a similar fashion but
-was specified to always be a timespec_t. Rather than incorrectly
-define this type all timespec_t types have been replaced by the new
-inode_timespec_t type.
-
-Finally, the kernel and user space 'sys/time.h' headers were aligned
-with each other. They define as appropriate for the context several
-constants as macros and include static inline implementation of
-gethrestime(), gethrestime_sec(), and gethrtime().
-
-Reviewed-by: Chunwei Chen <tuxoko@gmail.com>
-Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
-Closes #7643
-Backported-by: Richard Yao <ryao@gentoo.org>
----
- config/kernel-current-time.m4 | 7 +++---
- include/sys/dmu.h | 2 +-
- include/sys/dmu_objset.h | 2 +-
- include/sys/dsl_dir.h | 4 ++--
- include/sys/spa_impl.h | 2 +-
- include/sys/xvattr.h | 2 +-
- include/sys/zfs_context.h | 9 +------
- include/sys/zfs_znode.h | 33 ++++++++++++++++++-------
- include/sys/zpl.h | 9 +++++++
- lib/libspl/Makefile.am | 2 --
- lib/libspl/gethrestime.c | 38 -----------------------------
- lib/libspl/gethrtime.c | 45 -----------------------------------
- lib/libspl/include/sys/time.h | 37 +++++++++++++++++++++-------
- lib/libzpool/kernel.c | 4 ++--
- module/zfs/dmu_objset.c | 2 +-
- module/zfs/dsl_dir.c | 6 ++---
- module/zfs/fm.c | 2 +-
- module/zfs/zfs_ctldir.c | 2 +-
- module/zfs/zfs_vnops.c | 4 ++--
- module/zfs/zfs_znode.c | 4 ++--
- module/zfs/zpl_inode.c | 5 ++--
- 21 files changed, 88 insertions(+), 133 deletions(-)
- delete mode 100644 lib/libspl/gethrestime.c
- delete mode 100644 lib/libspl/gethrtime.c
-
-diff --git a/config/kernel-current-time.m4 b/config/kernel-current-time.m4
-index 2ede9ff38c4..c7d5c9b5200 100644
---- a/config/kernel-current-time.m4
-+++ b/config/kernel-current-time.m4
-@@ -1,15 +1,14 @@
- dnl #
- dnl # 4.9, current_time() added
-+dnl # 4.18, return type changed from timespec to timespec64
- dnl #
- AC_DEFUN([ZFS_AC_KERNEL_CURRENT_TIME],
- [AC_MSG_CHECKING([whether current_time() exists])
- ZFS_LINUX_TRY_COMPILE_SYMBOL([
- #include <linux/fs.h>
- ], [
-- struct inode ip;
-- struct timespec now __attribute__ ((unused));
--
-- now = current_time(&ip);
-+ struct inode ip __attribute__ ((unused));
-+ ip.i_atime = current_time(&ip);
- ], [current_time], [fs/inode.c], [
- AC_MSG_RESULT(yes)
- AC_DEFINE(HAVE_CURRENT_TIME, 1, [current_time() exists])
-diff --git a/include/sys/dmu.h b/include/sys/dmu.h
-index bcdf7d646fb..755a90561ae 100644
---- a/include/sys/dmu.h
-+++ b/include/sys/dmu.h
-@@ -891,7 +891,7 @@ uint64_t dmu_objset_fsid_guid(objset_t *os);
- /*
- * Get the [cm]time for an objset's snapshot dir
- */
--timestruc_t dmu_objset_snap_cmtime(objset_t *os);
-+inode_timespec_t dmu_objset_snap_cmtime(objset_t *os);
-
- int dmu_objset_is_snapshot(objset_t *os);
-
-diff --git a/include/sys/dmu_objset.h b/include/sys/dmu_objset.h
-index a836e03722c..531e81d4d88 100644
---- a/include/sys/dmu_objset.h
-+++ b/include/sys/dmu_objset.h
-@@ -179,7 +179,7 @@ int dmu_objset_find_dp(struct dsl_pool *dp, uint64_t ddobj,
- int func(struct dsl_pool *, struct dsl_dataset *, void *),
- void *arg, int flags);
- void dmu_objset_evict_dbufs(objset_t *os);
--timestruc_t dmu_objset_snap_cmtime(objset_t *os);
-+inode_timespec_t dmu_objset_snap_cmtime(objset_t *os);
-
- /* called from dsl */
- void dmu_objset_sync(objset_t *os, zio_t *zio, dmu_tx_t *tx);
-diff --git a/include/sys/dsl_dir.h b/include/sys/dsl_dir.h
-index 69b0b6a5355..80e83fdc4d3 100644
---- a/include/sys/dsl_dir.h
-+++ b/include/sys/dsl_dir.h
-@@ -103,7 +103,7 @@ struct dsl_dir {
- /* Protected by dd_lock */
- kmutex_t dd_lock;
- list_t dd_props; /* list of dsl_prop_record_t's */
-- timestruc_t dd_snap_cmtime; /* last time snapshot namespace changed */
-+ inode_timespec_t dd_snap_cmtime; /* last snapshot namespace change */
- uint64_t dd_origin_txg;
-
- /* gross estimate of space used by in-flight tx's */
-@@ -159,7 +159,7 @@ boolean_t dsl_dir_is_clone(dsl_dir_t *dd);
- void dsl_dir_new_refreservation(dsl_dir_t *dd, struct dsl_dataset *ds,
- uint64_t reservation, cred_t *cr, dmu_tx_t *tx);
- void dsl_dir_snap_cmtime_update(dsl_dir_t *dd);
--timestruc_t dsl_dir_snap_cmtime(dsl_dir_t *dd);
-+inode_timespec_t dsl_dir_snap_cmtime(dsl_dir_t *dd);
- void dsl_dir_set_reservation_sync_impl(dsl_dir_t *dd, uint64_t value,
- dmu_tx_t *tx);
- void dsl_dir_zapify(dsl_dir_t *dd, dmu_tx_t *tx);
-diff --git a/include/sys/spa_impl.h b/include/sys/spa_impl.h
-index b1e78c1d592..fa7490ace67 100644
---- a/include/sys/spa_impl.h
-+++ b/include/sys/spa_impl.h
-@@ -153,7 +153,7 @@ struct spa {
- uint64_t spa_freeze_txg; /* freeze pool at this txg */
- uint64_t spa_load_max_txg; /* best initial ub_txg */
- uint64_t spa_claim_max_txg; /* highest claimed birth txg */
-- timespec_t spa_loaded_ts; /* 1st successful open time */
-+ inode_timespec_t spa_loaded_ts; /* 1st successful open time */
- objset_t *spa_meta_objset; /* copy of dp->dp_meta_objset */
- kmutex_t spa_evicting_os_lock; /* Evicting objset list lock */
- list_t spa_evicting_os_list; /* Objsets being evicted. */
-diff --git a/include/sys/xvattr.h b/include/sys/xvattr.h
-index 4779b632163..5d38927cd4b 100644
---- a/include/sys/xvattr.h
-+++ b/include/sys/xvattr.h
-@@ -47,7 +47,7 @@
- * Structure of all optional attributes.
- */
- typedef struct xoptattr {
-- timestruc_t xoa_createtime; /* Create time of file */
-+ inode_timespec_t xoa_createtime; /* Create time of file */
- uint8_t xoa_archive;
- uint8_t xoa_system;
- uint8_t xoa_readonly;
-diff --git a/include/sys/zfs_context.h b/include/sys/zfs_context.h
-index 4fe35342dee..68c58f95531 100644
---- a/include/sys/zfs_context.h
-+++ b/include/sys/zfs_context.h
-@@ -527,7 +527,7 @@ extern char *vn_dumpdir;
- #define AV_SCANSTAMP_SZ 32 /* length of anti-virus scanstamp */
-
- typedef struct xoptattr {
-- timestruc_t xoa_createtime; /* Create time of file */
-+ inode_timespec_t xoa_createtime; /* Create time of file */
- uint8_t xoa_archive;
- uint8_t xoa_system;
- uint8_t xoa_readonly;
-@@ -640,13 +640,6 @@ extern void delay(clock_t ticks);
- #define USEC_TO_TICK(usec) ((usec) / (MICROSEC / hz))
- #define NSEC_TO_TICK(usec) ((usec) / (NANOSEC / hz))
-
--#define gethrestime_sec() time(NULL)
--#define gethrestime(t) \
-- do {\
-- (t)->tv_sec = gethrestime_sec();\
-- (t)->tv_nsec = 0;\
-- } while (0);
--
- #define max_ncpus 64
- #define boot_ncpus (sysconf(_SC_NPROCESSORS_ONLN))
-
-diff --git a/include/sys/zfs_znode.h b/include/sys/zfs_znode.h
-index c292f03739e..26d1eb37557 100644
---- a/include/sys/zfs_znode.h
-+++ b/include/sys/zfs_znode.h
-@@ -270,19 +270,36 @@ typedef struct znode_hold {
-
- extern unsigned int zfs_object_mutex_size;
-
--/* Encode ZFS stored time values from a struct timespec */
-+/*
-+ * Encode ZFS stored time values from a struct timespec / struct timespec64.
-+ */
- #define ZFS_TIME_ENCODE(tp, stmp) \
--{ \
-+do { \
- (stmp)[0] = (uint64_t)(tp)->tv_sec; \
- (stmp)[1] = (uint64_t)(tp)->tv_nsec; \
--}
-+} while (0)
-
--/* Decode ZFS stored time values to a struct timespec */
-+#if defined(HAVE_INODE_TIMESPEC64_TIMES)
-+/*
-+ * Decode ZFS stored time values to a struct timespec64
-+ * 4.18 and newer kernels.
-+ */
- #define ZFS_TIME_DECODE(tp, stmp) \
--{ \
-- (tp)->tv_sec = (time_t)(stmp)[0]; \
-- (tp)->tv_nsec = (long)(stmp)[1]; \
--}
-+do { \
-+ (tp)->tv_sec = (time64_t)(stmp)[0]; \
-+ (tp)->tv_nsec = (long)(stmp)[1]; \
-+} while (0)
-+#else
-+/*
-+ * Decode ZFS stored time values to a struct timespec
-+ * 4.17 and older kernels.
-+ */
-+#define ZFS_TIME_DECODE(tp, stmp) \
-+do { \
-+ (tp)->tv_sec = (time_t)(stmp)[0]; \
-+ (tp)->tv_nsec = (long)(stmp)[1]; \
-+} while (0)
-+#endif /* HAVE_INODE_TIMESPEC64_TIMES */
-
- /*
- * Timestamp defines
-diff --git a/include/sys/zpl.h b/include/sys/zpl.h
-index 65ed4313603..e433fbc64ea 100644
---- a/include/sys/zpl.h
-+++ b/include/sys/zpl.h
-@@ -189,4 +189,13 @@ zpl_dir_emit_dots(struct file *file, zpl_dir_context_t *ctx)
- }
- #endif /* HAVE_VFS_ITERATE */
-
-+/*
-+ * Linux 4.18, inode times converted from timespec to timespec64.
-+ */
-+#if defined(HAVE_INODE_TIMESPEC64_TIMES)
-+#define zpl_inode_timespec_trunc(ts, gran) timespec64_trunc(ts, gran)
-+#else
-+#define zpl_inode_timespec_trunc(ts, gran) timespec_trunc(ts, gran)
-+#endif
-+
- #endif /* _SYS_ZPL_H */
-diff --git a/lib/libspl/Makefile.am b/lib/libspl/Makefile.am
-index 59bc8ffb42f..a6e63cb8868 100644
---- a/lib/libspl/Makefile.am
-+++ b/lib/libspl/Makefile.am
-@@ -19,8 +19,6 @@ noinst_LTLIBRARIES = libspl.la
-
- USER_C = \
- getexecname.c \
-- gethrtime.c \
-- gethrestime.c \
- getmntany.c \
- list.c \
- mkdirp.c \
-diff --git a/lib/libspl/gethrestime.c b/lib/libspl/gethrestime.c
-deleted file mode 100644
-index d37cc2d5994..00000000000
---- a/lib/libspl/gethrestime.c
-+++ /dev/null
-@@ -1,38 +0,0 @@
--/*
-- * CDDL HEADER START
-- *
-- * The contents of this file are subject to the terms of the
-- * Common Development and Distribution License (the "License").
-- * You may not use this file except in compliance with the License.
-- *
-- * You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
-- * or http://www.opensolaris.org/os/licensing.
-- * See the License for the specific language governing permissions
-- * and limitations under the License.
-- *
-- * When distributing Covered Code, include this CDDL HEADER in each
-- * file and include the License file at usr/src/OPENSOLARIS.LICENSE.
-- * If applicable, add the following below this CDDL HEADER, with the
-- * fields enclosed by brackets "[]" replaced with your own identifying
-- * information: Portions Copyright [yyyy] [name of copyright owner]
-- *
-- * CDDL HEADER END
-- */
--
--/*
-- * Copyright 2008 Sun Microsystems, Inc. All rights reserved.
-- * Use is subject to license terms.
-- */
--
--#include <time.h>
--#include <sys/time.h>
--
--void
--gethrestime(timestruc_t *ts)
--{
-- struct timeval tv;
--
-- gettimeofday(&tv, NULL);
-- ts->tv_sec = tv.tv_sec;
-- ts->tv_nsec = tv.tv_usec * NSEC_PER_USEC;
--}
-diff --git a/lib/libspl/gethrtime.c b/lib/libspl/gethrtime.c
-deleted file mode 100644
-index 95ceb18e119..00000000000
---- a/lib/libspl/gethrtime.c
-+++ /dev/null
-@@ -1,45 +0,0 @@
--/*
-- * CDDL HEADER START
-- *
-- * The contents of this file are subject to the terms of the
-- * Common Development and Distribution License (the "License").
-- * You may not use this file except in compliance with the License.
-- *
-- * You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
-- * or http://www.opensolaris.org/os/licensing.
-- * See the License for the specific language governing permissions
-- * and limitations under the License.
-- *
-- * When distributing Covered Code, include this CDDL HEADER in each
-- * file and include the License file at usr/src/OPENSOLARIS.LICENSE.
-- * If applicable, add the following below this CDDL HEADER, with the
-- * fields enclosed by brackets "[]" replaced with your own identifying
-- * information: Portions Copyright [yyyy] [name of copyright owner]
-- *
-- * CDDL HEADER END
-- */
--
--/*
-- * Copyright 2008 Sun Microsystems, Inc. All rights reserved.
-- * Use is subject to license terms.
-- */
--
--#include <time.h>
--#include <sys/time.h>
--#include <stdlib.h>
--#include <stdio.h>
--
--hrtime_t
--gethrtime(void)
--{
-- struct timespec ts;
-- int rc;
--
-- rc = clock_gettime(CLOCK_MONOTONIC, &ts);
-- if (rc) {
-- fprintf(stderr, "Error: clock_gettime() = %d\n", rc);
-- abort();
-- }
--
-- return ((((u_int64_t)ts.tv_sec) * NANOSEC) + ts.tv_nsec);
--}
-diff --git a/lib/libspl/include/sys/time.h b/lib/libspl/include/sys/time.h
-index dc645fa5c37..04b3ba87bd3 100644
---- a/lib/libspl/include/sys/time.h
-+++ b/lib/libspl/include/sys/time.h
-@@ -27,8 +27,9 @@
- #ifndef _LIBSPL_SYS_TIME_H
- #define _LIBSPL_SYS_TIME_H
-
--#include_next <sys/time.h>
-+#include <time.h>
- #include <sys/types.h>
-+#include_next <sys/time.h>
-
- #ifndef SEC
- #define SEC 1
-@@ -70,13 +71,33 @@
- #define SEC2NSEC(m) ((hrtime_t)(m) * (NANOSEC / SEC))
- #endif
-
--
- typedef long long hrtime_t;
--typedef struct timespec timestruc_t;
--typedef struct timespec timespec_t;
--
--
--extern hrtime_t gethrtime(void);
--extern void gethrestime(timestruc_t *);
-+typedef struct timespec timespec_t;
-+typedef struct timespec inode_timespec_t;
-+
-+static inline void
-+gethrestime(inode_timespec_t *ts)
-+{
-+ struct timeval tv;
-+ (void) gettimeofday(&tv, NULL);
-+ ts->tv_sec = tv.tv_sec;
-+ ts->tv_nsec = tv.tv_usec * NSEC_PER_USEC;
-+}
-+
-+static inline time_t
-+gethrestime_sec(void)
-+{
-+ struct timeval tv;
-+ (void) gettimeofday(&tv, NULL);
-+ return (tv.tv_sec);
-+}
-+
-+static inline hrtime_t
-+gethrtime(void)
-+{
-+ struct timespec ts;
-+ (void) clock_gettime(CLOCK_MONOTONIC, &ts);
-+ return ((((u_int64_t)ts.tv_sec) * NANOSEC) + ts.tv_nsec);
-+}
-
- #endif /* _LIBSPL_SYS_TIME_H */
-diff --git a/lib/libzpool/kernel.c b/lib/libzpool/kernel.c
-index e67d13c9265..3ea8778b1dc 100644
---- a/lib/libzpool/kernel.c
-+++ b/lib/libzpool/kernel.c
-@@ -498,7 +498,7 @@ cv_timedwait(kcondvar_t *cv, kmutex_t *mp, clock_t abstime)
- {
- int error;
- struct timeval tv;
-- timestruc_t ts;
-+ struct timespec ts;
- clock_t delta;
-
- ASSERT3U(cv->cv_magic, ==, CV_MAGIC);
-@@ -536,7 +536,7 @@ cv_timedwait_hires(kcondvar_t *cv, kmutex_t *mp, hrtime_t tim, hrtime_t res,
- {
- int error;
- struct timeval tv;
-- timestruc_t ts;
-+ struct timespec ts;
- hrtime_t delta;
-
- ASSERT(flag == 0 || flag == CALLOUT_FLAG_ABSOLUTE);
-diff --git a/module/zfs/dmu_objset.c b/module/zfs/dmu_objset.c
-index 3425d542f98..449ebedfa16 100644
---- a/module/zfs/dmu_objset.c
-+++ b/module/zfs/dmu_objset.c
-@@ -860,7 +860,7 @@ dmu_objset_evict_done(objset_t *os)
- kmem_free(os, sizeof (objset_t));
- }
-
--timestruc_t
-+inode_timespec_t
- dmu_objset_snap_cmtime(objset_t *os)
- {
- return (dsl_dir_snap_cmtime(os->os_dsl_dataset->ds_dir));
-diff --git a/module/zfs/dsl_dir.c b/module/zfs/dsl_dir.c
-index a3ef5896a3f..deecf6bc53e 100644
---- a/module/zfs/dsl_dir.c
-+++ b/module/zfs/dsl_dir.c
-@@ -1975,10 +1975,10 @@ dsl_dir_transfer_possible(dsl_dir_t *sdd, dsl_dir_t *tdd,
- return (0);
- }
-
--timestruc_t
-+inode_timespec_t
- dsl_dir_snap_cmtime(dsl_dir_t *dd)
- {
-- timestruc_t t;
-+ inode_timespec_t t;
-
- mutex_enter(&dd->dd_lock);
- t = dd->dd_snap_cmtime;
-@@ -1990,7 +1990,7 @@ dsl_dir_snap_cmtime(dsl_dir_t *dd)
- void
- dsl_dir_snap_cmtime_update(dsl_dir_t *dd)
- {
-- timestruc_t t;
-+ inode_timespec_t t;
-
- gethrestime(&t);
- mutex_enter(&dd->dd_lock);
-diff --git a/module/zfs/fm.c b/module/zfs/fm.c
-index cb148149376..9d26cc99eeb 100644
---- a/module/zfs/fm.c
-+++ b/module/zfs/fm.c
-@@ -508,8 +508,8 @@ zfs_zevent_insert(zevent_t *ev)
- int
- zfs_zevent_post(nvlist_t *nvl, nvlist_t *detector, zevent_cb_t *cb)
- {
-+ inode_timespec_t tv;
- int64_t tv_array[2];
-- timestruc_t tv;
- uint64_t eid;
- size_t nvl_size = 0;
- zevent_t *ev;
-diff --git a/module/zfs/zfs_ctldir.c b/module/zfs/zfs_ctldir.c
-index 3b5fb196f1d..3ff2c101b67 100644
---- a/module/zfs/zfs_ctldir.c
-+++ b/module/zfs/zfs_ctldir.c
-@@ -451,7 +451,7 @@ static struct inode *
- zfsctl_inode_alloc(zfsvfs_t *zfsvfs, uint64_t id,
- const struct file_operations *fops, const struct inode_operations *ops)
- {
-- struct timespec now;
-+ inode_timespec_t now;
- struct inode *ip;
- znode_t *zp;
-
-diff --git a/module/zfs/zfs_vnops.c b/module/zfs/zfs_vnops.c
-index 0d2b61a1c31..34ea751c320 100644
---- a/module/zfs/zfs_vnops.c
-+++ b/module/zfs/zfs_vnops.c
-@@ -3158,7 +3158,7 @@ zfs_setattr(struct inode *ip, vattr_t *vap, int flags, cred_t *cr)
-
- if (mask & (ATTR_MTIME | ATTR_SIZE)) {
- ZFS_TIME_ENCODE(&vap->va_mtime, mtime);
-- ZTOI(zp)->i_mtime = timespec_trunc(vap->va_mtime,
-+ ZTOI(zp)->i_mtime = zpl_inode_timespec_trunc(vap->va_mtime,
- ZTOI(zp)->i_sb->s_time_gran);
-
- SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_MTIME(zfsvfs), NULL,
-@@ -3167,7 +3167,7 @@ zfs_setattr(struct inode *ip, vattr_t *vap, int flags, cred_t *cr)
-
- if (mask & (ATTR_CTIME | ATTR_SIZE)) {
- ZFS_TIME_ENCODE(&vap->va_ctime, ctime);
-- ZTOI(zp)->i_ctime = timespec_trunc(vap->va_ctime,
-+ ZTOI(zp)->i_ctime = zpl_inode_timespec_trunc(vap->va_ctime,
- ZTOI(zp)->i_sb->s_time_gran);
- SA_ADD_BULK_ATTR(bulk, count, SA_ZPL_CTIME(zfsvfs), NULL,
- ctime, sizeof (ctime));
-diff --git a/module/zfs/zfs_znode.c b/module/zfs/zfs_znode.c
-index f508a248f0f..e222c791183 100644
---- a/module/zfs/zfs_znode.c
-+++ b/module/zfs/zfs_znode.c
-@@ -700,7 +700,7 @@ zfs_mknode(znode_t *dzp, vattr_t *vap, dmu_tx_t *tx, cred_t *cr,
- uint64_t rdev = 0;
- zfsvfs_t *zfsvfs = ZTOZSB(dzp);
- dmu_buf_t *db;
-- timestruc_t now;
-+ inode_timespec_t now;
- uint64_t gen, obj;
- int bonuslen;
- int dnodesize;
-@@ -1349,7 +1349,7 @@ void
- zfs_tstamp_update_setup(znode_t *zp, uint_t flag, uint64_t mtime[2],
- uint64_t ctime[2])
- {
-- timestruc_t now;
-+ inode_timespec_t now;
-
- gethrestime(&now);
-
-diff --git a/module/zfs/zpl_inode.c b/module/zfs/zpl_inode.c
-index 3b5643d0917..41b91cabcb9 100644
---- a/module/zfs/zpl_inode.c
-+++ b/module/zfs/zpl_inode.c
-@@ -384,9 +384,10 @@ zpl_setattr(struct dentry *dentry, struct iattr *ia)
- vap->va_mtime = ia->ia_mtime;
- vap->va_ctime = ia->ia_ctime;
-
-- if (vap->va_mask & ATTR_ATIME)
-- ip->i_atime = timespec_trunc(ia->ia_atime,
-+ if (vap->va_mask & ATTR_ATIME) {
-+ ip->i_atime = zpl_inode_timespec_trunc(ia->ia_atime,
- ip->i_sb->s_time_gran);
-+ }
-
- cookie = spl_fstrans_mark();
- error = -zfs_setattr(ip, vap, 0, cr);
-From ecaa72315658207608c9e43df870de77af36c1ff Mon Sep 17 00:00:00 2001
-From: Brian Behlendorf <behlendorf1@llnl.gov>
-Date: Fri, 15 Jun 2018 15:05:21 -0700
-Subject: [PATCH] Linux compat 4.18: check_disk_size_change()
-
-Added support for the bops->check_events() interface which was
-added in the 2.6.38 kernel to replace bops->media_changed().
-Fully implementing this functionality allows the volume resize
-code to rely on revalidate_disk(), which is the preferred
-mechanism, and removes the need to use check_disk_size_change().
-
-In order for bops->check_events() to lookup the zvol_state_t
-stored in the disk->private_data the zvol_state_lock needs to
-be held. Since the check events interface may poll the mutex
-has been converted to a rwlock for better concurrently. The
-rwlock need only be taken as a writer in the zvol_free() path
-when disk->private_data is set to NULL.
-
-The configure checks for the block_device_operations structure
-were consolidated in a single kernel-block-device-operations.m4
-file.
-
-The ZFS_AC_KERNEL_BDEV_BLOCK_DEVICE_OPERATIONS configure checks
-and assoicated dead code was removed. This interface was added
-to the 2.6.28 kernel which predates the oldest supported 2.6.32
-kernel and will therefore always be available.
-
-Updated maximum Linux version in META file. The 4.17 kernel
-was released on 2018-06-03 and ZoL is compatible with the
-finalized kernel.
-
-Reviewed-by: Boris Protopopov <boris.protopopov@actifio.com>
-Reviewed-by: Sara Hartse <sara.hartse@delphix.com>
-Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
-Closes #7611
-Backported-by: Richard Yao <ryao@gentoo.org>
----
- config/kernel-bdev-block-device-operations.m4 | 34 ---
- ...el-block-device-operations-release-void.m4 | 29 --
- config/kernel-block-device-operations.m4 | 57 ++++
- config/kernel.m4 | 2 +-
- include/linux/blkdev_compat.h | 1 +
- module/zfs/zvol.c | 286 ++++++++----------
- 6 files changed, 192 insertions(+), 217 deletions(-)
- delete mode 100644 config/kernel-bdev-block-device-operations.m4
- delete mode 100644 config/kernel-block-device-operations-release-void.m4
- create mode 100644 config/kernel-block-device-operations.m4
-
-diff --git a/config/kernel-bdev-block-device-operations.m4 b/config/kernel-bdev-block-device-operations.m4
-deleted file mode 100644
-index faacc195da9..00000000000
---- a/config/kernel-bdev-block-device-operations.m4
-+++ /dev/null
-@@ -1,34 +0,0 @@
--dnl #
--dnl # 2.6.x API change
--dnl #
--AC_DEFUN([ZFS_AC_KERNEL_BDEV_BLOCK_DEVICE_OPERATIONS], [
-- AC_MSG_CHECKING([block device operation prototypes])
-- tmp_flags="$EXTRA_KCFLAGS"
-- EXTRA_KCFLAGS="${NO_UNUSED_BUT_SET_VARIABLE}"
-- ZFS_LINUX_TRY_COMPILE([
-- #include <linux/blkdev.h>
--
-- int blk_open(struct block_device *bdev, fmode_t mode)
-- { return 0; }
-- int blk_ioctl(struct block_device *bdev, fmode_t mode,
-- unsigned x, unsigned long y) { return 0; }
-- int blk_compat_ioctl(struct block_device * bdev, fmode_t mode,
-- unsigned x, unsigned long y) { return 0; }
--
-- static const struct block_device_operations
-- bops __attribute__ ((unused)) = {
-- .open = blk_open,
-- .release = NULL,
-- .ioctl = blk_ioctl,
-- .compat_ioctl = blk_compat_ioctl,
-- };
-- ],[
-- ],[
-- AC_MSG_RESULT(struct block_device)
-- AC_DEFINE(HAVE_BDEV_BLOCK_DEVICE_OPERATIONS, 1,
-- [struct block_device_operations use bdevs])
-- ],[
-- AC_MSG_RESULT(struct inode)
-- ])
-- EXTRA_KCFLAGS="$tmp_flags"
--])
-diff --git a/config/kernel-block-device-operations-release-void.m4 b/config/kernel-block-device-operations-release-void.m4
-deleted file mode 100644
-index a73f858722a..00000000000
---- a/config/kernel-block-device-operations-release-void.m4
-+++ /dev/null
-@@ -1,29 +0,0 @@
--dnl #
--dnl # 3.10.x API change
--dnl #
--AC_DEFUN([ZFS_AC_KERNEL_BLOCK_DEVICE_OPERATIONS_RELEASE_VOID], [
-- AC_MSG_CHECKING([whether block_device_operations.release is void])
-- tmp_flags="$EXTRA_KCFLAGS"
-- EXTRA_KCFLAGS="${NO_UNUSED_BUT_SET_VARIABLE}"
-- ZFS_LINUX_TRY_COMPILE([
-- #include <linux/blkdev.h>
--
-- void blk_release(struct gendisk *g, fmode_t mode) { return; }
--
-- static const struct block_device_operations
-- bops __attribute__ ((unused)) = {
-- .open = NULL,
-- .release = blk_release,
-- .ioctl = NULL,
-- .compat_ioctl = NULL,
-- };
-- ],[
-- ],[
-- AC_MSG_RESULT(void)
-- AC_DEFINE(HAVE_BLOCK_DEVICE_OPERATIONS_RELEASE_VOID, 1,
-- [struct block_device_operations.release returns void])
-- ],[
-- AC_MSG_RESULT(int)
-- ])
-- EXTRA_KCFLAGS="$tmp_flags"
--])
-diff --git a/config/kernel-block-device-operations.m4 b/config/kernel-block-device-operations.m4
-new file mode 100644
-index 00000000000..5f2811c1534
---- /dev/null
-+++ b/config/kernel-block-device-operations.m4
-@@ -0,0 +1,57 @@
-+dnl #
-+dnl # 2.6.38 API change
-+dnl #
-+AC_DEFUN([ZFS_AC_KERNEL_BLOCK_DEVICE_OPERATIONS_CHECK_EVENTS], [
-+ AC_MSG_CHECKING([whether bops->check_events() exists])
-+ tmp_flags="$EXTRA_KCFLAGS"
-+ EXTRA_KCFLAGS="${NO_UNUSED_BUT_SET_VARIABLE}"
-+ ZFS_LINUX_TRY_COMPILE([
-+ #include <linux/blkdev.h>
-+
-+ unsigned int blk_check_events(struct gendisk *disk,
-+ unsigned int clearing) { return (0); }
-+
-+ static const struct block_device_operations
-+ bops __attribute__ ((unused)) = {
-+ .check_events = blk_check_events,
-+ };
-+ ],[
-+ ],[
-+ AC_MSG_RESULT(yes)
-+ AC_DEFINE(HAVE_BLOCK_DEVICE_OPERATIONS_CHECK_EVENTS, 1,
-+ [bops->check_events() exists])
-+ ],[
-+ AC_MSG_RESULT(no)
-+ ])
-+ EXTRA_KCFLAGS="$tmp_flags"
-+])
-+
-+dnl #
-+dnl # 3.10.x API change
-+dnl #
-+AC_DEFUN([ZFS_AC_KERNEL_BLOCK_DEVICE_OPERATIONS_RELEASE_VOID], [
-+ AC_MSG_CHECKING([whether bops->release() is void])
-+ tmp_flags="$EXTRA_KCFLAGS"
-+ EXTRA_KCFLAGS="${NO_UNUSED_BUT_SET_VARIABLE}"
-+ ZFS_LINUX_TRY_COMPILE([
-+ #include <linux/blkdev.h>
-+
-+ void blk_release(struct gendisk *g, fmode_t mode) { return; }
-+
-+ static const struct block_device_operations
-+ bops __attribute__ ((unused)) = {
-+ .open = NULL,
-+ .release = blk_release,
-+ .ioctl = NULL,
-+ .compat_ioctl = NULL,
-+ };
-+ ],[
-+ ],[
-+ AC_MSG_RESULT(void)
-+ AC_DEFINE(HAVE_BLOCK_DEVICE_OPERATIONS_RELEASE_VOID, 1,
-+ [bops->release() returns void])
-+ ],[
-+ AC_MSG_RESULT(int)
-+ ])
-+ EXTRA_KCFLAGS="$tmp_flags"
-+])
-diff --git a/config/kernel.m4 b/config/kernel.m4
-index 375e4b79ae6..c7ca260c5f0 100644
---- a/config/kernel.m4
-+++ b/config/kernel.m4
-@@ -12,7 +12,7 @@ AC_DEFUN([ZFS_AC_CONFIG_KERNEL], [
- ZFS_AC_KERNEL_CURRENT_BIO_TAIL
- ZFS_AC_KERNEL_SUPER_USER_NS
- ZFS_AC_KERNEL_SUBMIT_BIO
-- ZFS_AC_KERNEL_BDEV_BLOCK_DEVICE_OPERATIONS
-+ ZFS_AC_KERNEL_BLOCK_DEVICE_OPERATIONS_CHECK_EVENTS
- ZFS_AC_KERNEL_BLOCK_DEVICE_OPERATIONS_RELEASE_VOID
- ZFS_AC_KERNEL_TYPE_FMODE_T
- ZFS_AC_KERNEL_3ARG_BLKDEV_GET
-diff --git a/include/linux/blkdev_compat.h b/include/linux/blkdev_compat.h
-index f99980ab3e0..27f05662662 100644
---- a/include/linux/blkdev_compat.h
-+++ b/include/linux/blkdev_compat.h
-@@ -32,6 +32,7 @@
- #include <linux/blkdev.h>
- #include <linux/elevator.h>
- #include <linux/backing-dev.h>
-+#include <linux/msdos_fs.h> /* for SECTOR_* */
-
- #ifndef HAVE_FMODE_T
- typedef unsigned __bitwise__ fmode_t;
-diff --git a/module/zfs/zvol.c b/module/zfs/zvol.c
-index 3e7059b340b..e57d2e70332 100644
---- a/module/zfs/zvol.c
-+++ b/module/zfs/zvol.c
-@@ -99,7 +99,7 @@ unsigned long zvol_max_discard_blocks = 16384;
- unsigned int zvol_volmode = ZFS_VOLMODE_GEOM;
-
- static taskq_t *zvol_taskq;
--static kmutex_t zvol_state_lock;
-+static krwlock_t zvol_state_lock;
- static list_t zvol_state_list;
-
- #define ZVOL_HT_SIZE 1024
-@@ -176,17 +176,17 @@ zvol_find_by_dev(dev_t dev)
- {
- zvol_state_t *zv;
-
-- mutex_enter(&zvol_state_lock);
-+ rw_enter(&zvol_state_lock, RW_READER);
- for (zv = list_head(&zvol_state_list); zv != NULL;
- zv = list_next(&zvol_state_list, zv)) {
- mutex_enter(&zv->zv_state_lock);
- if (zv->zv_dev == dev) {
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
- return (zv);
- }
- mutex_exit(&zv->zv_state_lock);
- }
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
-
- return (NULL);
- }
-@@ -204,7 +204,7 @@ zvol_find_by_name_hash(const char *name, uint64_t hash, int mode)
- zvol_state_t *zv;
- struct hlist_node *p = NULL;
-
-- mutex_enter(&zvol_state_lock);
-+ rw_enter(&zvol_state_lock, RW_READER);
- hlist_for_each(p, ZVOL_HT_HEAD(hash)) {
- zv = hlist_entry(p, zvol_state_t, zv_hlink);
- mutex_enter(&zv->zv_state_lock);
-@@ -227,12 +227,12 @@ zvol_find_by_name_hash(const char *name, uint64_t hash, int mode)
- strncmp(zv->zv_name, name, MAXNAMELEN)
- == 0);
- }
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
- return (zv);
- }
- mutex_exit(&zv->zv_state_lock);
- }
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
-
- return (NULL);
- }
-@@ -339,24 +339,6 @@ zvol_get_stats(objset_t *os, nvlist_t *nv)
- return (SET_ERROR(error));
- }
-
--static void
--zvol_size_changed(zvol_state_t *zv, uint64_t volsize)
--{
-- struct block_device *bdev;
--
-- ASSERT(MUTEX_HELD(&zv->zv_state_lock));
--
-- bdev = bdget_disk(zv->zv_disk, 0);
-- if (bdev == NULL)
-- return;
--
-- set_capacity(zv->zv_disk, volsize >> 9);
-- zv->zv_volsize = volsize;
-- check_disk_size_change(zv->zv_disk, bdev);
--
-- bdput(bdev);
--}
--
- /*
- * Sanity check volume size.
- */
-@@ -409,31 +391,17 @@ zvol_update_volsize(uint64_t volsize, objset_t *os)
- return (error);
- }
-
--static int
--zvol_update_live_volsize(zvol_state_t *zv, uint64_t volsize)
--{
-- zvol_size_changed(zv, volsize);
--
-- /*
-- * We should post a event here describing the expansion. However,
-- * the zfs_ereport_post() interface doesn't nicely support posting
-- * events for zvols, it assumes events relate to vdevs or zios.
-- */
--
-- return (0);
--}
--
- /*
-- * Set ZFS_PROP_VOLSIZE set entry point.
-+ * Set ZFS_PROP_VOLSIZE set entry point. Note that modifying the volume
-+ * size will result in a udev "change" event being generated.
- */
- int
- zvol_set_volsize(const char *name, uint64_t volsize)
- {
-- zvol_state_t *zv = NULL;
- objset_t *os = NULL;
-- int error;
-- dmu_object_info_t *doi;
-+ struct gendisk *disk = NULL;
- uint64_t readonly;
-+ int error;
- boolean_t owned = B_FALSE;
-
- error = dsl_prop_get_integer(name,
-@@ -443,7 +411,7 @@ zvol_set_volsize(const char *name, uint64_t volsize)
- if (readonly)
- return (SET_ERROR(EROFS));
-
-- zv = zvol_find_by_name(name, RW_READER);
-+ zvol_state_t *zv = zvol_find_by_name(name, RW_READER);
-
- ASSERT(zv == NULL || (MUTEX_HELD(&zv->zv_state_lock) &&
- RW_READ_HELD(&zv->zv_suspend_lock)));
-@@ -464,16 +432,18 @@ zvol_set_volsize(const char *name, uint64_t volsize)
- os = zv->zv_objset;
- }
-
-- doi = kmem_alloc(sizeof (dmu_object_info_t), KM_SLEEP);
-+ dmu_object_info_t *doi = kmem_alloc(sizeof (*doi), KM_SLEEP);
-
- if ((error = dmu_object_info(os, ZVOL_OBJ, doi)) ||
- (error = zvol_check_volsize(volsize, doi->doi_data_block_size)))
- goto out;
-
- error = zvol_update_volsize(volsize, os);
--
-- if (error == 0 && zv != NULL)
-- error = zvol_update_live_volsize(zv, volsize);
-+ if (error == 0 && zv != NULL) {
-+ zv->zv_volsize = volsize;
-+ zv->zv_changed = 1;
-+ disk = zv->zv_disk;
-+ }
- out:
- kmem_free(doi, sizeof (dmu_object_info_t));
-
-@@ -488,6 +458,9 @@ zvol_set_volsize(const char *name, uint64_t volsize)
- if (zv != NULL)
- mutex_exit(&zv->zv_state_lock);
-
-+ if (disk != NULL)
-+ revalidate_disk(disk);
-+
- return (SET_ERROR(error));
- }
-
-@@ -543,8 +516,8 @@ zvol_set_volblocksize(const char *name, uint64_t volblocksize)
- if (zv == NULL)
- return (SET_ERROR(ENXIO));
-
-- ASSERT(MUTEX_HELD(&zv->zv_state_lock) &&
-- RW_READ_HELD(&zv->zv_suspend_lock));
-+ ASSERT(MUTEX_HELD(&zv->zv_state_lock));
-+ ASSERT(RW_READ_HELD(&zv->zv_suspend_lock));
-
- if (zv->zv_flags & ZVOL_RDONLY) {
- mutex_exit(&zv->zv_state_lock);
-@@ -1120,7 +1093,7 @@ zvol_get_data(void *arg, lr_write_t *lr, char *buf, zio_t *zio)
- static void
- zvol_insert(zvol_state_t *zv)
- {
-- ASSERT(MUTEX_HELD(&zvol_state_lock));
-+ ASSERT(RW_WRITE_HELD(&zvol_state_lock));
- ASSERT3U(MINOR(zv->zv_dev) & ZVOL_MINOR_MASK, ==, 0);
- list_insert_head(&zvol_state_list, zv);
- hlist_add_head(&zv->zv_hlink, ZVOL_HT_HEAD(zv->zv_hash));
-@@ -1132,7 +1105,7 @@ zvol_insert(zvol_state_t *zv)
- static void
- zvol_remove(zvol_state_t *zv)
- {
-- ASSERT(MUTEX_HELD(&zvol_state_lock));
-+ ASSERT(RW_WRITE_HELD(&zvol_state_lock));
- list_remove(&zvol_state_list, zv);
- hlist_del(&zv->zv_hlink);
- }
-@@ -1148,8 +1121,8 @@ zvol_setup_zv(zvol_state_t *zv)
- uint64_t ro;
- objset_t *os = zv->zv_objset;
-
-- ASSERT(MUTEX_HELD(&zv->zv_state_lock) &&
-- RW_LOCK_HELD(&zv->zv_suspend_lock));
-+ ASSERT(MUTEX_HELD(&zv->zv_state_lock));
-+ ASSERT(RW_LOCK_HELD(&zv->zv_suspend_lock));
-
- error = dsl_prop_get_integer(zv->zv_name, "readonly", &ro, NULL);
- if (error)
-@@ -1227,8 +1200,8 @@ zvol_suspend(const char *name)
- return (NULL);
-
- /* block all I/O, release in zvol_resume. */
-- ASSERT(MUTEX_HELD(&zv->zv_state_lock) &&
-- RW_WRITE_HELD(&zv->zv_suspend_lock));
-+ ASSERT(MUTEX_HELD(&zv->zv_state_lock));
-+ ASSERT(RW_WRITE_HELD(&zv->zv_suspend_lock));
-
- atomic_inc(&zv->zv_suspend_ref);
-
-@@ -1349,9 +1322,7 @@ zvol_open(struct block_device *bdev, fmode_t flag)
- int error = 0;
- boolean_t drop_suspend = B_FALSE;
-
-- ASSERT(!mutex_owned(&zvol_state_lock));
--
-- mutex_enter(&zvol_state_lock);
-+ rw_enter(&zvol_state_lock, RW_READER);
- /*
- * Obtain a copy of private_data under the zvol_state_lock to make
- * sure that either the result of zvol free code path setting
-@@ -1360,7 +1331,7 @@ zvol_open(struct block_device *bdev, fmode_t flag)
- */
- zv = bdev->bd_disk->private_data;
- if (zv == NULL) {
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
- return (SET_ERROR(-ENXIO));
- }
-
-@@ -1378,8 +1349,7 @@ zvol_open(struct block_device *bdev, fmode_t flag)
- } else {
- rw_exit(&zv->zv_suspend_lock);
- }
--
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
-
- if (zv->zv_open_count == 0) {
- error = zvol_first_open(zv);
-@@ -1394,11 +1364,18 @@ zvol_open(struct block_device *bdev, fmode_t flag)
-
- zv->zv_open_count++;
-
-+ mutex_exit(&zv->zv_state_lock);
-+ if (drop_suspend)
-+ rw_exit(&zv->zv_suspend_lock);
-+
- check_disk_change(bdev);
-
-+ return (0);
-+
- out_open_count:
- if (zv->zv_open_count == 0)
- zvol_last_close(zv);
-+
- out_mutex:
- mutex_exit(&zv->zv_state_lock);
- if (drop_suspend)
-@@ -1419,26 +1396,34 @@ zvol_release(struct gendisk *disk, fmode_t mode)
- zvol_state_t *zv;
- boolean_t drop_suspend = B_FALSE;
-
-- ASSERT(!mutex_owned(&zvol_state_lock));
--
-- mutex_enter(&zvol_state_lock);
-+ rw_enter(&zvol_state_lock, RW_READER);
- zv = disk->private_data;
-- ASSERT(zv && zv->zv_open_count > 0);
--
-- /* take zv_suspend_lock before zv_state_lock */
-- rw_enter(&zv->zv_suspend_lock, RW_READER);
-
- mutex_enter(&zv->zv_state_lock);
-- mutex_exit(&zvol_state_lock);
-+ ASSERT(zv && zv->zv_open_count > 0);
-
- /*
- * make sure zvol is not suspended during last close
- * (hold zv_suspend_lock), otherwise, drop the lock
- */
-- if (zv->zv_open_count == 1)
-- drop_suspend = B_TRUE;
-- else
-- rw_exit(&zv->zv_suspend_lock);
-+ if (zv->zv_open_count == 1) {
-+ if (!rw_tryenter(&zv->zv_suspend_lock, RW_READER)) {
-+ mutex_exit(&zv->zv_state_lock);
-+ rw_enter(&zv->zv_suspend_lock, RW_READER);
-+ mutex_enter(&zv->zv_state_lock);
-+ /* check to see if zv_suspend_lock is needed */
-+ if (zv->zv_open_count != 1) {
-+ rw_exit(&zv->zv_suspend_lock);
-+ drop_suspend = B_FALSE;
-+ }
-+ }
-+ } else {
-+ drop_suspend = B_FALSE;
-+ }
-+ rw_exit(&zvol_state_lock);
-+
-+ ASSERT(MUTEX_HELD(&zv->zv_state_lock));
-+ ASSERT(zv->zv_open_count != 1 || RW_READ_HELD(&zv->zv_suspend_lock));
-
- zv->zv_open_count--;
- if (zv->zv_open_count == 0)
-@@ -1461,7 +1446,7 @@ zvol_ioctl(struct block_device *bdev, fmode_t mode,
- zvol_state_t *zv = bdev->bd_disk->private_data;
- int error = 0;
-
-- ASSERT(zv && zv->zv_open_count > 0);
-+ ASSERT3U(zv->zv_open_count, >, 0);
-
- switch (cmd) {
- case BLKFLSBUF:
-@@ -1501,23 +1486,62 @@ zvol_compat_ioctl(struct block_device *bdev, fmode_t mode,
- #define zvol_compat_ioctl NULL
- #endif
-
-+/*
-+ * Linux 2.6.38 preferred interface.
-+ */
-+#ifdef HAVE_BLOCK_DEVICE_OPERATIONS_CHECK_EVENTS
-+static unsigned int
-+zvol_check_events(struct gendisk *disk, unsigned int clearing)
-+{
-+ unsigned int mask = 0;
-+
-+ rw_enter(&zvol_state_lock, RW_READER);
-+
-+ zvol_state_t *zv = disk->private_data;
-+ if (zv != NULL) {
-+ mutex_enter(&zv->zv_state_lock);
-+ mask = zv->zv_changed ? DISK_EVENT_MEDIA_CHANGE : 0;
-+ zv->zv_changed = 0;
-+ mutex_exit(&zv->zv_state_lock);
-+ }
-+
-+ rw_exit(&zvol_state_lock);
-+
-+ return (mask);
-+}
-+#else
- static int zvol_media_changed(struct gendisk *disk)
- {
-+ int changed = 0;
-+
-+ rw_enter(&zvol_state_lock, RW_READER);
-+
- zvol_state_t *zv = disk->private_data;
-+ if (zv != NULL) {
-+ mutex_enter(&zv->zv_state_lock);
-+ changed = zv->zv_changed;
-+ zv->zv_changed = 0;
-+ mutex_exit(&zv->zv_state_lock);
-+ }
-
-- ASSERT(zv && zv->zv_open_count > 0);
-+ rw_exit(&zvol_state_lock);
-
-- return (zv->zv_changed);
-+ return (changed);
- }
-+#endif
-
- static int zvol_revalidate_disk(struct gendisk *disk)
- {
-- zvol_state_t *zv = disk->private_data;
-+ rw_enter(&zvol_state_lock, RW_READER);
-
-- ASSERT(zv && zv->zv_open_count > 0);
-+ zvol_state_t *zv = disk->private_data;
-+ if (zv != NULL) {
-+ mutex_enter(&zv->zv_state_lock);
-+ set_capacity(zv->zv_disk, zv->zv_volsize >> SECTOR_BITS);
-+ mutex_exit(&zv->zv_state_lock);
-+ }
-
-- zv->zv_changed = 0;
-- set_capacity(zv->zv_disk, zv->zv_volsize >> 9);
-+ rw_exit(&zvol_state_lock);
-
- return (0);
- }
-@@ -1534,7 +1558,7 @@ zvol_getgeo(struct block_device *bdev, struct hd_geometry *geo)
- zvol_state_t *zv = bdev->bd_disk->private_data;
- sector_t sectors;
-
-- ASSERT(zv && zv->zv_open_count > 0);
-+ ASSERT3U(zv->zv_open_count, >, 0);
-
- sectors = get_capacity(zv->zv_disk);
-
-@@ -1567,68 +1591,20 @@ zvol_probe(dev_t dev, int *part, void *arg)
- return (kobj);
- }
-
--#ifdef HAVE_BDEV_BLOCK_DEVICE_OPERATIONS
- static struct block_device_operations zvol_ops = {
- .open = zvol_open,
- .release = zvol_release,
- .ioctl = zvol_ioctl,
- .compat_ioctl = zvol_compat_ioctl,
-- .media_changed = zvol_media_changed,
-- .revalidate_disk = zvol_revalidate_disk,
-- .getgeo = zvol_getgeo,
-- .owner = THIS_MODULE,
--};
--
--#else /* HAVE_BDEV_BLOCK_DEVICE_OPERATIONS */
--
--static int
--zvol_open_by_inode(struct inode *inode, struct file *file)
--{
-- return (zvol_open(inode->i_bdev, file->f_mode));
--}
--
--static int
--zvol_release_by_inode(struct inode *inode, struct file *file)
--{
-- return (zvol_release(inode->i_bdev->bd_disk, file->f_mode));
--}
--
--static int
--zvol_ioctl_by_inode(struct inode *inode, struct file *file,
-- unsigned int cmd, unsigned long arg)
--{
-- if (file == NULL || inode == NULL)
-- return (SET_ERROR(-EINVAL));
--
-- return (zvol_ioctl(inode->i_bdev, file->f_mode, cmd, arg));
--}
--
--#ifdef CONFIG_COMPAT
--static long
--zvol_compat_ioctl_by_inode(struct file *file,
-- unsigned int cmd, unsigned long arg)
--{
-- if (file == NULL)
-- return (SET_ERROR(-EINVAL));
--
-- return (zvol_compat_ioctl(file->f_dentry->d_inode->i_bdev,
-- file->f_mode, cmd, arg));
--}
-+#ifdef HAVE_BLOCK_DEVICE_OPERATIONS_CHECK_EVENTS
-+ .check_events = zvol_check_events,
- #else
--#define zvol_compat_ioctl_by_inode NULL
--#endif
--
--static struct block_device_operations zvol_ops = {
-- .open = zvol_open_by_inode,
-- .release = zvol_release_by_inode,
-- .ioctl = zvol_ioctl_by_inode,
-- .compat_ioctl = zvol_compat_ioctl_by_inode,
- .media_changed = zvol_media_changed,
-+#endif
- .revalidate_disk = zvol_revalidate_disk,
- .getgeo = zvol_getgeo,
- .owner = THIS_MODULE,
- };
--#endif /* HAVE_BDEV_BLOCK_DEVICE_OPERATIONS */
-
- /*
- * Allocate memory for a new zvol_state_t and setup the required
-@@ -1681,6 +1657,10 @@ zvol_alloc(dev_t dev, const char *name)
- rw_init(&zv->zv_suspend_lock, NULL, RW_DEFAULT, NULL);
-
- zv->zv_disk->major = zvol_major;
-+#ifdef HAVE_BLOCK_DEVICE_OPERATIONS_CHECK_EVENTS
-+ zv->zv_disk->events = DISK_EVENT_MEDIA_CHANGE;
-+#endif
-+
- if (volmode == ZFS_VOLMODE_DEV) {
- /*
- * ZFS_VOLMODE_DEV disable partitioning on ZVOL devices: set
-@@ -1725,7 +1705,6 @@ zvol_free(void *arg)
- {
- zvol_state_t *zv = arg;
-
-- ASSERT(!MUTEX_HELD(&zvol_state_lock));
- ASSERT(!RW_LOCK_HELD(&zv->zv_suspend_lock));
- ASSERT(!MUTEX_HELD(&zv->zv_state_lock));
- ASSERT(zv->zv_open_count == 0);
-@@ -1852,9 +1831,9 @@ zvol_create_minor_impl(const char *name)
- kmem_free(doi, sizeof (dmu_object_info_t));
-
- if (error == 0) {
-- mutex_enter(&zvol_state_lock);
-+ rw_enter(&zvol_state_lock, RW_WRITER);
- zvol_insert(zv);
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
- add_disk(zv->zv_disk);
- } else {
- ida_simple_remove(&zvol_ida, idx);
-@@ -1871,7 +1850,7 @@ zvol_rename_minor(zvol_state_t *zv, const char *newname)
- {
- int readonly = get_disk_ro(zv->zv_disk);
-
-- ASSERT(MUTEX_HELD(&zvol_state_lock));
-+ ASSERT(RW_LOCK_HELD(&zvol_state_lock));
- ASSERT(MUTEX_HELD(&zv->zv_state_lock));
-
- strlcpy(zv->zv_name, newname, sizeof (zv->zv_name));
-@@ -2111,7 +2090,7 @@ zvol_remove_minors_impl(const char *name)
- list_create(&free_list, sizeof (zvol_state_t),
- offsetof(zvol_state_t, zv_next));
-
-- mutex_enter(&zvol_state_lock);
-+ rw_enter(&zvol_state_lock, RW_WRITER);
-
- for (zv = list_head(&zvol_state_list); zv != NULL; zv = zv_next) {
- zv_next = list_next(&zvol_state_list, zv);
-@@ -2136,15 +2115,15 @@ zvol_remove_minors_impl(const char *name)
- zvol_remove(zv);
-
- /*
-- * clear this while holding zvol_state_lock so
-- * zvol_open won't open it
-+ * Cleared while holding zvol_state_lock as a writer
-+ * which will prevent zvol_open() from opening it.
- */
- zv->zv_disk->private_data = NULL;
-
- /* Drop zv_state_lock before zvol_free() */
- mutex_exit(&zv->zv_state_lock);
-
-- /* try parallel zv_free, if failed do it in place */
-+ /* Try parallel zv_free, if failed do it in place */
- t = taskq_dispatch(system_taskq, zvol_free, zv,
- TQ_SLEEP);
- if (t == TASKQID_INVALID)
-@@ -2155,11 +2134,9 @@ zvol_remove_minors_impl(const char *name)
- mutex_exit(&zv->zv_state_lock);
- }
- }
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
-
-- /*
-- * Drop zvol_state_lock before calling zvol_free()
-- */
-+ /* Drop zvol_state_lock before calling zvol_free() */
- while ((zv = list_head(&free_list)) != NULL) {
- list_remove(&free_list, zv);
- zvol_free(zv);
-@@ -2178,7 +2155,7 @@ zvol_remove_minor_impl(const char *name)
- if (zvol_inhibit_dev)
- return;
-
-- mutex_enter(&zvol_state_lock);
-+ rw_enter(&zvol_state_lock, RW_WRITER);
-
- for (zv = list_head(&zvol_state_list); zv != NULL; zv = zv_next) {
- zv_next = list_next(&zvol_state_list, zv);
-@@ -2198,7 +2175,10 @@ zvol_remove_minor_impl(const char *name)
- }
- zvol_remove(zv);
-
-- /* clear this so zvol_open won't open it */
-+ /*
-+ * Cleared while holding zvol_state_lock as a writer
-+ * which will prevent zvol_open() from opening it.
-+ */
- zv->zv_disk->private_data = NULL;
-
- mutex_exit(&zv->zv_state_lock);
-@@ -2209,7 +2189,7 @@ zvol_remove_minor_impl(const char *name)
- }
-
- /* Drop zvol_state_lock before calling zvol_free() */
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
-
- if (zv != NULL)
- zvol_free(zv);
-@@ -2230,7 +2210,7 @@ zvol_rename_minors_impl(const char *oldname, const char *newname)
- oldnamelen = strlen(oldname);
- newnamelen = strlen(newname);
-
-- mutex_enter(&zvol_state_lock);
-+ rw_enter(&zvol_state_lock, RW_READER);
-
- for (zv = list_head(&zvol_state_list); zv != NULL; zv = zv_next) {
- zv_next = list_next(&zvol_state_list, zv);
-@@ -2258,7 +2238,7 @@ zvol_rename_minors_impl(const char *oldname, const char *newname)
- mutex_exit(&zv->zv_state_lock);
- }
-
-- mutex_exit(&zvol_state_lock);
-+ rw_exit(&zvol_state_lock);
- }
-
- typedef struct zvol_snapdev_cb_arg {
-@@ -2635,7 +2615,7 @@ zvol_init(void)
-
- list_create(&zvol_state_list, sizeof (zvol_state_t),
- offsetof(zvol_state_t, zv_next));
-- mutex_init(&zvol_state_lock, NULL, MUTEX_DEFAULT, NULL);
-+ rw_init(&zvol_state_lock, NULL, RW_DEFAULT, NULL);
- ida_init(&zvol_ida);
-
- zvol_taskq = taskq_create(ZVOL_DRIVER, threads, maxclsyspri,
-@@ -2672,7 +2652,7 @@ zvol_init(void)
- taskq_destroy(zvol_taskq);
- out:
- ida_destroy(&zvol_ida);
-- mutex_destroy(&zvol_state_lock);
-+ rw_destroy(&zvol_state_lock);
- list_destroy(&zvol_state_list);
-
- return (SET_ERROR(error));
-@@ -2689,7 +2669,7 @@ zvol_fini(void)
-
- taskq_destroy(zvol_taskq);
- list_destroy(&zvol_state_list);
-- mutex_destroy(&zvol_state_lock);
-+ rw_destroy(&zvol_state_lock);
-
- ida_destroy(&zvol_ida);
- }