From: Hannes Reinecke <hare@suse.de>
To: KY Srinivasan <kys@microsoft.com>
Cc: James Bottomley <jbottomley@parallels.com>,
"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"devel@linuxdriverproject.org" <devel@linuxdriverproject.org>,
"ohering@suse.com" <ohering@suse.com>,
"hch@infradead.org" <hch@infradead.org>,
"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>
Subject: Re: scanning for LUNs
Date: Mon, 08 Apr 2013 16:42:06 +0200 [thread overview]
Message-ID: <5162D73E.4000906@suse.de> (raw)
In-Reply-To: <5c58dfba0d564ed4b6959a0d478268b9@SN2PR03MB061.namprd03.prod.outlook.com>
On 04/04/2013 07:12 PM, KY Srinivasan wrote:
>
>
>> -----Original Message-----
>> From: James Bottomley [mailto:jbottomley@parallels.com]
>> Sent: Thursday, April 04, 2013 11:15 AM
>> To: KY Srinivasan
>> Cc: gregkh@linuxfoundation.org; linux-kernel@vger.kernel.org;
>> devel@linuxdriverproject.org; ohering@suse.com; hch@infradead.org; linux-
>> scsi@vger.kernel.org
>> Subject: Re: scanning for LUNs
>>
>> On Thu, 2013-04-04 at 08:12 -0700, K. Y. Srinivasan wrote:
>>> Here is the code snippet for scanning LUNS (drivers/scsi/scsi_scan.c in function
>>> __scsi_scan_target()):
>>>
>>> /*
>>> * Scan LUN 0, if there is some response, scan further. Ideally, we
>>> * would not configure LUN 0 until all LUNs are scanned.
>>> */
>>> res = scsi_probe_and_add_lun(starget, 0, &bflags, NULL, rescan, NULL);
>>> if (res == SCSI_SCAN_LUN_PRESENT || res ==
>> SCSI_SCAN_TARGET_PRESENT) {
>>> if (scsi_report_lun_scan(starget, bflags, rescan) != 0)
>>>
>>>
>>> So, if we don't get a response while scanning LUN0, we will not use
>>> scsi_report_lun_scan().
>>> On Hyper-V, the scsi emulation on the host does not treat LUN0 as
>>> anything special and we
>>> could have situations where the only device under a scsi controller is
>>> at a location other than 0
>>> or 1. In this case the standard LUN scanning code in Linux fails to
>>> detect this device. Is this
>>> behaviour expected? Why is LUN0 treated differently here. Looking at
>>> the scsi spec, I am not sure
>>> if this is what is specified. Any help/guidance will be greatly
>>> appreciated.
>>
>> Why don't you describe the problem. We can't scan randomly a bunch of
>> LUNs hoping for a response (the space is 10^19). SAM thinks you use
>> LUNW for this, but that's not well supported. We can't annoy USB
>> devices by probing with REPORT LUNS, so conventionally most arrays
>> return something for LUN0 even if they don't actually have one (That's
>> what the peripheral qualifier codes are supposed to be about). We
>> translate PQ1 and PQ2 to SCSI_SCAN_TARGET_PRESENT, which means no LUN,
>> but there is a target to scan here.
>>
>> If you're sending back an error to an INQUIRY to LUN0, then you're out
>> of spec. The SCSI standards say:
>>
>> SPC3 6.4.1: In response to an INQUIRY command received by an
>> incorrect logical unit, the SCSI target device shall return the
>> INQUIRY data with the peripheral qualifier set to the value
>> defined in 6.4.2. The INQUIRY command shall return CHECK
>> CONDITION status only when the device server is unable to return
>> the requested INQUIRY data
>
> Thanks James. I will further investigate the issue on our platform.
>
Or check if you can use W_LUN for scanning.
I've done a patchset for this (check the mailing list).
Using W_LUN is precisely for this type of setup.
(And would provide me with another scenario for using W_LUNs :-)
Cheers,
Hannes
--
Dr. Hannes Reinecke zSeries & Storage
hare@suse.de +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Hannes Reinecke <hare@suse.de>
To: KY Srinivasan <kys@microsoft.com>
Cc: James Bottomley <jbottomley@parallels.com>,
"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"devel@linuxdriverproject.org" <devel@linuxdriverproject.org>,
"ohering@suse.com" <ohering@suse.com>,
"hch@infradead.org" <hch@infradead.org>,
"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>
Subject: Re: scanning for LUNs
Date: Mon, 08 Apr 2013 16:42:06 +0200 [thread overview]
Message-ID: <5162D73E.4000906@suse.de> (raw)
In-Reply-To: <5c58dfba0d564ed4b6959a0d478268b9@SN2PR03MB061.namprd03.prod.outlook.com>
On 04/04/2013 07:12 PM, KY Srinivasan wrote:
>
>
>> -----Original Message-----
>> From: James Bottomley [mailto:jbottomley@parallels.com]
>> Sent: Thursday, April 04, 2013 11:15 AM
>> To: KY Srinivasan
>> Cc: gregkh@linuxfoundation.org; linux-kernel@vger.kernel.org;
>> devel@linuxdriverproject.org; ohering@suse.com; hch@infradead.org; linux-
>> scsi@vger.kernel.org
>> Subject: Re: scanning for LUNs
>>
>> On Thu, 2013-04-04 at 08:12 -0700, K. Y. Srinivasan wrote:
>>> Here is the code snippet for scanning LUNS (drivers/scsi/scsi_scan.c in function
>>> __scsi_scan_target()):
>>>
>>> /*
>>> * Scan LUN 0, if there is some response, scan further. Ideally, we
>>> * would not configure LUN 0 until all LUNs are scanned.
>>> */
>>> res = scsi_probe_and_add_lun(starget, 0, &bflags, NULL, rescan, NULL);
>>> if (res == SCSI_SCAN_LUN_PRESENT || res ==
>> SCSI_SCAN_TARGET_PRESENT) {
>>> if (scsi_report_lun_scan(starget, bflags, rescan) != 0)
>>>
>>>
>>> So, if we don't get a response while scanning LUN0, we will not use
>>> scsi_report_lun_scan().
>>> On Hyper-V, the scsi emulation on the host does not treat LUN0 as
>>> anything special and we
>>> could have situations where the only device under a scsi controller is
>>> at a location other than 0
>>> or 1. In this case the standard LUN scanning code in Linux fails to
>>> detect this device. Is this
>>> behaviour expected? Why is LUN0 treated differently here. Looking at
>>> the scsi spec, I am not sure
>>> if this is what is specified. Any help/guidance will be greatly
>>> appreciated.
>>
>> Why don't you describe the problem. We can't scan randomly a bunch of
>> LUNs hoping for a response (the space is 10^19). SAM thinks you use
>> LUNW for this, but that's not well supported. We can't annoy USB
>> devices by probing with REPORT LUNS, so conventionally most arrays
>> return something for LUN0 even if they don't actually have one (That's
>> what the peripheral qualifier codes are supposed to be about). We
>> translate PQ1 and PQ2 to SCSI_SCAN_TARGET_PRESENT, which means no LUN,
>> but there is a target to scan here.
>>
>> If you're sending back an error to an INQUIRY to LUN0, then you're out
>> of spec. The SCSI standards say:
>>
>> SPC3 6.4.1: In response to an INQUIRY command received by an
>> incorrect logical unit, the SCSI target device shall return the
>> INQUIRY data with the peripheral qualifier set to the value
>> defined in 6.4.2. The INQUIRY command shall return CHECK
>> CONDITION status only when the device server is unable to return
>> the requested INQUIRY data
>
> Thanks James. I will further investigate the issue on our platform.
>
Or check if you can use W_LUN for scanning.
I've done a patchset for this (check the mailing list).
Using W_LUN is precisely for this type of setup.
(And would provide me with another scenario for using W_LUNs :-)
Cheers,
Hannes
--
Dr. Hannes Reinecke zSeries & Storage
hare@suse.de +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
next prev parent reply other threads:[~2013-04-08 14:42 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-04 15:12 scanning for LUNs K. Y. Srinivasan
2013-04-04 15:15 ` James Bottomley
2013-04-04 17:12 ` KY Srinivasan
2013-04-08 14:42 ` Hannes Reinecke [this message]
2013-04-08 14:42 ` Hannes Reinecke
2013-04-08 17:34 ` KY Srinivasan
2013-04-08 17:34 ` KY Srinivasan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5162D73E.4000906@suse.de \
--to=hare@suse.de \
--cc=devel@linuxdriverproject.org \
--cc=gregkh@linuxfoundation.org \
--cc=hch@infradead.org \
--cc=jbottomley@parallels.com \
--cc=kys@microsoft.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=ohering@suse.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.