[stgt] [PATCH] iscsi: target redirect support
FUJITA Tomonori
fujita.tomonori at lab.ntt.co.jp
Wed Jun 16 05:36:54 CEST 2010
On Tue, 15 Jun 2010 12:13:32 -0700
Chandra Seetharaman <sekharan at us.ibm.com> wrote:
> > > In effect, (1) I think we need to be able to mark the redirector tgtd as
> > > such (with an option in the command line or such),
> >
> > I'm not sure why we need this since sounds like you can avoid the
> > issue with a proper targets.conf.
>
> Looks like using the targets.conf file itself avoids the problem...
> but...
>
> having a separate "designated" redirector would allow
> - to have single node to which the initiators always connect
> - to run the "redirector" in a node which doesn't actually have the
> backing store
You can do it now. You can configure a target to be redirected
somewhere without adding luns.
> - avoid unnecessary creation of the threads for the luns (with 1 lun in
> both of my targets, i see 10 more threads og tgtd).
What version of tgt do you use?
As I wrote above, you can configure a target having only lun0 to be
redirected. We create only one pthread per target. We don't create any
threads for lun0.
> - Have the redirector definition in targets.conf itself (which would
> mostly be the case ?!)
That's a different issue. We can configure the redirect via tgtadm
now. All you need to do is modifying tgt-admin to support it. No need
to modify tgtd or tgtadm.
> What do you think ?
> >
> > btw, I think that we need to modify the redirect configuration on the
> > fly so I avoided the command line.
>
> I see that as a plus, but in a deployed environment won't the user want
> to have these in a static place, like targets.conf ?
See above. That's a different issue. Please send a patch for tgt-admin
to support to configure redirection. I'm happy to apply it.
--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
More information about the stgt
mailing list