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
