Re: [PATCH 02/11] qla4xxx: added IPv6 support.

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

 



On 04/06/2010 05:21 AM, ravi.anand wrote:

On Feb 1, 2010, at 10:15 AM, Mike Christie wrote:

On 01/30/2010 12:28 AM, Ravi Anand wrote:


+ DEBUG2(dev_info(&ha->pdev->dev, "%s: DDB[%d] osIdx = %d "
+ "State %04x ConnErr %08x "
+ NIPQUAD_FMT ":%04d \"%s\"\n",


Do you go down this path for ipv6 (looked like it did and did in other
places)? NIPQUAD_FMT will not work, and we are trying to not use it
NIPQUAD_FMT/NIPQUAD. See %pI4 use in qla4xxx_conn_get_param. There is
also a %pI6 for ipv6.

Also you need to update qla4xxx_conn_get_param and
qla4xxx_host_get_param ip handling.



While trying to address the IPV6 part for conn_get_param() and
host_get_param(), we ran into
the dilemma how to populate IPV4 and IPV6 address at the same time.
Currently on our adapter IPV4 and IPV6 can be enabled at the same time
and can be active.

So we wanted to find out if for following param mask :

ISCSI_HOST_PARAM_IPADDRESS

should we just fill the buffer for IPV4 and IPV6 address or should
we need define new param mask sepeartely for IPV6 ?

Basically we will like to follow up with another patch to address this part
on to of the patches which we have just pushed.


Right now iscsi_tcp supports ipv4 and ipv6 for that param. I think people have been saying it might be easier to break it up into a ipv4 and ipv6 param.

Karen, it seems like you are only supporting ipv4 right now. Any thoughts?

Jay, it seems be2iscsi handling is incorrect. Is your driver supporting ipv6 right now?
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux