Michael Tokarev wrote: > [Sorry for repost: was typo in linux-scsi@vger > email, and open-iscsi@ didn't accept email > either, as it says "i'm not subscribed" (which > isn't _entirely_ true ;)] > > Ravi Anand wrote: >> All, >> >> Today in the following email thread , Mike Christie outlined the >> modifications that has been done to qla4xx driver from the last submission: >> >>> http://marc.theaimsgroup.com/?l=linux-scsi&m=115151443814051&w=2 > [...] >> All the patches has been uploaded at the following URL: >> >> ftp://ftp.qlogic.com/outgoing/linux/iSCSI/upstream/5.00.05b6-k/ >> >> Complete driver package and the diff from previous submission for review >> has also been uploaded at the above mentioned link. > [...] > > Hi. > > I'm not of much help in reviewing the code and/or making code > suggestions. Instead, I'm trying to build the driver as we've > several new servers with ISP4010 adapters, which should work > somehow... ;) > > Except of the obvious prob wrt ql4_netlink.h file which you > already fixed by subsequent patch.. well.... > > ....Ok. I'm new in this area (just joined open-iscsi list). > And my first question is: which [i]scsi implementation you > were used as a base for this driver? > > I've tried against curren open-iscsi svn repo, against > scsi-misc kernel git tree, and against open-iscsi-1.0-485 > tarball. Whch all are mostly similar (in API anyway). > ql4 driver does not compile - even after "fixing" some > "missing" #includes (like iscsi_if.h defining most of > the ISCSI_* constants used in ql4_os.c), there are alot > of functions missing (iscsi_{add,remove,free}_session() > etc), undeclared structures used (*dscvr*) etc. For the kernel code you need to get the scsi-misc tree (see kernel.org/git for details). For the userspace tools you need the open-iscsi.org svn tree with this patch http://www.cs.wisc.edu/~michaelc/iscsi/open-iscsi/linux-scsi/tmp-qla4xxx-hook-into-userpace3.patch For the userspace tools, we do not have the init scripts finished up so you have to build the svn userspace tools, install them, then run iscsid and iscsiadm by hand. The iscsiadm command to run with qla4xxx is: iscsiadm -m discovery -t st -a $host_no_of_qla4xxx_card -p $discovery_address:$portal - : 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