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

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

 



On 01/30/2010 12:28 AM, Ravi Anand wrote:
+       uint16_t iscsi_max_burst_len;
+       uint16_t iscsi_max_outsnd_r2t;
+       uint16_t iscsi_first_burst_len;
+       uint16_t iscsi_max_rcv_data_seg_len;
+       uint16_t iscsi_max_snd_data_seg_len;


Probably do not need those settings above. Did not really see them used.



+
+       struct in6_addr remote_ipv6_addr;
+       struct in6_addr link_local_ipv6_addr;
  };

  /*
@@ -275,6 +284,16 @@ struct ddb_entry {
  #include "ql4_fw.h"
  #include "ql4_nvram.h"

+/* shortcut to print ISID */
+#define ISID(addr) \
+       ((unsigned char *)&addr)[5], \
+       ((unsigned char *)&addr)[4], \
+       ((unsigned char *)&addr)[3], \
+       ((unsigned char *)&addr)[2], \
+       ((unsigned char *)&addr)[1], \
+       ((unsigned char *)&addr)[0]
+#define ISID_FMT "0x%02x%02x%02x%02x%02x%02x"


Also not used. And if you add it, it should go in iscsi_proto.h.


+       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.


+       if (qla4xxx_mailbox_command(ha, 6, 6, mbox_cmd, mbox_sts)
+               != QLA_SUCCESS) {


I think we normally put the != on the end of the other line.

--
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