Re: [PATCH] iscsi: target redirect support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 15 Jun 2010 12:13:32 -0700
Chandra Seetharaman <sekharan@xxxxxxxxxx> 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SCSI]     [Linux RAID]     [Linux Clusters]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]

  Powered by Linux