From 409b4127eea9bcf276f3bdaf5c2b8a907ccd1d59 Mon Sep 17 00:00:00 2001
From: Matthew Ahrens <mahrens@delphix.com>
Date: Fri, 23 Sep 2016 20:44:15 -0700
Subject: [PATCH] OpenZFS 7337 - inherit_001_pos occasionally times out

Authored by: Matthew Ahrens <mahrens@delphix.com>
Reviewed by: John Kennedy <john.kennedy@delphix.com>
Reviewed by: George Wilson <george.wilson@delphix.com>
Approved by: Robert Mustacchi <rm@joyent.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Giuseppe Di Natale <dinatale2@llnl.gov>
Ported-by: George Melikov <mail@gmelikov.ru>

OpenZFS-issue: https://www.illumos.org/issues/7337
OpenZFS-commit: https://github.com/openzfs/openzfs/commit/b021ac0
Closes #5800

Porting notes:
- Additional code refactor for better Zol and OpenZFS codebase sync
---
 .../inheritance/inherit_001_pos.ksh           | 76 +++++++++----------
 1 file changed, 36 insertions(+), 40 deletions(-)

diff --git a/tests/zfs-tests/tests/functional/inheritance/inherit_001_pos.ksh b/tests/zfs-tests/tests/functional/inheritance/inherit_001_pos.ksh
index afcbfde0272a..96a07eb1bc3f 100755
--- a/tests/zfs-tests/tests/functional/inheritance/inherit_001_pos.ksh
+++ b/tests/zfs-tests/tests/functional/inheritance/inherit_001_pos.ksh
@@ -25,7 +25,7 @@
 # Use is subject to license terms.
 
 #
-# Copyright (c) 2013 by Delphix. All rights reserved.
+# Copyright (c) 2013, 2015 by Delphix. All rights reserved.
 #
 
 . $STF_SUITE/include/libtest.shlib
@@ -370,53 +370,49 @@ function scan_state { #state-file
 	done
 }
 
+#
+# Note that we keep this list relatively short so that this test doesn't
+# time out (after taking more than 10 minutes).
+#
+set -A prop "checksum" "" \
+	"compression" "" \
+	"atime" "" \
+	"sharenfs" "" \
+	"recordsize" "recsize" \
+	"snapdir" "" \
+	"readonly" ""
+
 #
 # Note except for the mountpoint default value (which is handled in
 # the routine itself), each property specified in the 'prop' array
 # above must have a corresponding entry in the two arrays below.
 #
-if is_linux; then
-	set -A prop "checksum" "" \
-		"compression" "compress" \
-		"atime" "" \
-		"devices" "" \
-		"exec" "" \
-		"setuid" "" \
-		"recordsize" "recsize" \
-		"snapdir" "" \
-		"acltype"
 
-	set -A def_val "on" "off" "on" "on" "on" \
-		"on" "" \
-		"hidden" "off"
+set -A def_val "on" "off" "on" \
+	"off" "" \
+	"hidden" \
+	"off"
 
-	set -A local_val "off" "on" "off" "off" "off" \
-		"off" "" \
-		"visible" "off"
+set -A local_val "off" "on" "off" \
+	"on" "" \
+	"visible" \
+	"off"
+
+#
+# Add system specific values
+#
+
+if ! is_linux; then
+	prop+=("aclmode" "" \
+		"mountpoint" "")
+	def_val+=("discard" \
+		"")
+	local_val+=("groupmask" \
+		"$TESTDIR")
 else
-	set -A prop "checksum" "" \
-		"compression" "compress" \
-		"atime" "" \
-		"devices" "" \
-		"exec" "" \
-		"setuid" "" \
-		"sharenfs", "" \
-		"recordsize" "recsize" \
-		"mountpoint" "" \
-		"snapdir" "" \
-		"aclmode", "" \
-		"aclinherit" "" \
-		"readonly" "rdonly"
-
-	set -A def_val "on" "off" "on" "on" "on" \
-		"on" "off" "" \
-		"" "hidden" "discard" "secure" \
-		"off"
-
-	set -A local_val "off" "on" "off" "off" "off" \
-		"off" "on" "" \
-		"$TESTDIR" "visible" "groupmask" "discard" \
-		"off"
+	prop+=("acltype" "")
+	def_val+=("off")
+	local_val+=("off")
 fi