* [patch 0/1] [PATCH] scsi: fix check of PQ and PDT bits for WLUNs
@ 2008-08-12 11:40 Martin Petermann
2008-08-12 11:40 ` [patch 1/1] " Martin Petermann
0 siblings, 1 reply; 4+ messages in thread
From: Martin Petermann @ 2008-08-12 11:40 UTC (permalink / raw)
To: linux-scsi
--
^ permalink raw reply [flat|nested] 4+ messages in thread
* [patch 1/1] scsi: fix check of PQ and PDT bits for WLUNs
2008-08-12 11:40 [patch 0/1] [PATCH] scsi: fix check of PQ and PDT bits for WLUNs Martin Petermann
@ 2008-08-12 11:40 ` Martin Petermann
2008-08-21 20:14 ` James Bottomley
0 siblings, 1 reply; 4+ messages in thread
From: Martin Petermann @ 2008-08-12 11:40 UTC (permalink / raw)
To: linux-scsi; +Cc: Martin Petermann
[-- Attachment #1: scsi_scan.diff --]
[-- Type: text/plain, Size: 1568 bytes --]
From: Martin Petermann <martin@linux.vnet.ibm.com>
With kernel version 2.6.19 a check was introduced not to create a generic
SCSI device for devices that return PQ=1 and PDT=0x1f. For WLUNs (see
SAM-3, p. 41ff) generic SCSI devices should be created unconditionally
without looking at the PQ bit. I'm referring here to the thread "With kernel
2.6.19 no sg devices for devices that return PQ=1, PDT=0x1f" of the
linux-scsi forum. This is my proposed fix.
Signed-off-by: Martin Petermann <martin@linux.vnet.ibm.com>
---
drivers/scsi/scsi_scan.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
--- a/drivers/scsi/scsi_scan.c 2008-08-12 12:51:32.000000000 +0200
+++ b/drivers/scsi/scsi_scan.c 2008-08-12 12:58:16.000000000 +0200
@@ -75,6 +75,11 @@
#define SCSI_SCAN_TARGET_PRESENT 1
#define SCSI_SCAN_LUN_PRESENT 2
+#define SCSI_W_LUN_BASE 0xc100
+#define SCSI_W_LUN_REPORT_LUNS (SCSI_W_LUN_BASE + 1)
+#define SCSI_W_LUN_ACCESS_CONTROL (SCSI_W_LUN_BASE + 2)
+#define SCSI_W_LUN_TARGET_LOG_PAGE (SCSI_W_LUN_BASE + 3)
+
static const char *scsi_null_device_strs = "nullnullnullnull";
#define MAX_SCSI_LUNS 512
@@ -1080,7 +1085,7 @@ static int scsi_probe_and_add_lun(struct
* PDT=1Fh none (no FDD connected to the requested logical unit)
*/
if (((result[0] >> 5) == 1 || starget->pdt_1f_for_no_lun) &&
- (result[0] & 0x1f) == 0x1f) {
+ (result[0] & 0x1f) == 0x1f && (lun & 0xff00) != SCSI_W_LUN_BASE) {
SCSI_LOG_SCAN_BUS(3, printk(KERN_INFO
"scsi scan: peripheral device type"
" of 31, no device added\n"));
--
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [patch 1/1] scsi: fix check of PQ and PDT bits for WLUNs
2008-08-12 11:40 ` [patch 1/1] " Martin Petermann
@ 2008-08-21 20:14 ` James Bottomley
2008-08-25 8:41 ` Swen Schillig
0 siblings, 1 reply; 4+ messages in thread
From: James Bottomley @ 2008-08-21 20:14 UTC (permalink / raw)
To: Martin Petermann; +Cc: linux-scsi
On Tue, 2008-08-12 at 13:40 +0200, Martin Petermann wrote:
> plain text document attachment (scsi_scan.diff)
> From: Martin Petermann <martin@linux.vnet.ibm.com>
>
> With kernel version 2.6.19 a check was introduced not to create a generic
> SCSI device for devices that return PQ=1 and PDT=0x1f. For WLUNs (see
> SAM-3, p. 41ff) generic SCSI devices should be created unconditionally
> without looking at the PQ bit. I'm referring here to the thread "With kernel
> 2.6.19 no sg devices for devices that return PQ=1, PDT=0x1f" of the
> linux-scsi forum. This is my proposed fix.
>
> Signed-off-by: Martin Petermann <martin@linux.vnet.ibm.com>
>
> ---
> drivers/scsi/scsi_scan.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> --- a/drivers/scsi/scsi_scan.c 2008-08-12 12:51:32.000000000 +0200
> +++ b/drivers/scsi/scsi_scan.c 2008-08-12 12:58:16.000000000 +0200
> @@ -75,6 +75,11 @@
> #define SCSI_SCAN_TARGET_PRESENT 1
> #define SCSI_SCAN_LUN_PRESENT 2
>
> +#define SCSI_W_LUN_BASE 0xc100
> +#define SCSI_W_LUN_REPORT_LUNS (SCSI_W_LUN_BASE + 1)
> +#define SCSI_W_LUN_ACCESS_CONTROL (SCSI_W_LUN_BASE + 2)
> +#define SCSI_W_LUN_TARGET_LOG_PAGE (SCSI_W_LUN_BASE + 3)
These need to be in a header file, I think, probably scsi.h
> static const char *scsi_null_device_strs = "nullnullnullnull";
>
> #define MAX_SCSI_LUNS 512
> @@ -1080,7 +1085,7 @@ static int scsi_probe_and_add_lun(struct
> * PDT=1Fh none (no FDD connected to the requested logical unit)
> */
> if (((result[0] >> 5) == 1 || starget->pdt_1f_for_no_lun) &&
> - (result[0] & 0x1f) == 0x1f) {
> + (result[0] & 0x1f) == 0x1f && (lun & 0xff00) != SCSI_W_LUN_BASE) {
And this should probably be wrapped nicely too.
> SCSI_LOG_SCAN_BUS(3, printk(KERN_INFO
> "scsi scan: peripheral device type"
> " of 31, no device added\n"));
Does this look OK?
James
---
diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c
index 84b4879..34d0de6 100644
--- a/drivers/scsi/scsi_scan.c
+++ b/drivers/scsi/scsi_scan.c
@@ -1080,7 +1080,8 @@ static int scsi_probe_and_add_lun(struct scsi_target *starget,
* PDT=1Fh none (no FDD connected to the requested logical unit)
*/
if (((result[0] >> 5) == 1 || starget->pdt_1f_for_no_lun) &&
- (result[0] & 0x1f) == 0x1f) {
+ (result[0] & 0x1f) == 0x1f &&
+ !scsi_is_wlun(lun)) {
SCSI_LOG_SCAN_BUS(3, printk(KERN_INFO
"scsi scan: peripheral device type"
" of 31, no device added\n"));
diff --git a/include/scsi/scsi.h b/include/scsi/scsi.h
index 5c40cc5..192f871 100644
--- a/include/scsi/scsi.h
+++ b/include/scsi/scsi.h
@@ -309,6 +309,20 @@ struct scsi_lun {
};
/*
+ * The Well Known LUNS (SAM-3) in our int representation of a LUN
+ */
+#define SCSI_W_LUN_BASE 0xc100
+#define SCSI_W_LUN_REPORT_LUNS (SCSI_W_LUN_BASE + 1)
+#define SCSI_W_LUN_ACCESS_CONTROL (SCSI_W_LUN_BASE + 2)
+#define SCSI_W_LUN_TARGET_LOG_PAGE (SCSI_W_LUN_BASE + 3)
+
+static inline int scsi_is_wlun(unsigned int lun)
+{
+ return (lun & 0xff00) == SCSI_W_LUN_BASE;
+}
+
+
+/*
* MESSAGE CODES
*/
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [patch 1/1] scsi: fix check of PQ and PDT bits for WLUNs
2008-08-21 20:14 ` James Bottomley
@ 2008-08-25 8:41 ` Swen Schillig
0 siblings, 0 replies; 4+ messages in thread
From: Swen Schillig @ 2008-08-25 8:41 UTC (permalink / raw)
To: James Bottomley; +Cc: Martin Petermann, linux-scsi
On Thursday 21 August 2008 22:14, James Bottomley wrote:
> On Tue, 2008-08-12 at 13:40 +0200, Martin Petermann wrote:
> > plain text document attachment (scsi_scan.diff)
> > From: Martin Petermann <martin@linux.vnet.ibm.com>
> >
> > With kernel version 2.6.19 a check was introduced not to create a generic
> > SCSI device for devices that return PQ=1 and PDT=0x1f. For WLUNs (see
> > SAM-3, p. 41ff) generic SCSI devices should be created unconditionally
> > without looking at the PQ bit. I'm referring here to the thread "With kernel
> > 2.6.19 no sg devices for devices that return PQ=1, PDT=0x1f" of the
> > linux-scsi forum. This is my proposed fix.
> >
> > Signed-off-by: Martin Petermann <martin@linux.vnet.ibm.com>
> >
> > ---
> > drivers/scsi/scsi_scan.c | 7 ++++++-
> > 1 file changed, 6 insertions(+), 1 deletion(-)
> >
> > --- a/drivers/scsi/scsi_scan.c 2008-08-12 12:51:32.000000000 +0200
> > +++ b/drivers/scsi/scsi_scan.c 2008-08-12 12:58:16.000000000 +0200
> > @@ -75,6 +75,11 @@
> > #define SCSI_SCAN_TARGET_PRESENT 1
> > #define SCSI_SCAN_LUN_PRESENT 2
> >
> > +#define SCSI_W_LUN_BASE 0xc100
> > +#define SCSI_W_LUN_REPORT_LUNS (SCSI_W_LUN_BASE + 1)
> > +#define SCSI_W_LUN_ACCESS_CONTROL (SCSI_W_LUN_BASE + 2)
> > +#define SCSI_W_LUN_TARGET_LOG_PAGE (SCSI_W_LUN_BASE + 3)
>
> These need to be in a header file, I think, probably scsi.h
>
> > static const char *scsi_null_device_strs = "nullnullnullnull";
> >
> > #define MAX_SCSI_LUNS 512
> > @@ -1080,7 +1085,7 @@ static int scsi_probe_and_add_lun(struct
> > * PDT=1Fh none (no FDD connected to the requested logical unit)
> > */
> > if (((result[0] >> 5) == 1 || starget->pdt_1f_for_no_lun) &&
> > - (result[0] & 0x1f) == 0x1f) {
> > + (result[0] & 0x1f) == 0x1f && (lun & 0xff00) != SCSI_W_LUN_BASE) {
>
> And this should probably be wrapped nicely too.
>
> > SCSI_LOG_SCAN_BUS(3, printk(KERN_INFO
> > "scsi scan: peripheral device type"
> > " of 31, no device added\n"));
>
> Does this look OK?
>
> James
>
> ---
>
> diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c
> index 84b4879..34d0de6 100644
> --- a/drivers/scsi/scsi_scan.c
> +++ b/drivers/scsi/scsi_scan.c
> @@ -1080,7 +1080,8 @@ static int scsi_probe_and_add_lun(struct scsi_target *starget,
> * PDT=1Fh none (no FDD connected to the requested logical unit)
> */
> if (((result[0] >> 5) == 1 || starget->pdt_1f_for_no_lun) &&
> - (result[0] & 0x1f) == 0x1f) {
> + (result[0] & 0x1f) == 0x1f &&
> + !scsi_is_wlun(lun)) {
> SCSI_LOG_SCAN_BUS(3, printk(KERN_INFO
> "scsi scan: peripheral device type"
> " of 31, no device added\n"));
> diff --git a/include/scsi/scsi.h b/include/scsi/scsi.h
> index 5c40cc5..192f871 100644
> --- a/include/scsi/scsi.h
> +++ b/include/scsi/scsi.h
> @@ -309,6 +309,20 @@ struct scsi_lun {
> };
>
> /*
> + * The Well Known LUNS (SAM-3) in our int representation of a LUN
> + */
> +#define SCSI_W_LUN_BASE 0xc100
> +#define SCSI_W_LUN_REPORT_LUNS (SCSI_W_LUN_BASE + 1)
> +#define SCSI_W_LUN_ACCESS_CONTROL (SCSI_W_LUN_BASE + 2)
> +#define SCSI_W_LUN_TARGET_LOG_PAGE (SCSI_W_LUN_BASE + 3)
> +
> +static inline int scsi_is_wlun(unsigned int lun)
> +{
> + return (lun & 0xff00) == SCSI_W_LUN_BASE;
> +}
> +
> +
> +/*
> * MESSAGE CODES
> */
Looks ok.
So ACK from my side, if required.
Cheers Swen
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2008-08-25 8:41 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-08-12 11:40 [patch 0/1] [PATCH] scsi: fix check of PQ and PDT bits for WLUNs Martin Petermann
2008-08-12 11:40 ` [patch 1/1] " Martin Petermann
2008-08-21 20:14 ` James Bottomley
2008-08-25 8:41 ` Swen Schillig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).