> @@ -282,6 +305,16 @@ static void core_tmr_drain_state_list( > if (prout_cmd == cmd) > continue; > > + sess = cmd->se_sess; > + if (WARN_ON_ONCE(!sess)) > + continue; > + > + spin_lock(&sess->sess_cmd_lock); > + rc = __target_check_io_state(cmd); > + spin_unlock(&sess->sess_cmd_lock); > + if (!rc) > + continue; > +
I still don't understand why we care about the session or sess_cmd_lock here. There isn't anything in __target_check_io_state that relies on them (except for the assert checking the lock). -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
