Commit ce75af21 authored by GangCao's avatar GangCao Committed by Ben Walker
Browse files

QoS: remove the limit on unmap kinds of I/O 



Unmap, discard, write zeros will be sent down from
higher stack. Remove these IOs for the QoS limit.

Change-Id: Ieb3cc19f31c43f8ddf8f8d2fd338f442ef48b679
Signed-off-by: default avatarGangCao <gang.cao@intel.com>
Reviewed-on: https://review.gerrithub.io/c/442673


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarLiang Yan <liang.z.yan@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
parent 4a6f4552

lib/bdev/bdev.c

+0 −4
Original line number Diff line number Diff line
@@ -1235,8 +1235,6 @@ _spdk_bdev_qos_io_to_limit(struct spdk_bdev_io *bdev_io) 
	case SPDK_BDEV_IO_TYPE_NVME_IO_MD:

	case SPDK_BDEV_IO_TYPE_READ:

	case SPDK_BDEV_IO_TYPE_WRITE:

	case SPDK_BDEV_IO_TYPE_UNMAP:

	case SPDK_BDEV_IO_TYPE_WRITE_ZEROES:

		return true;

	default:

		return false;
@@ -1273,8 +1271,6 @@ _spdk_bdev_get_io_size_in_byte(struct spdk_bdev_io *bdev_io) 
		return bdev_io->u.nvme_passthru.nbytes;

	case SPDK_BDEV_IO_TYPE_READ:

	case SPDK_BDEV_IO_TYPE_WRITE:

	case SPDK_BDEV_IO_TYPE_UNMAP:

	case SPDK_BDEV_IO_TYPE_WRITE_ZEROES:

		return bdev_io->u.bdev.num_blocks * bdev->blocklen;

	default:

		return 0;