>-----Original Message----- >From: Eddie Wai [mailto:eddie.wai@xxxxxxxxxxxx] >Sent: Monday, May 02, 2011 5:19 AM >To: open-iscsi@xxxxxxxxxxxxxxxx >Cc: James.Bottomley@xxxxxxx; michaelc@xxxxxxxxxxx; linux- >scsi@xxxxxxxxxxxxxxx; Vikas Chaudhary; Lalit Chandivade; Ravi Anand; Harish >Zunjarrao >Subject: Re: [RFC-V2 PATCH] iscsiadm: Add netconfig support in iscsiadm and >iscsid > >Hello Vikas, > >The patch looks good but I have a few questions/comments in relation to >the general scheme of setting these iface net parameters: > >1. As far as I understand, each iface file of the same hw must now >incorporate an unique iface_num which will be used to key off the >corresponding iface settings in the transport. Iscsiadm will send a >UEVENT along with the net contents to the transport only when it is >explicitly asked to apply the update via the command line. Perhaps it >would be beneficial to have the initiator apply the changes implicitly >like how it is currently making the call to the >iscsi_host_set_net_params just prior to all the ep_connect calls. This >will allow the current iface parameters to be set for the corresponding >ep_connect call. Yes, it makes sense. We can add flag similar to "t->template->set_host_ip" and allow implicit network configuration for each ep_connect. Mike C, do you see any issues? However, when we apply network settings for qla4xxx, it will do logout and login all devices, so to avoid all devices getting logout and login for each ep_connect call we do not want to apply network settings for each ep_connect. > >2. In the qla4xxx code changes, it looks like each Scsi_Host is only >allowing 1 IPv4 and 2 IPv6 addresses to the applied to the HBA. This is >conceivable as each interface can only assume that number of addresses. >But in the case of VLANs, how do you handle the case when the VLANs >assume different IP addresses and subnets when they are based off of the >same HBA? > >This can be resolved easily if we have a direct reference to the iface >parameters upon the ep_connect call. Perhaps embed this info (or >iface_num) to the Scsi_Host struct? Or just do as proposed in (1). > >3. Its probably a good idea to bump the ISCSI_UEVENT_MAX #define since >the netlink.c in the initiator uses it to error out undefined events. >> ISCSI_UEVENT_PATH_UPDATE = UEVENT_BASE + 20, >> + ISCSI_UEVENT_SET_NET_CONFIG = UEVENT_BASE + 21, >> >> ISCSI_UEVENT_MAX = ISCSI_UEVENT_PATH_UPDATE, >+ ISCSI_UEVENT_MAX = ISCSI_UEVENT_SET_NET_CONFIG, >> > Yes, we will fix this. >4. Although the newly defined "set_net_config" to the iscsi_transport >structure in the driver is a new augmentation, its interesting that the >iscsi_transport_template in the initiator already has a "set_net_config" >entry which is currently used for uip_broadcast_params. We probably >should change the name of one of them. We are not able to find duplication of "set_net_config" on master branch of open_iscsi. Can you please point out exact location?. Thanks, Vikas. This message and any attached documents contain information from QLogic Corporation or its wholly-owned subsidiaries that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message. -- 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
