Commit 7d427678 authored by Dariusz Stojaczyk's avatar Dariusz Stojaczyk Committed by Daniel Verkamp
Browse files

vhost: removed redundant check in get_scsi_lun() 



Each call to get_scsi_lun is preceeded with the same check.

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


Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarPawel Wodkowski <pawelx.wodkowski@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent 9548caa6

lib/vhost/vhost_scsi.c

+1 −5
Original line number Diff line number Diff line
@@ -227,13 +227,9 @@ get_scsi_lun(struct spdk_scsi_dev *scsi_dev, const __u8 *lun) 
	uint16_t lun_id = (((uint16_t)lun[2] << 8) | lun[3]) & 0x3FFF;



	/* For now only one LUN per controller is allowed so no need to search LUN IDs */

	if (likely(scsi_dev != NULL)) {

	return spdk_scsi_dev_get_lun(scsi_dev, lun_id);

}



	return NULL;

}



static void

process_ctrl_request(struct spdk_vhost_scsi_dev *svdev, struct rte_vhost_vring *controlq,

		     uint16_t req_idx)