On Mon, 2012-07-16 at 18:56 -0700, Nicholas A. Bellinger wrote: > On Mon, 2012-07-16 at 18:40 -0700, Roland Dreier wrote: > > > OK, I'll take a look at how you handle this... > > > > So looking at commit bc187ea6c3b3 in the tree you just pushed out > > ("target: Check sess_tearing_down in target_get_sess_cmd()") it looks > > like you just return from target_submit_cmd() if we fail to add the > > command to sess_cmd_list -- doesn't this mean we just leak those > > commands and possibly leave the HW sitting there with open exchanges? > > > > With dropping patch #5 for now, I assume that would be the case.. > Or with patch #4 even.. -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html