> On Thu, Aug 14, 2014 at 04:30:58PM +0300, Dolev Raviv wrote:
>> From: Subhash Jadavani <subhashj@xxxxxxxxxxxxxx>
>> REPORT LUNS command has "SELECT REPORT" field which controls what type of
>> logical units to be reported by device server. According to UFS device
standard, if this field is set to 0, REPORT LUNS would report only
report
>> standard logical units. If it's set to 1 then it would report only well
known logical unit and if it's set to 2 then device would report both
standard and well known logical units.
>> Some well-known logical units might not have scsi upper-layer drivers. In
>> such cases, the runtime PM reference count increased during device
enumeration will not be decreased, causing the parent device (host) to
always be on even during idle.
>> This change allows the SCSI LLD (Low Level Driver) to choose which type
of logical units should be detected.
>> Signed-off-by: Subhash Jadavani <subhashj@xxxxxxxxxxxxxx>
>> Signed-off-by: Sujit Reddy Thumma <sthumma@xxxxxxxxxxxxxx>
>> Signed-off-by: Dolev Raviv <draviv@xxxxxxxxxxxxxx>
>> diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c index
4a6e4ba..5a0e164 100644
>> --- a/drivers/scsi/scsi_scan.c
>> +++ b/drivers/scsi/scsi_scan.c
>> @@ -802,6 +802,14 @@ static int scsi_add_lun(struct scsi_device *sdev,
unsigned char *inq_result,
>> } else {
>> sdev->type = (inq_result[0] & 0x1f);
>> sdev->removable = (inq_result[1] & 0x80) >> 7;
>> +
>> + /*
>> + * some devices may respond with wrong type for
>> + * well-known logical units. Force well-known type
>> + * to enumerate them correctly.
>> + */
>> + if (scsi_is_wlun(sdev->lun) && (sdev->type != TYPE_WLUN))
>> + sdev->type = TYPE_WLUN;
> no need to test for TYPE_WLUN here.]
Agreed, will fix it.
>> switch (sdev->type) {
>> @@ -817,6 +825,7 @@ static int scsi_add_lun(struct scsi_device *sdev,
unsigned char *inq_result,
>> case TYPE_COMM:
>> case TYPE_RAID:
>> case TYPE_OSD:
>> + case TYPE_WLUN:
>> sdev->writeable = 1;
>> break;
>> case TYPE_ROM:
> This switch statements has been removed in 3.17-rc1, please rebase your
series on top of it.
Sure.
> While you're at it please make this patch, which
> introduceÑ? core scsi changes first in the series.
Ok. Will move these core changes first in the current patch series.
>> @@ -1412,6 +1421,13 @@ static int scsi_report_lun_scan(struct
>> scsi_target *starget, int bflags,
>> */
>> memset(&scsi_cmd[1], 0, 5);
>> + if (shost->report_wlus)
>> + /*
>> + * Set "SELECT REPORT" field to 0x2 which will make device to + *
report well known logical units along with standard LUs.
>> + */
>> + scsi_cmd[2] = 0x2;
> So we're finally getting well known LUN support. I think we should key
this
> off the SBC compliance level and not have the drivers opt in.
If i understood this correctly, we want to make all devices report well
known LUs irrespective of device driver (LLD)'s preference? If yes then i
will knock off "if (shost->report_wlus)" check.
>> diff --git a/drivers/scsi/scsi_sysfs.c b/drivers/scsi/scsi_sysfs.c
index 5f36788..ba9d0f0 100644
>> --- a/drivers/scsi/scsi_sysfs.c
>> +++ b/drivers/scsi/scsi_sysfs.c
>> @@ -1060,6 +1060,13 @@ int scsi_sysfs_add_sdev(struct scsi_device
*sdev)
>> }
>> }
>> + /*
>> + * put runtime pm reference for well-known logical units,
>> + * drivers are expected to _get_* again during probe.
>> + */
>> + if (scsi_is_wlun(sdev->lun))
>> + scsi_autopm_put_device(sdev);
> Special casing the well known LUNs here seems wrong. Shouldn't we do
this
> for any devices that don't get a driver attached to them?
Agreed, i will replace "if (scsi_is_wlun(sdev->lun))" check with "if
(sdev->sdev_gendev.driver)".
Regards,
Subhash
--
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
