target: Fix target_submit_tmr se_tmr_req allocation failures
[linux-2.6.git] / drivers / target / target_core_transport.c
index d44cbc2..96e78f3 100644 (file)
@@ -1704,11 +1704,12 @@ EXPORT_SYMBOL(target_submit_cmd);
  * @unpacked_lun: unpacked LUN to reference for struct se_lun
  * @fabric_context: fabric context for TMR req
  * @tm_type: Type of TM request
+ * @flags: submit cmd flags
  *
  * Callable from all contexts.
  **/
 
-void target_submit_tmr(struct se_cmd *se_cmd, struct se_session *se_sess,
+int target_submit_tmr(struct se_cmd *se_cmd, struct se_session *se_sess,
                unsigned char *sense, u32 unpacked_lun,
                void *fabric_tmr_ptr, unsigned char tm_type, int flags)
 {
@@ -1720,25 +1721,26 @@ void target_submit_tmr(struct se_cmd *se_cmd, struct se_session *se_sess,
 
        transport_init_se_cmd(se_cmd, se_tpg->se_tpg_tfo, se_sess,
                              0, DMA_NONE, MSG_SIMPLE_TAG, sense);
+       /*
+        * FIXME: Currently expect caller to handle se_cmd->se_tmr_req
+        * allocation failure.
+        */
+       ret = core_tmr_alloc_req(se_cmd, fabric_tmr_ptr, tm_type, GFP_KERNEL);
+       if (ret < 0)
+               return -ENOMEM;
 
        /* See target_submit_cmd for commentary */
        target_get_sess_cmd(se_sess, se_cmd, (flags & TARGET_SCF_ACK_KREF));
 
-       ret = core_tmr_alloc_req(se_cmd, fabric_tmr_ptr, tm_type, GFP_KERNEL);
-       if (ret < 0) {
-               dump_stack();
-               /* FIXME XXX */
-               return;
-       }
-
        ret = transport_lookup_tmr_lun(se_cmd, unpacked_lun);
        if (ret) {
                se_cmd->se_tmr_req->response = TMR_LUN_DOES_NOT_EXIST;
                se_cmd->se_tfo->queue_tm_rsp(se_cmd);
                transport_generic_free_cmd(se_cmd, 0);
-               return;
+               return 0;
        }
        transport_generic_handle_tmr(se_cmd);
+       return 0;
 }
 EXPORT_SYMBOL(target_submit_tmr);