From: Brian Behlendorf Date: Mon, 15 Oct 2012 16:27:48 +0000 (-0700) Subject: Limit zfs_vdev_aggregation_limit to SPA_MAXBLOCKSIZE X-Git-Url: https://git.camperquake.de/gitweb.cgi?a=commitdiff_plain;h=c418410393325eb01be5b6cc5ff0e6e8aa24af93;hp=-c;p=zfs.git Limit zfs_vdev_aggregation_limit to SPA_MAXBLOCKSIZE Prevent users from setting the zfs_vdev_aggregation_limit tuning larger than SPA_MAXBLOCKSIZE. Signed-off-by: Brian Behlendorf Closes #520 --- c418410393325eb01be5b6cc5ff0e6e8aa24af93 diff --git a/module/zfs/vdev_queue.c b/module/zfs/vdev_queue.c index 7ba6389..e2096fa 100644 --- a/module/zfs/vdev_queue.c +++ b/module/zfs/vdev_queue.c @@ -201,7 +201,7 @@ vdev_queue_io_to_issue(vdev_queue_t *vq, uint64_t pending_limit) avl_tree_t *t; vdev_io_t *vi; int flags; - uint64_t maxspan = zfs_vdev_aggregation_limit; + uint64_t maxspan = MIN(zfs_vdev_aggregation_limit, SPA_MAXBLOCKSIZE); uint64_t maxgap; int stretch; @@ -312,7 +312,7 @@ again: if (fio != lio) { uint64_t size = IO_SPAN(fio, lio); - ASSERT(size <= zfs_vdev_aggregation_limit); + ASSERT(size <= maxspan); ASSERT(vi != NULL); aio = zio_vdev_delegated_io(fio->io_vd, fio->io_offset,