Commit c9e5777e authored by Dariusz Stojaczyk's avatar Dariusz Stojaczyk Committed by Jim Harris
Browse files

virtio: clarify feature negotiation failure 



Change-Id: Ifb1566033569af0b3cf4a9dfbc8094a9883c7622
Signed-off-by: default avatarDariusz Stojaczyk <dariuszx.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/420980


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent eeb1650c

lib/virtio/virtio.c

+3 −0
Original line number Diff line number Diff line
@@ -266,6 +266,9 @@ virtio_negotiate_features(struct virtio_dev *dev, uint64_t req_features) 
	virtio_dev_set_status(dev, VIRTIO_CONFIG_S_FEATURES_OK);

	if (!(virtio_dev_get_status(dev) & VIRTIO_CONFIG_S_FEATURES_OK)) {

		SPDK_ERRLOG("failed to set FEATURES_OK status!\n");

		/* either the device failed, or we offered some features that

		 * depend on other, not offered features.

		 */

		return -EINVAL;

	}