On 08/25/2017 01:46 PM, Bart Van Assche wrote: > This patch avoids that gcc reports the following warning when > building with W=1: > > drivers/scsi/iscsi_tcp.c:166:24: warning: variable ?session? set but not used > [-Wunused-but-set-variable] > > Signed-off-by: Bart Van Assche <bart.vanass...@wdc.com> > Reviewed-by: Christoph Hellwig <h...@lst.de> > Reviewed-by: Hannes Reinecke <h...@suse.com> > Cc: Lee Duncan <ldun...@suse.com> > Cc: Johannes Thumshirn <jthumsh...@suse.de> > --- > drivers/scsi/iscsi_tcp.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/scsi/iscsi_tcp.c b/drivers/scsi/iscsi_tcp.c > index 4842fc0e809d..4d934d6c3e13 100644 > --- a/drivers/scsi/iscsi_tcp.c > +++ b/drivers/scsi/iscsi_tcp.c > @@ -163,7 +163,6 @@ static void iscsi_sw_tcp_state_change(struct sock *sk) > struct iscsi_tcp_conn *tcp_conn; > struct iscsi_sw_tcp_conn *tcp_sw_conn; > struct iscsi_conn *conn; > - struct iscsi_session *session; > void (*old_state_change)(struct sock *); > > read_lock_bh(&sk->sk_callback_lock); > @@ -172,7 +171,6 @@ static void iscsi_sw_tcp_state_change(struct sock *sk) > read_unlock_bh(&sk->sk_callback_lock); > return; > } > - session = conn->session; > > iscsi_sw_sk_state_check(sk); > >
Reviewed-by: Lee Duncan <ldun...@suse.com> -- Lee Duncan
