summaryrefslogtreecommitdiffstats
path: root/scripts/zpios-survey.sh
diff options
context:
space:
mode:
authorBrian Behlendorf <[email protected]>2015-02-06 13:37:02 -0800
committerBrian Behlendorf <[email protected]>2015-02-10 16:08:49 -0800
commit6442f3cfe3454b458456823fff49a14bf0badf65 (patch)
tree948338fa05ba572e8f1f092c87d51a8cf43fe6b3 /scripts/zpios-survey.sh
parent534759fad30ca55608bcd42a0af5a87c8c4feb36 (diff)
Retire zio_bulk_flags
Long ago the zio_bulk_flags module parameter was introduced to facilitate debugging and profiling the zio_buf_caches. Today this code works well and there's no compelling reason to keep this functionality. In fact it's preferable to revert this so the code is more consistent with other ZFS implementations. Signed-off-by: Brian Behlendorf <[email protected]> Signed-off-by: Ned Bass <[email protected]> Issue #3063
Diffstat (limited to 'scripts/zpios-survey.sh')
-rwxr-xr-xscripts/zpios-survey.sh30
1 files changed, 1 insertions, 29 deletions
diff --git a/scripts/zpios-survey.sh b/scripts/zpios-survey.sh
index cb751b467..78601695f 100755
--- a/scripts/zpios-survey.sh
+++ b/scripts/zpios-survey.sh
@@ -120,40 +120,13 @@ zpios_survey_pending() {
tee -a ${ZPIOS_SURVEY_LOG}
}
-# To avoid memory fragmentation issues our slab implementation can be
-# based on a virtual address space. Interestingly, we take a pretty
-# substantial performance penalty for this somewhere in the low level
-# IO drivers. If we back the slab with kmem pages we see far better
-# read performance numbers at the cost of memory fragmention and general
-# system instability due to large allocations. This may be because of
-# an optimization in the low level drivers due to the contigeous kmem
-# based memory. This needs to be explained. The good news here is that
-# with zerocopy interfaces added at the DMU layer we could gaurentee
-# kmem based memory for a pool of pages.
-#
-# 0x100 = KMC_KMEM - Force kmem_* based slab
-# 0x200 = KMC_VMEM - Force vmem_* based slab
-zpios_survey_kmem() {
- TEST_NAME="${ZPOOL_CONFIG}+${ZPIOS_TEST}+kmem"
- print_header ${TEST_NAME}
-
- ${ZFS_SH} ${VERBOSE_FLAG} \
- zfs="zio_bulk_flags=0x100" | \
- tee -a ${ZPIOS_SURVEY_LOG}
- ${ZPIOS_SH} ${VERBOSE_FLAG} -c ${ZPOOL_CONFIG} -t ${ZPIOS_TEST} | \
- tee -a ${ZPIOS_SURVEY_LOG}
- ${ZFS_SH} -u ${VERBOSE_FLAG} | \
- tee -a ${ZPIOS_SURVEY_LOG}
-}
-
# Apply all possible turning concurrently to get a best case number
zpios_survey_all() {
TEST_NAME="${ZPOOL_CONFIG}+${ZPIOS_TEST}+all"
print_header ${TEST_NAME}
${ZFS_SH} ${VERBOSE_FLAG} \
- zfs="zfs_vdev_max_pending=1024" \
- zfs="zio_bulk_flags=0x100" | \
+ zfs="zfs_vdev_max_pending=1024" | \
tee -a ${ZPIOS_SURVEY_LOG}
${ZPIOS_SH} ${VERBOSE_FLAG} -c ${ZPOOL_CONFIG} -t ${ZPIOS_TEST} \
-o "--noprefetch --zerocopy" \
@@ -209,7 +182,6 @@ zpios_survey_prefetch
zpios_survey_zerocopy
zpios_survey_checksum
zpios_survey_pending
-zpios_survey_kmem
zpios_survey_all
exit 0