Re: Function conversion of Task Management Function Request

[FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx>]

On Thu, 23 Jun 2011 09:18:14 +0900
nishino syuuzou <nishino.syuuzou@xxxxxxxxxxxxxx> wrote:

> 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;

I think that it's a bug. Please send a patch.


> 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".

Sounds a bug too. We should return NOT_SUPPORT response about tmf that
we don't support. Can you send a patch too?

It would be great if you can send a patch to implement the above
feature that tgt doesn't support.

Thanks!

--
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