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>
Apparently Analagous 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