Function conversion of Task Management Function Request

[nishino syuuzou <nishino.syuuzou@xxxxxxxxxxxxxx>]

Hi, 

I would like to confirm about Function conversion of TMFR.
I think that the Function is changed to the internal variable 
with the following code but it seems to CLEAR_TAST_SET(*1) 
and CLEAR_ACA(*2) are placed crossly.

./usr/iscsi/iscsid.c
static int iscsi_tm_execute(struct iscsi_task *task) {
   :
        switch (req->flags & ISCSI_FLAG_TM_FUNC_MASK) {
              :
        case ISCSI_TM_FUNC_ABORT_TASK:
                fn = ABORT_TASK;
                break;
        case ISCSI_TM_FUNC_ABORT_TASK_SET:
                fn = ABORT_TASK_SET;
                break;
        case ISCSI_TM_FUNC_CLEAR_ACA:
*1              fn = CLEAR_TASK_SET;
                break;
        case ISCSI_TM_FUNC_CLEAR_TASK_SET:
*2              fn = CLEAR_ACA;
                break;

After the above determination (in target_mgmt_request function in
./usr/target.c),
the case "ISCSI_TM_FUNC_CLEAR_ACA" is performed about Task aborting and 
the case "ISCSI_TM_FUNC_CLEAR_TASK_SET" is performed with the comment 
"We don't support ACA".

What do you think of this?

Best Regards,
Syuuzou Nishino


--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html