Re: New portal feature breaks systems without IPv6 in kernel

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

 



On Wed, 11 May 2011 17:06:35 +1000
ronnie sahlberg <ronniesahlberg@xxxxxxxxx> wrote:

> By default, if no portal is specified, tgtd will try to bind to both
> IPv6 and IPv4 wildcard.
> This caused the start to fail with an error on systems with no IPv6 available.

The root problem is that the current code assumes network
configurations in the following way:

if (list_empty(&iscsi_portals_list)) {
        iscsi_add_portal("0::0", 0, 1, 0);
        iscsi_add_portal("0.0.0.0", 0, 1, 0);
}

Which is exactly what network programing text books tell us not to do,
I think.

What we should do here is asking a system about available network
addresses by passing NULL to getaddrinfo() like the old code did.

I think that we should move the portal allocation code in
iscsi_add_portal() to iscsi_tcp_init_potal(). iscsi_tcp_init_potal()
takes addr and port, then creates iscsi_portal struct.

Can you fix this in such way?


> Note that even on those systrems that lack IPv6 support, tgtd can be
> used and start, just that it must be started with deliberate
> non-support for IPv6 by ONLY specifying an IPv4 portal,   example
> 
> tgtd --iscsi portal=0.0.0.0:3260
> 
> This allows tgtd to run on those systems that are non-IPv6 aware even
> without this patch.

If the latest version can't work with the previous configuration, then
it's a regression. We need to fix that.


> Since being from AsiaPac, where ARIN did run out of IPv4 addresses a
> while back,  I dont know if I want this patch applied.
> My portal stuff did cause a regression, but on the other hand I see
> good reason too why forcing IPv4 users to become aware.
> And they can use a manual override, like specifying IPv4 portal
> manually on the command line.

Well, my company have enough money to get new IPv4 addresses in
several ways even if ARIN can't allocate IPV4 addresses. :)

Seriously, I don't think we can make IPv6 mandatory ever.


Please fix the issue in a different way.


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


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

  Powered by Linux