search for: target_depend_item

...y(*vc->lunp != 1)) { > /* virtio-scsi spec requires byte 0 of the lun to be 1 */ > vq_err(vq, "Illegal virtio-scsi lun: %u\n", *vc->lunp); > @@ -1441,7 +1441,7 @@ static void vhost_scsi_flush(struct vhost_scsi *vs) > se_tpg = &tpg->se_tpg; > ret = target_depend_item(&se_tpg->tpg_group.cg_item); > if (ret) { > - pr_warn("configfs_depend_item() failed: %d\n", ret); > + pr_warn("target_depend_item() failed: %d\n", ret); > kfree(vs_tpg); > mutex_unlock(&tpg->tv_tpg_mutex); > goto out; &gt...

...> mutex_unlock(&tpg->tv_tpg_mutex); > ret = -EEXIST; > - goto out; > + goto undepend; > } > /* > * In order to ensure individual vhost-scsi configfs > @@ -1442,9 +1522,8 @@ static void vhost_scsi_flush(struct vhost_scsi *vs) > ret = target_depend_item(&se_tpg->tpg_group.cg_item); > if (ret) { > pr_warn("target_depend_item() failed: %d\n", ret); > - kfree(vs_tpg); > mutex_unlock(&tpg->tv_tpg_mutex); > - goto out; > + goto undepend; > } > tpg->tv_tpg_vhost_count++; &...

The following patches were made over Linus tree and also apply over mst tree's vhost branch. The patches fix an issue where management operations like LUN mapping/unmapping and device addition hang for 30 seconds or up to N minutes depending on the device. The problem is that we use a global mutex to protect the list of tpgs but we hold that mutex during those management operations. So if you

The following patches were made over Linus tree. The patches fix 3 issues: 1. If a user performs LIO LUN unmapping before the endpoint has been cleared then we can end up trying to free a bogus tmf struct if the TMF is still exucuting when we do the unmap. 2. If vhost_scsi_setup_vq_cmds fails we can leave the tpg->vhost_scsi pointer set and we can end up trying to access a freed struct. 3.