Remove the unused but set variable se_tpg in vhost_scsi_nexus_cb() to
fix the following GCC warning when building with 'W=1':
drivers/vhost/scsi.c:1752:26: warning: variable ?se_tpg? set but not used
Signed-off-by: Tobias Klauser <tklauser at distanz.ch>
---
drivers/vhost/scsi.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/vhost/scsi.c b/drivers/vhost/scsi.c
index 6e29d053843d..e2be447752c2 100644
--- a/drivers/vhost/scsi.c
+++ b/drivers/vhost/scsi.c
@@ -1749,7 +1749,6 @@ static int vhost_scsi_nexus_cb(struct se_portal_group
*se_tpg,
static int vhost_scsi_make_nexus(struct vhost_scsi_tpg *tpg,
const char *name)
{
- struct se_portal_group *se_tpg;
struct vhost_scsi_nexus *tv_nexus;
mutex_lock(&tpg->tv_tpg_mutex);
@@ -1758,7 +1757,6 @@ static int vhost_scsi_make_nexus(struct vhost_scsi_tpg
*tpg,
pr_debug("tpg->tpg_nexus already exists\n");
return -EEXIST;
}
- se_tpg = &tpg->se_tpg;
tv_nexus = kzalloc(sizeof(struct vhost_scsi_nexus), GFP_KERNEL);
if (!tv_nexus) {
--
2.11.0.rc0.7.gbe5a750
On Fri, Nov 11, 2016 at 02:27:10PM +0100, Tobias Klauser wrote:> Remove the unused but set variable se_tpg in vhost_scsi_nexus_cb() to > fix the following GCC warning when building with 'W=1': > > drivers/vhost/scsi.c:1752:26: warning: variable ?se_tpg? set but not used > > Signed-off-by: Tobias Klauser <tklauser at distanz.ch> > --- > drivers/vhost/scsi.c | 2 -- > 1 file changed, 2 deletions(-)Reviewed-by: Stefan Hajnoczi <stefanha at redhat.com> -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 455 bytes Desc: not available URL: <http://lists.linuxfoundation.org/pipermail/virtualization/attachments/20161115/2de75d2b/attachment.sig>
Possibly Parallel Threads
- [PATCH] vhost/scsi: Remove unused but set variable
- [PATCH 3/8] vhost scsi: alloc cmds per vq instead of session
- [PATCH 2/2] vhost/scsi: Delete error messages for failed memory allocations in five functions
- [PATCH 1/2] vhost/scsi: Improve a size determination in four functions
- [PATCH] tcm_vhost: Post-merge review changes requested by MST