From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Christie Subject: Re: [PATCH] iscsi driver update 4.0.1.8 Date: Thu, 15 Jul 2004 10:24:23 -0700 Sender: linux-scsi-owner@vger.kernel.org Message-ID: <40F6BDC7.2090405@us.ibm.com> References: <200407151602.45830.naveenb@cisco.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from e3.ny.us.ibm.com ([32.97.182.103]:34963 "EHLO e3.ny.us.ibm.com") by vger.kernel.org with ESMTP id S266254AbUGOR0z (ORCPT ); Thu, 15 Jul 2004 13:26:55 -0400 In-Reply-To: <200407151602.45830.naveenb@cisco.com> List-Id: linux-scsi@vger.kernel.org To: naveenb@cisco.com Cc: Linux-SCSI Mailing List Naveen Burmi wrote: > This version of the iSCSI driver (4.0.1.8) contains the following changes > since 4.0.1.7: > > 1. The issue of deadlock due to multiple kmaps has been partially resolved. > We now use sendpage along the xmit path. > 2. The driver used to overwrite scsi midlayer naming. This has been removed. > Now proc_name field is used for iSCSI device lookup in sysfs. > 3. The driver's queuecommand now returns SCSI_MLQUEUE_DEVICE_BUSY and > SCSI_MLQUEUE_HOST_BUSY on failure. > 4. We have removed bus reset handler from the code. > > The following items are being worked on/debated: > > a) Per target probe via hotplug. > In earlier reviews, it was debated that the hotplug mechanism should be used > for target activation. Though scsi_scan_host() is exported, > scsi_scan_host_selected() isn't. So, an adapter scan is exported to LLDD > whereas anything else (target or lun scan) has to go through hotplug. > Network oriented storage like iSCSI has to deal with delays in targets > getting discovered through SendTargets discovery and poll for SLP > discoveries. > It would help a lot to use scsi_scan_host_selected() in such cases and needs > to be exported for such LLDDs. What is wrong with doing everything from userspace? If we do discovery in userspace and send down that info from usrspace, why not kick off the scanning from userspace once the establish session ioctl has completed? The driver today does not handle userspace initiated scans correctly, so this needs to be addressed regardless? The other question is are you advocating that the report luns data changed event no longer be handled from userspace? If scsi_scan_host_selected() is exported are you going to detect the event in the driver, kick off to userpace, then have some event agent go back down to the driver to use scsi_scan_host_selected()? Or are you wanting scsi_scan_host_selected() for your lun masking. I sent patches to kill the latter and it will hopefully not exists in this lld in the near future.