From fbb1fc41f38d58958718d596cef24a2568fda4d0 Mon Sep 17 00:00:00 2001 From: Jim Harris Date: Fri, 25 Jan 2019 08:55:21 -0700 Subject: [PATCH] scsi: squash errlog for THIN_PROVISION INQUIRY page It is perfectly valid for a bdev to not support the unmap command - there's no need to print an ERRLOG when a SCSI INQUIRY 0xB2 (LOGICAL BLOCK PROVISIONING) command is sent to query if the LUN supports it. Signed-off-by: Jim Harris Change-Id: I18389df4d55a1ac186707d624ddea292a5470e80 Reviewed-on: https://review.gerrithub.io/c/442104 Tested-by: SPDK CI Jenkins Reviewed-by: Ben Walker Reviewed-by: Shuhei Matsumoto Reviewed-by: Changpeng Liu Chandler-Test-Pool: SPDK Automated Test System --- lib/scsi/scsi_bdev.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/scsi/scsi_bdev.c b/lib/scsi/scsi_bdev.c index 4b766bd6b..641ca1315 100644 --- a/lib/scsi/scsi_bdev.c +++ b/lib/scsi/scsi_bdev.c @@ -644,7 +644,6 @@ spdk_bdev_scsi_inquiry(struct spdk_bdev *bdev, struct spdk_scsi_task *task, case SPDK_SPC_VPD_BLOCK_THIN_PROVISION: { if (!spdk_bdev_io_type_supported(bdev, SPDK_BDEV_IO_TYPE_UNMAP)) { - SPDK_ERRLOG("unsupported INQUIRY VPD page 0x%x\n", pc); goto inq_error; }