[stgt] New portal feature breaks systems without IPv6 in kernel

ronnie sahlberg ronniesahlberg at gmail.com
Fri May 13 11:36:04 CEST 2011

On Thu, May 12, 2011 at 8:22 PM, FUJITA Tomonori
<fujita.tomonori at lab.ntt.co.jp> wrote:
> On Wed, 11 May 2011 17:06:35 +1000
> ronnie sahlberg <ronniesahlberg at gmail.com> 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, 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?

I can and I will
patch eta tomorrow
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