On 08/23/2017 11:40 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.vanassche@xxxxxxx> > Cc: Lee Duncan <lduncan@xxxxxxxx> > Cc: Christoph Hellwig <hch@xxxxxx> > Cc: Hannes Reinecke <hare@xxxxxxxx> > Cc: Johannes Thumshirn <jthumshirn@xxxxxxx> > --- > 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: Hannes Reinecke <hare@xxxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke Teamlead Storage & Networking hare@xxxxxxx +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg)
